[PATCH 00/11] make struct of_device_id.data const

Grant Likely grant.likely at secretlab.ca
Sat Jul 14 05:36:59 EST 2012


On Fri, Jul 13, 2012 at 6:41 AM, Rob Herring <robherring2 at gmail.com> wrote:
> On 07/13/2012 07:32 AM, y at pengutronix.de wrote:
>> From: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
>>
>> Hello,
>>
>> this series' goal is to make struct of_device_id.data const.
>> Conceptually a driver must not modify the data contained there so making
>> it const is the right thing.
>>
>> This change might introduce warnings because drivers don't use const to
>> access the data pointed to by of_device_id.data. In most cases the fix
>> is to add some consts.
>>
>> This series (apart from adding this const in patch 10) fixes all drivers
>> covered by all arm defconfigs (based on v3.5-rc6).
>>
>
> What about other arches?

This whole series is of questionable value to me.  While I agree that
.data should be const, I don't think changing it has is measurably
better than what we have now.  Particularly when the series isn't
complete for all users.  And "building all defconfigs" is not the
right way to look for affected users.

g.


More information about the devicetree-discuss mailing list