| Message ID | 20240326103620.298298-4-tudor.ambarus@linaro.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 028a87e91fcd8c485afcf8bd0d26ae34a0872438 |
| Headers | show |
| Series | arm64: dts: exynos: gs101: define all PERIC USI nodes | expand |
Hi Tudor > -----Original Message----- > From: Tudor Ambarus <tudor.ambarus@linaro.org> > Sent: Tuesday, March 26, 2024 4:06 PM > To: peter.griffin@linaro.org; robh+dt@kernel.org; > krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org > Cc: alim.akhtar@samsung.com; linux-arm-kernel@lists.infradead.org; linux- > samsung-soc@vger.kernel.org; devicetree@vger.kernel.org; linux- > kernel@vger.kernel.org; andre.draszik@linaro.org; > willmcvicker@google.com; kernel-team@android.com; Tudor Ambarus > <tudor.ambarus@linaro.org> > Subject: [PATCH v2 3/4] arm64: dts: exynos: gs101: join lines close to 80 chars > > These lines fit 81 characters, which is pretty close to 80. > Join the lines. > Does this breaks checkpatch flow? > Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org> > --- For better readability, this looks good. Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> > arch/arm64/boot/dts/exynos/google/gs101.dtsi | 12 ++++-------- > 1 file changed, 4 insertions(+), 8 deletions(-) >
On 3/26/24 11:10, Alim Akhtar wrote: > Hi Tudor Hi, Alim! > >> -----Original Message----- >> From: Tudor Ambarus <tudor.ambarus@linaro.org> >> Sent: Tuesday, March 26, 2024 4:06 PM >> To: peter.griffin@linaro.org; robh+dt@kernel.org; >> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org >> Cc: alim.akhtar@samsung.com; linux-arm-kernel@lists.infradead.org; linux- >> samsung-soc@vger.kernel.org; devicetree@vger.kernel.org; linux- >> kernel@vger.kernel.org; andre.draszik@linaro.org; >> willmcvicker@google.com; kernel-team@android.com; Tudor Ambarus >> <tudor.ambarus@linaro.org> >> Subject: [PATCH v2 3/4] arm64: dts: exynos: gs101: join lines close to 80 > chars >> >> These lines fit 81 characters, which is pretty close to 80. >> Join the lines. >> > Does this breaks checkpatch flow? ./scripts/checkpatch --strict does not complain > >> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org> >> --- > For better readability, this looks good. > > Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> Thank you for reviewing the series! Cheers, ta
On 26/03/2024 15:48, Tudor Ambarus wrote: > > > On 3/26/24 11:10, Alim Akhtar wrote: >> Hi Tudor > > Hi, Alim! >> >>> -----Original Message----- >>> From: Tudor Ambarus <tudor.ambarus@linaro.org> >>> Sent: Tuesday, March 26, 2024 4:06 PM >>> To: peter.griffin@linaro.org; robh+dt@kernel.org; >>> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org >>> Cc: alim.akhtar@samsung.com; linux-arm-kernel@lists.infradead.org; linux- >>> samsung-soc@vger.kernel.org; devicetree@vger.kernel.org; linux- >>> kernel@vger.kernel.org; andre.draszik@linaro.org; >>> willmcvicker@google.com; kernel-team@android.com; Tudor Ambarus >>> <tudor.ambarus@linaro.org> >>> Subject: [PATCH v2 3/4] arm64: dts: exynos: gs101: join lines close to 80 >> chars >>> >>> These lines fit 81 characters, which is pretty close to 80. >>> Join the lines. >>> >> Does this breaks checkpatch flow? > > ./scripts/checkpatch --strict does not complain Because checkpatch does not have limit of 80... Coding style has, but for readability it is fine to stretch or even break this rule. Best regards, Krzysztof
Hi Tudor, On Tue, 26 Mar 2024 at 10:36, Tudor Ambarus <tudor.ambarus@linaro.org> wrote: > > These lines fit 81 characters, which is pretty close to 80. > Join the lines. > > Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org> > --- Reviewed-by: Peter Griffin <peter.griffin@linaro.org> > arch/arm64/boot/dts/exynos/google/gs101.dtsi | 12 ++++-------- > 1 file changed, 4 insertions(+), 8 deletions(-) > > diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi > index cfb3ddc7f885..690deca37e4f 100644 > --- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi > +++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi > @@ -374,8 +374,7 @@ pinctrl_peric0: pinctrl@10840000 { > }; > > usi8: usi@109700c0 { > - compatible = "google,gs101-usi", > - "samsung,exynos850-usi"; > + compatible = "google,gs101-usi", "samsung,exynos850-usi"; > reg = <0x109700c0 0x20>; > ranges; > #address-cells = <1>; > @@ -403,8 +402,7 @@ hsi2c_8: i2c@10970000 { > }; > > usi_uart: usi@10a000c0 { > - compatible = "google,gs101-usi", > - "samsung,exynos850-usi"; > + compatible = "google,gs101-usi", "samsung,exynos850-usi"; > reg = <0x10a000c0 0x20>; > ranges; > #address-cells = <1>; > @@ -419,8 +417,7 @@ usi_uart: usi@10a000c0 { > serial_0: serial@10a00000 { > compatible = "google,gs101-uart"; > reg = <0x10a00000 0xc0>; > - interrupts = <GIC_SPI 634 > - IRQ_TYPE_LEVEL_HIGH 0>; > + interrupts = <GIC_SPI 634 IRQ_TYPE_LEVEL_HIGH 0>; > clocks = <&cmu_peric0 CLK_GOUT_PERIC0_PERIC0_TOP1_PCLK_0>, > <&cmu_peric0 CLK_GOUT_PERIC0_PERIC0_TOP1_IPCLK_0>; > clock-names = "uart", "clk_uart_baud0"; > @@ -454,8 +451,7 @@ pinctrl_peric1: pinctrl@10c40000 { > }; > > usi12: usi@10d500c0 { > - compatible = "google,gs101-usi", > - "samsung,exynos850-usi"; > + compatible = "google,gs101-usi", "samsung,exynos850-usi"; > reg = <0x10d500c0 0x20>; > ranges; > #address-cells = <1>; > -- > 2.44.0.396.g6e790dbe36-goog >
diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi index cfb3ddc7f885..690deca37e4f 100644 --- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi +++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi @@ -374,8 +374,7 @@ pinctrl_peric0: pinctrl@10840000 { }; usi8: usi@109700c0 { - compatible = "google,gs101-usi", - "samsung,exynos850-usi"; + compatible = "google,gs101-usi", "samsung,exynos850-usi"; reg = <0x109700c0 0x20>; ranges; #address-cells = <1>; @@ -403,8 +402,7 @@ hsi2c_8: i2c@10970000 { }; usi_uart: usi@10a000c0 { - compatible = "google,gs101-usi", - "samsung,exynos850-usi"; + compatible = "google,gs101-usi", "samsung,exynos850-usi"; reg = <0x10a000c0 0x20>; ranges; #address-cells = <1>; @@ -419,8 +417,7 @@ usi_uart: usi@10a000c0 { serial_0: serial@10a00000 { compatible = "google,gs101-uart"; reg = <0x10a00000 0xc0>; - interrupts = <GIC_SPI 634 - IRQ_TYPE_LEVEL_HIGH 0>; + interrupts = <GIC_SPI 634 IRQ_TYPE_LEVEL_HIGH 0>; clocks = <&cmu_peric0 CLK_GOUT_PERIC0_PERIC0_TOP1_PCLK_0>, <&cmu_peric0 CLK_GOUT_PERIC0_PERIC0_TOP1_IPCLK_0>; clock-names = "uart", "clk_uart_baud0"; @@ -454,8 +451,7 @@ pinctrl_peric1: pinctrl@10c40000 { }; usi12: usi@10d500c0 { - compatible = "google,gs101-usi", - "samsung,exynos850-usi"; + compatible = "google,gs101-usi", "samsung,exynos850-usi"; reg = <0x10d500c0 0x20>; ranges; #address-cells = <1>;
These lines fit 81 characters, which is pretty close to 80. Join the lines. Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org> --- arch/arm64/boot/dts/exynos/google/gs101.dtsi | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-)