2024-02-08 10:54:24

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup

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

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
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



2024-02-08 10:54:37

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 2/2] ARM: dts: ti: omap: minor whitespace cleanup

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

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
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


2024-03-31 10:26:21

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup


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 <[email protected]>

2024-04-09 19:37:16

by Nishanth Menon

[permalink] [raw]
Subject: Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup

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


2024-04-09 20:14:44

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup

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


2024-04-10 00:40:20

by Nishanth Menon

[permalink] [raw]
Subject: Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup

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

2024-04-10 06:26:05

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: (subset) [PATCH 2/2] ARM: dts: ti: omap: minor whitespace cleanup


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 <[email protected]>


2024-04-10 07:16:20

by Tony Lindgren

[permalink] [raw]
Subject: Re: (subset) [PATCH 1/2] arm64: dts: ti: k3-am62p5-sk: minor whitespace cleanup

* Nishanth Menon <[email protected]> [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

2024-04-10 09:53:00

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH 2/2] ARM: dts: ti: omap: minor whitespace cleanup

* Krzysztof Kozlowski <[email protected]> [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 <[email protected]>