Re: [PATCH] ep93xx_eth.c: general cleanup

From: Lennert Buytenhek
Date: Wed Dec 16 2009 - 13:08:58 EST


On Wed, Dec 16, 2009 at 01:06:10PM -0500, H Hartley Sweeten wrote:

> >> General cleanup of the ep93xx_eth driver.
> >
> > Apart from keeping DRV_MODULE_NAME and DRV_MODULE_VERSION, I have
> > no strong opinion about this one way or the other. So I guess that
> > means ACK.
>
> I will add back the DRV_MODULE_NAME and DRV_MODULE_VERSION and repost.
>
> What about the message in ep93xx_eth_init_module()? Would you prefer
> that one to stay?

Most drivers I'm familiar with print a version message when they are
first instantiated -- perhaps that makes more sense.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/