Re: [PATCH v2 3/5] configs: am62px_evm_a53_defconfig: enable USB DFU support

[Roger Quadros <rogerq@kernel.org>]

Hi Siddharth,

On 19/12/2024 07:43, Siddharth Vadapalli wrote:
> On Wed, Dec 18, 2024 at 03:47:20PM +0200, Roger Quadros wrote:
> 
> Hello Roger,
> 
>> Hi Siddharth,
>>
>> On 18/12/2024 14:45, Siddharth Vadapalli wrote:
>>> The config fragment "am62x_a53_usbdfu.config" which adds USB DFU support
>>> for AM62x SoC is applicable to the AM62Px SoC as well. Hence, include it
>>
>> Apparently "am62x_a53_usbdfu.config" adds USB Host and Gadget and DFU support.
> 
> Do you propose that the Host configs should be moved into the respective
> defconfigs of SoCs which include am62x_a53_usbdfu.config, while dropping
> them from am62x_a53_usbdfu.config? Please let me know. This can be
> implemented by a separate series if that is what you are suggesting.

No. The commit that introduced am62x_a53_usbdfu.config mentions that it is
for USB Host and Gadget and DFU.

Your commit message seems to indicate that it is for DFU.
I leave it to you if you want to update the commit message or not.

> 
> Regards,
> Siddharth.

-- 
cheers,
-roger