[driver-core:debugfs_cleanup] BUILD SUCCESS f7dc69c6674f761485cbd5574c35a7769a1ad31d

2020-08-18 Thread kernel test robot
allnoconfig i386 randconfig-a005-20200818 i386 randconfig-a002-20200818 i386 randconfig-a001-20200818 i386 randconfig-a006-20200818 i386 randconfig-a003-20200818 i386 randconfig-a004

[driver-core:readfile] BUILD SUCCESS d4b51a26f12c944f70a685fbadca460e24b8cdd6

2020-08-18 Thread kernel test robot
allnoconfig i386 randconfig-a005-20200818 i386 randconfig-a002-20200818 i386 randconfig-a001-20200818 i386 randconfig-a006-20200818 i386 randconfig-a003-20200818 i386 randconfig-a004-20200818 x86_64

[staging:staging-linus] BUILD SUCCESS 1dffeb8b8b4c261c45416d53c75ea51e6ece1770

2020-08-18 Thread kernel test robot
allnoconfig i386 randconfig-a005-20200818 i386 randconfig-a002-20200818 i386 randconfig-a001-20200818 i386 randconfig-a006-20200818 i386 randconfig-a003-20200818 i386 randconfig-a004-20200818 x86_64

Re: [PATCH 00/16] IOMMU driver for Kirin 960/970

2020-08-18 Thread John Stultz
On Tue, Aug 18, 2020 at 9:26 AM Robin Murphy wrote: > On 2020-08-18 16:29, Mauro Carvalho Chehab wrote: > > Em Tue, 18 Aug 2020 15:47:55 +0100 > > Basically, the DT binding has this, for IOMMU: > > > > > > smmu_lpae { > > compatible = "hisilicon,smmu-lpae"; > > }; > > > >

Re: [PATCH] block: convert tasklets to use new tasklet_setup() API

2020-08-18 Thread James Bottomley
On Tue, 2020-08-18 at 13:10 -0700, Kees Cook wrote: > On Tue, Aug 18, 2020 at 01:00:33PM -0700, James Bottomley wrote: > > On Mon, 2020-08-17 at 13:02 -0700, Jens Axboe wrote: > > > On 8/17/20 12:48 PM, Kees Cook wrote: > > > > On Mon, Aug 17, 2020 at 12:44:34PM -0700, Jens Axboe wrote: > > > > > O

Re: [PATCH] block: convert tasklets to use new tasklet_setup() API

2020-08-18 Thread Kees Cook
On Tue, Aug 18, 2020 at 01:00:33PM -0700, James Bottomley wrote: > On Mon, 2020-08-17 at 13:02 -0700, Jens Axboe wrote: > > On 8/17/20 12:48 PM, Kees Cook wrote: > > > On Mon, Aug 17, 2020 at 12:44:34PM -0700, Jens Axboe wrote: > > > > On 8/17/20 12:29 PM, Kees Cook wrote: > > > > > On Mon, Aug 17,

Re: [PATCH] block: convert tasklets to use new tasklet_setup() API

2020-08-18 Thread James Bottomley
On Mon, 2020-08-17 at 13:02 -0700, Jens Axboe wrote: > On 8/17/20 12:48 PM, Kees Cook wrote: > > On Mon, Aug 17, 2020 at 12:44:34PM -0700, Jens Axboe wrote: > > > On 8/17/20 12:29 PM, Kees Cook wrote: > > > > On Mon, Aug 17, 2020 at 06:56:47AM -0700, Jens Axboe wrote: > > > > > On 8/17/20 2:15 AM,

Re: [PATCH 1/6] staging: hikey9xx: fix Kconfig dependency chain

2020-08-18 Thread Greg Kroah-Hartman
On Tue, Aug 18, 2020 at 06:07:16PM +0200, Mauro Carvalho Chehab wrote: > Em Tue, 18 Aug 2020 17:07:04 +0200 > Greg Kroah-Hartman escreveu: > > > On Tue, Aug 18, 2020 at 04:58:53PM +0200, Mauro Carvalho Chehab wrote: > > > Both the SPMI controller and the SPMI PMIC driver > > > depends on the SPMI

Re: [PATCH] staging: kpc2000: kpc_dma: fix spelling mistake "for for" -> "for"

2020-08-18 Thread Joe Perches
On Tue, 2020-08-18 at 17:46 +0100, Colin King wrote: > There are a couple of duplicated "for" spelling mistakes in dev_err > error messages. Fix these. Might as well remove the messages instead. > diff --git a/drivers/staging/kpc2000/kpc_dma/fileops.c > b/drivers/staging/kpc2000/kpc_dma/fileops.

[PATCH] staging: kpc2000: kpc_dma: fix spelling mistake "for for" -> "for"

2020-08-18 Thread Colin King
From: Colin Ian King There are a couple of duplicated "for" spelling mistakes in dev_err error messages. Fix these. Signed-off-by: Colin Ian King --- drivers/staging/kpc2000/kpc_dma/fileops.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/kpc2000/kpc_dm

Re: [PATCH 00/16] IOMMU driver for Kirin 960/970

2020-08-18 Thread Robin Murphy
On 2020-08-18 16:29, Mauro Carvalho Chehab wrote: Hi Robin, Em Tue, 18 Aug 2020 15:47:55 +0100 Robin Murphy escreveu: On 2020-08-17 08:49, Mauro Carvalho Chehab wrote: Add a driver for the Kirin 960/970 iommu. As on the past series, this starts from the original 4.9 driver from the 96boards

Re: [PATCH 1/6] staging: hikey9xx: fix Kconfig dependency chain

2020-08-18 Thread Mauro Carvalho Chehab
Em Tue, 18 Aug 2020 17:07:04 +0200 Greg Kroah-Hartman escreveu: > On Tue, Aug 18, 2020 at 04:58:53PM +0200, Mauro Carvalho Chehab wrote: > > Both the SPMI controller and the SPMI PMIC driver > > depends on the SPMI bus support. > > > > The dependency for the regulator is also wrong: > > it shoul

[PATCH] staging: hikey9xx: Kconfig: add regulator dependency

2020-08-18 Thread Mauro Carvalho Chehab
The regulator driver needs it, as otherwise it will produce errors when creating vmlinux. Reported-by: Greg Kroah-Hartman Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/hikey9xx/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/hikey9xx/Kconfig b/drivers/stag

Re: [PATCH 00/16] IOMMU driver for Kirin 960/970

2020-08-18 Thread Mauro Carvalho Chehab
Hi Robin, Em Tue, 18 Aug 2020 15:47:55 +0100 Robin Murphy escreveu: > On 2020-08-17 08:49, Mauro Carvalho Chehab wrote: > > Add a driver for the Kirin 960/970 iommu. > > > > As on the past series, this starts from the original 4.9 driver from > > the 96boards tree: > > > > https://github.c

[PATCH] staging: emxx_udc: Allow for building on !ARM

2020-08-18 Thread Alex Dewar
Currently the module can only be test built on ARM, although it seems to build fine on x86. Change this to allow for broader test coverage. Signed-off-by: Alex Dewar --- drivers/staging/emxx_udc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/emxx_udc/

Re: [PATCH 1/6] staging: hikey9xx: fix Kconfig dependency chain

2020-08-18 Thread Greg Kroah-Hartman
On Tue, Aug 18, 2020 at 04:58:53PM +0200, Mauro Carvalho Chehab wrote: > Both the SPMI controller and the SPMI PMIC driver > depends on the SPMI bus support. > > The dependency for the regulator is also wrong: > it should depends on the SPMI version of the HiSilicon 6421, > and not on the normal o

Re: [PATCH] staging: emxx_udc: Use standard BIT() macro

2020-08-18 Thread Alex Dewar
On Tue, Aug 18, 2020 at 07:29:02AM -0700, Randy Dunlap wrote: > On 8/18/20 6:49 AM, Alex Dewar wrote: > > Currently emxx_udc.h defines bit values using local macros. Use the > > standard one instead. > > > > Also, combine bit values with bitwise-or rather than addition, as > > suggested by Coccine

[PATCH 5/6] dt: document HiSilicon SPMI controller and mfd/regulator properties

2020-08-18 Thread Mauro Carvalho Chehab
Add documentation for the properties needed by the HiSilicon 6421v600 driver, and by the SPMI controller used to access the chipset. Signed-off-by: Mauro Carvalho Chehab --- .../hikey9xx/hisilicon,hi6421-spmi-pmic.yaml | 159 ++ .../hisilicon,hisi-spmi-controller.yaml | 6

[PATCH 0/6] Do some cleanups at the HiSilicon 6421 regulator

2020-08-18 Thread Mauro Carvalho Chehab
Hi Greg, Patch 1 contains a Kconfig fixup, addressing the problem you noticed when building those drivers; Patches 2 to 4 addresses some issues at the device tree, pointed by Rob's review. Patch 5 adds the DT documentation. As requested, I moved those also to the staging dir. Patch 6 adds an

[PATCH 4/6] staging: mfd: hi6421-spmi-pmic: Simplify the compatible string

2020-08-18 Thread Mauro Carvalho Chehab
It is clear that this driver is for PMIC. So, get rid of it at the compatible. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/hikey9xx/hi6421-spmi-pmic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/hikey9xx/hi6421-spmi-pmic.c b/drivers/staging/h

[PATCH 3/6] staging: spmi: hisi-spmi-controller: change compatible string

2020-08-18 Thread Mauro Carvalho Chehab
Add the chipset name at the compatible string, as other HiSilicon chipsets with SPMI bus might require something different. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/hikey9xx/hisi-spmi-controller.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/stagi

[PATCH 2/6] staging: mfd: hi6421-spmi-pmic: get rid of interrupt properties

2020-08-18 Thread Mauro Carvalho Chehab
Both irqnum and irqarray properties reflect the same thing: the number of bits and bytes for interrupts at this chipset. E. g.: irqnum = 8 x irqarray This can be seen by the way pending interrupts are handled: /* During probe time */ pmic->irqs = devm_kzalloc(dev, pmic->i

[PATCH 1/6] staging: hikey9xx: fix Kconfig dependency chain

2020-08-18 Thread Mauro Carvalho Chehab
Both the SPMI controller and the SPMI PMIC driver depends on the SPMI bus support. The dependency for the regulator is also wrong: it should depends on the SPMI version of the HiSilicon 6421, and not on the normal one. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/hikey9xx/Kconfig |

Re: [PATCH 00/16] IOMMU driver for Kirin 960/970

2020-08-18 Thread Robin Murphy
On 2020-08-17 08:49, Mauro Carvalho Chehab wrote: Add a driver for the Kirin 960/970 iommu. As on the past series, this starts from the original 4.9 driver from the 96boards tree: https://github.com/96boards-hikey/linux/tree/hikey970-v4.9 The remaining patches add SPDX headers and make

Re: [PATCH] staging: emxx_udc: Use standard BIT() macro

2020-08-18 Thread Randy Dunlap
On 8/18/20 6:49 AM, Alex Dewar wrote: > Currently emxx_udc.h defines bit values using local macros. Use the > standard one instead. > > Also, combine bit values with bitwise-or rather than addition, as > suggested by Coccinelle. > > Signed-off-by: Alex Dewar Hi, Does this build? Just checking

Re: [PATCH v3 00/44] SPMI patches needed by Hikey 970

2020-08-18 Thread Mauro Carvalho Chehab
Em Tue, 18 Aug 2020 16:17:50 +0200 Greg Kroah-Hartman escreveu: > On Mon, Aug 17, 2020 at 09:10:19AM +0200, Mauro Carvalho Chehab wrote: > > Hi Greg, > > > > This patch series is part of a work I'm doing in order to be able to support > > a HiKey 970 board that I recently got on my hands. > >

Re: [PATCH v2 1/4] staging: android: Replace BUG_ON with WARN_ON

2020-08-18 Thread Greg Kroah-Hartman
On Tue, Aug 18, 2020 at 05:19:40PM +0300, Tomer Samara wrote: > On Tue, Aug 18, 2020 at 04:11:06PM +0200, Greg Kroah-Hartman wrote: > > On Sun, Aug 16, 2020 at 10:23:25PM +0300, Tomer Samara wrote: > > > BUG_ON() is replaced with WARN_ON at ion_page_pool.c > > > > Why? > > > > > Fixes the followi

Re: [PATCH v2 1/4] staging: android: Replace BUG_ON with WARN_ON

2020-08-18 Thread Tomer Samara
On Tue, Aug 18, 2020 at 04:11:06PM +0200, Greg Kroah-Hartman wrote: > On Sun, Aug 16, 2020 at 10:23:25PM +0300, Tomer Samara wrote: > > BUG_ON() is replaced with WARN_ON at ion_page_pool.c > > Why? > > > Fixes the following issue: > > Avoid crashing the kernel - try using WARN_ON & recovery code

Re: [PATCH v3 00/44] SPMI patches needed by Hikey 970

2020-08-18 Thread Greg Kroah-Hartman
On Mon, Aug 17, 2020 at 09:10:19AM +0200, Mauro Carvalho Chehab wrote: > Hi Greg, > > This patch series is part of a work I'm doing in order to be able to support > a HiKey 970 board that I recently got on my hands. With this applied, I get the following build error: ERROR: modpost: "__spmi_drive

Re: [PATCH v2 4/4] staging: android: Add error handling to order_to_index callers

2020-08-18 Thread Greg Kroah-Hartman
On Sun, Aug 16, 2020 at 10:31:22PM +0300, Tomer Samara wrote: > Add error check to: > - free_buffer_page > - alloc_buffer_page > after calling order_to_index, due to converting BUG to WARN at > order_to_index. You are fixing a bug you caused in a previous patch, not good :) thanks, greg k-h

Re: [PATCH v2 3/4] staging: android: Convert BUG to WARN

2020-08-18 Thread Greg Kroah-Hartman
On Sun, Aug 16, 2020 at 10:30:10PM +0300, Tomer Samara wrote: > replace BUG() with WARN() at ion_sytem_heap.c, this > fix the following checkpatch issue: > Avoid crashing the kernel - try using WARN_ON & > recovery code ratherthan BUG() or BUG_ON(). > > Signed-off-by: Tomer Samara > --- > driver

Re: [PATCH v2 1/4] staging: android: Replace BUG_ON with WARN_ON

2020-08-18 Thread Greg Kroah-Hartman
On Sun, Aug 16, 2020 at 10:23:25PM +0300, Tomer Samara wrote: > BUG_ON() is replaced with WARN_ON at ion_page_pool.c Why? > Fixes the following issue: > Avoid crashing the kernel - try using WARN_ON & recovery code ratherthan > BUG() or BUG_ON(). Ideally you can get rid of WARN_ON() too, right?

Re: [PATCH v2 2/4] staging: android: Add error handling to ion_page_pool_shrink

2020-08-18 Thread Greg Kroah-Hartman
On Sun, Aug 16, 2020 at 10:24:22PM +0300, Tomer Samara wrote: > Add error check to ion_page_pool_shrink after calling > ion_page_pool_remove, due to converting BUG_ON to WARN_ON. > > Signed-off-by: Tomer Samara So this fixes a previous patch? That's not good, please merge them together so you d

Re: [PATCH] staging: rtl8723bs: os_dep: fix function-name print using __func__

2020-08-18 Thread Greg KH
On Wed, Aug 12, 2020 at 09:37:45PM +0530, Mohammed Rushad wrote: > This patch to the os_intfs.c fixes the printing of function names using > the preferred '"%s...", __func__' and alignment issues as pointed out by > the checkpatch.pl tool. > > Signed-off-by: Mohammed Rushad > --- > drivers/stagi

Re: [PATCH] staging: rtl8723bs: os_dep: fix coding style issue in xmit_linux.c

2020-08-18 Thread Greg KH
On Mon, Aug 03, 2020 at 12:26:44AM +0530, Mohammed Rushad wrote: > This is a patch to the xmit_linux.c file that fixes brace and missing > line warning found by checkpatch.pl tool > > Signed-off-by: Mohammed Rushad > --- > drivers/staging/rtl8723bs/os_dep/xmit_linux.c | 4 ++-- > 1 file changed,

Re: [PATCH v2] staging: wfx: refactor to avoid duplication at hif_tx.c

2020-08-18 Thread Greg KH
On Mon, Aug 10, 2020 at 11:38:33AM +0200, Jérôme Pouiller wrote: > Hello Tomer, > > On Wednesday 5 August 2020 14:14:42 CEST Tomer Samara wrote: > > > > Add functions wfx_full_send(), wfx_full_send_no_reply_async(), > > wfx_full_send_no_reply() and wfx_full_send_no_reply_free() > > which works as

Re: [PATCH] binder: Modify comments

2020-08-18 Thread Martijn Coenen
On Tue, Aug 18, 2020 at 3:34 AM hui yang wrote: > > From: YangHui > > The function name should is binder_alloc_new_buf() > Reviewed-by: Martijn Coenen > Signed-off-by: YangHui > --- > drivers/android/binder_alloc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drive

Re: [PATCH v2] staging: wfx: fixed misspelled word in comment

2020-08-18 Thread Greg KH
On Tue, Aug 04, 2020 at 06:17:47PM +0530, Aditya Bansal wrote: > From: Aditya Bansal > > Subject: [PATCH v2] fixed typo in driver/staging/wfx/hif_tx.c file > > Correct the spelling of word function and careful > > Signed-off-by: Aditya Bansal > --- > > diff --git a/drivers/staging/wfx/hif_tx.

[PATCH] staging: emxx_udc: Use standard BIT() macro

2020-08-18 Thread Alex Dewar
Currently emxx_udc.h defines bit values using local macros. Use the standard one instead. Also, combine bit values with bitwise-or rather than addition, as suggested by Coccinelle. Signed-off-by: Alex Dewar --- drivers/staging/emxx_udc/emxx_udc.h | 456 +--- 1 file chang

Re: [PATCH] char: ipmi: convert tasklets to use new tasklet_setup() API

2020-08-18 Thread Corey Minyard
On Tue, Aug 18, 2020 at 02:46:23PM +0530, Allen wrote: > > > > > > Signed-off-by: Romain Perier > > > Signed-off-by: Allen Pais > > > > This looks good to me. > > > > Reviewed-by: Corey Minyard > > > > Are you planning to push this, or do you want me to take it? If you > > want me to take it, w

Re: [PATCH -next] binderfs: make symbol 'binderfs_fs_parameters' static

2020-08-18 Thread Christian Brauner
On Tue, Aug 18, 2020 at 07:22:45PM +0800, Wei Yongjun wrote: > The sparse tool complains as follows: > > drivers/android/binderfs.c:66:32: warning: > symbol 'binderfs_fs_parameters' was not declared. Should it be static? > > This variable is not used outside of binderfs.c, so this commit > marks

[PATCH -next] binderfs: make symbol 'binderfs_fs_parameters' static

2020-08-18 Thread Wei Yongjun
The sparse tool complains as follows: drivers/android/binderfs.c:66:32: warning: symbol 'binderfs_fs_parameters' was not declared. Should it be static? This variable is not used outside of binderfs.c, so this commit marks it static. Fixes: 095cf502b31e ("binderfs: port to new mount api") Report

Re: [PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
On Tue, Aug 18, 2020 at 11:50:35AM +0200, Greg Kroah-Hartman wrote: > On Tue, Aug 18, 2020 at 12:17:08PM +0300, Tomer Samara wrote: > > *** BLURB HERE *** > > Really? > > And your subject line could use some work too :( > sorry for that, i've made a script for sending patchset and accidently it

Re: [PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Greg Kroah-Hartman
On Tue, Aug 18, 2020 at 12:17:08PM +0300, Tomer Samara wrote: > *** BLURB HERE *** Really? And your subject line could use some work too :( ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/dri

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers drivers/staging/android/ion/ion_page_pool.c

Re: [PATCH] char: ipmi: convert tasklets to use new tasklet_setup() API

2020-08-18 Thread Allen
> > > > Signed-off-by: Romain Perier > > Signed-off-by: Allen Pais > > This looks good to me. > > Reviewed-by: Corey Minyard > > Are you planning to push this, or do you want me to take it? If you > want me to take it, what is the urgency? Thanks. Well, not hurry, as long as it goes into 5.9

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers drivers/staging/android/ion/ion_page_pool.c

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers drivers/staging/android/ion/ion_page_pool.c

[PATCH 0/4] *** SUBJECT HERE ***

2020-08-18 Thread Tomer Samara
*** BLURB HERE *** Tomer Samara (4): staging: android: Replace BUG_ON with WARN_ON staging: android: Add error handling to ion_page_pool_shrink staging: android: Convert BUG to WARN staging: android: Add error handling to order_to_index callers drivers/staging/android/ion/ion_page_pool.c

Re: [PATCH RESEND v10 07/11] device-mapping: Introduce DMA range map, supplanting dma_pfn_offset

2020-08-18 Thread Andy Shevchenko
On Mon, Aug 17, 2020 at 05:53:09PM -0400, Jim Quinlan wrote: > The new field 'dma_range_map' in struct device is used to facilitate the > use of single or multiple offsets between mapping regions of cpu addrs and > dma addrs. It subsumes the role of "dev->dma_pfn_offset" which was only > capable o