| Message ID | 20210922195208.1734936-1-festevam@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 07b2fb60467203a6525415022a6eefdddebe6b14 |
| Headers | show |
| Series | arm64: dts: sm6125: Remove leading zeroes | expand |
Reviewed-by: Martin Botka <martin.botka@somainline.org> On Wed, Sep 22 2021 at 04:52:08 PM -0300, Fabio Estevam <festevam@gmail.com> wrote: > dtc complains about the leading zeroes: > > arch/arm64/boot/dts/qcom/sm6125.dtsi:497.19-503.6: Warning > (unit_address_format): /soc/timer@f120000/frame@0f121000: unit name > should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:505.19-510.6: Warning > (unit_address_format): /soc/timer@f120000/frame@0f123000: unit name > should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:512.19-517.6: Warning > (unit_address_format): /soc/timer@f120000/frame@0f124000: unit name > should not have leading 0 > > Remove them. > > Signed-off-by: Fabio Estevam <festevam@gmail.com> > --- > arch/arm64/boot/dts/qcom/sm6125.dtsi | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/sm6125.dtsi > b/arch/arm64/boot/dts/qcom/sm6125.dtsi > index 2b37ce6a9f9c..0c1057456597 100644 > --- a/arch/arm64/boot/dts/qcom/sm6125.dtsi > +++ b/arch/arm64/boot/dts/qcom/sm6125.dtsi > @@ -494,7 +494,7 @@ timer@f120000 { > reg = <0x0f120000 0x1000>; > clock-frequency = <19200000>; > > - frame@0f121000 { > + frame@f121000 { > frame-number = <0>; > interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>, > <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>; > @@ -502,14 +502,14 @@ frame@0f121000 { > <0x0f122000 0x1000>; > }; > > - frame@0f123000 { > + frame@f123000 { > frame-number = <1>; > interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>; > reg = <0x0f123000 0x1000>; > status = "disabled"; > }; > > - frame@0f124000 { > + frame@f124000 { > frame-number = <2>; > interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>; > reg = <0x0f124000 0x1000>; > -- > 2.25.1
On Wed, 22 Sep 2021 16:52:08 -0300, Fabio Estevam wrote: > dtc complains about the leading zeroes: > > arch/arm64/boot/dts/qcom/sm6125.dtsi:497.19-503.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f121000: unit name should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:505.19-510.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f123000: unit name should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:512.19-517.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f124000: unit name should not have leading 0 > > Remove them. > > [...] Applied, thanks! [1/1] arm64: dts: sm6125: Remove leading zeroes commit: 2e7586bab95f5d284867c35ca46c0f0c19ccbf7f Best regards,
Hello: This patch was applied to qcom/linux.git (refs/heads/for-next): On Wed, 22 Sep 2021 16:52:08 -0300 you wrote: > dtc complains about the leading zeroes: > > arch/arm64/boot/dts/qcom/sm6125.dtsi:497.19-503.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f121000: unit name should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:505.19-510.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f123000: unit name should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:512.19-517.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f124000: unit name should not have leading 0 > > Remove them. > > [...] Here is the summary with links: - arm64: dts: sm6125: Remove leading zeroes https://git.kernel.org/qcom/c/07b2fb604672 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
On Wed, Sep 22, 2021 at 9:52 PM Fabio Estevam <festevam@gmail.com> wrote: > dtc complains about the leading zeroes: > > arch/arm64/boot/dts/qcom/sm6125.dtsi:497.19-503.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f121000: unit name should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:505.19-510.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f123000: unit name should not have leading 0s > arch/arm64/boot/dts/qcom/sm6125.dtsi:512.19-517.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f124000: unit name should not have leading 0 > > Remove them. > > Signed-off-by: Fabio Estevam <festevam@gmail.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
diff --git a/arch/arm64/boot/dts/qcom/sm6125.dtsi b/arch/arm64/boot/dts/qcom/sm6125.dtsi index 2b37ce6a9f9c..0c1057456597 100644 --- a/arch/arm64/boot/dts/qcom/sm6125.dtsi +++ b/arch/arm64/boot/dts/qcom/sm6125.dtsi @@ -494,7 +494,7 @@ timer@f120000 { reg = <0x0f120000 0x1000>; clock-frequency = <19200000>; - frame@0f121000 { + frame@f121000 { frame-number = <0>; interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>, <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>; @@ -502,14 +502,14 @@ frame@0f121000 { <0x0f122000 0x1000>; }; - frame@0f123000 { + frame@f123000 { frame-number = <1>; interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>; reg = <0x0f123000 0x1000>; status = "disabled"; }; - frame@0f124000 { + frame@f124000 { frame-number = <2>; interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>; reg = <0x0f124000 0x1000>;
dtc complains about the leading zeroes: arch/arm64/boot/dts/qcom/sm6125.dtsi:497.19-503.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f121000: unit name should not have leading 0s arch/arm64/boot/dts/qcom/sm6125.dtsi:505.19-510.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f123000: unit name should not have leading 0s arch/arm64/boot/dts/qcom/sm6125.dtsi:512.19-517.6: Warning (unit_address_format): /soc/timer@f120000/frame@0f124000: unit name should not have leading 0 Remove them. Signed-off-by: Fabio Estevam <festevam@gmail.com> --- arch/arm64/boot/dts/qcom/sm6125.dtsi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)