Re: [PATCH 4.19 118/219] rfkill: allocate static minor

From: Greg Kroah-Hartman
Date: Fri Jan 10 2020 - 06:51:09 EST


On Fri, Jan 10, 2020 at 12:00:33PM +0100, Pavel Machek wrote:
> On Sun 2019-12-29 18:18:40, Greg Kroah-Hartman wrote:
> > From: Marcel Holtmann <marcel@xxxxxxxxxxxx>
> >
> > [ Upstream commit 8670b2b8b029a6650d133486be9d2ace146fd29a ]
> >
> > udev has a feature of creating /dev/<node> device-nodes if it finds
> > a devnode:<node> modalias. This allows for auto-loading of modules that
> > provide the node. This requires to use a statically allocated minor
> > number for misc character devices.
> >
> > However, rfkill uses dynamic minor numbers and prevents auto-loading
> > of the module. So allocate the next static misc minor number and use
> > it for rfkill.
>
> Is this good idea for stable?

Yes.

> I don't see this major/minor allocated in devices.txt in
> mainline. Should something like this be added?
>
> Signed-off-by: Pavel Machek <pavel@xxxxxxx>

Good idea, can you resend this as a "real" patch so that I can apply it?

thanks,

greg k-h