linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
@ 2024-02-08 10:51 Krzysztof Kozlowski
  2024-02-08 10:51 ` [PATCH 2/2] ARM: dts: ti: omap: " Krzysztof Kozlowski
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-02-08 10:51 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Nishanth Menon,
	Vignesh Raghavendra, Tero Kristo, linux-omap, devicetree,
	linux-kernel, linux-arm-kernel
  Cc: Krzysztof Kozlowski

The DTS code coding style expects exactly one space before '{'
character.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/ti/k3-am62p5-sk.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts b/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
index 95a0146279b1..21aa40878cd1 100644
--- a/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
@@ -494,7 +494,7 @@ &ospi0 {
 	pinctrl-0 = <&ospi0_pins_default>;
 	bootph-all;
 
-	flash@0{
+	flash@0 {
 		compatible = "jedec,spi-nor";
 		reg = <0x0>;
 		spi-tx-bus-width = <8>;
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 2/2] ARM: dts: ti: omap: minor whitespace cleanup
  2024-02-08 10:51 [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup Krzysztof Kozlowski
@ 2024-02-08 10:51 ` Krzysztof Kozlowski
  2024-04-10  6:12   ` Tony Lindgren
  2024-04-10  6:21   ` (subset) " Krzysztof Kozlowski
  2024-03-31 10:26 ` [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: " Krzysztof Kozlowski
  2024-04-09 19:36 ` (subset) " Nishanth Menon
  2 siblings, 2 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-02-08 10:51 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Nishanth Menon,
	Vignesh Raghavendra, Tero Kristo, linux-omap, devicetree,
	linux-kernel, linux-arm-kernel
  Cc: Krzysztof Kozlowski

The DTS code coding style expects exactly one space before '{'
character.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/ti/omap/am33xx.dtsi | 8 ++++----
 arch/arm/boot/dts/ti/omap/am4372.dtsi | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/ti/omap/am33xx.dtsi b/arch/arm/boot/dts/ti/omap/am33xx.dtsi
index 989d5a6edeed..0614ffdc1578 100644
--- a/arch/arm/boot/dts/ti/omap/am33xx.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am33xx.dtsi
@@ -80,7 +80,7 @@ cpu0_opp_table: opp-table {
 		 * because the can not be enabled simultaneously on a
 		 * single SoC.
 		 */
-		opp-50-300000000{
+		opp-50-300000000 {
 			/* OPP50 */
 			opp-hz = /bits/ 64 <300000000>;
 			opp-microvolt = <950000 931000 969000>;
@@ -88,7 +88,7 @@ opp-50-300000000{
 			opp-suspend;
 		};
 
-		opp-100-275000000{
+		opp-100-275000000 {
 			/* OPP100-1 */
 			opp-hz = /bits/ 64 <275000000>;
 			opp-microvolt = <1100000 1078000 1122000>;
@@ -96,7 +96,7 @@ opp-100-275000000{
 			opp-suspend;
 		};
 
-		opp-100-300000000{
+		opp-100-300000000 {
 			/* OPP100-2 */
 			opp-hz = /bits/ 64 <300000000>;
 			opp-microvolt = <1100000 1078000 1122000>;
@@ -104,7 +104,7 @@ opp-100-300000000{
 			opp-suspend;
 		};
 
-		opp-100-500000000{
+		opp-100-500000000 {
 			/* OPP100-3 */
 			opp-hz = /bits/ 64 <500000000>;
 			opp-microvolt = <1100000 1078000 1122000>;
diff --git a/arch/arm/boot/dts/ti/omap/am4372.dtsi b/arch/arm/boot/dts/ti/omap/am4372.dtsi
index 5fd1b380ece6..0a1df30f2818 100644
--- a/arch/arm/boot/dts/ti/omap/am4372.dtsi
+++ b/arch/arm/boot/dts/ti/omap/am4372.dtsi
@@ -92,7 +92,7 @@ opp-120-720000000 {
 			opp-supported-hw = <0xFF 0x08>;
 		};
 
-		opp-800000000{
+		opp-800000000 {
 			/* OPP Turbo */
 			opp-hz = /bits/ 64 <800000000>;
 			opp-microvolt = <1260000 1234800 1285200>;
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
  2024-02-08 10:51 [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup Krzysztof Kozlowski
  2024-02-08 10:51 ` [PATCH 2/2] ARM: dts: ti: omap: " Krzysztof Kozlowski
@ 2024-03-31 10:26 ` Krzysztof Kozlowski
  2024-04-09 19:36 ` (subset) " Nishanth Menon
  2 siblings, 0 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-31 10:26 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Nishanth Menon,
	Vignesh Raghavendra, Tero Kristo, linux-omap, devicetree,
	linux-kernel, linux-arm-kernel, Krzysztof Kozlowski


On Thu, 08 Feb 2024 11:51:45 +0100, Krzysztof Kozlowski wrote:
> The DTS code coding style expects exactly one space before '{'
> character.
> 
> 

This is waiting on the lists for almost two months, so I just picked it up. Let
me know if anyone prefers to take it instead.

Applied, thanks!

[1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
      https://git.kernel.org/krzk/linux-dt/c/9d0ee097b3e5873e4e98770b94f11481f485e7c9
[2/2] ARM: dts: ti: omap: minor whitespace cleanup
      https://git.kernel.org/krzk/linux-dt/c/021bc7094e8c8ac1380527d3f53561c9a234a190

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
  2024-02-08 10:51 [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup Krzysztof Kozlowski
  2024-02-08 10:51 ` [PATCH 2/2] ARM: dts: ti: omap: " Krzysztof Kozlowski
  2024-03-31 10:26 ` [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: " Krzysztof Kozlowski
@ 2024-04-09 19:36 ` Nishanth Menon
  2024-04-09 20:04   ` Krzysztof Kozlowski
  2 siblings, 1 reply; 9+ messages in thread
From: Nishanth Menon @ 2024-04-09 19:36 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vignesh Raghavendra,
	Tero Kristo, linux-omap, devicetree, linux-kernel,
	linux-arm-kernel, Krzysztof Kozlowski
  Cc: Nishanth Menon

Hi Krzysztof Kozlowski,

On Thu, 08 Feb 2024 11:51:45 +0100, Krzysztof Kozlowski wrote:
> The DTS code coding style expects exactly one space before '{'
> character.
> 
> 
As discussed offline, I am picking this patch up.

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
      commit: 45ab8daed512258c07fd14536a3633440dabfe84

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
  2024-04-09 19:36 ` (subset) " Nishanth Menon
@ 2024-04-09 20:04   ` Krzysztof Kozlowski
  2024-04-10  0:39     ` Nishanth Menon
  0 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-09 20:04 UTC (permalink / raw)
  To: Nishanth Menon, Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vignesh Raghavendra,
	Tero Kristo, linux-omap, devicetree, linux-kernel,
	linux-arm-kernel

On 09/04/2024 21:36, Nishanth Menon wrote:
> Hi Krzysztof Kozlowski,
> 
> On Thu, 08 Feb 2024 11:51:45 +0100, Krzysztof Kozlowski wrote:
>> The DTS code coding style expects exactly one space before '{'
>> character.
>>
>>
> As discussed offline, I am picking this patch up.
> 
> I have applied the following to branch ti-k3-dts-next on [1].
> Thank you!
> 
> [1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
>       commit: 45ab8daed512258c07fd14536a3633440dabfe84

What about the omap one (second in the series)? Shall I take it?

Best regards,
Krzysztof


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
  2024-04-09 20:04   ` Krzysztof Kozlowski
@ 2024-04-10  0:39     ` Nishanth Menon
  2024-04-10  6:13       ` Tony Lindgren
  0 siblings, 1 reply; 9+ messages in thread
From: Nishanth Menon @ 2024-04-10  0:39 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vignesh Raghavendra,
	Tero Kristo, linux-omap, devicetree, linux-kernel,
	linux-arm-kernel

On 22:04-20240409, Krzysztof Kozlowski wrote:
> On 09/04/2024 21:36, Nishanth Menon wrote:
> > Hi Krzysztof Kozlowski,
> > 
> > On Thu, 08 Feb 2024 11:51:45 +0100, Krzysztof Kozlowski wrote:
> >> The DTS code coding style expects exactly one space before '{'
> >> character.
> >>
> >>
> > As discussed offline, I am picking this patch up.
> > 
> > I have applied the following to branch ti-k3-dts-next on [1].
> > Thank you!
> > 
> > [1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
> >       commit: 45ab8daed512258c07fd14536a3633440dabfe84
> 
> What about the omap one (second in the series)? Shall I take it?

I had poked Tony about it (he is the maintainer for OMAP). Tony: could
you comment?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] ARM: dts: ti: omap: minor whitespace cleanup
  2024-02-08 10:51 ` [PATCH 2/2] ARM: dts: ti: omap: " Krzysztof Kozlowski
@ 2024-04-10  6:12   ` Tony Lindgren
  2024-04-10  6:21   ` (subset) " Krzysztof Kozlowski
  1 sibling, 0 replies; 9+ messages in thread
From: Tony Lindgren @ 2024-04-10  6:12 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Benoît Cousson, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Nishanth Menon, Vignesh Raghavendra, Tero Kristo,
	linux-omap, devicetree, linux-kernel, linux-arm-kernel

* Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [240208 10:51]:
> The DTS code coding style expects exactly one space before '{'
> character.

Krzysztof, sorry for the confusion on getting this merged, please go ahead
and take it:

Acked-by: Tony Lindgren <tony@atomide.com>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
  2024-04-10  0:39     ` Nishanth Menon
@ 2024-04-10  6:13       ` Tony Lindgren
  0 siblings, 0 replies; 9+ messages in thread
From: Tony Lindgren @ 2024-04-10  6:13 UTC (permalink / raw)
  To: Nishanth Menon
  Cc: Krzysztof Kozlowski, Benoît Cousson, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vignesh Raghavendra,
	Tero Kristo, linux-omap, devicetree, linux-kernel,
	linux-arm-kernel

* Nishanth Menon <nm@ti.com> [240410 00:39]:
> On 22:04-20240409, Krzysztof Kozlowski wrote:
> > On 09/04/2024 21:36, Nishanth Menon wrote:
> > > Hi Krzysztof Kozlowski,
> > > 
> > > On Thu, 08 Feb 2024 11:51:45 +0100, Krzysztof Kozlowski wrote:
> > >> The DTS code coding style expects exactly one space before '{'
> > >> character.
> > >>
> > >>
> > > As discussed offline, I am picking this patch up.
> > > 
> > > I have applied the following to branch ti-k3-dts-next on [1].
> > > Thank you!
> > > 
> > > [1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup
> > >       commit: 45ab8daed512258c07fd14536a3633440dabfe84
> > 
> > What about the omap one (second in the series)? Shall I take it?
> 
> I had poked Tony about it (he is the maintainer for OMAP). Tony: could
> you comment?

Krzysztof, please go ahead apply it, it's not confilicting with anything
I have. I just acked the patch.

Regards,

Tony

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: (subset) [PATCH 2/2] ARM: dts: ti: omap: minor whitespace cleanup
  2024-02-08 10:51 ` [PATCH 2/2] ARM: dts: ti: omap: " Krzysztof Kozlowski
  2024-04-10  6:12   ` Tony Lindgren
@ 2024-04-10  6:21   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-10  6:21 UTC (permalink / raw)
  To: Benoît Cousson, Tony Lindgren, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Nishanth Menon,
	Vignesh Raghavendra, Tero Kristo, linux-omap, devicetree,
	linux-kernel, linux-arm-kernel, Krzysztof Kozlowski


On Thu, 08 Feb 2024 11:51:46 +0100, Krzysztof Kozlowski wrote:
> The DTS code coding style expects exactly one space before '{'
> character.
> 
> 

Applied, thanks!

[2/2] ARM: dts: ti: omap: minor whitespace cleanup
      https://git.kernel.org/krzk/linux-dt/c/71413bcb66e018e54afec47a9ce1199130d6fa38

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2024-04-10  6:30 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-08 10:51 [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup Krzysztof Kozlowski
2024-02-08 10:51 ` [PATCH 2/2] ARM: dts: ti: omap: " Krzysztof Kozlowski
2024-04-10  6:12   ` Tony Lindgren
2024-04-10  6:21   ` (subset) " Krzysztof Kozlowski
2024-03-31 10:26 ` [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: " Krzysztof Kozlowski
2024-04-09 19:36 ` (subset) " Nishanth Menon
2024-04-09 20:04   ` Krzysztof Kozlowski
2024-04-10  0:39     ` Nishanth Menon
2024-04-10  6:13       ` Tony Lindgren

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).