On Thu, Nov 07, 2019 at 11:45:53AM +0000, Mark Brown wrote: > On Thu, Nov 07, 2019 at 09:00:25AM +0100, Linus Walleij wrote: > > > I wonder if it's worth to look at the static checkers like checkpatch > > to warn for this? > > I noticed this due to the warning emitted in the build process so I'm > not sure there'd be much more chance that people would notice (though > admittedly the warning during the build doesn't suggest a boot problem). > There's hundreds of other examples in the tree that don't trigger > crashes for whatever reason, I think this one was just noticable because > it got built in. Is it worth hving a attribute for 'can't be initdata' to add to sparse to catch things like this. Not sure if checkpatch would be able to do the sort of full level of catch for this. -- Ben Dooks, ben@xxxxxxxxx, http://www.fluff.org/ben/ Large Hadron Colada: A large Pina Colada that makes the universe disappear.
