| Message ID | 9875ec0211187e4f5e2a4379c63eacdb69b31d7a.1682252615.git.stano.jakubek@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2,1/2] ARM: dts: bcm23550: Add SoC-specific I2C compatible | expand |
On Sun, 23 Apr 2023 14:40:18 +0200, Stanislav Jakubek wrote: > Convert Broadcom Kona family I2C bindings to DT schema. > > Changes during conversion: > - add used, but previously undocumented SoC-specific compatibles > - drop references to SoCs that are not upstream > - add supported clock frequencies according to the Linux driver [1] > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/i2c/busses/i2c-bcm-kona.c#n731 > > Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> > --- > Changes in v2: > - add missing arm-gic.h include > - fix indentation to 4 spaces > > .../devicetree/bindings/i2c/brcm,kona-i2c.txt | 35 ----------- > .../bindings/i2c/brcm,kona-i2c.yaml | 59 +++++++++++++++++++ > 2 files changed, 59 insertions(+), 35 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/i2c/brcm,kona-i2c.txt > create mode 100644 Documentation/devicetree/bindings/i2c/brcm,kona-i2c.yaml > Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings. Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future. Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/72ba28004afb733224f7294a146fc2a6a5a834a7.1682252615.git.stano.jakubek@gmail.com i2c@16000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@16000: compatible: ['brcm,kona-i2c'] is too short arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@17000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@17000: compatible: ['brcm,kona-i2c'] is too short arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@18000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@18000: compatible: ['brcm,kona-i2c'] is too short arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@1c000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] arch/arm/boot/dts/bcm23550-sparrow.dtb i2c@1c000: compatible: ['brcm,kona-i2c'] is too short arch/arm/boot/dts/bcm23550-sparrow.dtb
On 23/04/2023 14:40, Stanislav Jakubek wrote: > Convert Broadcom Kona family I2C bindings to DT schema. > > Changes during conversion: > - add used, but previously undocumented SoC-specific compatibles > - drop references to SoCs that are not upstream > - add supported clock frequencies according to the Linux driver [1] > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/i2c/busses/i2c-bcm-kona.c#n731 This line points to moving reference in next, so might no be accurate later. If you need external reference, then it should be some stable tag. But anyway if this is in mainline (is this?) then, why referencing via external link? Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Mon, Apr 24, 2023 at 03:53:07PM +0200, Krzysztof Kozlowski wrote: > On 23/04/2023 14:40, Stanislav Jakubek wrote: > > Convert Broadcom Kona family I2C bindings to DT schema. > > > > Changes during conversion: > > - add used, but previously undocumented SoC-specific compatibles > > - drop references to SoCs that are not upstream > > - add supported clock frequencies according to the Linux driver [1] > > > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/i2c/busses/i2c-bcm-kona.c#n731 > > This line points to moving reference in next, so might no be accurate > later. If you need external reference, then it should be some stable > tag. But anyway if this is in mainline (is this?) then, why referencing > via external link? Yes, it's in mainline since 2013. I just pointed to linux-next since that was what I had open at the time. Just to make sure if I understood correctly, if it's in mainline then there's no need to reference it via link? Or should I point to some tag instead, e.g.: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/i2c/busses/i2c-bcm-kona.c?h=v6.3#n731 Thank you for the review, Stanislav > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Best regards, > Krzysztof >
On Mon, Apr 24, 2023 at 08:28:08AM -0500, Rob Herring wrote: > > On Sun, 23 Apr 2023 14:40:18 +0200, Stanislav Jakubek wrote: > > Convert Broadcom Kona family I2C bindings to DT schema. > > > > Changes during conversion: > > - add used, but previously undocumented SoC-specific compatibles > > - drop references to SoCs that are not upstream > > - add supported clock frequencies according to the Linux driver [1] > > > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/i2c/busses/i2c-bcm-kona.c#n731 > > > > Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> > > --- > > Changes in v2: > > - add missing arm-gic.h include > > - fix indentation to 4 spaces > > > > .../devicetree/bindings/i2c/brcm,kona-i2c.txt | 35 ----------- > > .../bindings/i2c/brcm,kona-i2c.yaml | 59 +++++++++++++++++++ > > 2 files changed, 59 insertions(+), 35 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/i2c/brcm,kona-i2c.txt > > create mode 100644 Documentation/devicetree/bindings/i2c/brcm,kona-i2c.yaml > > > > Running 'make dtbs_check' with the schema in this patch gives the > following warnings. Consider if they are expected or the schema is > incorrect. These may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for dtbs_check. > This will change in the future. > > Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/72ba28004afb733224f7294a146fc2a6a5a834a7.1682252615.git.stano.jakubek@gmail.com > > > i2c@16000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@16000: compatible: ['brcm,kona-i2c'] is too short > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@17000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@17000: compatible: ['brcm,kona-i2c'] is too short > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@18000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@18000: compatible: ['brcm,kona-i2c'] is too short > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@1c000: compatible:0: 'brcm,kona-i2c' is not one of ['brcm,bcm11351-i2c', 'brcm,bcm21664-i2c', 'brcm,bcm23550-i2c'] > arch/arm/boot/dts/bcm23550-sparrow.dtb > > i2c@1c000: compatible: ['brcm,kona-i2c'] is too short > arch/arm/boot/dts/bcm23550-sparrow.dtb > These warnings are fixed by the first patch in the series. Maybe it didn't apply? Regards, Stanislav
On 26/04/2023 19:16, Stanislav Jakubek wrote: > On Mon, Apr 24, 2023 at 03:53:07PM +0200, Krzysztof Kozlowski wrote: >> On 23/04/2023 14:40, Stanislav Jakubek wrote: >>> Convert Broadcom Kona family I2C bindings to DT schema. >>> >>> Changes during conversion: >>> - add used, but previously undocumented SoC-specific compatibles >>> - drop references to SoCs that are not upstream >>> - add supported clock frequencies according to the Linux driver [1] >>> >>> [1] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/i2c/busses/i2c-bcm-kona.c#n731 >> >> This line points to moving reference in next, so might no be accurate >> later. If you need external reference, then it should be some stable >> tag. But anyway if this is in mainline (is this?) then, why referencing >> via external link? > > Yes, it's in mainline since 2013. I just pointed to linux-next since > that was what I had open at the time. > > Just to make sure if I understood correctly, if it's in mainline then > there's no need to reference it via link? Or should I point to some tag > instead, e.g.: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/i2c/busses/i2c-bcm-kona.c?h=v6.3#n731 Yes, this would be better but I propose still to drop the link entirely. If you know the commit SHA which introduced this, mention it. Best regards, Krzysztof
> > i2c@1c000: compatible: ['brcm,kona-i2c'] is too short > > arch/arm/boot/dts/bcm23550-sparrow.dtb > > > > These warnings are fixed by the first patch in the series. > Maybe it didn't apply? So, this depends on patch 1 but I2C doesn't pick up DT changes. So, shall this go via arm-soc, then? Or DT? Anyway, for any route: Acked-by: Wolfram Sang <wsa@kernel.org>
On 30/04/2023 08:55, Wolfram Sang wrote: > >>> i2c@1c000: compatible: ['brcm,kona-i2c'] is too short >>> arch/arm/boot/dts/bcm23550-sparrow.dtb >>> >> >> These warnings are fixed by the first patch in the series. >> Maybe it didn't apply? > > So, this depends on patch 1 but I2C doesn't pick up DT changes. So, > shall this go via arm-soc, then? Or DT? Anyway, for any route: > > Acked-by: Wolfram Sang <wsa@kernel.org> The dtbs warnings - if fixed via any other pactch - can be safely ignored. They do not affect bisectability. Please grab this patch via I2C. The DTS will go separately via arm-soc. Best regards, Krzysztof
> The dtbs warnings - if fixed via any other pactch - can be safely > ignored. They do not affect bisectability. Please grab this patch via > I2C. The DTS will go separately via arm-soc. OK, thanks for the heads up!
On Sun, 23 Apr 2023 14:39:12 +0200, Stanislav Jakubek <stano.jakubek@gmail.com> wrote: > Add BCM23550-specific compatible for brcm,kona-i2c nodes. While not > currently used by the i2c-bcm-kona driver, they can serve for further > customization, if required. > Done in preparation for dt-binding coversion to DT schema. > > Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> > --- Applied to https://github.com/Broadcom/stblinux/commits/devicetree/next, thanks! -- Florian
diff --git a/arch/arm/boot/dts/bcm23550.dtsi b/arch/arm/boot/dts/bcm23550.dtsi index a36c9b1d23c8..32b244b8546c 100644 --- a/arch/arm/boot/dts/bcm23550.dtsi +++ b/arch/arm/boot/dts/bcm23550.dtsi @@ -159,7 +159,7 @@ uartb3: serial@2000 { }; bsc1: i2c@16000 { - compatible = "brcm,kona-i2c"; + compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c"; reg = <0x00016000 0x70>; interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>; #address-cells = <1>; @@ -169,7 +169,7 @@ bsc1: i2c@16000 { }; bsc2: i2c@17000 { - compatible = "brcm,kona-i2c"; + compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c"; reg = <0x00017000 0x70>; interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>; #address-cells = <1>; @@ -179,7 +179,7 @@ bsc2: i2c@17000 { }; bsc3: i2c@18000 { - compatible = "brcm,kona-i2c"; + compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c"; reg = <0x00018000 0x70>; interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>; #address-cells = <1>; @@ -189,7 +189,7 @@ bsc3: i2c@18000 { }; bsc4: i2c@1c000 { - compatible = "brcm,kona-i2c"; + compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c"; reg = <0x0001c000 0x70>; interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>; #address-cells = <1>;
Add BCM23550-specific compatible for brcm,kona-i2c nodes. While not currently used by the i2c-bcm-kona driver, they can serve for further customization, if required. Done in preparation for dt-binding coversion to DT schema. Signed-off-by: Stanislav Jakubek <stano.jakubek@gmail.com> --- Changes in v2: - none arch/arm/boot/dts/bcm23550.dtsi | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)