Re: [PATCH 1/1] w1: ds2482: Use named initializers for arrays of i2c_device_data

["Uwe Kleine-König (The Capable Hub)" <u.kleine-koenig@baylibre.com>]

[-- Attachment #1: Type: text/plain, Size: 1437 bytes --]

Hello,

On Thu, Jun 04, 2026 at 06:50:28PM +0200, Krzysztof Kozlowski wrote:
> From: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>
> 
> While being less compact, using named initializers allows to more easily
> see which members of the structs are assigned which value without having
> to lookup the declaration of the struct. And it's also more robust
> against changes to the struct definition.
> 
> The mentioned robustness is relevant for a planned change to struct
> i2c_device_id that replaces .driver_data by an anonymous union.
> 
> This patch doesn't modify the compiled arrays, only their representation
> in source form benefits. The former was confirmed with x86 and arm64
> builds.
> 
> Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>
> Link: https://patch.msgid.link/20260518171456.872736-2-u.kleine-koenig@baylibre.com
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

I wonder about this patch because it was included in next as
10cc9de3d4fc between next-20260526 and next-20260604 (inclusive) but
isn't included any more in next-20260605.

I assume this mail is part of the submit path of w1 to mainline and
Krzysztof dropped it from his tree to not create a conflict in next when
Greg adds a slightly modified variant?

If so, the expectation that this patch makes it into 7.2-rc1 is still
justified, right?

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]