diff mbox series

[meta-ti,kirkstone,v2] conf: machine: am57xx: export ov10635 camera device tree overlay

Message ID 20240312171132.1330699-1-u-kumar1@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,kirkstone,v2] conf: machine: am57xx: export ov10635 camera device tree overlay | expand

Commit Message

Kumar, Udit March 12, 2024, 5:11 p.m. UTC
Signed-off-by: Udit Kumar <u-kumar1@ti.com>
---
Changelog:
Changes in v2:
1) Updated subject of patch
2) moved overlay to KERNEL_DEVICETREE_PREFIX
link to v1:
https://lore.kernel.org/all/20240311111359.1556157-1-u-kumar1@ti.com/

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

Comments

Denys Dmytriyenko March 12, 2024, 9:50 p.m. UTC | #1
On Tue, Mar 12, 2024 at 10:41:32PM +0530, Udit Kumar via lists.yoctoproject.org wrote:
> 
> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> ---
> Changelog:
> Changes in v2:
> 1) Updated subject of patch
> 2) moved overlay to KERNEL_DEVICETREE_PREFIX
> link to v1:
> https://lore.kernel.org/all/20240311111359.1556157-1-u-kumar1@ti.com/
> 
>  meta-ti-bsp/conf/machine/am57xx-evm.conf | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-ti-bsp/conf/machine/am57xx-evm.conf b/meta-ti-bsp/conf/machine/am57xx-evm.conf
> index 43a6a833..8bcf20ea 100644
> --- a/meta-ti-bsp/conf/machine/am57xx-evm.conf
> +++ b/meta-ti-bsp/conf/machine/am57xx-evm.conf
> @@ -10,6 +10,7 @@ SERIAL_CONSOLES = "115200;ttyS2"
>  
>  KERNEL_DEVICETREE_PREFIX = " \
>      am57 \


> +    ov10635 \
>      ti/ov10635 \

What is the difference between these 2 entries? Which one is correct?
Kumar, Udit March 13, 2024, 4:43 a.m. UTC | #2
On 3/13/2024 3:20 AM, Denys Dmytriyenko wrote:
> On Tue, Mar 12, 2024 at 10:41:32PM +0530, Udit Kumar via lists.yoctoproject.org wrote:
>> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
>> ---
>> Changelog:
>> Changes in v2:
>> 1) Updated subject of patch
>> 2) moved overlay to KERNEL_DEVICETREE_PREFIX
>> link to v1:
>> https://lore.kernel.org/all/20240311111359.1556157-1-u-kumar1@ti.com/
>>
>>   meta-ti-bsp/conf/machine/am57xx-evm.conf | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/meta-ti-bsp/conf/machine/am57xx-evm.conf b/meta-ti-bsp/conf/machine/am57xx-evm.conf
>> index 43a6a833..8bcf20ea 100644
>> --- a/meta-ti-bsp/conf/machine/am57xx-evm.conf
>> +++ b/meta-ti-bsp/conf/machine/am57xx-evm.conf
>> @@ -10,6 +10,7 @@ SERIAL_CONSOLES = "115200;ttyS2"
>>   
>>   KERNEL_DEVICETREE_PREFIX = " \
>>       am57 \
>
>> +    ov10635 \
>>       ti/ov10635 \
> What is the difference between these 2 entries? Which one is correct?


Kernel 5.10 is hosting inside ti folder (arch/arm/boot/dts/ti/ov10635.dts)

whereas in 6.1 moved to base folder. (./arch/arm/boot/dts/ov10635.dtso)



>
Denys Dmytriyenko March 14, 2024, 6:30 p.m. UTC | #3
On Wed, Mar 13, 2024 at 10:13:47AM +0530, Kumar, Udit wrote:
> 
> On 3/13/2024 3:20 AM, Denys Dmytriyenko wrote:
> >On Tue, Mar 12, 2024 at 10:41:32PM +0530, Udit Kumar via lists.yoctoproject.org wrote:
> >>Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> >>---
> >>Changelog:
> >>Changes in v2:
> >>1) Updated subject of patch
> >>2) moved overlay to KERNEL_DEVICETREE_PREFIX
> >>link to v1:
> >>https://lore.kernel.org/all/20240311111359.1556157-1-u-kumar1@ti.com/
> >>
> >>  meta-ti-bsp/conf/machine/am57xx-evm.conf | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >>diff --git a/meta-ti-bsp/conf/machine/am57xx-evm.conf b/meta-ti-bsp/conf/machine/am57xx-evm.conf
> >>index 43a6a833..8bcf20ea 100644
> >>--- a/meta-ti-bsp/conf/machine/am57xx-evm.conf
> >>+++ b/meta-ti-bsp/conf/machine/am57xx-evm.conf
> >>@@ -10,6 +10,7 @@ SERIAL_CONSOLES = "115200;ttyS2"
> >>  KERNEL_DEVICETREE_PREFIX = " \
> >>      am57 \
> >
> >>+    ov10635 \
> >>      ti/ov10635 \
> >What is the difference between these 2 entries? Which one is correct?
> 
> 
> Kernel 5.10 is hosting inside ti folder (arch/arm/boot/dts/ti/ov10635.dts)
> 
> whereas in 6.1 moved to base folder. (./arch/arm/boot/dts/ov10635.dtso)

There's no 5.10 kernel in meta-ti any more.
Ryan Eatmon March 14, 2024, 6:51 p.m. UTC | #4
On 3/14/24 13:30, Denys Dmytriyenko wrote:
> On Wed, Mar 13, 2024 at 10:13:47AM +0530, Kumar, Udit wrote:
>> On 3/13/2024 3:20 AM, Denys Dmytriyenko wrote:
>>> On Tue, Mar 12, 2024 at 10:41:32PM +0530, Udit Kumar via lists.yoctoproject.org wrote:
>>>> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
>>>> ---
>>>> Changelog:
>>>> Changes in v2:
>>>> 1) Updated subject of patch
>>>> 2) moved overlay to KERNEL_DEVICETREE_PREFIX
>>>> link to v1:
>>>> https://lore.kernel.org/all/20240311111359.1556157-1-u-kumar1@ti.com/
>>>>
>>>>   meta-ti-bsp/conf/machine/am57xx-evm.conf | 1 +
>>>>   1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/meta-ti-bsp/conf/machine/am57xx-evm.conf b/meta-ti-bsp/conf/machine/am57xx-evm.conf
>>>> index 43a6a833..8bcf20ea 100644
>>>> --- a/meta-ti-bsp/conf/machine/am57xx-evm.conf
>>>> +++ b/meta-ti-bsp/conf/machine/am57xx-evm.conf
>>>> @@ -10,6 +10,7 @@ SERIAL_CONSOLES = "115200;ttyS2"
>>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>>       am57 \
>>>> +    ov10635 \
>>>>       ti/ov10635 \
>>> What is the difference between these 2 entries? Which one is correct?
>>
>> Kernel 5.10 is hosting inside ti folder (arch/arm/boot/dts/ti/ov10635.dts)
>>
>> whereas in 6.1 moved to base folder. (./arch/arm/boot/dts/ov10635.dtso)
> There's no 5.10 kernel in meta-ti any more.
>
That is very true.  I need to go through all of the machines and see if 
we can clean up anything as we move through the kernels. I'll take a 
note to do that next week.
diff mbox series

Patch

diff --git a/meta-ti-bsp/conf/machine/am57xx-evm.conf b/meta-ti-bsp/conf/machine/am57xx-evm.conf
index 43a6a833..8bcf20ea 100644
--- a/meta-ti-bsp/conf/machine/am57xx-evm.conf
+++ b/meta-ti-bsp/conf/machine/am57xx-evm.conf
@@ -10,6 +10,7 @@  SERIAL_CONSOLES = "115200;ttyS2"
 
 KERNEL_DEVICETREE_PREFIX = " \
     am57 \
+    ov10635 \
     ti/am57 \
     ti/lcd \
     ti/ov10635 \