RE: [PATCH] staging: most: video: fixed a parentheses coding style issue.

2020-12-18 Thread David Laight
From: Joe Perches > Sent: 18 December 2020 10:09 > On Fri, 2020-12-18 at 09:49 +, David Laight wrote: > > From: Joe Perches > > checkpatch probably shouldn't complain about lines that end in ( > > if they are function definitions. > > Opinons vary. > > Very few function declaration/definition

Re: [PATCH] staging: most: video: fixed a parentheses coding style issue.

2020-12-18 Thread Joe Perches
On Fri, 2020-12-18 at 09:49 +, David Laight wrote: > From: Joe Perches > > Sent: 17 December 2020 23:58 > > > > On Thu, 2020-12-17 at 15:45 -0800, Daniel West wrote: > > > Fixed a coding style issue. > > > > It may pass checkpatch without warning, but it's uncommon kernel coding > > style. >

RE: [PATCH] staging: most: video: fixed a parentheses coding style issue.

2020-12-18 Thread David Laight
From: Joe Perches > Sent: 17 December 2020 23:58 > > On Thu, 2020-12-17 at 15:45 -0800, Daniel West wrote: > > Fixed a coding style issue. > > It may pass checkpatch without warning, but it's uncommon kernel coding style. checkpatch probably shouldn't complain about lines that end in ( if they a

Re: [PATCH] staging: most: video: fixed a parentheses coding style issue.

2020-12-17 Thread Joe Perches
On Thu, 2020-12-17 at 15:45 -0800, Daniel West wrote: > Fixed a coding style issue. It may pass checkpatch without warning, but it's uncommon kernel coding style. > diff --git a/drivers/staging/most/video/video.c > b/drivers/staging/most/video/video.c [] > @@ -365,8 +365,8 @@ static const struct

[PATCH] staging: most: video: fixed a parentheses coding style issue.

2020-12-17 Thread Daniel West
Fixed a coding style issue. Signed-off-by: Daniel West --- drivers/staging/most/video/video.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/most/video/video.c b/drivers/staging/most/video/video.c index 829df899b993..c58192ab0c2a 100644 --- a/drivers/sta