Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > From: Johannes Berg <johannes.berg@xxxxxxxxx> > > 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 overwritten (accidentally or otherwise), which would > otherwise cause an arbitrary function or bad pointer to > be called. > > Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx> > Acked-by: Stanislaw Gruszka <sgruszka@xxxxxxxxxx> Patch applied to wireless-drivers-next.git, thanks. ae3cf4764506 iwlegacy: make il3945_mac_ops __ro_after_init -- https://patchwork.kernel.org/patch/9463999/ Documentation about submitting wireless patches and checking status from patchwork: https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
