Thread (4 messages) 4 messages, 3 authors, 2012-11-29

Re: [PATCH v2] drivers/of: Constify device_node->name and ->path_component_name

From: Grant Likely <hidden>
Date: 2012-11-29 16:34:21
Also in: linux-next, lkml, sparclinux

On Wed, 28 Nov 2012 10:46:04 +1100, Michael Neuling [off-list ref] wrote:
quoted
Neither of these should ever be changed once set. Make them const and
fix up the users that try to modify it in-place. In one case
kmalloc+memcpy is replaced with kstrdup() to avoid modifying the string.

Build tested with defconfigs on ARM, PowerPC, Sparc, MIPS, x86 among
others.
Grant,

This breaks powerpc chroma_defconfig in next-20121127 with:

arch/powerpc/sysdev/scom.c:160:17: error: assignment discards 'const' qualifier from pointer target type [-Werror]

The following fixes it.  The change is to generic code, so I'm not sure
it's the right fix as it may break other configs/archs.
That is rather risky. Post it as a patch to core code, but in the mean
time I'll patch the powerpc scom code.

g.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help