Message ID | 20200124114914.27065-1-dafna.hirschfeld@collabora.com |
---|---|
State | Not Applicable, archived |
Headers | show |
Series | dt-bindings: fix compilation error of the example in intel,lgm-emmc-phy.yaml | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | warning | "total: 0 errors, 1 warnings, 8 lines checked" |
On Fri, Jan 24, 2020 at 5:49 AM Dafna Hirschfeld <dafna.hirschfeld@collabora.com> wrote: > > Running: > export DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml > 'make dt_binding_check' > > gives a compilation error. This is because in the example there > is the label 'emmc-phy' but labels are not allowed to have '-' sing. > Replace the '-' with '_' to fix the error. > > Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> There's a fix from the author, but you're first to get the fix correct, so: Fixes: 5bc999108025 ("dt-bindings: phy: intel-emmc-phy: Add YAML schema for LGM eMMC PHY") Acked-by: Rob Herring <robh@kernel.org> Kishon, Please apply these soon as linux-next is broken. Rob
On 24.01.20 15:03, Rob Herring wrote: > On Fri, Jan 24, 2020 at 5:49 AM Dafna Hirschfeld > <dafna.hirschfeld@collabora.com> wrote: >> >> Running: >> export DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml >> 'make dt_binding_check' >> >> gives a compilation error. This is because in the example there >> is the label 'emmc-phy' but labels are not allowed to have '-' sing. >> Replace the '-' with '_' to fix the error. >> >> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> > > There's a fix from the author, but you're first to get the fix correct, so: Oh, sorry, I was not aware of that. Dafna > > Fixes: 5bc999108025 ("dt-bindings: phy: intel-emmc-phy: Add YAML > schema for LGM eMMC PHY") > Acked-by: Rob Herring <robh@kernel.org> > > Kishon, Please apply these soon as linux-next is broken. > > Rob >
On 24/01/20 7:59 pm, Dafna Hirschfeld wrote: > > > On 24.01.20 15:03, Rob Herring wrote: >> On Fri, Jan 24, 2020 at 5:49 AM Dafna Hirschfeld >> <dafna.hirschfeld@collabora.com> wrote: >>> >>> Running: >>> export >>> DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml >>> >>> 'make dt_binding_check' >>> >>> gives a compilation error. This is because in the example there >>> is the label 'emmc-phy' but labels are not allowed to have '-' sing. >>> Replace the '-' with '_' to fix the error. >>> >>> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> >> >> There's a fix from the author, but you're first to get the fix >> correct, so: > Oh, sorry, I was not aware of that. > Dafna > >> >> Fixes: 5bc999108025 ("dt-bindings: phy: intel-emmc-phy: Add YAML >> schema for LGM eMMC PHY") >> Acked-by: Rob Herring <robh@kernel.org> >> >> Kishon, Please apply these soon as linux-next is broken. >> merged now, Thanks! -Kishon
On Mon, Jan 27, 2020 at 6:21 AM Kishon Vijay Abraham I <kishon@ti.com> wrote: > > > > On 24/01/20 7:59 pm, Dafna Hirschfeld wrote: > > > > > > On 24.01.20 15:03, Rob Herring wrote: > >> On Fri, Jan 24, 2020 at 5:49 AM Dafna Hirschfeld > >> <dafna.hirschfeld@collabora.com> wrote: > >>> > >>> Running: > >>> export > >>> DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml > >>> > >>> 'make dt_binding_check' > >>> > >>> gives a compilation error. This is because in the example there > >>> is the label 'emmc-phy' but labels are not allowed to have '-' sing. > >>> Replace the '-' with '_' to fix the error. > >>> > >>> Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> > >> > >> There's a fix from the author, but you're first to get the fix > >> correct, so: > > Oh, sorry, I was not aware of that. > > Dafna > > > >> > >> Fixes: 5bc999108025 ("dt-bindings: phy: intel-emmc-phy: Add YAML > >> schema for LGM eMMC PHY") > >> Acked-by: Rob Herring <robh@kernel.org> > >> > >> Kishon, Please apply these soon as linux-next is broken. > >> > > merged now, Thanks! And please drop or revert this. It's still has failures: Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.example.dts:23.13-33: Warning (reg_format): /example-0/chiptop@e0200000/emmc-phy@a8:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1) Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.example.dt.yaml: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.example.dt.yaml: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.example.dt.yaml: Warning (spi_bus_reg): Failed prerequisite 'reg_format' Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.example.dts:21.33-26.13: Warning (avoid_default_addr_size): /example-0/chiptop@e0200000/emmc-phy@a8: Relying on default #address-cells value Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.example.dts:21.33-26.13: Warning (avoid_default_addr_size): /example-0/chiptop@e0200000/emmc-phy@a8: Relying on default #size-cells value I'll send out a complete fix and take via the DT tree so maybe we can have an rc1 that works. Rob
diff --git a/Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml b/Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml index ff7959c21af0..0de6b0f128b8 100644 --- a/Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml +++ b/Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml @@ -46,7 +46,7 @@ examples: compatible = "intel,lgm-syscon", "syscon"; reg = <0xe0200000 0x100>; - emmc-phy: emmc-phy@a8 { + emmc_phy: emmc-phy@a8 { compatible = "intel,lgm-emmc-phy"; reg = <0x00a8 0x10>; clocks = <&emmc>;
Running: export DT_SCHEMA_FILES=Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml 'make dt_binding_check' gives a compilation error. This is because in the example there is the label 'emmc-phy' but labels are not allowed to have '-' sing. Replace the '-' with '_' to fix the error. Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> --- Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)