[PATCH] Stagingdriver cctree: Fix for checkpatch warning

bincy bincy_k_philip at yahoo.co.in
Wed Jul 12 08:53:56 UTC 2017


Hi Gilad,

I have added your comments and resent the patch.Thanks

On 11/07/17 17:16, Gilad Ben-Yossef wrote:
> Hello Philip,
>
> Thank your patch.
>
> Your patch subject line is not descriptive and not formatted well.
>
> A better subject would be something like:
>
> staging: ccree: move comment to fit coding style
>
> Thanks,
> Gilad
>
> On Fri, Jun 30, 2017 at 7:32 AM,  <bincy_k_philip at yahoo.co.in> wrote:
>> From: Bincy K Philip <bincy_k_philip at yahoo.co.in>
>>
>> Trivial fix for Line over 80 characters
>>
>> Moved the comment to top of the definition
>>
>> Signed-off-by: Bincy K Philip <bincy_k_philip at yahoo.co.in>
>> ---
>>  drivers/staging/ccree/cc_hw_queue_defs.h | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/staging/ccree/cc_hw_queue_defs.h b/drivers/staging/ccree/cc_hw_queue_defs.h
>> index aaa56c8..a18e6c9 100644
>> --- a/drivers/staging/ccree/cc_hw_queue_defs.h
>> +++ b/drivers/staging/ccree/cc_hw_queue_defs.h
>> @@ -27,7 +27,8 @@
>>  ******************************************************************************/
>>
>>  #define HW_DESC_SIZE_WORDS             6
>> -#define HW_QUEUE_SLOTS_MAX              15 /* Max. available slots in HW queue */
>> +/* Define max. available slots in HW queue */
>> +#define HW_QUEUE_SLOTS_MAX              15
>>
>>  #define CC_REG_NAME(word, name) DX_DSCRPTR_QUEUE_WORD ## word ## _ ## name
>>
>> --
>> 1.8.3.1
>>
>
>
>


More information about the devel mailing list