Re: [PATCH 2/4] ath9k: move hw code to its own module

[Greg KH <greg@kroah.com>]

On Sat, Sep 26, 2009 at 02:04:54PM +0200, Christian Lamparter wrote:
> On Thursday 24 September 2009 05:07:00 Luis R. Rodriguez wrote:
> > hw code for Atheros 802.11n hardware is commmon between
> > different chipsets. This moves this code into a separate
> > module, the next expected user of this code will be
> > the ath9k_htc module.
> > 
> > The ath9k/ dir is now selected by ATH9K_HW, an option which
> > gets selected by either ath9k or ath9k_htc, but remains
> > invisible for user menuconfig configuration. If either
> > ath9k or ath9k_htc will be compiled into the kernel
> > ath9k_hw will also be compiled in.
> >
> > diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
> > index 7a4de3d..7d167a1 100644
> > --- a/drivers/net/wireless/ath/ath9k/hw.c
> > +++ b/drivers/net/wireless/ath/ath9k/hw.c
> > @@ -33,6 +33,23 @@ static u32 ath9k_hw_ini_fixup(struct ath_hw *ah,
> >  static void ath9k_hw_9280_spur_mitigate(struct ath_hw *ah, struct ath9k_channel *chan);
> >  static void ath9k_hw_spur_mitigate(struct ath_hw *ah, struct ath9k_channel *chan);
> >  
> > +MODULE_AUTHOR("Atheros Communications");
> [...]
> > +
> > +static int __init ath9k_init(void)
> > +{
> > +	return 0;
> > +}
> > +module_init(ath9k_init);
> > +
> > +static void __exit ath9k_exit(void)
> > +{
> > +	return;
> > +}
> > +module_exit(ath9k_exit);
> > +
> >  /********************/
> >  /* Helper Functions */
> >  /********************/
> 
> FYI: there's no need for module_init/module_exit stubs.
> ( http://osdir.com/ml/linux-wireless/2009-04/msg00478.html )

But it's nicer if they are there, I'd recommend keeping them.

thanks,

greg k-h