Re: [PATCH] ks7010: removed checkpatch.pl format errors

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 01:28:45PM +0530, shivang upadhyay wrote: > Signed-off-by: shivang upadhyay > --- > drivers/staging/ks7010/ks_hostif.h | 24 > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/staging/ks7010/ks_hostif.h > b/drivers/stagi

Re: [PATCH] ks7010: removed checkpatch.pl format errors

2021-02-12 Thread Frans Klaver
Hi, On Fri, Feb 12, 2021 at 8:59 AM shivang upadhyay wrote: > > Signed-off-by: shivang upadhyay There's no description here. Even though this seems like a trivial patch, it should still be there. Your commit message would look something like ks7010: enclose non-trivial defines in parenthes

[driver-core:driver-core-testing] BUILD SUCCESS 471b12c43f376d5203dbff0e91316eea11f6f4df

2021-02-12 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git driver-core-testing branch HEAD: 471b12c43f376d5203dbff0e91316eea11f6f4df driver core: auxiliary bus: Fix calling stage for auxiliary bus init elapsed time: 728m configs tested: 161 configs skipped: 2 The fol

Re: [PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Johan Hovold
On Fri, Feb 12, 2021 at 01:48:35PM +0530, Pritthijit Nath wrote: > Fixed the spelling of 'transfered' to 'transferred'. > > Signed-off-by: Pritthijit Nath > --- > drivers/staging/greybus/hid.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/greybus/hid.c

Re: [RESEND PATCH v5 5/6] staging: mt7621-dts: use valid vendor 'mediatek' instead of invalid 'mtk'

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 08:43:29AM +0100, Sergio Paracuellos wrote: > Vendor listed for mediatek in kernel vendor file 'vendor-prefixes.yaml' > contains 'mediatek' as a valid vendor string. Some nodes in the device > tree are using an invalid vendor string vfor 'mtk' instead. Fix all of > them in d

Re: [RESEND PATCH v5 4/6] staging: mt7621-dts: make use of new 'mt7621-clk'

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 08:43:28AM +0100, Sergio Paracuellos wrote: > Clocks for SoC mt7621 have been properly integrated so there is > no need to declare fixed clocks at all in the device tree. Remove > all of them, add new device tree nodes for mt7621-clk and update > the rest of the nodes to use

Re: [PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 09:44:04AM +0100, Johan Hovold wrote: > On Fri, Feb 12, 2021 at 01:48:35PM +0530, Pritthijit Nath wrote: > > Fixed the spelling of 'transfered' to 'transferred'. > > > > Signed-off-by: Pritthijit Nath > > --- > > drivers/staging/greybus/hid.c | 2 +- > > 1 file changed, 1

Re: [PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Viresh Kumar
On 12-02-21, 09:54, Greg KH wrote: > On Fri, Feb 12, 2021 at 09:44:04AM +0100, Johan Hovold wrote: > > On Fri, Feb 12, 2021 at 01:48:35PM +0530, Pritthijit Nath wrote: > > > Fixed the spelling of 'transfered' to 'transferred'. > > > > > > Signed-off-by: Pritthijit Nath > > > --- > > > drivers/st

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Viresh Kumar
On 12-02-21, 13:48, Pritthijit Nath wrote: > This change fixes a checkpatch CHECK style issue for "Alignment should match > open parenthesis". > > Signed-off-by: Pritthijit Nath > --- > drivers/staging/greybus/hid.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/dr

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 02:39:26PM +0530, Viresh Kumar wrote: > On 12-02-21, 13:48, Pritthijit Nath wrote: > > This change fixes a checkpatch CHECK style issue for "Alignment should match > > open parenthesis". > > > > Signed-off-by: Pritthijit Nath > > --- > > drivers/staging/greybus/hid.c | 4

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Viresh Kumar
On 12-02-21, 10:17, Greg KH wrote: > On Fri, Feb 12, 2021 at 02:39:26PM +0530, Viresh Kumar wrote: > > On 12-02-21, 13:48, Pritthijit Nath wrote: > > > This change fixes a checkpatch CHECK style issue for "Alignment should > > > match > > > open parenthesis". > > > > > > Signed-off-by: Pritthijit

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 02:51:30PM +0530, Viresh Kumar wrote: > On 12-02-21, 10:17, Greg KH wrote: > > On Fri, Feb 12, 2021 at 02:39:26PM +0530, Viresh Kumar wrote: > > > On 12-02-21, 13:48, Pritthijit Nath wrote: > > > > This change fixes a checkpatch CHECK style issue for "Alignment should > > >

Re: [PATCH v4 2/2] staging: rtl8723bs: remove obsolete commented out code

2021-02-12 Thread Dan Carpenter
On Thu, Feb 11, 2021 at 06:36:04PM +0530, karthik alapati wrote: > There is a bunch of messy, commented out code. Just delete it. > > Suggested-by: Dan Carpenter > Signed-off-by: karthik alapati Perfect, thanks. Greg already merged this as well so good job. regards, dan carepnter __

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Johan Hovold
On Fri, Feb 12, 2021 at 02:51:30PM +0530, Viresh Kumar wrote: > On 12-02-21, 10:17, Greg KH wrote: > > On Fri, Feb 12, 2021 at 02:39:26PM +0530, Viresh Kumar wrote: > > > On 12-02-21, 13:48, Pritthijit Nath wrote: > > > > This change fixes a checkpatch CHECK style issue for "Alignment should > > >

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Viresh Kumar
On 12-02-21, 10:52, Johan Hovold wrote: > But what will the checkpatch crew then work on? Lol.. > Better staging than the rest of the kernel. [ /me wondering on who stops them from sending patches for rest of the kernel ? ] -- viresh ___ devel mailin

Re: [PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 03:20:08PM +0530, Pritthijit Nath wrote: > Fixed the spelling of 'transfered' to 'transferred'. > > Signed-off-by: Pritthijit Nath > --- > Fixed the typo in the patch which was meant to fix that very typo. > Really sorry for last time. > Hope this does not have any othe

Re: [PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Johan Hovold
On Fri, Feb 12, 2021 at 03:25:29PM +0530, Viresh Kumar wrote: > On 12-02-21, 10:52, Johan Hovold wrote: > > But what will the checkpatch crew then work on? > > Lol.. > > > Better staging than the rest of the kernel. > > [ /me wondering on who stops them from sending patches for rest of the > ker

[PATCH 1/2] staging: greybus: Fixed alignment issue in hid.c

2021-02-12 Thread Pritthijit Nath
This change fixes a checkpatch CHECK style issue for "Alignment should match open parenthesis". Signed-off-by: Pritthijit Nath --- drivers/staging/greybus/hid.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/hid.c b/drivers/staging/greybus/hid.c i

[PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Pritthijit Nath
Fixed the spelling of 'transfered' to 'transferred'. Signed-off-by: Pritthijit Nath --- drivers/staging/greybus/hid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/hid.c b/drivers/staging/greybus/hid.c index a56c3fb5d35a..6b19ff4743a9 100644 --- a/dr

[PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Pritthijit Nath
Fixed the spelling of 'transfered' to 'transferred'. Signed-off-by: Pritthijit Nath --- Fixed the typo in the patch which was meant to fix that very typo. Really sorry for last time. Hope this does not have any other typo. drivers/staging/greybus/hid.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH 2/2] staging: greybus: Fixed a misspelling in hid.c

2021-02-12 Thread Pritthijit Nath
Fixed the spelling of 'transfered' to 'transferred'. Signed-off-by: Pritthijit Nath --- Fixed the typo in 'transferred' which crept in last time. drivers/staging/greybus/hid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/hid.c b/drivers/staging/gr

Re: [PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Anirudh Rayabharam
On Thu, Feb 11, 2021 at 09:35:27PM +0100, Greg KH wrote: > On Fri, Feb 12, 2021 at 01:59:08AM +0530, Anirudh Rayabharam wrote: > > Fix sparse byte-order warnings in the i2400m_bm_cmd_prepare() > > function: > > > > wimax/i2400m/fw.c:194:36: warning: restricted __le32 degrades to integer > > wimax/

Re: [PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 08:00:25PM +0530, Anirudh Rayabharam wrote: > On Thu, Feb 11, 2021 at 09:35:27PM +0100, Greg KH wrote: > > On Fri, Feb 12, 2021 at 01:59:08AM +0530, Anirudh Rayabharam wrote: > > > Fix sparse byte-order warnings in the i2400m_bm_cmd_prepare() > > > function: > > > > > > wim

[PATCH] Fix space prohibited issue in fbtft-bus.c

2021-02-12 Thread Alaa Emad
This change fixes a checkpatch error for "space prohibited before that close parenthesis ')'" Signed-off-by: Alaa Emad --- certs/x509_revocation_list| 0 drivers/staging/fbtft/fbtft-bus.c | 4 ++-- 2 files changed, 2 insertions(+), 2 deletions(-) create mode 100644 certs/x509_revocatio

Re: [PATCH] Fix space prohibited issue in fbtft-bus.c

2021-02-12 Thread Greg KH
On Fri, Feb 12, 2021 at 04:58:33PM +0200, Alaa Emad wrote: > This change fixes a checkpatch error for "space prohibited before that close > parenthesis ')'" > > Signed-off-by: Alaa Emad > --- > certs/x509_revocation_list| 0 > drivers/staging/fbtft/fbtft-bus.c | 4 ++-- > 2 files change

[PATCH v2] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Anirudh Rayabharam
Fix sparse byte-order warnings in the i2400m_bm_cmd_prepare() function: wimax/i2400m/fw.c:194:36: warning: restricted __le32 degrades to integer wimax/i2400m/fw.c:195:34: warning: invalid assignment: += wimax/i2400m/fw.c:195:34:left side has type unsigned int wimax/i2400m/fw.c:195:34:right

working together

2021-02-12 Thread MR Ho-Seok Yang
Good Day I am Ho-Seok Yang,my principal’s wish, to make huge financial investment in your home country Please, I will provide more details about the transaction if you are sure you can handle classified information and also let me know your entitlement for the solicited role I shall be expecting

[PATCH v2] ks7010: enclosed non-trivial defines in parentheses

2021-02-12 Thread shivang upadhyay
checkpatch.pl complains about missing parentheses "Macros with complex values should be enclosed in parentheses". fixed that Signed-off-by: shivang upadhyay --- drivers/staging/ks7010/ks_hostif.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/driv

Re: [PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-12 Thread Anirudh Rayabharam
On Fri, Feb 12, 2021 at 03:43:10PM +0100, Greg KH wrote: > On Fri, Feb 12, 2021 at 08:00:25PM +0530, Anirudh Rayabharam wrote: > > On Thu, Feb 11, 2021 at 09:35:27PM +0100, Greg KH wrote: > > > On Fri, Feb 12, 2021 at 01:59:08AM +0530, Anirudh Rayabharam wrote: > > > > Fix sparse byte-order warning

Re: [PATCH v7 0/7] clk: clk-wizard: clock-wizard: Driver updates

2021-02-12 Thread Michal Simek
Hi Miquel, On 1/21/21 3:39 PM, Miquel Raynal wrote: > Hi Shubhrajyoti, > > Shubhrajyoti Datta wrote on Wed, 4 Nov > 2020 20:36:40 +0530: > >> In the thread [1] Greg suggested that we move the driver >> to the clk from the staging. >> Add patches to address the concerns regarding the fractional

[PATCH] staging: greybus: minor code style fix

2021-02-12 Thread Manikantan Ravichandran
checkpatch warning fix for string split across lines Signed-off-by: Manikantan Ravichandran --- drivers/staging/greybus/audio_manager_sysfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/audio_manager_sysfs.c b/drivers/staging/greybus/audio_man

Re: [PATCH] Fix space prohibited issue in fbtft-bus.c

2021-02-12 Thread kernel test robot
e' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Alaa-Emad/Fix-space-prohibited-issue-in-fbtft-bus-c/20210212-230034 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git a2ea4e1d9091cd8bc69f1c42c15bedc38618f

[staging:staging-next] BUILD SUCCESS 6522ad26df0e85c4b52cc39fde269b0c64f9172a

2021-02-12 Thread kernel test robot
-a005-20210209 i386 randconfig-a003-20210209 i386 randconfig-a002-20210209 i386 randconfig-a006-20210209 i386 randconfig-a004-20210209 i386 randconfig-a003-20210212 i386 randconfig-a005-20210212 i386

[staging:staging-testing] BUILD SUCCESS a2ea4e1d9091cd8bc69f1c42c15bedc38618f04c

2021-02-12 Thread kernel test robot
i386 randconfig-a003-20210209 i386 randconfig-a002-20210209 i386 randconfig-a006-20210209 i386 randconfig-a004-20210209 i386 randconfig-a003-20210212 i386 randconfig-a005-20210212 i386

[PATCH] staging: wimax: i2400m: fix some incorrect type warnings

2021-02-12 Thread Ayush
2400m/tx.c:897:32:got unsigned int [usertype] wimax/i2400m/tx.c:899:15: warning: cast to restricted __le32 wimax/i2400m/tx.c:899:15: warning: cast from restricted __le16 Signed-off-by: Ayush --- compile tested on linux-next (tag: next-20210212). drivers/staging/wimax/i2400m/tx.c | 8 --