[PATCH] dt-bindings: Fix inconsistent quoting
Rob Herring
robh at kernel.org
Thu Oct 16 04:46:02 PDT 2025
On Thu, Oct 16, 2025 at 2:57 AM Geert Uytterhoeven <geert at linux-m68k.org> wrote:
>
> Hi Rob,
>
> On Thu, 16 Oct 2025 at 01:20, Rob Herring (Arm) <robh at kernel.org> wrote:
> > yamllint has gained a new check which checks for inconsistent quoting
> > (mixed " and ' quotes within a file). Fix all the cases yamllint found
> > so we can enable the check (once the check is in a release). Use
> > whichever quoting is dominate in the file.
> >
> > Signed-off-by: Rob Herring (Arm) <robh at kernel.org>
>
> Thanks for your patch!
>
> Since you are mentioning mixed quotes, is one or the other preferred?
I have a slight preference for single quotes.
> Shouldn't we try to be consistent across all files?
I don't particularly care to change 915 files. And if the tools don't
enforce it, I'm not going to do so in reviews.
> > --- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> > @@ -129,7 +129,7 @@ additionalProperties:
> >
> > - type: object
> > additionalProperties:
> > - $ref: "#/additionalProperties/anyOf/0"
> > + $ref: '#/additionalProperties/anyOf/0'
> >
> > examples:
> > - |
> > @@ -190,7 +190,7 @@ examples:
> >
> > sdhi0_pins: sd0 {
> > groups = "sdhi0_data4", "sdhi0_ctrl";
> > - function = "sdhi0";
> > + function = "sdhi0';
>
> Huh?
One too many search and replace. Fixed.
Rob
More information about the linux-phy
mailing list