diff mbox series

[kirkstone] conf: machine: am62*: extend picking common overlays by prefix

Message ID 20231107235316.29646-1-praneeth@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [kirkstone] conf: machine: am62*: extend picking common overlays by prefix | expand

Commit Message

Bajjuri, Praneeth Nov. 7, 2023, 11:53 p.m. UTC
commit f5851386 ("conf: machine: am62*: Pick common overlays by prefix")
picks common HDMI and CSI overlays for am62 platform variants.

Adding capability to extend the same prefix for common overlays for
am62 family in more generic way to include new mcan overlays added [1]

[1]: https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/commit/?&id=3b01c392627f

given the differences in SK-AM62-LP containing  B2B HDMI Samtec connector,
just added the kernel prefix to pick common mcan overlay.

Signed-off-by: Praneeth Bajjuri <praneeth@ti.com>
---
 meta-ti-bsp/conf/machine/am62axx-evm.conf   | 3 +--
 meta-ti-bsp/conf/machine/am62pxx-evm.conf   | 3 +--
 meta-ti-bsp/conf/machine/am62xx-evm.conf    | 3 +--
 meta-ti-bsp/conf/machine/am62xx-lp-evm.conf | 1 +
 4 files changed, 4 insertions(+), 6 deletions(-)

Comments

Jai Luthra Nov. 8, 2023, 7:07 a.m. UTC | #1
Hi Praneeth,

Thanks for the patch.

On Nov 07, 2023 at 17:53:16 -0600, Praneeth Bajjuri wrote:
> commit f5851386 ("conf: machine: am62*: Pick common overlays by prefix")
> picks common HDMI and CSI overlays for am62 platform variants.
> 
> Adding capability to extend the same prefix for common overlays for
> am62 family in more generic way to include new mcan overlays added [1]
> 
> [1]: https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/commit/?&id=3b01c392627f
> 
> given the differences in SK-AM62-LP containing  B2B HDMI Samtec connector,
> just added the kernel prefix to pick common mcan overlay.
> 
> Signed-off-by: Praneeth Bajjuri <praneeth@ti.com>
> ---
>  meta-ti-bsp/conf/machine/am62axx-evm.conf   | 3 +--
>  meta-ti-bsp/conf/machine/am62pxx-evm.conf   | 3 +--
>  meta-ti-bsp/conf/machine/am62xx-evm.conf    | 3 +--
>  meta-ti-bsp/conf/machine/am62xx-lp-evm.conf | 1 +
>  4 files changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/meta-ti-bsp/conf/machine/am62axx-evm.conf b/meta-ti-bsp/conf/machine/am62axx-evm.conf
> index ef60e9d1..22f61401 100644
> --- a/meta-ti-bsp/conf/machine/am62axx-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62axx-evm.conf
> @@ -6,8 +6,7 @@ require conf/machine/include/am62axx.inc
>  
>  KERNEL_DEVICETREE_PREFIX = " \
>      ti/k3-am62a7 \
> -    ti/k3-am62x-sk-csi2 \
> -    ti/k3-am62x-sk-hdmi \
> +    ti/k3-am62x-sk \
>      ti/k3-fpdlink \
>  "
>  
> diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm.conf b/meta-ti-bsp/conf/machine/am62pxx-evm.conf
> index 931de953..5e0bab97 100644
> --- a/meta-ti-bsp/conf/machine/am62pxx-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62pxx-evm.conf
> @@ -6,8 +6,7 @@ require conf/machine/include/am62pxx.inc
>  
>  KERNEL_DEVICETREE_PREFIX = " \
>      ti/k3-am62p5 \
> -    ti/k3-am62x-sk-csi2 \
> -    ti/k3-am62x-sk-hdmi \
> +    ti/k3-am62x-sk \
>  "
>  
>  KERNEL_DEVICETREE = " \
> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf b/meta-ti-bsp/conf/machine/am62xx-evm.conf
> index b60ba156..fadbc6f4 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf

I would guess we can also do this for am62xxsip-evm.conf.
Or is MCAN not available on the SIP board for some reason?

With that handled -
Reviewed-by: Jai Luthra <j-luthra@ti.com>

> @@ -7,8 +7,7 @@ require conf/machine/include/ti-extras.inc
>  
>  KERNEL_DEVICETREE_PREFIX = " \
>      ti/k3-am625 \
> -    ti/k3-am62x-sk-csi2 \
> -    ti/k3-am62x-sk-hdmi \
> +    ti/k3-am62x-sk \
>  "
>  
>  KERNEL_DEVICETREE = " \
> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> index 0d2f89ca..afa5e061 100644
> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
> @@ -8,6 +8,7 @@ KERNEL_DEVICETREE_PREFIX = " \
>      ti/k3-am62-lp \
>      ti/k3-am625-sk \
>      ti/k3-am62x-sk-hdmi \
> +    ti/k3-am62x-sk-mcan \
>  "
>  
>  KERNEL_DEVICETREE = " \
> -- 
> 2.17.1
> 

Thanks,
Jai
Bajjuri, Praneeth Nov. 8, 2023, 3:54 p.m. UTC | #2
On 11/8/2023 1:07 AM, Jai Luthra wrote:
> Hi Praneeth,
> 
> Thanks for the patch.
> 
> On Nov 07, 2023 at 17:53:16 -0600, Praneeth Bajjuri wrote:
>> commit f5851386 ("conf: machine: am62*: Pick common overlays by prefix")
>> picks common HDMI and CSI overlays for am62 platform variants.
>>
>> Adding capability to extend the same prefix for common overlays for
>> am62 family in more generic way to include new mcan overlays added [1]
>>
>> [1]: https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/commit/?&id=3b01c392627f
>>
>> given the differences in SK-AM62-LP containing  B2B HDMI Samtec connector,
>> just added the kernel prefix to pick common mcan overlay.
>>
>> Signed-off-by: Praneeth Bajjuri <praneeth@ti.com>
>> ---
>>   meta-ti-bsp/conf/machine/am62axx-evm.conf   | 3 +--
>>   meta-ti-bsp/conf/machine/am62pxx-evm.conf   | 3 +--
>>   meta-ti-bsp/conf/machine/am62xx-evm.conf    | 3 +--
>>   meta-ti-bsp/conf/machine/am62xx-lp-evm.conf | 1 +
>>   4 files changed, 4 insertions(+), 6 deletions(-)
>>
>> diff --git a/meta-ti-bsp/conf/machine/am62axx-evm.conf b/meta-ti-bsp/conf/machine/am62axx-evm.conf
>> index ef60e9d1..22f61401 100644
>> --- a/meta-ti-bsp/conf/machine/am62axx-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62axx-evm.conf
>> @@ -6,8 +6,7 @@ require conf/machine/include/am62axx.inc
>>   
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am62a7 \
>> -    ti/k3-am62x-sk-csi2 \
>> -    ti/k3-am62x-sk-hdmi \
>> +    ti/k3-am62x-sk \
>>       ti/k3-fpdlink \
>>   "
>>   
>> diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm.conf b/meta-ti-bsp/conf/machine/am62pxx-evm.conf
>> index 931de953..5e0bab97 100644
>> --- a/meta-ti-bsp/conf/machine/am62pxx-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62pxx-evm.conf
>> @@ -6,8 +6,7 @@ require conf/machine/include/am62pxx.inc
>>   
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am62p5 \
>> -    ti/k3-am62x-sk-csi2 \
>> -    ti/k3-am62x-sk-hdmi \
>> +    ti/k3-am62x-sk \
>>   "
>>   
>>   KERNEL_DEVICETREE = " \
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf b/meta-ti-bsp/conf/machine/am62xx-evm.conf
>> index b60ba156..fadbc6f4 100644
>> --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
> 
> I would guess we can also do this for am62xxsip-evm.conf.
> Or is MCAN not available on the SIP board for some reason?

Thanks for catching it.
Will send a v2 with sip


> 
> With that handled -
> Reviewed-by: Jai Luthra <j-luthra@ti.com>
> 
>> @@ -7,8 +7,7 @@ require conf/machine/include/ti-extras.inc
>>   
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am625 \
>> -    ti/k3-am62x-sk-csi2 \
>> -    ti/k3-am62x-sk-hdmi \
>> +    ti/k3-am62x-sk \
>>   "
>>   
>>   KERNEL_DEVICETREE = " \
>> diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>> index 0d2f89ca..afa5e061 100644
>> --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
>> @@ -8,6 +8,7 @@ KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-am62-lp \
>>       ti/k3-am625-sk \
>>       ti/k3-am62x-sk-hdmi \
>> +    ti/k3-am62x-sk-mcan \
>>   "
>>   
>>   KERNEL_DEVICETREE = " \
>> -- 
>> 2.17.1
>>
> 
> Thanks,
> Jai
diff mbox series

Patch

diff --git a/meta-ti-bsp/conf/machine/am62axx-evm.conf b/meta-ti-bsp/conf/machine/am62axx-evm.conf
index ef60e9d1..22f61401 100644
--- a/meta-ti-bsp/conf/machine/am62axx-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62axx-evm.conf
@@ -6,8 +6,7 @@  require conf/machine/include/am62axx.inc
 
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am62a7 \
-    ti/k3-am62x-sk-csi2 \
-    ti/k3-am62x-sk-hdmi \
+    ti/k3-am62x-sk \
     ti/k3-fpdlink \
 "
 
diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm.conf b/meta-ti-bsp/conf/machine/am62pxx-evm.conf
index 931de953..5e0bab97 100644
--- a/meta-ti-bsp/conf/machine/am62pxx-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62pxx-evm.conf
@@ -6,8 +6,7 @@  require conf/machine/include/am62pxx.inc
 
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am62p5 \
-    ti/k3-am62x-sk-csi2 \
-    ti/k3-am62x-sk-hdmi \
+    ti/k3-am62x-sk \
 "
 
 KERNEL_DEVICETREE = " \
diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf b/meta-ti-bsp/conf/machine/am62xx-evm.conf
index b60ba156..fadbc6f4 100644
--- a/meta-ti-bsp/conf/machine/am62xx-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf
@@ -7,8 +7,7 @@  require conf/machine/include/ti-extras.inc
 
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am625 \
-    ti/k3-am62x-sk-csi2 \
-    ti/k3-am62x-sk-hdmi \
+    ti/k3-am62x-sk \
 "
 
 KERNEL_DEVICETREE = " \
diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
index 0d2f89ca..afa5e061 100644
--- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf
@@ -8,6 +8,7 @@  KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-am62-lp \
     ti/k3-am625-sk \
     ti/k3-am62x-sk-hdmi \
+    ti/k3-am62x-sk-mcan \
 "
 
 KERNEL_DEVICETREE = " \