* [PATCH v2] arm: dts: omap: Apply am57xx-idk overlays to base dtbs
@ 2023-10-10 21:19 Rob Herring
2023-10-18 5:20 ` Tony Lindgren
0 siblings, 1 reply; 4+ messages in thread
From: Rob Herring @ 2023-10-10 21:19 UTC (permalink / raw)
To: Andrew Davis, Benoît Cousson, Tony Lindgren,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-omap, devicetree, linux-kernel
DT overlays in tree need to be applied to a base DTB to validate they
apply, to run schema checks on them, and to catch any errors at compile
time.
Signed-off-by: Rob Herring <robh@kernel.org>
---
v2:
- Only test applying overlays when building all DTBs
---
arch/arm/boot/dts/ti/omap/Makefile | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/ti/omap/Makefile b/arch/arm/boot/dts/ti/omap/Makefile
index d2b590004fed..d6ee4a247100 100644
--- a/arch/arm/boot/dts/ti/omap/Makefile
+++ b/arch/arm/boot/dts/ti/omap/Makefile
@@ -129,6 +129,16 @@ dtb-$(CONFIG_SOC_AM43XX) += \
am57xx-evm-dtbs := am57xx-beagle-x15.dtb am57xx-evm.dtbo
am57xx-evm-reva3-dtbs := am57xx-beagle-x15-revc.dtb am57xx-evm.dtbo
+am571x-idk-overlays-dtbs := am571x-idk.dtb \
+ am571x-idk-touchscreen.dtbo am57xx-idk-lcd-osd101t2587.dtbo
+am572x-idk-overlays-dtbs := am572x-idk.dtb \
+ am572x-idk-touchscreen.dtbo am57xx-idk-lcd-osd101t2045.dtbo
+
+# Build time test only, enabled by CONFIG_OF_ALL_DTBS
+dtb- += \
+ am571x-idk-overlays.dtb \
+ am572x-idk-overlays.dtb
+
dtb-$(CONFIG_SOC_DRA7XX) += \
am57xx-beagle-x15.dtb \
am57xx-beagle-x15-revb1.dtb \
--
2.42.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] arm: dts: omap: Apply am57xx-idk overlays to base dtbs
2023-10-10 21:19 [PATCH v2] arm: dts: omap: Apply am57xx-idk overlays to base dtbs Rob Herring
@ 2023-10-18 5:20 ` Tony Lindgren
2023-10-18 14:05 ` Andrew Davis
0 siblings, 1 reply; 4+ messages in thread
From: Tony Lindgren @ 2023-10-18 5:20 UTC (permalink / raw)
To: Rob Herring
Cc: Andrew Davis, Benoît Cousson, Krzysztof Kozlowski,
Conor Dooley, linux-omap, devicetree, linux-kernel
* Rob Herring <robh@kernel.org> [231011 00:19]:
> DT overlays in tree need to be applied to a base DTB to validate they
> apply, to run schema checks on them, and to catch any errors at compile
> time.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> v2:
> - Only test applying overlays when building all DTBs
Andrew, any comments or care to ack?
Regards,
Tony
> ---
> arch/arm/boot/dts/ti/omap/Makefile | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/ti/omap/Makefile b/arch/arm/boot/dts/ti/omap/Makefile
> index d2b590004fed..d6ee4a247100 100644
> --- a/arch/arm/boot/dts/ti/omap/Makefile
> +++ b/arch/arm/boot/dts/ti/omap/Makefile
> @@ -129,6 +129,16 @@ dtb-$(CONFIG_SOC_AM43XX) += \
> am57xx-evm-dtbs := am57xx-beagle-x15.dtb am57xx-evm.dtbo
> am57xx-evm-reva3-dtbs := am57xx-beagle-x15-revc.dtb am57xx-evm.dtbo
>
> +am571x-idk-overlays-dtbs := am571x-idk.dtb \
> + am571x-idk-touchscreen.dtbo am57xx-idk-lcd-osd101t2587.dtbo
> +am572x-idk-overlays-dtbs := am572x-idk.dtb \
> + am572x-idk-touchscreen.dtbo am57xx-idk-lcd-osd101t2045.dtbo
> +
> +# Build time test only, enabled by CONFIG_OF_ALL_DTBS
> +dtb- += \
> + am571x-idk-overlays.dtb \
> + am572x-idk-overlays.dtb
> +
> dtb-$(CONFIG_SOC_DRA7XX) += \
> am57xx-beagle-x15.dtb \
> am57xx-beagle-x15-revb1.dtb \
> --
> 2.42.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] arm: dts: omap: Apply am57xx-idk overlays to base dtbs
2023-10-18 5:20 ` Tony Lindgren
@ 2023-10-18 14:05 ` Andrew Davis
2023-11-28 9:53 ` Tony Lindgren
0 siblings, 1 reply; 4+ messages in thread
From: Andrew Davis @ 2023-10-18 14:05 UTC (permalink / raw)
To: Tony Lindgren, Rob Herring
Cc: Benoît Cousson, Krzysztof Kozlowski, Conor Dooley,
linux-omap, devicetree, linux-kernel
On 10/18/23 12:20 AM, Tony Lindgren wrote:
> * Rob Herring <robh@kernel.org> [231011 00:19]:
>> DT overlays in tree need to be applied to a base DTB to validate they
>> apply, to run schema checks on them, and to catch any errors at compile
>> time.
>>
>> Signed-off-by: Rob Herring <robh@kernel.org>
>> ---
>> v2:
>> - Only test applying overlays when building all DTBs
>
> Andrew, any comments or care to ack?
As before, this is a valid combination, but it is only one of
many more. For now this is a reasonable start though,
Acked-by: Andrew Davis <afd@ti.com>
>
> Regards,
>
> Tony
>
>> ---
>> arch/arm/boot/dts/ti/omap/Makefile | 10 ++++++++++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/ti/omap/Makefile b/arch/arm/boot/dts/ti/omap/Makefile
>> index d2b590004fed..d6ee4a247100 100644
>> --- a/arch/arm/boot/dts/ti/omap/Makefile
>> +++ b/arch/arm/boot/dts/ti/omap/Makefile
>> @@ -129,6 +129,16 @@ dtb-$(CONFIG_SOC_AM43XX) += \
>> am57xx-evm-dtbs := am57xx-beagle-x15.dtb am57xx-evm.dtbo
>> am57xx-evm-reva3-dtbs := am57xx-beagle-x15-revc.dtb am57xx-evm.dtbo
>>
>> +am571x-idk-overlays-dtbs := am571x-idk.dtb \
>> + am571x-idk-touchscreen.dtbo am57xx-idk-lcd-osd101t2587.dtbo
>> +am572x-idk-overlays-dtbs := am572x-idk.dtb \
>> + am572x-idk-touchscreen.dtbo am57xx-idk-lcd-osd101t2045.dtbo
>> +
>> +# Build time test only, enabled by CONFIG_OF_ALL_DTBS
>> +dtb- += \
>> + am571x-idk-overlays.dtb \
>> + am572x-idk-overlays.dtb
>> +
>> dtb-$(CONFIG_SOC_DRA7XX) += \
>> am57xx-beagle-x15.dtb \
>> am57xx-beagle-x15-revb1.dtb \
>> --
>> 2.42.0
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] arm: dts: omap: Apply am57xx-idk overlays to base dtbs
2023-10-18 14:05 ` Andrew Davis
@ 2023-11-28 9:53 ` Tony Lindgren
0 siblings, 0 replies; 4+ messages in thread
From: Tony Lindgren @ 2023-11-28 9:53 UTC (permalink / raw)
To: Andrew Davis
Cc: Rob Herring, Benoît Cousson, Krzysztof Kozlowski,
Conor Dooley, linux-omap, devicetree, linux-kernel
* Andrew Davis <afd@ti.com> [231018 14:05]:
> On 10/18/23 12:20 AM, Tony Lindgren wrote:
> > * Rob Herring <robh@kernel.org> [231011 00:19]:
> > > DT overlays in tree need to be applied to a base DTB to validate they
> > > apply, to run schema checks on them, and to catch any errors at compile
> > > time.
> > >
> > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > ---
> > > v2:
> > > - Only test applying overlays when building all DTBs
> >
> > Andrew, any comments or care to ack?
>
> As before, this is a valid combination, but it is only one of
> many more. For now this is a reasonable start though,
>
> Acked-by: Andrew Davis <afd@ti.com>
OK thanks applying into omap-for-v6.8/dt.
Tony
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-11-28 9:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-10 21:19 [PATCH v2] arm: dts: omap: Apply am57xx-idk overlays to base dtbs Rob Herring
2023-10-18 5:20 ` Tony Lindgren
2023-10-18 14:05 ` Andrew Davis
2023-11-28 9:53 ` 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).