Re: [BUG] staging: r8822be: RTL8822be can't find any wireless AP

2018-07-05 Thread Pkshih
On Thu, 2018-07-05 at 12:06 -0500, Larry Finger wrote: > On 07/05/2018 02:36 AM, Pkshih wrote: > > On Wed, 2018-07-04 at 10:33 -0500, Daniel Drake wrote: > >> On Wed, Jul 4, 2018 at 10:13 AM, Larry Finger > >> wrote: > >>> We will have to agree to disagree. > >>> > >>> I have no idea what the

RE: [PATCH 1/1] X86/Hyper-V:: Fix the circular dependency in IPI enlightenment.

2018-07-05 Thread KY Srinivasan
> -Original Message- > From: Thomas Gleixner > Sent: Thursday, July 5, 2018 3:46 PM > To: Ingo Molnar > Cc: KY Srinivasan ; x...@kernel.org; > gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; >

[PATCH] staging: mt7621-pinctrl: Replace unsigned with unsigned int

2018-07-05 Thread Peter Vernia
Replaces instances of "unsigned" with "unsigned int" in pinctrl-mt7621.c Signed-off-by: Peter Vernia --- drivers/staging/mt7621-pinctrl/pinctrl-rt2880.c | 27 + 1 file changed, 14 insertions(+), 13 deletions(-) diff --git

Re: [PATCH v4 2/2] dt-bindings: document gpio-mt7621 bindings

2018-07-05 Thread Rob Herring
On Thu, Jul 05, 2018 at 03:43:11PM +0200, Sergio Paracuellos wrote: > Add a devicetree binding documentation for the mt7621 gpio. > > Signed-off-by: Sergio Paracuellos > Reviewed-by: NeilBrown > --- > .../bindings/gpio/mediatek,mt7621-gpio.txt | 35 > ++ > 1 file

Re: [PATCH 1/1] X86/Hyper-V:: Fix the circular dependency in IPI enlightenment.

2018-07-05 Thread Thomas Gleixner
On Fri, 6 Jul 2018, Ingo Molnar wrote: > * KY Srinivasan wrote: > > I am confused. The label ipi_mask_done was introduced in this patch > > (the patch under question fixes a circular dependency in this patch): > > > > commit 68bb7bfb7985df2bd15c2dc975cb68b7a901488a > > Author: K. Y. Srinivasan

Re: [PATCH 1/1] X86/Hyper-V:: Fix the circular dependency in IPI enlightenment.

2018-07-05 Thread Ingo Molnar
* KY Srinivasan wrote: > > > > -Original Message- > > From: Ingo Molnar On Behalf Of Ingo Molnar > > Sent: Thursday, July 5, 2018 8:38 AM > > To: KY Srinivasan > > Cc: x...@kernel.org; gre...@linuxfoundation.org; linux- > > ker...@vger.kernel.org; de...@linuxdriverproject.org;

RE: [PATCH 1/1] X86/Hyper-V:: Fix the circular dependency in IPI enlightenment.

2018-07-05 Thread KY Srinivasan
> -Original Message- > From: Ingo Molnar On Behalf Of Ingo Molnar > Sent: Thursday, July 5, 2018 8:38 AM > To: KY Srinivasan > Cc: x...@kernel.org; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com;

[PATCH 1/2] staging:fsl-mc: Move DPIO from staging to drivers/soc/fsl

2018-07-05 Thread Roy Pledge
Move the NXP DPIO (Datapath I/O Driver) out of the drivers/staging directory and into the drivers/soc/fsl directory. The DPIO driver enables access to Queue and Buffer Manager (QBMAN) hardware on NXP DPAA2 devices. This is a prerequisite to moving the DPAA2 Ethernet driver out of staging.

[PATCH 2/2] drivers/staging: Remove fsl-mc driver from staging

2018-07-05 Thread Roy Pledge
Remove the staging/drivers/fsl-mc directory from the staging area now that all the components have been moved to the main kernel areas. Signed-off-by: Roy Pledge --- drivers/staging/Kconfig | 2 -- drivers/staging/Makefile| 1 - drivers/staging/fsl-mc/Kconfig | 2 --

[PATCH 0/2] staging/fsl-mc/bus: Move DPIO from staging to drivers/soc/fsl

2018-07-05 Thread Roy Pledge
Move the NXP DPIO (Datapath I/O driver) from the staging/fsl-mc/bus/dpio directory to the drivers/soc/fsl directory. The DPIO driver enables access to the Queue and Buffer Managemer (QBMAN) hardware of NXP DPAA2 devices. This is a prerequiste for moving the DPAA2 Ethernet device driver from the

Re: [BUG] staging: r8822be: RTL8822be can't find any wireless AP

2018-07-05 Thread Larry Finger
On 07/05/2018 02:36 AM, Pkshih wrote: On Wed, 2018-07-04 at 10:33 -0500, Daniel Drake wrote: On Wed, Jul 4, 2018 at 10:13 AM, Larry Finger wrote: We will have to agree to disagree. I have no idea what the vendors are doing that cause some motherboards to need a different aspm value. What I

Re: [PATCH 1/1] X86/Hyper-V:: Fix the circular dependency in IPI enlightenment.

2018-07-05 Thread Ingo Molnar
* KY Srinivasan wrote: > > > > -Original Message- > > From: Ingo Molnar On Behalf Of Ingo Molnar > > Sent: Wednesday, July 4, 2018 9:11 AM > > To: KY Srinivasan > > Cc: x...@kernel.org; gre...@linuxfoundation.org; linux- > > ker...@vger.kernel.org; de...@linuxdriverproject.org;

RE: [PATCH 1/1] X86/Hyper-V:: Fix the circular dependency in IPI enlightenment.

2018-07-05 Thread KY Srinivasan
> -Original Message- > From: Ingo Molnar On Behalf Of Ingo Molnar > Sent: Wednesday, July 4, 2018 9:11 AM > To: KY Srinivasan > Cc: x...@kernel.org; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com;

[PATCH v4 1/2] gpio: mediatek: add driver for MT7621

2018-07-05 Thread Sergio Paracuellos
Add driver support for gpio of MT7621 SoC. Signed-off-by: Sergio Paracuellos Reviewed-by: NeilBrown --- drivers/gpio/Kconfig | 8 ++ drivers/gpio/Makefile | 1 + drivers/gpio/gpio-mt7621.c | 331 + 3 files changed, 340 insertions(+)

[PATCH v4 0/2] gpio: mediatek: driver for gpio chip in MT7621 SoC

2018-07-05 Thread Sergio Paracuellos
This patch series add support for gpio driver in mediatek MT7621 SoC. This driver has been in staging for a while and after some cleanups cycles we consider to give it a new try to get mainlined. Previous comments from Linus Walleij are here:

[PATCH v4 2/2] dt-bindings: document gpio-mt7621 bindings

2018-07-05 Thread Sergio Paracuellos
Add a devicetree binding documentation for the mt7621 gpio. Signed-off-by: Sergio Paracuellos Reviewed-by: NeilBrown --- .../bindings/gpio/mediatek,mt7621-gpio.txt | 35 ++ 1 file changed, 35 insertions(+) create mode 100644

[PATCH] staging: mt7621-mmc: Fix printk() facility level

2018-07-05 Thread Sergej Perschin
The patch fixes the following issue: WARNING: printk() should include KERN_ facility level Signed-off-by: Sergej Perschin Signed-off-by: Marcel Budoj --- drivers/staging/mt7621-mmc/dbg.c | 54 1 file changed, 27 insertions(+), 27 deletions(-) diff

Re: [PATCH 3/3] drivers/speakup: Fix style and coding warnings

2018-07-05 Thread Dan Carpenter
On Tue, Jul 03, 2018 at 09:10:55AM +0100, Justin Skists wrote: > > @@ -787,7 +787,7 @@ static ssize_t message_store_helper(const char *buf, > > size_t count, > > continue; > > } > > > > - index = simple_strtoul(cp, , 10); > > + index =

Re: [PATCH 3/3] drivers/speakup: Fix style and coding warnings

2018-07-05 Thread Dan Carpenter
On Wed, Jul 04, 2018 at 10:46:05PM +0300, Tamir Suliman wrote: > > Interesting construct...  Yeah I'm little bit rusty on my C /programming  . > I understand  proper coding style may be should end with else so I wasn't > sure .. however this resolved the warnings. :) The code you wrote was

[PATCH v2] staging: wilc1000: fix static checker warning to unlock mutex in wilc_deinit()

2018-07-05 Thread Ajay Singh
Fix for static checker warning inconsistent returns of 'hif_deinit_lock'(more details [1]). "drivers/staging/wilc1000/host_interface.c:3390 wilc_deinit() warn: inconsistent returns 'hif_deinit_lock'." Fixes: ff52a57a7a42 ("staging: wilc1000: move the allocation of cmd out of

Re: [PATCH] staging: wilc1000: fix static checker warning to unlock mutex in wilc_deinit()

2018-07-05 Thread Ajay Singh
Hi Dan, On Thu, 5 Jul 2018 11:41:26 +0300 Dan Carpenter wrote: > On Thu, Jul 05, 2018 at 09:18:53AM +0530, Ajay Singh wrote: > > Fix for static checker warning inconsistent returns of > > 'hif_deinit_lock'(more details [1]). > > > > "drivers/staging/wilc1000/host_interface.c:3390 wilc_deinit()

Re: [PATCH v6 1/4] resource: Move reparent_resources() to kernel/resource.c and make it public

2018-07-05 Thread Baoquan He
On 07/04/18 at 07:46pm, Andy Shevchenko wrote: > On Wed, Jul 4, 2018 at 7:10 AM, Baoquan He wrote: > > reparent_resources() is duplicated in arch/microblaze/pci/pci-common.c > > and arch/powerpc/kernel/pci-common.c, so move it to kernel/resource.c > > so that it's shared. > > With couple of

Re: [BUG] staging: r8822be: RTL8822be can't find any wireless AP

2018-07-05 Thread Jian-Hong Pan
2018-07-05 17:32 GMT+08:00 Dan Carpenter : > On Thu, Jul 05, 2018 at 05:26:28PM +0800, Jian-Hong Pan wrote: >> 2018-07-05 15:36 GMT+08:00 Pkshih : >> > On Wed, 2018-07-04 at 10:33 -0500, Daniel Drake wrote: >> >> On Wed, Jul 4, 2018 at 10:13 AM, Larry Finger >> >> wrote: >> >> > We will have to

Re: [BUG] staging: r8822be: RTL8822be can't find any wireless AP

2018-07-05 Thread Dan Carpenter
On Thu, Jul 05, 2018 at 05:26:28PM +0800, Jian-Hong Pan wrote: > 2018-07-05 15:36 GMT+08:00 Pkshih : > > On Wed, 2018-07-04 at 10:33 -0500, Daniel Drake wrote: > >> On Wed, Jul 4, 2018 at 10:13 AM, Larry Finger > >> wrote: > >> > We will have to agree to disagree. > >> > > >> > I have no idea

Re: [BUG] staging: r8822be: RTL8822be can't find any wireless AP

2018-07-05 Thread Jian-Hong Pan
2018-07-05 15:36 GMT+08:00 Pkshih : > On Wed, 2018-07-04 at 10:33 -0500, Daniel Drake wrote: >> On Wed, Jul 4, 2018 at 10:13 AM, Larry Finger >> wrote: >> > We will have to agree to disagree. >> > >> > I have no idea what the vendors are doing that cause some motherboards to >> > need a

Re: [PATCH] staging: wilc1000: fix static checker warning to unlock mutex in wilc_deinit()

2018-07-05 Thread Dan Carpenter
On Thu, Jul 05, 2018 at 09:18:53AM +0530, Ajay Singh wrote: > Fix for static checker warning inconsistent returns of > 'hif_deinit_lock'(more details [1]). > > "drivers/staging/wilc1000/host_interface.c:3390 wilc_deinit() > warn: inconsistent returns 'hif_deinit_lock'." > >

Re: [BUG] staging: r8822be: RTL8822be can't find any wireless AP

2018-07-05 Thread Pkshih
On Wed, 2018-07-04 at 10:33 -0500, Daniel Drake wrote: > On Wed, Jul 4, 2018 at 10:13 AM, Larry Finger > wrote: > > We will have to agree to disagree. > > > > I have no idea what the vendors are doing that cause some motherboards to > > need a different aspm value. What I do know is that we have

Re: [PATCH 0/4] staging: mt7621-gpio: fix problem using one node in DT

2018-07-05 Thread Sergio Paracuellos
On Thu, Jul 05, 2018 at 04:27:31PM +1000, NeilBrown wrote: > On Thu, Jul 05 2018, Sergio Paracuellos wrote: > > > This series fix problem related with the last changes included > > to use only one node in the device tree and some gpio banks > > naming issues. > > > > Hope this helps. > > It sure

Re: [PATCH 0/4] staging: mt7621-gpio: fix problem using one node in DT

2018-07-05 Thread NeilBrown
On Thu, Jul 05 2018, Sergio Paracuellos wrote: > This series fix problem related with the last changes included > to use only one node in the device tree and some gpio banks > naming issues. > > Hope this helps. It sure does. Everything looks and works as expected now. all: Reviewed-by:

[PATCH 4/4] staging: mt7621-gpio: use devm_kasprintf to set gpio banks labels

2018-07-05 Thread Sergio Paracuellos
Instead of using a custom function to return desired name for gpio use the default assigned one and concat it '-bankN' suffix using devm_kasprintf kernel function. Signed-off-by: Sergio Paracuellos --- drivers/staging/mt7621-gpio/gpio-mt7621.c | 12 ++-- 1 file changed, 2 insertions(+),

[PATCH 2/4] staging: mt7621-gpio: use custom xlate function

2018-07-05 Thread Sergio Paracuellos
The default gpio.of_xlate function assumes there is one gpio chip for each devicetree node. Device tree had changed to only use one node, which corresponds to 3 different gpio chips now. For that approach to work we need a custom xlate function. Signed-off-by: Sergio Paracuellos ---

[PATCH 1/4] staging: mt7621-gpio: set irq chip name only once

2018-07-05 Thread Sergio Paracuellos
There is only one irq chip so set its name only once in driver probe function. Signed-off-by: Sergio Paracuellos --- drivers/staging/mt7621-gpio/gpio-mt7621.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/mt7621-gpio/gpio-mt7621.c

[PATCH 3/4] staging: mt7621-gpio: assign gpio chip custom changes after bgpio_init

2018-07-05 Thread Sergio Paracuellos
bgpio_init function set different data of the gpio chip, like the name. We want specific name for each bank so to get that not overwritten move all custom changes after the bgpio_init function call. Signed-off-by: Sergio Paracuellos --- drivers/staging/mt7621-gpio/gpio-mt7621.c | 7 --- 1

[PATCH 0/4] staging: mt7621-gpio: fix problem using one node in DT

2018-07-05 Thread Sergio Paracuellos
This series fix problem related with the last changes included to use only one node in the device tree and some gpio banks naming issues. Hope this helps. Best regards, Sergio Paracuellos Sergio Paracuellos (4): staging: mt7621-gpio: set irq chip name only once staging: mt7621-gpio: use