Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-21 Thread Johan Hovold
On Thu, May 20, 2021 at 08:21:39AM -0700, Joe Perches wrote: > On Thu, 2021-05-20 at 10:21 +0200, Johan Hovold wrote: > > On Tue, May 18, 2021 at 12:01:22PM +0800, Xiaofei Tan wrote: > > > On 2021/5/17 22:15, Johan Hovold wrote: > > > > How is this an improvement? First, the multi-line comment

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-20 Thread Xiaofei Tan
Hi Johan, On 2021/5/20 21:52, Johan Hovold wrote: On Thu, May 20, 2021 at 09:21:25PM +0800, Xiaofei Tan wrote: Checkpatch already has too many checks IMO and I'm a bit surprised that it doesn't check this already. Perhaps it's because you used the -f to run checkpatch on in-kernel code,

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-20 Thread Joe Perches
On Thu, 2021-05-20 at 10:21 +0200, Johan Hovold wrote: > On Tue, May 18, 2021 at 12:01:22PM +0800, Xiaofei Tan wrote: > > On 2021/5/17 22:15, Johan Hovold wrote: > > > On Mon, May 17, 2021 at 02:37:10PM +0800, Xiaofei Tan wrote: > > > > Fix coding style issues of block comments, reported by

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-20 Thread Johan Hovold
On Thu, May 20, 2021 at 09:21:25PM +0800, Xiaofei Tan wrote: > > Checkpatch already has too many checks IMO and I'm a bit surprised that > > it doesn't check this already. Perhaps it's because you used the -f to > > run checkpatch on in-kernel code, which you should not. > > > >>> Second, that

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-20 Thread Xiaofei Tan
Hi Johan, On 2021/5/20 16:21, Johan Hovold wrote: On Tue, May 18, 2021 at 12:01:22PM +0800, Xiaofei Tan wrote: Hi Johan, Thanks for reviewing this patch. On 2021/5/17 22:15, Johan Hovold wrote: On Mon, May 17, 2021 at 02:37:10PM +0800, Xiaofei Tan wrote: Fix coding style issues of block

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-20 Thread Johan Hovold
On Tue, May 18, 2021 at 12:01:22PM +0800, Xiaofei Tan wrote: > Hi Johan, > > Thanks for reviewing this patch. > > On 2021/5/17 22:15, Johan Hovold wrote: > > On Mon, May 17, 2021 at 02:37:10PM +0800, Xiaofei Tan wrote: > >> Fix coding style issues of block comments, reported by checkpatch.pl. >

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-17 Thread Xiaofei Tan
Hi Johan, Thanks for reviewing this patch. On 2021/5/17 22:15, Johan Hovold wrote: On Mon, May 17, 2021 at 02:37:10PM +0800, Xiaofei Tan wrote: Fix coding style issues of block comments, reported by checkpatch.pl. Besides, add a period at the end of the sentenses. Signed-off-by: Xiaofei Tan

Re: [PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-17 Thread Johan Hovold
On Mon, May 17, 2021 at 02:37:10PM +0800, Xiaofei Tan wrote: > Fix coding style issues of block comments, reported by checkpatch.pl. > Besides, add a period at the end of the sentenses. > > Signed-off-by: Xiaofei Tan > --- > drivers/tty/hvc/hvc_console.c | 15 ++- > 1 file changed,

[PATCH 6/9] tty: hvc_console: Fix coding style issues of block comments

2021-05-17 Thread Xiaofei Tan
Fix coding style issues of block comments, reported by checkpatch.pl. Besides, add a period at the end of the sentenses. Signed-off-by: Xiaofei Tan --- drivers/tty/hvc/hvc_console.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git