On Wed, 2014-12-10 at 19:32 +0000, Rustad, Mark D wrote: > On Dec 10, 2014, at 10:28 AM, Joe Perches <[email protected]> wrote: [] > > diff --git a/include/net/irda/parameters.h b/include/net/irda/parameters.h [] > > typedef struct { > > - pi_minor_info_t *pi_minor_call_table; > > + const pi_minor_info_t *pi_minor_call_table; > > Might you want to go a little further and make it: > const pi_minor_into_t * const pi_minor_call_table; > so that the pointer itself is also constant? That could apply to some others > below as well. > > > int len; > > } pi_major_info_t;
I don't think that's necessary as all the pi_major_info_t uses become const and this is a typedef, but if you want to, go ahead. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
