Messages in this thread | | | Date | Sun, 22 Sep 2019 18:14:01 +0200 | From | Pavel Machek <> | Subject | Re: [PATCH V2 2/5] input: keyboard: imx_sc: Add i.MX system controller power key support |
| |
Hi!
> >>Why is it global struct? It seems to be flexible configurable over devicetree. > >>So I would assume it should be able to handle more then one button. Please > >>remove global variables, make it allocatable per OF node. > > > >There is ONLY one button available for SC key, but yes, I think I can make the structure > >private and get all necessary data from the structure using container_of. > > And we will never need more then 640 kB RAM ;) > https://en.wikiquote.org/wiki/Talk:Bill_Gates
Right question is "is it worth complicating kernel now (and making it slower/bigger) for future flexibility?". And I believe answer is "no", but it is not a big deal either way. Pavel
-- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
| |