Re: [PATCH v1 03/26] tbs2910: prepare to synchronise device trees with linux

2022-07-25 Thread Soeren Moch




On 25.07.22 18:42, sba...@denx.de wrote:

From: Marcel Ziswiler 
As a preparatory step make sure to refer to absolute node paths where
labels were removed in Linux upstream.
This avoids the following error once synchronised:
+Error: arch/arm/dts/imx6q-tbs2910-u-boot.dtsi:3.1-7 Label or path aips1
  not found
+Error: arch/arm/dts/imx6q-tbs2910-u-boot.dtsi:11.1-5 Label or path soc
  not found
Signed-off-by: Marcel Ziswiler 

Applied to u-boot-imx, master, thanks !


What a nonsense!

We already discussed that it absolutely makes no sense to remove dt
parts in u-boot that are accepted for linux and just not applied yet.

Just to prepare a synchronization. Really stupid!

Soeren



Best regards,
Stefano Babic



Re: [PATCH v1 03/26] tbs2910: prepare to synchronise device trees with linux

2022-07-21 Thread Fabio Estevam
Hi Marcel,

On Thu, Jul 21, 2022 at 3:56 PM Marcel Ziswiler
 wrote:

> Oh, okay. No, haven't seen that one yet. So it might make sense to re-base 
> everything on latest -next then
> which would no longer require such change. Let me re-work this in a v2. 
> Thanks!

Yes, please sync with linux-next.

linux-next has also an imx8mm/mq/mn/mp job ring fix for a problem that
Andrey reported recently.

Thanks for working on this!

Cheers


Re: [PATCH v1 03/26] tbs2910: prepare to synchronise device trees with linux

2022-07-21 Thread Soeren Moch

On 21.07.22 15:27, Marcel Ziswiler wrote:

From: Marcel Ziswiler 

As a preparatory step make sure to refer to absolute node paths where
labels were removed in Linux upstream.

Are you sure that this was removed upstream?
It was just submitted for inclusion [1], and may not have landed yet.

Regards,
Soeren

[1]
https://lore.kernel.org/linux-arm-kernel/20220620003736.GN254723@dragon/T/


This avoids the following error once synchronised:

+Error: arch/arm/dts/imx6q-tbs2910-u-boot.dtsi:3.1-7 Label or path aips1
  not found
+Error: arch/arm/dts/imx6q-tbs2910-u-boot.dtsi:11.1-5 Label or path soc
  not found

Signed-off-by: Marcel Ziswiler 
---

  arch/arm/dts/imx6q-tbs2910-u-boot.dtsi | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/imx6q-tbs2910-u-boot.dtsi 
b/arch/arm/dts/imx6q-tbs2910-u-boot.dtsi
index 65ab052ac2c..d48719e7d59 100644
--- a/arch/arm/dts/imx6q-tbs2910-u-boot.dtsi
+++ b/arch/arm/dts/imx6q-tbs2910-u-boot.dtsi
@@ -1,6 +1,6 @@
  // SPDX-License-Identifier: GPL-2.0+

- {
+&{/soc/bus@200} { /* AIPS1 */
u-boot,dm-pre-reloc;
  };

@@ -8,7 +8,7 @@
u-boot,dm-pre-reloc;
  };

- {
+&{/soc} {
u-boot,dm-pre-reloc;
  };