[PATCH net-next] net/rds: Add missing include file

2019-10-06 Thread YueHaibing
Fix build error: net/rds/ib_cm.c: In function rds_dma_hdrs_alloc: net/rds/ib_cm.c:475:13: error: implicit declaration of function dma_pool_zalloc; did you mean mempool_alloc? [-Werror=implicit-function-declaration] hdrs[i] = dma_pool_zalloc(pool, GFP_KERNEL, &hdr_daddrs[i]); ^~~~

Re: [PATCH 1/2] arm64: dts: imx8mn: Add system counter node

2019-10-06 Thread Shawn Guo
On Tue, Sep 10, 2019 at 11:25:17AM -0400, Anson Huang wrote: > Add i.MX8MN system counter node to enable timer-imx-sysctr > broadcast timer driver. > > Signed-off-by: Anson Huang Applied both, thanks.

[PATCH -next] ASoc: tas2770: Fix build error without GPIOLIB

2019-10-06 Thread YueHaibing
If GPIOLIB is not set, building fails: sound/soc/codecs/tas2770.c: In function tas2770_reset: sound/soc/codecs/tas2770.c:38:3: error: implicit declaration of function gpiod_set_value_cansleep; did you mean gpio_set_value_cansleep? [-Werror=implicit-function-declaration] gpiod_set_value_cansle

[PATCH -next] media: cx231xx: remove duplicated include from cx231xx-417.c

2019-10-06 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/media/usb/cx231xx/cx231xx-417.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/usb/cx231xx/cx231xx-417.c b/drivers/media/usb/cx231xx/cx231xx-417.c index 46d0215deee6..1aec4459f50a 100644 --- a/drivers/media/usb/c

Re: [PATCH 1/2] arm64: dts: imx8mm: Remove incorrect fallback compatible for ocotp

2019-10-06 Thread Shawn Guo
On Wed, Sep 11, 2019 at 10:24:46AM -0400, Anson Huang wrote: > Compared to i.MX7D, i.MX8MM has different ocotp layout, so it should > NOT use "fsl,imx7d-ocotp" as ocotp's fallback compatible, remove it. > > Signed-off-by: Anson Huang Applied both, thanks.

Re: [PATCH v4 1/4] x86/kvm: Add "nopvspin" parameter to disable PV spinlocks

2019-10-06 Thread Zhenzhong Duan
On 2019/10/4 22:52, Boris Ostrovsky wrote: On 10/3/19 10:02 AM, Zhenzhong Duan wrote: void __init kvm_spinlock_init(void) { - /* Does host kernel support KVM_FEATURE_PV_UNHALT? */ - if (!kvm_para_has_feature(KVM_FEATURE_PV_UNHALT)) - return; - - if (kvm_para_

Re: [PATCH v4 3/4] xen: Mark "xen_nopvspin" parameter obsolete

2019-10-06 Thread Zhenzhong Duan
On 2019/10/4 22:57, Boris Ostrovsky wrote: On 10/3/19 10:02 AM, Zhenzhong Duan wrote: Map "xen_nopvspin" to "nopvspin", fix stale description of "xen_nopvspin" as we use qspinlock now. Signed-off-by: Zhenzhong Duan Cc: Jonathan Corbet Cc: Boris Ostrovsky Cc: Juergen Gross Cc: Stefano Stab

Re: [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings

2019-10-06 Thread Shawn Guo
On Thu, Sep 12, 2019 at 10:56:31AM +0800, Anson Huang wrote: > The 800MHz opp speed grading fuse mask should be 0xd instead > of 0xf according to fuse map definition: > > SPEED_GRADING[1:0]MHz > 00 800 > 01 500 > 10 1000 > 11

Re: [PATCH v5 01/10] mm/memunmap: Use the correct start and end pfn when removing pages from zone

2019-10-06 Thread David Hildenbrand
On 05.10.19 08:13, Aneesh Kumar K.V wrote: > On 10/4/19 2:33 PM, David Hildenbrand wrote: >> On 04.10.19 11:00, David Hildenbrand wrote: >>> On 03.10.19 18:48, Aneesh Kumar K.V wrote: On 10/1/19 8:33 PM, David Hildenbrand wrote: > On 01.10.19 16:57, David Hildenbrand wrote: >> On 01.10

Re: [PATCH] rsi: fix potential null dereference in rsi_probe()

2019-10-06 Thread Kalle Valo
Johan Hovold writes: > On Fri, Oct 04, 2019 at 01:47:36PM +, Kalle Valo wrote: >> Denis Efremov wrote: >> >> > The id pointer can be NULL in rsi_probe(). > > While the existing code in rsi_probe() may lead you to believe that, > this statement is false. > >> > It is checked everywhere exce

Re: [PATCH 1/2] Revert "rsi: fix potential null dereference in rsi_probe()"

2019-10-06 Thread Kalle Valo
Johan Hovold writes: > This reverts commit f170d44bc4ec2feae5f6206980e7ae7fbf0432a0. > > USB core will never call a USB-driver probe function with a NULL > device-id pointer. > > Reverting before removing the existing checks in order to document this > and prevent the offending commit from being

Re: [PATCH] ARM: dts: imx6: Extend support for Phytec phycore i.MX6ULL SoM

2019-10-06 Thread Shawn Guo
On Thu, Sep 12, 2019 at 10:29:28PM +0200, Parthiban Nallathambi wrote: > Extend Phycore i.MX6UL SoM for i.MX6ULL with on board eMMC. Phycore > i.MX6ULL is deployed with same carrier board Segin as the pins are > compatible with UL version. > > Signed-off-by: Parthiban Nallathambi What is the bas

Re: [PATCH] ARM: dts: imx7s: Correct GPT's ipg clock source

2019-10-06 Thread Shawn Guo
On Mon, Sep 16, 2019 at 04:29:09PM +0800, Anson Huang wrote: > i.MX7S/D's GPT ipg clock should be from GPT clock root and > controlled by CCM's GPT CCGR, using correct clock source for > GPT ipg clock instead of IMX7D_CLK_DUMMY. So both 'ipg' and 'per' clock are coming from GPT root clock? > Sign

Re: [PATCH 1/1] iio: (bma400) add driver for the BMA400

2019-10-06 Thread Jonathan Cameron
On Wed, 25 Sep 2019 01:39:41 + Dan Robertson wrote: > Add a IIO driver for the Bosch BMA400 3-axes ultra-low power accelerometer. > The driver supports reading from the acceleration and temperature > registers. The driver also supports reading and configuring the output data > rate, oversampl

Re: [PATCH] lib: test_user_copy: style cleanup

2019-10-06 Thread Christian Brauner
On Sun, Oct 06, 2019 at 10:30:28AM +1100, Aleksa Sarai wrote: > While writing the tests for copy_struct_from_user(), I used a construct > that Linus doesn't appear to be too fond of: > > On 2019-10-04, Linus Torvalds wrote: > > Hmm. That code is ugly, both before and after the fix. > > > > This j

Re: [PATCH v3 0/2] Simplify count_read/count_write/signal_read

2019-10-06 Thread Jonathan Cameron
On Sat, 5 Oct 2019 13:19:38 -0400 William Breathitt Gray wrote: > On Sat, Oct 05, 2019 at 03:33:08PM +0100, Jonathan Cameron wrote: > > Hi William, > > > > This all makes sense to me. Do you want to wait for some more reviews > > or should I pick them up now through IIO? We are really early in

Re: [PATCH] counter: stm32: clean up indentation issue

2019-10-06 Thread Jonathan Cameron
On Sat, 5 Oct 2019 13:30:04 -0400 William Breathitt Gray wrote: > On Wed, Sep 25, 2019 at 10:51:26AM +0100, Colin King wrote: > > From: Colin Ian King > > > > There is an if statement that is indented one level too deeply, > > remove the extraneous tabs. > > > > Signed-off-by: Colin Ian King

Re: [PATCH] iio: gyro: clean up indentation issue

2019-10-06 Thread Jonathan Cameron
On Wed, 25 Sep 2019 13:32:53 +0100 Colin King wrote: > From: Colin Ian King > > There is a return statement that is indented incorrectly, add in > the missing tab. > > Signed-off-by: Colin Ian King Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to ignor

Re: [PATCH] iio: imu: adis16480: clean up a condition

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:36:30 +0300 Dan Carpenter wrote: > On Thu, Sep 26, 2019 at 11:06:39AM +, Ardelean, Alexandru wrote: > > On Thu, 2019-09-26 at 11:10 +0300, Dan Carpenter wrote: > > > [External] > > > > > > The "t" variable is unsigned so it can't be less than zero. We really > > > a

Re: [PATCH 01/10] iio: imu: adis: rename txrx_lock -> state_lock

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:03 +0300 Alexandru Ardelean wrote: > The lock can be extended a bit to protect other elements that are not > particular to just TX/RX. Another idea would have been to just add a new > `state_lock`, but that would mean 2 locks which would be redundant, and > probably caus

[PATCH v6 04/10] mm/memory_hotplug: Don't access uninitialized memmaps in shrink_zone_span()

2019-10-06 Thread David Hildenbrand
Let's limit shrinking to !ZONE_DEVICE so we can fix the current code. We should never try to touch the memmap of offline sections where we could have uninitialized memmaps and could trigger BUGs when calling page_to_nid() on poisoned pages. There is no reliable way to distinguish an uninitialized

[PATCH v6 02/10] mm/memmap_init: Update variable name in memmap_init_zone

2019-10-06 Thread David Hildenbrand
From: "Aneesh Kumar K.V" The third argument is actually number of pages. Changes the variable name from size to nr_pages to indicate this better. No functional change in this patch. Cc: Andrew Morton Cc: Michal Hocko Cc: Vlastimil Babka Cc: Oscar Salvador Cc: Mel Gorman Cc: Mike Rapoport

[PATCH v6 03/10] mm/memory_hotplug: Don't access uninitialized memmaps in shrink_pgdat_span()

2019-10-06 Thread David Hildenbrand
We might use the nid of memmaps that were never initialized. For example, if the memmap was poisoned, we will crash the kernel in pfn_to_nid() right now. Let's use the calculated boundaries of the separate zones instead. This now also avoids having to iterate over a whole bunch of subsections again

[PATCH v6 01/10] mm/memunmap: Don't access uninitialized memmap in memunmap_pages()

2019-10-06 Thread David Hildenbrand
From: "Aneesh Kumar K.V" With an altmap, the memmap falling into the reserved altmap space are not initialized and, therefore, contain a garbage NID and a garbage zone. Make sure to read the NID/zone from a memmap that was initialzed. This fixes a kernel crash that is observed when destroying a

[PATCH v6 06/10] mm/memory_hotplug: Poison memmap in remove_pfn_range_from_zone()

2019-10-06 Thread David Hildenbrand
Let's poison the pages similar to when adding new memory in sparse_add_section(). Also call remove_pfn_range_from_zone() from memunmap_pages(), so we can poison the memmap from there as well. While at it, calculate the pfn in memunmap_pages() only once. Cc: Andrew Morton Cc: David Hildenbrand C

[PATCH v6 09/10] mm/memory_hotplug: Drop local variables in shrink_zone_span()

2019-10-06 Thread David Hildenbrand
Get rid of the unnecessary local variables. Cc: Andrew Morton Cc: Oscar Salvador Cc: David Hildenbrand Cc: Michal Hocko Cc: Pavel Tatashin Cc: Dan Williams Cc: Wei Yang Signed-off-by: David Hildenbrand --- mm/memory_hotplug.c | 15 ++- 1 file changed, 6 insertions(+), 9 deleti

[PATCH v6 07/10] mm/memory_hotplug: We always have a zone in find_(smallest|biggest)_section_pfn

2019-10-06 Thread David Hildenbrand
With shrink_pgdat_span() out of the way, we now always have a valid zone. Cc: Andrew Morton Cc: Oscar Salvador Cc: David Hildenbrand Cc: Michal Hocko Cc: Pavel Tatashin Cc: Dan Williams Cc: Wei Yang Signed-off-by: David Hildenbrand --- mm/memory_hotplug.c | 4 ++-- 1 file changed, 2 inser

[PATCH v6 08/10] mm/memory_hotplug: Don't check for "all holes" in shrink_zone_span()

2019-10-06 Thread David Hildenbrand
If we have holes, the holes will automatically get detected and removed once we remove the next bigger/smaller section. The extra checks can go. Cc: Andrew Morton Cc: Oscar Salvador Cc: Michal Hocko Cc: David Hildenbrand Cc: Pavel Tatashin Cc: Dan Williams Cc: Wei Yang Signed-off-by: David

[PATCH v6 10/10] mm/memory_hotplug: Cleanup __remove_pages()

2019-10-06 Thread David Hildenbrand
Let's drop the basically unused section stuff and simplify. Also, let's use a shorter variant to calculate the number of pages to the next section boundary. Cc: Andrew Morton Cc: Oscar Salvador Cc: Michal Hocko Cc: Pavel Tatashin Cc: Dan Williams Cc: Wei Yang Signed-off-by: David Hildenbran

Re: [PATCH 01/10] iio: imu: adis: rename txrx_lock -> state_lock

2019-10-06 Thread Jonathan Cameron
On Sun, 6 Oct 2019 09:53:33 +0100 Jonathan Cameron wrote: > On Thu, 26 Sep 2019 14:18:03 +0300 > Alexandru Ardelean wrote: > > > The lock can be extended a bit to protect other elements that are not > > particular to just TX/RX. Another idea would have been to just add a new > > `state_lock`, b

Re: [PATCH 02/10] iio: imu: adis: add unlocked read/write function versions

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:04 +0300 Alexandru Ardelean wrote: > This will allow more flexible control to group reads & writes into a single > lock (particularly the state_lock). > > The end-goal is to remove the indio_dev->mlock usage, and the simplest fix > would have been to just add another lo

Re: [PATCH 03/10] iio: imu: adis[16480]: group RW into a single lock in adis_enable_irq()

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:05 +0300 Alexandru Ardelean wrote: > The adis_enable_irq() does a read & a write. This change keeps a lock for > the duration of both operations vs for each op. > > The change is also needed in adis16480, since that has it's own > implementation for adis_enable_irq(). >

Re: [PATCH 04/10] iio: imu: adis: create an unlocked version of adis_check_status()

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:06 +0300 Alexandru Ardelean wrote: > This one also gets re-used in certain operations, so it makes sense to > have an unlocked version of this to group it with other > reads/writes/operations to have a single lock for the whole state change. > > Signed-off-by: Alexandru

Re: [PATCH 06/10] iio: imu: adis: protect initial startup routine with state lock

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:08 +0300 Alexandru Ardelean wrote: > The initial startup routine is called by some ADIS drivers during probe, > and before registering with IIO. Normally, userspace should not be able to > do any access to the device (as there shouldn't be any available). > > This chang

Linux 4.4.195

2019-10-06 Thread Greg KH
I'm announcing the release of the 4.4.195 kernel. All users of the 4.4 kernel series must upgrade. The updated 4.4.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.4.y and can be browsed at the normal kernel.org git web browser:

Re: [PATCH 08/10] iio: imu: adis16400: rework locks using ADIS library's state lock

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:10 +0300 Alexandru Ardelean wrote: > This change removes the use of indio_dev's mlock in favor using the state > lock from the ADIS library. > > The set_freq() & get_freq() hooks are unlocked, so they require specific > locking. That is because in some cases the get_fre

Linux 4.9.195

2019-10-06 Thread Greg KH
I'm announcing the release of the 4.9.195 kernel. All users of the 4.9 kernel series must upgrade. The updated 4.9.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.9.y and can be browsed at the normal kernel.org git web browser:

Re: [PATCH 07/10] iio: imu: adis: group single conversion under a single state lock

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:09 +0300 Alexandru Ardelean wrote: > Signed-off-by: Alexandru Ardelean Applied. Thanks, J > --- > drivers/iio/imu/adis.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c > index 7468294d1

Linux 4.19.77

2019-10-06 Thread Greg KH
I'm announcing the release of the 4.19.77 kernel. All users of the 4.19 kernel series must upgrade. The updated 4.19.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.19.y and can be browsed at the normal kernel.org git web browser:

Re: Linux 4.4.195

2019-10-06 Thread Greg KH
diff --git a/Makefile b/Makefile index bea8f3f591c4..721fa569a680 100644 --- a/Makefile +++ b/Makefile @@ -1,6 +1,6 @@ VERSION = 4 PATCHLEVEL = 4 -SUBLEVEL = 194 +SUBLEVEL = 195 EXTRAVERSION = NAME = Blurry Fish Butt diff --git a/arch/arm/mach-zynq/platsmp.c b/arch/arm/mach-zynq/platsmp.c ind

Re: Linux 4.9.195

2019-10-06 Thread Greg KH
diff --git a/Makefile b/Makefile index 6e3c81c3bf40..bee0218e3fb5 100644 --- a/Makefile +++ b/Makefile @@ -1,6 +1,6 @@ VERSION = 4 PATCHLEVEL = 9 -SUBLEVEL = 194 +SUBLEVEL = 195 EXTRAVERSION = NAME = Roaring Lionus diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/e

Re: [PATCH v4 1/2] dt-bindings: soc: al-pos: Amazon's Annapurna Labs POS

2019-10-06 Thread Shenhar, Talel
thanks for the review. On 10/4/2019 6:05 PM, Rob Herring wrote: On Thu, Oct 3, 2019 at 6:33 AM Talel Shenhar wrote: Fails 'make dt_binding_check' (drop the '-'): ack, shall be part of v5 + +examples: + - | +al_pos_edac@f0070084 { edac@... ack, shall be part of v5 + compatibl

Re: [PATCH 05/10] iio: imu: adis: create an unlocked version of adis_reset()

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:07 +0300 Alexandru Ardelean wrote: > The reset routine may also be important to be protected by the state-lock > and grouped with other operations, so create an unlocked version, so that > this can be done. > > Signed-off-by: Alexandru Ardelean Applied > --- > drivers

Linux 5.2.19

2019-10-06 Thread Greg KH
I'm announcing the release of the 5.2.19 kernel. All users of the 5.2 kernel series must upgrade. The updated 5.2.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-5.2.y and can be browsed at the normal kernel.org git web browser:

Linux 4.14.147

2019-10-06 Thread Greg KH
I'm announcing the release of the 4.14.147 kernel. All users of the 4.14 kernel series must upgrade. The updated 4.14.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.14.y and can be browsed at the normal kernel.org git web browser

Re: [PATCH 09/10] iio: gyro: adis16136: rework locks using ADIS library's state lock

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:11 +0300 Alexandru Ardelean wrote: > This replaces indio_dev's mlock with the state lock/mutex from the ADIS > library. > > The __adis16136_get_freq() function has been prefixed to mark it as > unlocked. The adis16136_{set,get}_filter() functions now hold the state > lo

Linux 5.3.4

2019-10-06 Thread Greg KH
I'm announcing the release of the 5.3.4 kernel. All users of the 5.3 kernel series must upgrade. The updated 5.3.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-5.3.y and can be browsed at the normal kernel.org git web browser:

Re: [PATCH 10/10] iio: imu: adis16480: use state lock for filter freq set

2019-10-06 Thread Jonathan Cameron
On Thu, 26 Sep 2019 14:18:12 +0300 Alexandru Ardelean wrote: > It's the only operation that does 2 operations (a read & a write), so the > unlocked functions can be used under a single state lock. > > Signed-off-by: Alexandru Ardelean Applied to the togreg branch of iio.git and pushed out as te

Re: [PATCH] iio: adc: sc27xx: Use devm_hwspin_lock_request_specific() to simplify code

2019-10-06 Thread Jonathan Cameron
On Fri, 27 Sep 2019 10:41:19 +0800 Baolin Wang wrote: > Change to use devm_hwspin_lock_request_specific() to help to simplify the > cleanup code for drivers requesting one hwlock. > > Signed-off-by: Baolin Wang Applied to the togreg branch of iio.git and pushed out as testing for the autobuilde

Question about using #ifdef CONFIG_PPC64 in driver code

2019-10-06 Thread Oded Gabbay
Hi Greg, a while ago we had an argument about identifying in my driver's code whether I'm running on x86 or powerpc. I tried to do something dynamically (based on parent pci bridge ID), and you and other people objected to it. I see in other drivers (more then a few) that they are using #ifdef CON

Re: [PATCH 1/4] iio: pressure: bmp280: use bulk regulator ops

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 17:57:30 +0200 Bartosz Golaszewski wrote: > śr., 2 paź 2019 o 15:06 kbuild test robot napisał(a): > > > > Hi Bartosz, > > > > I love your patch! Yet something to improve: > > > > [auto build test ERROR on iio/togreg] > > [cannot apply to v5.4-rc1 next-20191002] > > [if your pa

Re: [PATCH 1/4] iio: pressure: bmp280: use bulk regulator ops

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 10:57:56 +0200 Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > The vddd and vdda supplies are always operated on together. We can > shrink the code a bit by using the bulk regulator helpers. > > Signed-off-by: Bartosz Golaszewski Looks good. Will pick up once I

Re: [PATCH] staging: greybus: add blank line after declarations

2019-10-06 Thread Greg KH
On Sat, Oct 05, 2019 at 06:00:46PM -0300, Gabriela Bittencourt wrote: > Fix CHECK: add blank line after declarations > > Signed-off-by: Gabriela Bittencourt > --- > drivers/staging/greybus/control.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/staging/greybus/control.h > b/

Re: [PATCH 2/4] iio: pressure: bmp280: use devm_iio_device_register()

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 10:57:57 +0200 Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > We can use the managed variant of iio_device_register() and remove > the corresponding unregister operation from the remove callback. > > Signed-off-by: Bartosz Golaszewski This results in a race wh

Re: [PATCH 3/4] iio: pressure: bmp280: remove stray newline

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 10:57:58 +0200 Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > Remove a stray newline from the probe callback. > > Signed-off-by: Bartosz Golaszewski Applied to the togreg branch of iio.git and pushed out as testing for no particular reason (well for the other p

Re: [alsa-devel] [PATCH -next] ASoc: tas2770: Fix build error without GPIOLIB

2019-10-06 Thread Ladislav Michl
Dear YueHaibing, On Sun, Oct 06, 2019 at 03:22:41PM +0800, YueHaibing wrote: > If GPIOLIB is not set, building fails: > > sound/soc/codecs/tas2770.c: In function tas2770_reset: > sound/soc/codecs/tas2770.c:38:3: error: implicit declaration of function > gpiod_set_value_cansleep; did you mean gpi

Re: [PATCH 4/4] iio: pressure: bmp280: use devm action and remove labels from probe

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 10:57:59 +0200 Bartosz Golaszewski wrote: > From: Bartosz Golaszewski > > We can drop some duplicate code if we use devm_action for disabling > regulators and pm. This allows us to completely remove all remove() > callbacks from both i2c and spi code. > > Signed-off-by: Bar

Re: [PATCH] taskstats: fix data-race

2019-10-06 Thread Dmitry Vyukov
On Sat, Oct 5, 2019 at 1:28 PM Christian Brauner wrote: > > When assiging and testing taskstats in taskstats > taskstats_exit() there's a race around writing and reading sig->stats. > > cpu0: > task calls exit() > do_exit() > -> taskstats_exit() > -> taskstats_tgid_alloc()

Re: [PATCH 1/4] dt-bindings: samsung: Indent examples with four spaces

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 18:07:41 +0200 Krzysztof Kozlowski wrote: > Change the indentation of examples used in json-schema bindings from two > to four spaces as this makes the code easier to read and seems to be > preferred in other files. > I'm assuming Rob will pick these up given they span multi

Re: [PATCH 4/4] dt-bindings: iio: adc: exynos: Use defines instead of clock numbers

2019-10-06 Thread Jonathan Cameron
On Wed, 2 Oct 2019 18:07:44 +0200 Krzysztof Kozlowski wrote: > Make the examples in Exynos ADC bindings more readable and bring them > closer to real DTS by using defines for clocks. > > Signed-off-by: Krzysztof Kozlowski Acked-by: Jonathan Cameron > --- > .../devicetree/bindings/iio/adc/s

Re: [PATCH v3 1/2] tpm: Use GFP_KERNEL for allocating struct tpm_buf

2019-10-06 Thread Jarkko Sakkinen
On Thu, Oct 03, 2019 at 05:25:16PM -0700, James Bottomley wrote: > On Thu, 2019-10-03 at 21:51 +0300, Jarkko Sakkinen wrote: > > Switch from GFP_HIGHUSER to GFP_KERNEL. On 32-bit platforms kmap() > > space > > could be unnecessarily wasted because of using GFP_HIGHUSER by taking > > a > > page of f

Re: [PATCH v2 1/7] iio: adc: max1027: Add debugfs register read support

2019-10-06 Thread Jonathan Cameron
On Thu, 3 Oct 2019 19:33:55 +0200 Miquel Raynal wrote: > Until now, only write operations were supported. Force two bytes read > operation when reading from this register (might be wrong when reading > the temperature, but will work with any other value). That's worrying as comments go. Just r

[PATCH -next] media: i2c: ov5695: Fix randbuild error

2019-10-06 Thread YueHaibing
If VIDEO_OV5695 is y and V4L2_FWNODE is m, building fails: drivers/media/i2c/ov5695.o: In function `ov5695_probe': ov5695.c:(.text+0xf4c): undefined reference to `v4l2_async_register_subdev_sensor_common' Select V4L2_FWNODE like OV5675 does. Fixes: 623df5d710fe ("media: i2c: ov5695: Modify the

Re: [alsa-devel] [PATCH -next] ASoc: tas2770: Fix build error without GPIOLIB

2019-10-06 Thread Yuehaibing
On 2019/10/6 17:57, Ladislav Michl wrote: > Dear YueHaibing, > > On Sun, Oct 06, 2019 at 03:22:41PM +0800, YueHaibing wrote: >> If GPIOLIB is not set, building fails: >> >> sound/soc/codecs/tas2770.c: In function tas2770_reset: >> sound/soc/codecs/tas2770.c:38:3: error: implicit declaration of fun

Re: [PATCH] staging: greybus: add blank line after declarations

2019-10-06 Thread Joe Perches
On Sun, 2019-10-06 at 11:50 +0200, Greg KH wrote: > On Sat, Oct 05, 2019 at 06:00:46PM -0300, Gabriela Bittencourt wrote: > > Fix CHECK: add blank line after declarations [] > > diff --git a/drivers/staging/greybus/control.h > > b/drivers/staging/greybus/control.h [] > > @@ -24,6 +24,7 @@ struct g

Re: [PATCH v2 2/7] iio: adc: max1027: Make it optional to use interrupts

2019-10-06 Thread Jonathan Cameron
On Thu, 3 Oct 2019 19:33:56 +0200 Miquel Raynal wrote: > The chip has a 'start conversion' and a 'end of conversion' pair of > pins. They can be used but this is absolutely not mandatory as regular > polling of the value is totally fine with the current internal > clocking setup. Turn the interr

Re: [PATCH v2 4/7] iio: adc: max1027: Prepare the introduction of different resolutions

2019-10-06 Thread Jonathan Cameron
On Thu, 3 Oct 2019 19:33:58 +0200 Miquel Raynal wrote: > Maxim's max1027/29/31 series returns the measured voltages with a > resolution of 10 bits. There is a very similar series, max1227/29/31 > which works identically but uses a resolution of 12 bits. Prepare the > support for these chips by t

Re: [PATCH v2 5/7] iio: adc: max1027: Introduce 12-bit devices support

2019-10-06 Thread Jonathan Cameron
On Thu, 3 Oct 2019 19:33:59 +0200 Miquel Raynal wrote: > Maxim's max12xx series is very similar to the max10xx series, with the > difference of the measurements depth which is upgraded from 10 to 12 > bits per channel. Everything else looks the same. > > Signed-off-by: Miquel Raynal Looks good

Re: [PATCH v2 7/7] dt-bindings: iio: adc: max1027: Document max12xx series compatibles

2019-10-06 Thread Jonathan Cameron
On Thu, 3 Oct 2019 19:34:01 +0200 Miquel Raynal wrote: > Update the bindings documentation with new Maxim ADCs compatibles. > > Signed-off-by: Miquel Raynal Both DT patches look fine to me... If you happened to fancy doing the yaml conversion it would be appreciated... :) If not we'll get

[PATCH -next 03/34] rtc: brcmstb-waketimer: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-brcmstb-waketimer.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-brcmstb-waketimer.c b/drivers/rtc/rtc-brcmstb

[PATCH -next 01/34] rtc: asm9260: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-asm9260.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-asm9260.c b/drivers/rtc/rtc-asm9260.c index 10413d8..100

[PATCH -next 10/34] rtc: ds1511: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-ds1511.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-ds1511.c b/drivers/rtc/rtc-ds1511.c index b6a4775..a63872

[PATCH -next 17/34] rtc: mt7622: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-mt7622.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-mt7622.c b/drivers/rtc/rtc-mt7622.c index 16bd26b..f1e356

[PATCH -next 08/34] rtc: digicolor: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-digicolor.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-digicolor.c b/drivers/rtc/rtc-digicolor.c index 0aecc3

[PATCH -next 04/34] rtc: at91sam9: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-at91sam9.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-at91sam9.c b/drivers/rtc/rtc-at91sam9.c index bb3ba7b..

[PATCH -next 12/34] rtc: ep93xx: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-ep93xx.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-ep93xx.c b/drivers/rtc/rtc-ep93xx.c index 1766496..8ec9ea

[PATCH -next 15/34] rtc: lpc32xx: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-lpc32xx.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-lpc32xx.c b/drivers/rtc/rtc-lpc32xx.c index ac39323..b6a

[PATCH -next 16/34] rtc: meson: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-meson.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-meson.c b/drivers/rtc/rtc-meson.c index e08b981..9bd8478 1

[PATCH -next 06/34] rtc: coh901331: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-coh901331.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-coh901331.c b/drivers/rtc/rtc-coh901331.c index 4ac850

[PATCH -next 13/34] rtc: jz4740: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-jz4740.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-jz4740.c b/drivers/rtc/rtc-jz4740.c index 3089645..18023e

[PATCH -next 14/34] rtc: lpc24xx: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-lpc24xx.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-lpc24xx.c b/drivers/rtc/rtc-lpc24xx.c index a8bb156..00e

[PATCH -next 18/34] rtc: mv: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-mv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-mv.c b/drivers/rtc/rtc-mv.c index ab9db57..d5f190e5 100644 --

[PATCH -next 07/34] rtc: davinci: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-davinci.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-davinci.c b/drivers/rtc/rtc-davinci.c index d8e0db2..390

[PATCH -next 09/34] rtc: ds1216: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-ds1216.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-ds1216.c b/drivers/rtc/rtc-ds1216.c index b225bcf..7eeb3f

[PATCH -next 11/34] rtc: ds1553: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-ds1553.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-ds1553.c b/drivers/rtc/rtc-ds1553.c index 219d6b5..cdf5e0

[PATCH -next 25/34] rtc: stk17ta8: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-stk17ta8.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-stk17ta8.c b/drivers/rtc/rtc-stk17ta8.c index a833ebc..

[PATCH -next 19/34] rtc: omap: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-omap.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-omap.c b/drivers/rtc/rtc-omap.c index a2941c8..988a4df 1006

[PATCH -next 20/34] rtc: pic32: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-pic32.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-pic32.c b/drivers/rtc/rtc-pic32.c index 17653ed..2b69467 1

[PATCH -next 29/34] rtc: sunxi: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-sunxi.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-sunxi.c b/drivers/rtc/rtc-sunxi.c index 9b6f248..f5d7f44 1

[PATCH -next 21/34] rtc: rtd119x: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-rtd119x.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-rtd119x.c b/drivers/rtc/rtc-rtd119x.c index b233559..bb9

[PATCH -next 00/34] rtc: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
devm_platform_ioremap_resource() internally have platform_get_resource() and devm_ioremap_resource() in it. So instead of calling them separately use devm_platform_ioremap_resource() directly. YueHaibing (34): rtc: asm9260: use devm_platform_ioremap_resource() to simplify code rtc: rtc-aspeed:

[PATCH -next 30/34] rtc: tegra: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-tegra.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-tegra.c b/drivers/rtc/rtc-tegra.c index 69d695b..0159069 1

[PATCH -next 31/34] rtc: tx4939: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-tx4939.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-tx4939.c b/drivers/rtc/rtc-tx4939.c index 5a29915..715b82

[PATCH -next 02/34] rtc: rtc-aspeed: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-aspeed.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-aspeed.c b/drivers/rtc/rtc-aspeed.c index e351d35..eacdd0

[PATCH -next 22/34] rtc: s3c: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-s3c.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-s3c.c b/drivers/rtc/rtc-s3c.c index 7801249..e1b50e6 100644

[PATCH -next 26/34] rtc: ds1286: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-ds1286.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-ds1286.c b/drivers/rtc/rtc-ds1286.c index a06508b..7acf84

[PATCH -next 05/34] rtc: cadence: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-cadence.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-cadence.c b/drivers/rtc/rtc-cadence.c index 592aae2..595

[PATCH -next 24/34] rtc: spear: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-spear.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-spear.c b/drivers/rtc/rtc-spear.c index 9f23b24..833daeb 1

[PATCH -next 28/34] rtc: stm32: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-stm32.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-stm32.c b/drivers/rtc/rtc-stm32.c index 2999e33..781cabb 1

[PATCH -next 23/34] rtc: sa1100: use devm_platform_ioremap_resource() to simplify code

2019-10-06 Thread YueHaibing
Use devm_platform_ioremap_resource() to simplify the code a bit. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/rtc/rtc-sa1100.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/rtc/rtc-sa1100.c b/drivers/rtc/rtc-sa1100.c index 86fa723..d37893

  1   2   3   4   5   6   7   8   9   10   >