Re: [PATCH 12/12] arm64: defconfig: Enable I3C and SPD5118 hwmon

On Wed, Mar 25, 2026 at 11:59:36AM +0100, Krzysztof Kozlowski wrote:
> On 25/03/2026 11:31, Thierry Reding wrote:
> > On Thu, Mar 19, 2026 at 06:15:14PM +0100, Krzysztof Kozlowski wrote:
> >> On 19/03/2026 18:09, Akhil R wrote:
> >>> On Thu, 19 Mar 2026 10:40:34 +0100, Krzysztof Kozlowski wrote:
> >>>> On Wed, Mar 18, 2026 at 10:57:25PM +0530, Akhil R wrote:
> >>>>> Add I3C subsystem support, DesignWare I3C master controller, and
> >>>>> SPD5118 hwmon sensor as modules to the defconfig.
> >>>>
> >>>> Why? If there is no user of that, why would we want it? Your commit msg
> >>>> should explain that.
> >>>
> >>> Ack. This is for Tegra410 which has a DesignWare I3C host controller.
> >>> I will add this in the commit message.
> >>
> >> Board or products. Not SoCs.
> > 
> > Is this a new requirement? I see a bit of both in defconfig changes.
> 
> Almost every review from me has it for 2-3 years... And it is a known
> thing since always in a bit different wording: we do not care about
> downstream things and downstream products. defconfig does not serve
> downstream at all, makes no sense outside of our (upstream) work.

I don't understand why you're turning this into a downstream vs.
upstream discussion. This is all code that is being submitted upstream,
because we want these new platforms with I3C support enabled upstream.
It's as simple as that.

> > Some mention specific products, other mention SoCs. Does this
> > requirement apply to DT platforms or also ACPI platforms?
> 
> Just like kernel, applies to all platforms, regardless of firmware
> interface.

Hm... again, I don't think there's every been a rule to the effect of
needing to specify a particular platform or product when adding a new
defconfig change. There's plenty of things that we're enabling in the
defconfigs because we think they are generally useful.

But alright, we'll either add more details to the commit message, or
drop this patch entirely.

Thierry

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmnD1+4ACgkQ3SOs138+
s6FDCg//TEr1ZBNYM0xe8kwbmPyZIx3Vx3e28JVg+qyEspS58LsOeq9esxtRuXxS
nbLyK6ZR2UMQSz8flsE3eTLe6uX2nS4M2ulMPPhiGmS68FWBMX87sjcgm838wRCY
81kws1U2fMSmqX/4cPJ1GX+t0v5h33bMZR6NZnfQnuZA4XAUMpX8tc/yR1N7WkUM
xx8snURBoY1HEyabSqye0F1kwIwBP4uD0Lp+BLb3WTcLnRlTShkmUiYN5XoIaZAr
qqgERKNeG48lK8XB07DSNEi/vZF0SxMUvlv5xKZlK+G0m+Wdd8QaNrMqfZPQfvzk
mT/ixvLEwmWhI6TjzbNeAh84MZoCNm6M1/khnUfiwECVZbiRubyO1f2MDwCC/Vej
D8mZm0IKzSS1Nni8cccKarGjtf7AtziB6cTwmihlqpBvYCDMMtF/Pkz8JVX4rCSR
F3F/dwnFWLHfzn9XIz0YFPQyeXYOjI8GpwdcCAYUv86M1c+b8hp6W2CDwdVhZrX6
wcSAxuep2zqjuQ9qDMD+l2DHSuC4raYwmHR4SwkumNAUGumrOP5tGHpwPvcfzdBv
W1vEzFjECkS6NxWfDxLtF3NbjKPbQ8WZHoKMmC8vQW8p5fd0WKHOfJjYSx9HSaET
3Tk9L+O4ChP4yfX+vwPicJqk8Vo9yZUmeGXIN9hKUBBfYHR+tLQ=
=TaJr
-----END PGP SIGNATURE-----