Re: [PATCH 2/2] dt-bindings: Drop incorrect usage of double '::'
Peter Rosin <[email protected]> Mon, 13 Jul 2026 13:57:01 +0200
| Newsgroups | org.kernel.vger.linux-mmc,org.infradead.lists.linux-phy,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-clk,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-gpio,org.kernel.vger.linux-i2c,org.kernel.vger.linux-kernel,org.kernel.vger.linux-leds,org.kernel.vger.linux-media,org.kernel.vger.linux-pm,org.kernel.vger.linux-renesas-soc,org.kernel.vger.linux-samsung-soc,org.kernel.vger.linux-serial,org.kernel.vger.linux-sound,org.kernel.vger.linux-usb |
|---|---|
| Message-ID | <[email protected]> |
On 2026-06-22 12:16, Krzysztof Kozlowski wrote: > There is no use of double colon '::' in YAML. OTOH, the literal style > block, e.g. using '|' treats all characters as content [1] therefore > single use of ':' in descriptions is perfectly fine, whenever '|' is > used. > > Cleanup existing code, so the confusing style won't be re-used in new > contributions. > > Link: https://yaml.org/spec/1.2.2/#literal-style [1] > Signed-off-by: Krzysztof Kozlowski <[email protected]> > > --- > > Intention for this patch is to go via Rob's tree. > --- > diff --git a/Documentation/devicetree/bindings/mux/mux-consumer.yaml b/Documentation/devicetree/bindings/mux/mux-consumer.yaml > index 9e2d78a78e40..769243a2bf04 100644 > --- a/Documentation/devicetree/bindings/mux/mux-consumer.yaml > +++ b/Documentation/devicetree/bindings/mux/mux-consumer.yaml > @@ -13,8 +13,8 @@ description: | > Mux controller consumers should specify a list of mux controllers that they > want to use with a property containing a 'mux-ctrl-list': > > - mux-ctrl-list ::= <single-mux-ctrl> [mux-ctrl-list] > - single-mux-ctrl ::= <mux-ctrl-phandle> [mux-ctrl-specifier] > + mux-ctrl-list := <single-mux-ctrl> [mux-ctrl-list] > + single-mux-ctrl := <mux-ctrl-phandle> [mux-ctrl-specifier] > mux-ctrl-phandle : phandle to mux controller node > mux-ctrl-specifier : array of #mux-control-cells specifying the > given mux controller (controller specific) This change does not match the others? I do not think ::= is some kind of malformed YAML that has crept into a literal block. Instead, I think it is syntax inspired from BNF or something such. IIRC, it was once adapted from Documentation/devicetree/bindings/pwm/pwm.txt Or, I am missing something... Cheers, Peter