linux-kernel - RE: Re: [PATCH] extcon: Drop explicit initialization of struct i2c_device_id::driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <556725552.2924666.1733371094684@v8mail-kr1-0.v8mail-kr1.knoxportal-kr-prod-green.svc.cluster.local>
Date: Thu, 05 Dec 2024 12:58:14 +0900
From: MyungJoo Ham <myungjoo.ham@...sung.com>
To: Uwe Kleine-König <u.kleine-koenig@...libre.com>,
	Chanwoo Choi <cw00.choi@...sung.com>
CC: Krzysztof Kozlowski <krzk@...nel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: Re: [PATCH] extcon: Drop explicit initialization of struct
 i2c_device_id::driver_data to 0

>Hello,
>
>On Wed, Sep 18, 2024 at 02:31:48PM +0200, Uwe Kleine-König wrote:
>> These drivers don't use the driver_data member of struct i2c_device_id,
>> so don't explicitly initialize this member.
>> 
>> This prepares putting driver_data in an anonymous union which requires
>> either no initialization or named designators. But it's also a nice
>> cleanup on its own.
>> 
>> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@...libre.com>
>
>That patch got some positive feedback by Krzysztof but wasn't applied.
>Is this still on someone's radar for application?
>
>Best regards
>Uwe

Can you provide a link to a commit that requires this change?

For now, I don't see any benefit from this change.

Cheers,
MyungJoo


Download attachment "signature.asc" of type "application/octet-stream" (499 bytes)