diff mbox series

[meta-ti,scarthgap] conf: machine: j722s: Change overlay in KERNEL_DEVICETREE_PREFIX

Message ID 20250211055530.16704-1-a-dutta@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,scarthgap] conf: machine: j722s: Change overlay in KERNEL_DEVICETREE_PREFIX | expand

Commit Message

Dutta, Anurag Feb. 11, 2025, 5:55 a.m. UTC
Currently, the overlay mentioned in KERNEL_DEVICETREE_PREFIX in case
of j722s is "k3-j7200-evm-mcspi-loopback" which is incorrect. Change
the overlay to "k3-am625-sk-mcspi-loopback.dtso" in order to carry
out SPI loopback tests successfully.

Fixes: 13b691a9bc2d ("conf: machine: j722s: Add overlay to
KERNEL_DEVICETREE_PREFIX")

Signed-off-by: Anurag Dutta <a-dutta@ti.com>
---

Test link: https://gist.github.com/anuragdutta731/f93908b31d524ff0c52c3fd314449948

 meta-ti-bsp/conf/machine/j722s-evm.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Denys Dmytriyenko Feb. 11, 2025, 8:27 p.m. UTC | #1
Is it really scarthgap-specific?


On Tue, Feb 11, 2025 at 11:25:30AM +0530, Anurag Dutta via lists.yoctoproject.org wrote:
> Currently, the overlay mentioned in KERNEL_DEVICETREE_PREFIX in case
> of j722s is "k3-j7200-evm-mcspi-loopback" which is incorrect. Change
> the overlay to "k3-am625-sk-mcspi-loopback.dtso" in order to carry
> out SPI loopback tests successfully.
> 
> Fixes: 13b691a9bc2d ("conf: machine: j722s: Add overlay to
> KERNEL_DEVICETREE_PREFIX")
> 
> Signed-off-by: Anurag Dutta <a-dutta@ti.com>
> ---
> 
> Test link: https://gist.github.com/anuragdutta731/f93908b31d524ff0c52c3fd314449948
> 
>  meta-ti-bsp/conf/machine/j722s-evm.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-ti-bsp/conf/machine/j722s-evm.conf b/meta-ti-bsp/conf/machine/j722s-evm.conf
> index 8f42866c..52a6cfbd 100644
> --- a/meta-ti-bsp/conf/machine/j722s-evm.conf
> +++ b/meta-ti-bsp/conf/machine/j722s-evm.conf
> @@ -7,7 +7,7 @@ require conf/machine/include/j722s.inc
>  KERNEL_DEVICETREE_PREFIX = " \
>      ti/k3-j722s \
>      ti/k3-fpdlink \
> -    ti/k3-j7200-evm-mcspi-loopback \
> +    ti/k3-am625-sk-mcspi-loopback \
>      ti/k3-v3link \
>  "
>  
> -- 
> 2.34.1
Dutta, Anurag Feb. 17, 2025, 5:27 a.m. UTC | #2
On 12-02-2025 01:57, Denys Dmytriyenko wrote:
> Is it really scarthgap-specific?
Yes, this patch is scarthgap-specific because overlays are not upstreamed.
>
> On Tue, Feb 11, 2025 at 11:25:30AM +0530, Anurag Dutta via lists.yoctoproject.org wrote:
>> Currently, the overlay mentioned in KERNEL_DEVICETREE_PREFIX in case
>> of j722s is "k3-j7200-evm-mcspi-loopback" which is incorrect. Change
>> the overlay to "k3-am625-sk-mcspi-loopback.dtso" in order to carry
>> out SPI loopback tests successfully.
>>
>> Fixes: 13b691a9bc2d ("conf: machine: j722s: Add overlay to
>> KERNEL_DEVICETREE_PREFIX")
>>
>> Signed-off-by: Anurag Dutta <a-dutta@ti.com>
>> ---
>>
>> Test link: https://gist.github.com/anuragdutta731/f93908b31d524ff0c52c3fd314449948
>>
>>   meta-ti-bsp/conf/machine/j722s-evm.conf | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-ti-bsp/conf/machine/j722s-evm.conf b/meta-ti-bsp/conf/machine/j722s-evm.conf
>> index 8f42866c..52a6cfbd 100644
>> --- a/meta-ti-bsp/conf/machine/j722s-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/j722s-evm.conf
>> @@ -7,7 +7,7 @@ require conf/machine/include/j722s.inc
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       ti/k3-j722s \
>>       ti/k3-fpdlink \
>> -    ti/k3-j7200-evm-mcspi-loopback \
>> +    ti/k3-am625-sk-mcspi-loopback \
>>       ti/k3-v3link \
>>   "
>>   
>> -- 
>> 2.34.1
Ryan Eatmon Feb. 17, 2025, 3:20 p.m. UTC | #3
On 2/16/2025 11:27 PM, Dutta, Anurag wrote:
> 
> On 12-02-2025 01:57, Denys Dmytriyenko wrote:
>> Is it really scarthgap-specific?
> Yes, this patch is scarthgap-specific because overlays are not upstreamed.

scarthgap and master are just two branches for the yocto project.  This 
has nothing to do with the upstream kernel.  I already applied this 
patch to master as well.  But in the future, this kind of patch should 
be made for both the current LTS and the master branch.


>> On Tue, Feb 11, 2025 at 11:25:30AM +0530, Anurag Dutta via 
>> lists.yoctoproject.org wrote:
>>> Currently, the overlay mentioned in KERNEL_DEVICETREE_PREFIX in case
>>> of j722s is "k3-j7200-evm-mcspi-loopback" which is incorrect. Change
>>> the overlay to "k3-am625-sk-mcspi-loopback.dtso" in order to carry
>>> out SPI loopback tests successfully.
>>>
>>> Fixes: 13b691a9bc2d ("conf: machine: j722s: Add overlay to
>>> KERNEL_DEVICETREE_PREFIX")
>>>
>>> Signed-off-by: Anurag Dutta <a-dutta@ti.com>
>>> ---
>>>
>>> Test link: 
>>> https://gist.github.com/anuragdutta731/f93908b31d524ff0c52c3fd314449948
>>>
>>>   meta-ti-bsp/conf/machine/j722s-evm.conf | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/meta-ti-bsp/conf/machine/j722s-evm.conf 
>>> b/meta-ti-bsp/conf/machine/j722s-evm.conf
>>> index 8f42866c..52a6cfbd 100644
>>> --- a/meta-ti-bsp/conf/machine/j722s-evm.conf
>>> +++ b/meta-ti-bsp/conf/machine/j722s-evm.conf
>>> @@ -7,7 +7,7 @@ require conf/machine/include/j722s.inc
>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>       ti/k3-j722s \
>>>       ti/k3-fpdlink \
>>> -    ti/k3-j7200-evm-mcspi-loopback \
>>> +    ti/k3-am625-sk-mcspi-loopback \
>>>       ti/k3-v3link \
>>>   "
>>> -- 
>>> 2.34.1
diff mbox series

Patch

diff --git a/meta-ti-bsp/conf/machine/j722s-evm.conf b/meta-ti-bsp/conf/machine/j722s-evm.conf
index 8f42866c..52a6cfbd 100644
--- a/meta-ti-bsp/conf/machine/j722s-evm.conf
+++ b/meta-ti-bsp/conf/machine/j722s-evm.conf
@@ -7,7 +7,7 @@  require conf/machine/include/j722s.inc
 KERNEL_DEVICETREE_PREFIX = " \
     ti/k3-j722s \
     ti/k3-fpdlink \
-    ti/k3-j7200-evm-mcspi-loopback \
+    ti/k3-am625-sk-mcspi-loopback \
     ti/k3-v3link \
 "