Re: [PATCH 01/21] Staging: bcm: Remove typedef from "eInterface_setting" in Adapter.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 21, 2012 at 10:12:47AM -0400, Kevin McKinney wrote:
> On Mon, May 21, 2012 at 8:54 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> > On Mon, May 21, 2012 at 08:36:38AM -0400, Kevin McKinney wrote:
> >> On Mon, May 21, 2012 at 3:15 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> >> > On Sun, May 20, 2012 at 04:09:51PM -0400, Kevin McKinney wrote:
> >> >> This patch removes the typedef from
> >> >> eInterface_setting, remames the enum
> >> >> to INTERFACE_SETTING.
> >> >>
> >> >
> >> > On these ones I would have prefered if we could have gone straight
> >> > to kernel style naming like "enum interface_setting".  We don't like
> >> > ALL_CAPS struct names and we don't like CamelCase names.
> >> >
> >> > How much work is it to redo these?
> >> >
> >> Dan, it is not much work to redo these. I will change the names to our
> >> kernel style naming convention.
> >>
> >
> > That would be so great.  Some of them might be sort of very
> > commonish type names so if you want to put a bcm_ infront like
> > enum bcm_interface_setting that would be ok.
> >
> > Use your own judgement on that.
> 
> Okay, In this case I will prefix all of them with "bcm_" to be very clear.
> 

Whatever you feel is good is Ok.  ;)

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux