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

On Fri, Jul 13, 2012 at 6:41 AM, Rob Herring [off-list ref] wrote:

On 07/13/2012 07:32 AM, y@pengutronix.de wrote:

quoted

From: Uwe Kleine-König <redacted>

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.