Re: [meta-ti][master][PATCH] conf: am62dxx-evm: Fixup packaged overlay list

On 6/9/26 5:46 PM, Denys Dmytriyenko wrote:
> On Tue, Jun 09, 2026 at 04:07:51PM -0500, Judith Mendez wrote:
>> Hi Denys,
>>
>> On 6/9/26 3:53 PM, Denys Dmytriyenko wrote:
>>> On Tue, Jun 09, 2026 at 03:45:05PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>>
>>>> I assume this is for wrynose as well?  If so, please make sure to
>>>> add it on the subject going forward.
>>>>
>>>>
>>>> On 6/9/2026 3:28 PM, Judith Mendez wrote:
>>>>> Some k3-am62x-sk* overlays no longer apply to AM62A/AM62D boards,
>>>>> update overlay list packaged for AM62D mostly to include the new
>>>>> k3-am62a7-sk-mcan.dtbo without dropping applicable overlays.
>>>>>
>>>>> Signed-off-by: Judith Mendez <[email protected]>
>>>>> ---
>>>>>   meta-ti-bsp/conf/machine/am62dxx-evm.conf | 9 ++++++++-
>>>>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/meta-ti-bsp/conf/machine/am62dxx-evm.conf b/meta-ti-bsp/conf/machine/am62dxx-evm.conf
>>>>> index ef9df2f7..c776ffbf 100644
>>>>> --- a/meta-ti-bsp/conf/machine/am62dxx-evm.conf
>>>>> +++ b/meta-ti-bsp/conf/machine/am62dxx-evm.conf
>>>>> @@ -7,7 +7,14 @@ require conf/machine/include/am62dxx.inc
>>>>>   KERNEL_DEVICETREE_PREFIX = " \
>>>>>       ti/k3-am62d2 \
>>>>>       ti/k3-am62a7-sk-edgeai \
>>>>> -    ti/k3-am62x-sk \
>>>>> +    ti/k3-am62x-sk-csi2 \
>>>>> +    ti/k3-am62x-sk-ecap-capture \
>>>>> +    ti/k3-am62x-sk-eqep \
>>>>> +    ti/k3-am62x-sk-hdmi \
>>>>> +    ti/k3-am62x-sk-lpm \
>>>>> +    ti/k3-am62x-sk-mcspi-loopback \
>>>>> +    ti/k3-am62x-sk-pwm \
>>>
>>> Why are these needed? Shouldn't the first removed more generic prefix cover
>>> them all?
>>
>> Its explained in the commit message, some of the am62x-sk* overlays
>> no longer apply to am62a or am62d. I did not go test each overlay,
>> but for now, I removed the overlays that no longer apply to am62a/
>> am62d with this change
> 
> Yeah, I saw the commit message, but I understood it backwards - it sounded
> like the prefix functionality no longer worked with a generic prefix and you
> needed to list the overlays more explicitly. While in reality the generic
> prefix now is too broad and it picks up unneeded overlays for this platform.
> 
> Please disregard my previous question, but if you are going to re-send the
> patch, you might want to rephrase the commit message a bit more clearer...
> 

Sorry denys, recieved your comment before sending out v2, if needed, i
will send v3.

~ Judith