[PATCH] drivers: staging: Fix spelling in comments

2020-10-29 Thread Bhaskar Chowdhury
Fixed two spelling in two different comments.

s/defalut/default/p

This is in linux-next.

Signed-off-by: Bhaskar Chowdhury 
---
 drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c 
b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
index 1af919ff6d93..391c59490718 100644
--- a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
+++ b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
@@ -52,7 +52,7 @@ static void Init_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

@@ -109,7 +109,7 @@ static void Update_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

--
2.26.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] drivers: staging: Fix spelling in comments

2020-10-29 Thread Greg KH
On Thu, Oct 29, 2020 at 03:37:59PM +0530, Bhaskar Chowdhury wrote:
> Fixed two spelling in two different comments.
> 
> s/defalut/default/p
> 
> This is in linux-next.

No need for this line, it doesn't make any sense here, right?

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] drivers: staging: Fix spelling in comments

2020-10-29 Thread Greg KH
On Thu, Oct 29, 2020 at 03:37:59PM +0530, Bhaskar Chowdhury wrote:
> Fixed two spelling in two different comments.
> 
> s/defalut/default/p
> 
> This is in linux-next.
> 
> Signed-off-by: Bhaskar Chowdhury 
> ---
>  drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Please put the driver name in the subject line, like other patches for
this file do.

thanks,

greg k-h
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH v2] staging: fieldbus: Use %pM format specifier for MAC addresses

2020-10-29 Thread Andy Shevchenko
On Wed, Oct 28, 2020 at 07:39:58PM -0400, Sven Van Asbroeck wrote:
> On Tue, Oct 27, 2020 at 2:34 PM Andy Shevchenko
>  wrote:
> >
> > +   return snprintf(buf, max_size, "%pM\n", response.addr);
> 
> Judging from a few Outreachy patches that have hit my inbox, snprintf() is
> considered unsafe in a sysfs_get callback. It should be replaced by
> scnprintf() or even better, sysfs_emit(), which was recently introduced
> to address sprintf-variant issues in sysfs callbacks.

It's already applied, I think you can send a follow up for above,
which is good point, but not the scope of the patch (although
they might have been unified together).

-- 
With Best Regards,
Andy Shevchenko


___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH] drivers: staging: rtl8188eu: Fix spelling in two comments i.e defalut to default

2020-10-29 Thread Bhaskar Chowdhury
Fixed two spelling in two different comments.

s/defalut/default/p

Signed-off-by: Bhaskar Chowdhury 
---
 Greg, I have included the driver name as you suggested.

 drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c 
b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
index 1af919ff6d93..391c59490718 100644
--- a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
+++ b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
@@ -52,7 +52,7 @@ static void Init_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

@@ -109,7 +109,7 @@ static void Update_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

--
2.26.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] drivers: staging: rtl8188eu: Fix spelling in two comments i.e defalut to default

2020-10-29 Thread Greg KH
On Thu, Oct 29, 2020 at 04:36:00PM +0530, Bhaskar Chowdhury wrote:
> Fixed two spelling in two different comments.
> 
> s/defalut/default/p
> 
> Signed-off-by: Bhaskar Chowdhury 
> ---
>  Greg, I have included the driver name as you suggested.

Suggested when, is this a v2 patch?

Remember, kernel maintainers have the short-term memory of a squirrel...

Please resend and properly version your patch as the Documentation asks
you to.

thanks,

greg k-h
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH] Staging: rtl8723bs: core: rtw_cmd: Fixed four if-statement coding style issues

2020-10-29 Thread Manuel Palenzuela
Fixed four cases where the if-statement coding style wasn't following
the guidelines.

Signed-off-by: Manuel Palenzuela 
---
 drivers/staging/rtl8723bs/core/rtw_cmd.c | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c 
b/drivers/staging/rtl8723bs/core/rtw_cmd.c
index 4cf09d947d32..55142faa2213 100644
--- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
+++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
@@ -344,7 +344,7 @@ int rtw_enqueue_cmd(struct cmd_priv *pcmdpriv, struct 
cmd_obj *cmd_obj)
cmd_obj->padapter = padapter;
 
res = rtw_cmd_filter(pcmdpriv, cmd_obj);
-   if (_FAIL == res) {
+   if (res == _FAIL) {
rtw_free_cmd_obj(cmd_obj);
goto exit;
}
@@ -460,7 +460,7 @@ int rtw_cmd_thread(void *context)
 
cmd_start_time = jiffies;
 
-   if (_FAIL == rtw_cmd_filter(pcmdpriv, pcmd)) {
+   if (rtw_cmd_filter(pcmdpriv, pcmd) == _FAIL) {
pcmd->res = H2C_DROPPED;
goto post_process;
}
@@ -908,7 +908,7 @@ u8 rtw_disassoc_cmd(struct adapter *padapter, u32 
deauth_timeout_ms, bool enqueu
res = rtw_enqueue_cmd(cmdpriv, cmdobj);
} else {
/* no need to enqueue, do the cmd hdl directly and free cmd 
parameter */
-   if (H2C_SUCCESS != disconnect_hdl(padapter, (u8 *)param))
+   if (disconnect_hdl(padapter, (u8 *)param) != H2C_SUCCESS)
res = _FAIL;
kfree(param);
}
@@ -1249,7 +1249,7 @@ u8 rtw_set_chplan_cmd(struct adapter *padapter, u8 
chplan, u8 enqueue, u8 swconf
res = rtw_enqueue_cmd(pcmdpriv, pcmdobj);
} else {
/* no need to enqueue, do the cmd hdl directly and free cmd 
parameter */
-   if (H2C_SUCCESS != set_chplan_hdl(padapter, (unsigned char 
*)setChannelPlan_param))
+   if (set_chplan_hdl(padapter, (unsigned char 
*)setChannelPlan_param) != H2C_SUCCESS)
res = _FAIL;
 
kfree(setChannelPlan_param);
-- 
2.26.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH V2] drivers: staging: rtl8188eu: Fix spelling in two comments i.e defalut to default

2020-10-29 Thread Bhaskar Chowdhury
Fixed two spelling in two different comments.

s/defalut/default/p

Signed-off-by: Bhaskar Chowdhury 
---
 drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c 
b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
index 1af919ff6d93..391c59490718 100644
--- a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
+++ b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
@@ -52,7 +52,7 @@ static void Init_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

@@ -109,7 +109,7 @@ static void Update_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

--
2.26.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH V2] drivers: staging: rtl8188eu: Fix spelling in two comments i.e defalut to default

2020-10-29 Thread Greg KH
On Thu, Oct 29, 2020 at 05:37:25PM +0530, Bhaskar Chowdhury wrote:
> Fixed two spelling in two different comments.
> 
> s/defalut/default/p
> 
> Signed-off-by: Bhaskar Chowdhury 
> ---
>  drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

What changed from v1?

That always goes below the --- line, please fix up and send a v3.

thanks,

greg k-h
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH V3] drivers: staging: rtl8188eu: Fix spelling in two comments i.e defalut to default

2020-10-29 Thread Bhaskar Chowdhury
Fixed two spelling in two different comments.

s/defalut/default/p

Signed-off-by: Bhaskar Chowdhury 
---
 Changes from V2 : version change to reflect the correct one.

 drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c 
b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
index 1af919ff6d93..391c59490718 100644
--- a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
+++ b/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
@@ -52,7 +52,7 @@ static void Init_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

@@ -109,7 +109,7 @@ static void Update_ODM_ComInfo_88E(struct adapter *Adapter)
 * The base index =
 * 12. +((12-n)/2)dB 13~?? = decrease tx pwr by -((n-12)/2)dB
 */
-   dm_odm->BbSwingIdxOfdm = 12; /*  Set defalut value as index 12. */
+   dm_odm->BbSwingIdxOfdm = 12; /*  Set default value as index 12. */
dm_odm->BbSwingIdxOfdmCurrent = 12;
dm_odm->BbSwingFlagOfdm = false;

--
2.26.2

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH, net -> staging, v2] wimax: move out to staging

2020-10-29 Thread Arnd Bergmann
From: Arnd Bergmann 

There are no known users of this driver as of October 2020, and it will
be removed unless someone turns out to still need it in future releases.

According to https://en.wikipedia.org/wiki/List_of_WiMAX_networks, there
have been many public wimax networks, but it appears that many of these
have migrated to LTE or discontinued their service altogether.
As most PCs and phones lack WiMAX hardware support, the remaining
networks tend to use standalone routers. These almost certainly
run Linux, but not a modern kernel or the mainline wimax driver stack.

NetworkManager appears to have dropped userspace support in 2015
https://bugzilla.gnome.org/show_bug.cgi?id=747846, the
www.linuxwimax.org
site had already shut down earlier.

WiMax is apparently still being deployed on airport campus networks
("AeroMACS"), but in a frequency band that was not supported by the old
Intel 2400m (used in Sandy Bridge laptops and earlier), which is the
only driver using the kernel's wimax stack.

Move all files into drivers/staging/wimax, including the uapi header
files and documentation, to make it easier to remove it when it gets
to that. Only minimal changes are made to the source files, in order
to make it possible to port patches across the move.

Also remove the MAINTAINERS entry that refers to a broken mailing
list and website.

Suggested-by: Inaky Perez-Gonzalez 
Signed-off-by: Arnd Bergmann 
---
changes in v2:
- fix a build regression
- add more information about remaining networks (Dan Carpenter)_

For v1, Greg said he'd appply the patch when he gets an Ack
from the maintainers.

Inaky, Johannes, Jakub: are you happy with this version?
---
 Documentation/admin-guide/index.rst   |  1 -
 Documentation/networking/kapi.rst | 21 --
 .../translations/zh_CN/admin-guide/index.rst  |  1 -
 MAINTAINERS   | 22 ---
 drivers/net/Kconfig   |  2 --
 drivers/net/Makefile  |  1 -
 drivers/net/wimax/Kconfig | 18 ---
 drivers/net/wimax/Makefile|  2 --
 drivers/staging/Kconfig   |  2 ++
 drivers/staging/Makefile  |  1 +
 .../staging/wimax/Documentation}/i2400m.rst   |  0
 .../staging/wimax/Documentation}/index.rst|  0
 .../staging/wimax/Documentation}/wimax.rst|  0
 {net => drivers/staging}/wimax/Kconfig|  6 +
 {net => drivers/staging}/wimax/Makefile   |  2 ++
 drivers/staging/wimax/TODO| 18 +++
 {net => drivers/staging}/wimax/debug-levels.h |  2 +-
 {net => drivers/staging}/wimax/debugfs.c  |  2 +-
 drivers/{net => staging}/wimax/i2400m/Kconfig |  0
 .../{net => staging}/wimax/i2400m/Makefile|  0
 .../{net => staging}/wimax/i2400m/control.c   |  2 +-
 .../wimax/i2400m/debug-levels.h   |  2 +-
 .../{net => staging}/wimax/i2400m/debugfs.c   |  0
 .../{net => staging}/wimax/i2400m/driver.c|  2 +-
 drivers/{net => staging}/wimax/i2400m/fw.c|  0
 .../wimax/i2400m/i2400m-usb.h |  0
 .../{net => staging}/wimax/i2400m/i2400m.h|  4 ++--
 .../staging/wimax/i2400m/linux-wimax-i2400m.h |  0
 .../{net => staging}/wimax/i2400m/netdev.c|  0
 .../{net => staging}/wimax/i2400m/op-rfkill.c |  2 +-
 drivers/{net => staging}/wimax/i2400m/rx.c|  0
 drivers/{net => staging}/wimax/i2400m/sysfs.c |  0
 drivers/{net => staging}/wimax/i2400m/tx.c|  0
 .../wimax/i2400m/usb-debug-levels.h   |  2 +-
 .../{net => staging}/wimax/i2400m/usb-fw.c|  0
 .../{net => staging}/wimax/i2400m/usb-notif.c |  0
 .../{net => staging}/wimax/i2400m/usb-rx.c|  0
 .../{net => staging}/wimax/i2400m/usb-tx.c|  0
 drivers/{net => staging}/wimax/i2400m/usb.c   |  2 +-
 {net => drivers/staging}/wimax/id-table.c |  2 +-
 .../staging/wimax/linux-wimax-debug.h |  2 +-
 .../staging/wimax/linux-wimax.h   |  0
 .../staging/wimax/net-wimax.h |  2 +-
 {net => drivers/staging}/wimax/op-msg.c   |  2 +-
 {net => drivers/staging}/wimax/op-reset.c |  4 ++--
 {net => drivers/staging}/wimax/op-rfkill.c|  4 ++--
 {net => drivers/staging}/wimax/op-state-get.c |  4 ++--
 {net => drivers/staging}/wimax/stack.c|  2 +-
 .../staging}/wimax/wimax-internal.h   |  2 +-
 net/Kconfig   |  2 --
 net/Makefile  |  1 -
 51 files changed, 51 insertions(+), 93 deletions(-)
 delete mode 100644 drivers/net/wimax/Kconfig
 delete mode 100644 drivers/net/wimax/Makefile
 rename {Documentation/admin-guide/wimax => 
drivers/staging/wimax/Documentation}/i2400m.rst (100%)
 rename {Documentation/admin-guide/wimax => 
drivers/staging/wimax/Documentation}/index.rst (100%)
 rename {Documentation/admin-guide/wimax => 
drivers/staging/wimax/Documentation}/wimax.rst (100%)
 rename {net => drivers/staging}/wimax/Kconfig (94%)
 rename {net

[PATCH] staging: comedi: cb_pcidas: reinstate delay removed from trimpot setting

2020-10-29 Thread Ian Abbott
Commit eddd2a4c675c ("staging: comedi: cb_pcidas: refactor
write_calibration_bitstream()") inadvertently removed one of the
`udelay(1)` calls when writing to the calibration register in
`cb_pcidas_calib_write()`.  Reinstate the delay.  It may seem strange
that the delay is placed before the register write, but this function is
called in a loop so the extra delay can make a difference.

This _might_ solve reported issues reading analog inputs on a
PCIe-DAS1602/16 card where the analog input values "were scaled in a
strange way that didn't make sense".  On the same hardware running a
system with a 3.13 kernel, and then a system with a 4.4 kernel, but with
the same application software, the system with the 3.13 kernel was fine,
but the one with the 4.4 kernel exhibited the problem.  Of the 90
changes to the driver between those kernel versions, this change looked
like the most likely culprit.

Fixes: eddd2a4c675c ("staging: comedi: cb_pcidas: refactor 
write_calibration_bitstream()")
Cc: 
Signed-off-by: Ian Abbott 
---
 drivers/staging/comedi/drivers/cb_pcidas.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/comedi/drivers/cb_pcidas.c 
b/drivers/staging/comedi/drivers/cb_pcidas.c
index 48ec2ee953dc..4f2ac39aa619 100644
--- a/drivers/staging/comedi/drivers/cb_pcidas.c
+++ b/drivers/staging/comedi/drivers/cb_pcidas.c
@@ -529,6 +529,7 @@ static void cb_pcidas_calib_write(struct comedi_device *dev,
if (trimpot) {
/* select trimpot */
calib_bits |= PCIDAS_CALIB_TRIM_SEL;
+   udelay(1);
outw(calib_bits, devpriv->pcibar1 + PCIDAS_CALIB_REG);
}
 
-- 
2.28.0

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH, net -> staging, v2] wimax: move out to staging

2020-10-29 Thread Johannes Berg
On Thu, 2020-10-29 at 14:43 +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann 
> 
> There are no known users of this driver as of October 2020, and it will
> be removed unless someone turns out to still need it in future releases.
> 
> According to https://en.wikipedia.org/wiki/List_of_WiMAX_networks, there
> have been many public wimax networks, but it appears that many of these
> have migrated to LTE or discontinued their service altogether.
> As most PCs and phones lack WiMAX hardware support, the remaining
> networks tend to use standalone routers. These almost certainly
> run Linux, but not a modern kernel or the mainline wimax driver stack.
> 
> NetworkManager appears to have dropped userspace support in 2015
> https://bugzilla.gnome.org/show_bug.cgi?id=747846, the
> www.linuxwimax.org
> site had already shut down earlier.
> 
> WiMax is apparently still being deployed on airport campus networks
> ("AeroMACS"), but in a frequency band that was not supported by the old
> Intel 2400m (used in Sandy Bridge laptops and earlier), which is the
> only driver using the kernel's wimax stack.
> 
> Move all files into drivers/staging/wimax, including the uapi header
> files and documentation, to make it easier to remove it when it gets
> to that. Only minimal changes are made to the source files, in order
> to make it possible to port patches across the move.
> 
> Also remove the MAINTAINERS entry that refers to a broken mailing
> list and website.
> 
> Suggested-by: Inaky Perez-Gonzalez 
> Signed-off-by: Arnd Bergmann 
> ---
> changes in v2:
> - fix a build regression
> - add more information about remaining networks (Dan Carpenter)_
> 
> For v1, Greg said he'd appply the patch when he gets an Ack
> from the maintainers.
> 
> Inaky, Johannes, Jakub: are you happy with this version?

Sure, looks fine to me.

Acked-by: Johannes Berg 

Not that I have much relation to this code other than having fixed up
genetlink stuff over the years :)

johannes

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH -next] binder: change error code from postive to negative in binder_transaction

2020-10-29 Thread Christian Brauner
On Mon, Oct 26, 2020 at 07:03:14PM +0800, Zhang Qilong wrote:
> Depending on the context, the error return value
> here (extra_buffers_size < added_size) should be
> negative.
> 
> Signed-off-by: Zhang Qilong 
> ---

Thanks!
Acked-by: Christian Brauner 
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [RFC] wimax: move out to staging

2020-10-29 Thread Jakub Kicinski
On Wed, 28 Oct 2020 06:56:28 +0100 Greg Kroah-Hartman wrote:
> On Tue, Oct 27, 2020 at 10:20:13PM +0100, Arnd Bergmann wrote:
> > From: Arnd Bergmann 
> > 
> > There are no known users of this driver as of October 2020, and it will
> > be removed unless someone turns out to still need it in future releases.
> > 
> > According to https://en.wikipedia.org/wiki/List_of_WiMAX_networks, there
> > have been many public wimax networks, but it appears that these entries
> > are all stale, after everyone has migrated to LTE or discontinued their
> > service altogether.
> > 
> > NetworkManager appears to have dropped userspace support in 2015
> > https://bugzilla.gnome.org/show_bug.cgi?id=747846, the
> > www.linuxwimax.org
> > site had already shut down earlier.
> > 
> > WiMax is apparently still being deployed on airport campus networks
> > ("AeroMACS"), but in a frequency band that was not supported by the old
> > Intel 2400m (used in Sandy Bridge laptops and earlier), which is the
> > only driver using the kernel's wimax stack.
> > 
> > Move all files into drivers/staging/wimax, including the uapi header
> > files and documentation, to make it easier to remove it when it gets
> > to that. Only minimal changes are made to the source files, in order
> > to make it possible to port patches across the move.
> > 
> > Also remove the MAINTAINERS entry that refers to a broken mailing
> > list and website.
> > 
> > Suggested-by: Inaky Perez-Gonzalez 
> > Signed-off-by: Arnd Bergmann   
> 
> Is this ok for me to take through the staging tree?  If so, I need an
> ack from the networking maintainers.
> 
> If not, feel free to send it through the networking tree and add:
> 
> Acked-by: Greg Kroah-Hartman 

Thinking about it now - we want this applied to -next, correct? 
In that case it may be better if we take it. The code is pretty dead
but syzbot and the trivial fix crowd don't know it, so I may slip,
apply something and we'll have a conflict.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] staging: rtl8723bs: Add 024c:0627 to the list of SDIO device-ids

2020-10-29 Thread Hans de Goede
Hi,

On 10/29/20 2:52 PM, Brian O'Keefe wrote:
> Add 024c:0627 to the list of SDIO device-ids, based on hardware found in
> the wild. This hardware exists on at least some Acer SW1-011 tablets.
> 
> Signed-off-by: Brian O'Keefe 

Patch looks good to me:

Reviewed-by: Hans de Goede 

Regards,

Hans

> ---
>  drivers/staging/rtl8723bs/os_dep/sdio_intf.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> index 5b1392deb0a7..7256d55fcc1b 100644
> --- a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> +++ b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
> @@ -21,6 +21,7 @@ static const struct sdio_device_id sdio_ids[] =
> { SDIO_DEVICE(0x024c, 0x0525), },
> { SDIO_DEVICE(0x024c, 0x0623), },
> { SDIO_DEVICE(0x024c, 0x0626), },
> +   { SDIO_DEVICE(0x024c, 0x0627), },
> { SDIO_DEVICE(0x024c, 0xb723), },
> { /* end: all zeroes */ },
>  };
> --
> 2.25.1
> 

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [RFC] wimax: move out to staging

2020-10-29 Thread Arnd Bergmann
n Thu, Oct 29, 2020 at 4:56 PM Jakub Kicinski  wrote:
> On Wed, 28 Oct 2020 06:56:28 +0100 Greg Kroah-Hartman wrote:
> > On Tue, Oct 27, 2020 at 10:20:13PM +0100, Arnd Bergmann wrote:
> >
> > Is this ok for me to take through the staging tree?  If so, I need an
> > ack from the networking maintainers.
> >
> > If not, feel free to send it through the networking tree and add:
> >
> > Acked-by: Greg Kroah-Hartman 
>
> Thinking about it now - we want this applied to -next, correct?
> In that case it may be better if we take it. The code is pretty dead
> but syzbot and the trivial fix crowd don't know it, so I may slip,
> apply something and we'll have a conflict.

I think git will deal with a merge between branches containing
the move vs fix, so it should work either way.

A downside of having the move in net-next would be that
you'd get trivial fixes send to Greg, but him being unable to
apply them to his tree because the code is elsewhere.

If you think it helps, I could prepare a pull request with this one
patch (and probably the bugfix I sent first that triggered it), and
then you can both merge the branch into net-next as well
as staging-next.

  Arnd
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [RFC] wimax: move out to staging

2020-10-29 Thread Jakub Kicinski
On Thu, 29 Oct 2020 17:26:09 +0100 Arnd Bergmann wrote:
> n Thu, Oct 29, 2020 at 4:56 PM Jakub Kicinski  wrote:
> > On Wed, 28 Oct 2020 06:56:28 +0100 Greg Kroah-Hartman wrote:  
> > > On Tue, Oct 27, 2020 at 10:20:13PM +0100, Arnd Bergmann wrote:
> > >
> > > Is this ok for me to take through the staging tree?  If so, I need an
> > > ack from the networking maintainers.
> > >
> > > If not, feel free to send it through the networking tree and add:
> > >
> > > Acked-by: Greg Kroah-Hartman   
> >
> > Thinking about it now - we want this applied to -next, correct?
> > In that case it may be better if we take it. The code is pretty dead
> > but syzbot and the trivial fix crowd don't know it, so I may slip,
> > apply something and we'll have a conflict.  
> 
> I think git will deal with a merge between branches containing
> the move vs fix, so it should work either way.
> 
> A downside of having the move in net-next would be that
> you'd get trivial fixes send to Greg, but him being unable to
> apply them to his tree because the code is elsewhere.
> 
> If you think it helps, I could prepare a pull request with this one
> patch (and probably the bugfix I sent first that triggered it), and
> then you can both merge the branch into net-next as well
> as staging-next.

If you wouldn't mind branch sounds like the best solution.

Acked-by: Jakub Kicinski 
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH] staging: rtl8723bs: Add 024c:0627 to the list of SDIO device-ids

2020-10-29 Thread Brian O'Keefe
Add 024c:0627 to the list of SDIO device-ids, based on hardware found in
the wild. This hardware exists on at least some Acer SW1-011 tablets.

Signed-off-by: Brian O'Keefe 
---
 drivers/staging/rtl8723bs/os_dep/sdio_intf.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
index 5b1392deb0a7..7256d55fcc1b 100644
--- a/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
+++ b/drivers/staging/rtl8723bs/os_dep/sdio_intf.c
@@ -21,6 +21,7 @@ static const struct sdio_device_id sdio_ids[] =
{ SDIO_DEVICE(0x024c, 0x0525), },
{ SDIO_DEVICE(0x024c, 0x0623), },
{ SDIO_DEVICE(0x024c, 0x0626), },
+   { SDIO_DEVICE(0x024c, 0x0627), },
{ SDIO_DEVICE(0x024c, 0xb723), },
{ /* end: all zeroes */ },
 };
--
2.25.1
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH v6 5/9] media: staging: rkisp1: remove unecessary clocks

2020-10-29 Thread Rob Herring
On Tue, Oct 20, 2020 at 2:39 PM Helen Koike  wrote:
>
> aclk_isp_wrap is a child of aclk_isp, and hclk_isp_wrap is a child of
> hclk_isp, thus we can remove parents from the list.
>
> Also, for the isp0, we only need the ISP clock, ACLK and HCLK.
> In the future we'll need a pixel clock for RK3288 and RK3399, and a JPEG
> clock for RK3288.
>
> So with the goal to cleanup the dt-bindings and remove it from staging,
> simplify clock names to isp, aclk and hclk.
>
> Assigned clocks are meant to refer to the full path in the clock tree,
> i.e. the leaf in the tree.
> For instance, in RK3399, the clock responsible for ACLK (ISP AXI CLOCK)
> is aclk_isp0_wrapper.
>
> For reference, this is the isp clock topology on RK3399:
>
>  xin24m
> pll_npll
>npll
>   clk_isp1
>   clk_isp0
> pll_cpll
>cpll
>   aclk_isp1
>  aclk_isp1_noc
>  hclk_isp1
> aclk_isp1_wrapper
> hclk_isp1_noc
>   aclk_isp0
>  hclk_isp1_wrapper
>  aclk_isp0_wrapper
>  aclk_isp0_noc
>  hclk_isp0
> hclk_isp0_wrapper
> hclk_isp0_noc
>  pclkin_isp1_wrapper
>
> Signed-off-by: Helen Koike 
> Reviewed-by: Tomasz Figa 
>
> ---
>
> Changes in V6:
> - Define clocks in the top level, and use if/else schema to define how
>   many for each compatible as sugested by Rob Herring on
>   
> https://patchwork.linuxtv.org/project/linux-media/patch/20200722155533.252844-6-helen.ko...@collabora.com/#122626
> ---
>  .../bindings/media/rockchip-isp1.yaml | 44 +--
>  drivers/staging/media/rkisp1/rkisp1-dev.c |  8 ++--
>  2 files changed, 33 insertions(+), 19 deletions(-)

Reviewed-by: Rob Herring 
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[GIT PULL, staging, net-next] wimax: move to staging

2020-10-29 Thread Arnd Bergmann
The following changes since commit 3650b228f83adda7e5ee532e2b90429c03f7b9ec:

  Linux 5.10-rc1 (2020-10-25 15:14:11 -0700)

are available in the Git repository at:

  git://git.kernel.org:/pub/scm/linux/kernel/git/arnd/playground.git
tags/wimax-staging

for you to fetch changes up to f54ec58fee837ec847cb8b50593e81bfaa46107f:

  wimax: move out to staging (2020-10-29 19:27:45 +0100)


wimax: move to staging

After I sent a fix for what appeared to be a harmless warning in
the wimax user interface code, the conclusion was that the whole
thing has most likely not been used in a very long time, and the
user interface possibly been broken since b61a5eea5904 ("wimax: use
genl_register_family_with_ops()").

Using a shared branch between net-next and staging should help
coordinate patches getting submitted against it.


Arnd Bergmann (2):
  wimax: fix duplicate initializer warning
  wimax: move out to staging

 Documentation/admin-guide/index.rst|  1 -
 Documentation/networking/kapi.rst  | 21 
 .../translations/zh_CN/admin-guide/index.rst   |  1 -
 MAINTAINERS| 22 
 drivers/net/Kconfig|  2 --
 drivers/net/Makefile   |  1 -
 drivers/net/wimax/Kconfig  | 18 --
 drivers/net/wimax/Makefile |  2 --
 drivers/staging/Kconfig|  2 ++
 drivers/staging/Makefile   |  1 +
 .../staging/wimax/Documentation}/i2400m.rst|  0
 .../staging/wimax/Documentation}/index.rst |  0
 .../staging/wimax/Documentation}/wimax.rst |  0
 {net => drivers/staging}/wimax/Kconfig |  6 +
 {net => drivers/staging}/wimax/Makefile|  2 ++
 drivers/staging/wimax/TODO | 18 ++
 {net => drivers/staging}/wimax/debug-levels.h  |  2 +-
 {net => drivers/staging}/wimax/debugfs.c   |  2 +-
 drivers/{net => staging}/wimax/i2400m/Kconfig  |  0
 drivers/{net => staging}/wimax/i2400m/Makefile |  0
 drivers/{net => staging}/wimax/i2400m/control.c|  2 +-
 .../{net => staging}/wimax/i2400m/debug-levels.h   |  2 +-
 drivers/{net => staging}/wimax/i2400m/debugfs.c|  0
 drivers/{net => staging}/wimax/i2400m/driver.c |  2 +-
 drivers/{net => staging}/wimax/i2400m/fw.c |  0
 drivers/{net => staging}/wimax/i2400m/i2400m-usb.h |  0
 drivers/{net => staging}/wimax/i2400m/i2400m.h |  4 +--
 .../staging/wimax/i2400m/linux-wimax-i2400m.h  |  0
 drivers/{net => staging}/wimax/i2400m/netdev.c |  0
 drivers/{net => staging}/wimax/i2400m/op-rfkill.c  |  2 +-
 drivers/{net => staging}/wimax/i2400m/rx.c |  0
 drivers/{net => staging}/wimax/i2400m/sysfs.c  |  0
 drivers/{net => staging}/wimax/i2400m/tx.c |  0
 .../wimax/i2400m/usb-debug-levels.h|  2 +-
 drivers/{net => staging}/wimax/i2400m/usb-fw.c |  0
 drivers/{net => staging}/wimax/i2400m/usb-notif.c  |  0
 drivers/{net => staging}/wimax/i2400m/usb-rx.c |  0
 drivers/{net => staging}/wimax/i2400m/usb-tx.c |  0
 drivers/{net => staging}/wimax/i2400m/usb.c|  2 +-
 {net => drivers/staging}/wimax/id-table.c  |  2 +-
 .../staging/wimax/linux-wimax-debug.h  |  2 +-
 .../wimax.h => drivers/staging/wimax/linux-wimax.h |  0
 .../wimax.h => drivers/staging/wimax/net-wimax.h   |  2 +-
 {net => drivers/staging}/wimax/op-msg.c|  2 +-
 {net => drivers/staging}/wimax/op-reset.c  |  4 +--
 {net => drivers/staging}/wimax/op-rfkill.c |  4 +--
 {net => drivers/staging}/wimax/op-state-get.c  |  4 +--
 {net => drivers/staging}/wimax/stack.c | 29 ++
 {net => drivers/staging}/wimax/wimax-internal.h|  2 +-
 net/Kconfig|  2 --
 net/Makefile   |  1 -
 51 files changed, 68 insertions(+), 103 deletions(-)
 delete mode 100644 drivers/net/wimax/Kconfig
 delete mode 100644 drivers/net/wimax/Makefile
 rename {Documentation/admin-guide/wimax =>
drivers/staging/wimax/Documentation}/i2400m.rst (100%)
 rename {Documentation/admin-guide/wimax =>
drivers/staging/wimax/Documentation}/index.rst (100%)
 rename {Documentation/admin-guide/wimax =>
drivers/staging/wimax/Documentation}/wimax.rst (100%)
 rename {net => drivers/staging}/wimax/Kconfig (94%)
 rename {net => drivers/staging}/wimax/Makefile (83%)
 create mode 100644 drivers/staging/wimax/TODO
 rename {net => drivers/staging}/wimax/debug-levels.h (96%)
 rename {net => drivers/staging}/wimax/debugfs.c (97%)
 rename drivers/{net => staging}/wimax/i2400m/Kconfig (100%)
 rename drivers/{net => staging}/wimax/i2400m/Makefile (100%)
 rename drivers/{net => staging}/

[driver-core:debugfs_cleanup] BUILD SUCCESS eb27f3832891c8db753d3acb3a2a02d9930ef911

2020-10-29 Thread kernel test robot
tree/branch: 
https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git  
debugfs_cleanup
branch HEAD: eb27f3832891c8db753d3acb3a2a02d9930ef911  debugfs: remove return 
value of debugfs_create_devm_seqfile()

elapsed time: 780m

configs tested: 102
configs skipped: 2

The following configs have been built successfully.
More configs may be tested in the coming days.

gcc tested configs:
arm defconfig
arm64allyesconfig
arm64   defconfig
arm  allyesconfig
arm  allmodconfig
powerpc mpc5200_defconfig
powerpc   maple_defconfig
c6xevmc6678_defconfig
powerpc  arches_defconfig
shapsh4ad0a_defconfig
mips  rb532_defconfig
arm hackkit_defconfig
arm  gemini_defconfig
powerpc mpc8313_rdb_defconfig
sparc   sparc64_defconfig
shdreamcast_defconfig
shedosk7705_defconfig
powerpc  mpc885_ads_defconfig
sh  rsk7264_defconfig
sh sh03_defconfig
arm  prima2_defconfig
mips   capcella_defconfig
c6x  alldefconfig
mipsmalta_kvm_guest_defconfig
sh  sdk7780_defconfig
arm  zx_defconfig
ia64  tiger_defconfig
mips   rbtx49xx_defconfig
arm ebsa110_defconfig
ia64generic_defconfig
arm assabet_defconfig
arm  simpad_defconfig
arm  jornada720_defconfig
ia64 allmodconfig
ia64defconfig
ia64 allyesconfig
m68k allmodconfig
m68kdefconfig
m68k allyesconfig
nios2   defconfig
arc  allyesconfig
nds32 allnoconfig
c6x  allyesconfig
nds32   defconfig
nios2allyesconfig
cskydefconfig
alpha   defconfig
alphaallyesconfig
xtensa   allyesconfig
h8300allyesconfig
arc defconfig
sh   allmodconfig
parisc  defconfig
s390 allyesconfig
parisc   allyesconfig
s390defconfig
i386 allyesconfig
sparcallyesconfig
sparc   defconfig
i386defconfig
mips allyesconfig
mips allmodconfig
powerpc  allyesconfig
powerpc  allmodconfig
powerpc   allnoconfig
x86_64   randconfig-a001-20201029
x86_64   randconfig-a002-20201029
x86_64   randconfig-a003-20201029
x86_64   randconfig-a006-20201029
x86_64   randconfig-a005-20201029
x86_64   randconfig-a004-20201029
i386 randconfig-a002-20201029
i386 randconfig-a005-20201029
i386 randconfig-a003-20201029
i386 randconfig-a001-20201029
i386 randconfig-a004-20201029
i386 randconfig-a006-20201029
i386 randconfig-a016-20201029
i386 randconfig-a014-20201029
i386 randconfig-a015-20201029
i386 randconfig-a013-20201029
i386 randconfig-a012-20201029
i386 randconfig-a011-20201029
riscvnommu_k210_defconfig
riscvallyesconfig
riscvnommu_virt_defconfig
riscv allnoconfig
riscv   defconfig
riscv  rv32_defconfig
riscvallmodconfig
x86_64   rhel
x86_64   allyesconfig
x86_64rhel-7.6-kselftests
x86_64  defconfig
x86_64   rhel-8.3
x86_64  kexec

clang tested configs:
x86_64   randconfig-a011-20201029
x86_64   randconfig-a013-20201029
x86_64   randconfig-a016-20201029
x86_64