Re: [RFC/PATCH] sh: Remove cast of atomic_t in ATOMIC_INIT macro

[Simon Horman <horms@verge.net.au>]

On Fri, Apr 27, 2012 at 09:35:50AM +0900, Paul Mundt wrote:
> On Wed, Apr 25, 2012 at 12:11:14PM +0900, Nobuhiro Iwamatsu wrote:
> > And this problem occurs by STATIC_KEY_INIT_TRUE and
> > STATIC_KEY_INIT_FALSE of include/linux/jump_label.h.
> > The architecture by which jumplabel   is defined following,
> > 
> > > #define STATIC_KEY_INIT_TRUE ((struct static_key) \
> > >     { .enabled = ATOMIC_INIT(1), .entries = (void *)1 })
> > > #define STATIC_KEY_INIT_FALSE ((struct static_key) \
> > >     { .enabled = ATOMIC_INIT(0), .entries = (void *)0 })
> > 
> > and does not define following.
> > 
> > > #define STATIC_KEY_INIT_TRUE ((struct static_key) \
> > >                { .enabled = ATOMIC_INIT(1) })
> > > #define STATIC_KEY_INIT_FALSE ((struct static_key) \
> > >                { .enabled = ATOMIC_INIT(0) })
> > 
> > Although a problem is solved by considering it following.
> > 
> > > #define STATIC_KEY_INIT_TRUE {ATOMIC_INIT(1)}
> > > #define STATIC_KEY_INIT_FALSE {ATOMIC_INIT(0)}
> > 
> > However, since definitions differ by the same macro name,
> > extendibility is lost.
> > I think that it is better to remove the cast from ATOMIC_INIT.
> > 
> Yes, agreed. I'll queue up the patch, thanks.

Excellent.