Re: [PATCH v2] arm64: defconfig: Enable various configs for TI platforms

From: Nishanth Menon
Date: Wed Jul 26 2023 - 09:43:57 EST


on $subject: I think mentioning K3 will help narrow things down a bit.

On 19:00-20230726, Udit Kumar wrote:
> Enable TI ECAP, DP83869 driver, TI OMAP2, K3 remote proc

I think you mean to state TI mailbox and not omap2 :)

> SND_SOC_J721E_EVM, MCAN, UFS and RTI driver to be built
> as module.
> These are needed on different TI platforms.

Please elaborate the list of boards this benefits TI platforms are a
big variant list. TI Platforms is too generic a term that scales
architectures.. that is not the point you are trying to make.
>
> Signed-off-by: Aradhya Bhatia <a-bhatia1@xxxxxx>
> Signed-off-by: Hari Nagalla <hnagalla@xxxxxx>
> Signed-off-by: MD Danish Anwar <danishanwar@xxxxxx>
> Signed-off-by: Jayesh Choudhary <j-choudhary@xxxxxx>
> Signed-off-by: Vignesh Raghavendra <vigneshr@xxxxxx>
> Signed-off-by: Judith Mendez <jm@xxxxxx>
> Signed-off-by: Udit Kumar <u-kumar1@xxxxxx>

If the various contributers are ok, just my 2 cents:
Could you drop all other sign-offs other than yours? I understand the
contributions from various members on the TI SDK tree, but I'vent seen
them attempt to upstream and you took the effort in ensuring the new
squashed patch contains all the relevant components.

[..]
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D