RE: [PATCH -next 5/5] misc: microchip: pci1xxxx: use module_auxiliary_driver

[<Kumaravel.Thiagarajan@xxxxxxxxxxxxx>]

> -----Original Message-----
> From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
> Sent: Wednesday, September 7, 2022 8:17 PM
> To: Kumaravel Thiagarajan - I21417 <Kumaravel.Thiagarajan@xxxxxxxxxxxxx>
> Cc: weiyongjun@xxxxxxxxxxxxxxx; arnd@xxxxxxxx;
> weiyongjun1@xxxxxxxxxx; linux-gpio@xxxxxxxxxxxxxxx; kernel-
> janitors@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH -next 5/5] misc: microchip: pci1xxxx: use
> module_auxiliary_driver
> 
> On Wed, Sep 07, 2022 at 02:35:14PM +0000,
> Kumaravel.Thiagarajan@xxxxxxxxxxxxx wrote:
> > > -----Original Message-----
> > > From: Wei Yongjun <weiyongjun@xxxxxxxxxxxxxxx>
> > > Sent: Wednesday, September 7, 2022 2:05 PM
> > > To: Kumaravel Thiagarajan - I21417
> > > <Kumaravel.Thiagarajan@xxxxxxxxxxxxx>; Arnd Bergmann
> > > <arnd@xxxxxxxx>; Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > > Cc: Wei Yongjun <weiyongjun1@xxxxxxxxxx>;
> > > linux-gpio@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx
> > > Subject: [PATCH -next 5/5] misc: microchip: pci1xxxx: use
> > > module_auxiliary_driver
> > >
> > >
> > > Use the module_auxiliary_driver() macro to make the code simpler by
> > > eliminating module_init and module_exit calls.
> > I needed this during the experimentation stage. But now these
> > functions only do register and unregister. Hence, can be replaced with
> module_auxiliary_driver.
> > Thanks for your patch.
> > >
> > Add this tag -> Fixes: 7d3e4d807df2 ("misc: microchip: pci1xxxx: load gpio
> driver for the gpio controller auxiliary device enumerated by the auxiliary bus
> driver.")?
> 
> As you are the maintainer of this driver, you can add these markings to the
> patches and sign off on them when you forward them on to me.  No need to
> force the original developer to do this if you are already going to do it as well.
Sure. Understand this now. Thank you, Greg.

Regards,
Kumaravel