Re: [PATCH] iwlegacy: make il3945_mac_ops __ro_after_init

2016-12-07 Thread Stanislaw Gruszka
On Wed, Dec 07, 2016 at 07:36:46AM +0100, Johannes Berg wrote: > From: Johannes Berg > > There's no need for this to be only __read_mostly, since > it's only written in a single way depending on the module > parameter, so that can be moved into the module's __init > function, and the ops can be _

[PATCH] iwlegacy: make il3945_mac_ops __ro_after_init

2016-12-06 Thread Johannes Berg
From: Johannes Berg There's no need for this to be only __read_mostly, since it's only written in a single way depending on the module parameter, so that can be moved into the module's __init function, and the ops can be __ro_after_init. This is a little bit safer since it means the ops can't be