Re: [PATCH 5/8] Staging: bcm: Remove typedef for _stServiceFlowParamSI and call directly.

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

 



On Sat, Sep 22, 2012 at 7:09 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> On Sat, Sep 22, 2012 at 01:24:36AM -0400, Kevin McKinney wrote:
>> This patch removes typedef for _stServiceFlowParamSI,
>> and changes the name of the struct to
>> bcm_service_flow. In addition, any
>> calls to typedefs "stServiceFlowParamSI,
>> *pstServiceFlowParamSI, CServiceFlowParamSI"
>> are changed to call the struct directly.
>>
>
> This name isn't right.  These varaibles are all Service Flow
> variables, it's just that the other ones used SF instead of typing
> it all out.  (I have no idea what Service Flow means).
>
> I also don't know what SI means.
>
> I don't know what the right name is here...  I think it should
> involve "param" or something similar but I'm going to let you figure
> it out.  :P  Can you redo this one?
>
> Also the comments need to be updated.

Okay, I will change the name and resubmit.

>
> 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