lkml.org 
[lkml]   [2013]   [Sep]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH trivial] block/ioctl.c: let code match 'kernel code style'

It seems necessary to explain my goal (why I intended to send this patch).

1. support the 'work' of Joe and trivial@kernel.org.

in my opinion, 'trivial' means none-urgent, but it also means important.
(if both none-urgent and none-important, why need it?)

so I intend to send 1 or 2 trivial patches per month to express my opinion and support their 'work'.

2. I have planned to provide some contributes to block sub-system in 2nd half of 2013.

I have mentioned about it in my some original discussions (if necessary, I will provide the proof).
but at least now, I have done nothing for block sub system.
it is time to do something for it.

3. avoid the misunderstanding of the other related discussion in kernel mailing list.

the discussion is about whether need "#ifdef" for a patch.
maybe some members (e.g. Guenter) misunderstand me: "I do not care about the code style".
(it is only my 'guess', may he does not misunderstand, and this patch wants to make it clear).


BTW:

at least for me, I don't think "more patches is more contributions".
but it is true that I am trying to send more patches and get more discussions in kernel mailing list.
and I am (should be) also trying to avoid send waste mails to kernel mailing list.


Welcome any additional suggestions or completions.

Thanks.

On 09/03/2013 05:06 PM, Chen Gang wrote:
> On 09/03/2013 04:55 PM, Li Zefan wrote:
>> Please don't. Pure colding style cleanup is discouraged.
>>
>
> Pardon?
>
> Do you mean "coding style", not "colding style"? (or it is my
> misunderstanding?).
>
>
>> You're not going to run checkpatch.pl on the whole kernel tree and fix
>> all the complaints, are you?
>>
>
> I am not going to, it seems that may be the 'job' of Joe and
> trivial@kernel.org?
>
>
> BTW: it seems you did not redirect my mails to "/dev/null". ;-)
>
> Thanks.
>
>> On 2013/9/3 16:29, Chen Gang wrote:
>>> For 'switch case', remove redundancy '\t' (also can let related lines
>>> within 80 columns) and remove redundancy empty lines, just like other
>>> 'switch case' which match 'kernel code style' within the file.
>>>
>>> Let blkpg_ioctl() within 80 columns. Let 2nd line of blkdev_ioctl() and
>>> __blkdev_driver_ioctl() align 1st line parameter's start position, just
>>> like blk_ioctl_discard() and blk_ioctl_zeroout() within the file.
>>>
>>> For is_unrecognized_ioctl(), can shrink the 'return' statement into one
>>> line (so can save 2 lines), it still matches 'kernel code style' and it
>>> is no conflict with others within the file.
>>>
>>>
>>> Signed-off-by: Chen Gang <gang.chen@asianux.com>
>>> ---
>>> block/ioctl.c | 216 ++++++++++++++++++++++++++++-----------------------------
>>> 1 files changed, 105 insertions(+), 111 deletions(-)
>>
>>
>>
>
>


--
Chen Gang


\
 
 \ /
  Last update: 2013-09-03 12:21    [W:0.069 / U:0.884 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site