linux-next: Signed-off-by missing for commit in the v4l-dvb tree

2017-12-10 Thread Stephen Rothwell
Hi Mauro, Commit fdda01090cc2 ("media: cx231xx: Add support for The Imaging Source DFG/USB2pro") is missing a Signed-off-by from its committer. -- Cheers, Stephen Rothwell

Re: [GIT PULL] Documentation: Add a file explaining the requested Linux kernel license enforcement policy

2017-12-10 Thread Pavel Machek
Hi! > > I don't feel that is true at all, what we are doing here is providing a > > well-documented way toward compliance and the reinstatement of our > > license.  That's a key issue with regards to the existing trolls we are > > currently facing today, which we have to address in order to preser

v4.15-rc2 on thinkpad x60: ethernet stopped working

2017-12-10 Thread Pavel Machek
Hi! In v4.15-rc2+, network manager can not see my ethernet card, and manual attempts to ifconfig it up did not really help, either. Card is: 02:00.0 Ethernet controller: Intel Corporation 82573L Gigabit Ethernet Controller Dmesg says: dmesg | grep eth [0.648931] e1000e :02:00.0 eth0:

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Philippe Ombredanne
Andreas, On Sun, Dec 10, 2017 at 12:25 AM, Andreas Färber wrote: > The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, > which allows to detach the sensor as a breakout board. > > Cc: Marius Tarcatu > Signed-of

Re: [alsa-devel][PATCH v3] ASoC: TSCS42xx: Add support for Tempo Semiconductor's TSCS42xx audio CODEC

2017-12-10 Thread Philippe Ombredanne
Steven, On Sat, Dec 9, 2017 at 11:42 PM, Steven Eckhoff wrote: > Currently there is no support for the TSCS42xx audio CODEC. > > Add support for it. > > Below is the link to the v2 patch in case the threading is broken. This > patch addressed each issue raised in the last review. > > https://pat

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Peter Meerwald-Stadler
> The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. comments below > It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, > which allows to detach the sensor as a breakout board. > > Cc: Marius Tarcatu > Signed-off-by: Andreas Färber > --- > drivers/iio/magnet

Re: [PATCH] mm: Release a semaphore in 'get_vaddr_frames()'

2017-12-10 Thread Michal Hocko
On Sat 09-12-17 08:09:41, Christophe JAILLET wrote: > A semaphore is acquired before this check, so we must release it before > leaving. > > Fixes: b7f0554a56f2 ("mm: fail get_vaddr_frames() for filesystem-dax > mappings") > Signed-off-by: Christophe JAILLET > --- > -- Untested -- > > The wordi

[PATCH] mm, oom_reaper: fix memory corruption

2017-12-10 Thread Michal Hocko
From: Michal Hocko David Rientjes has reported the following memory corruption while the oom reaper tries to unmap the victims address space BUG: Bad page map in process oom_reaper pte:63538263 pmd: addr:7f50cab1d000 vm_flags:08100073 anon_vma:9eea335603f0 mapping:

Re: [PATCH v4] mm, thp: introduce generic transparent huge page allocation interfaces

2017-12-10 Thread Michal Hocko
On Sat 09-12-17 11:26:58, Du, Changbin wrote: > On Fri, Dec 08, 2017 at 09:27:37AM +0100, Michal Hocko wrote: > > On Fri 08-12-17 12:42:55, changbin...@intel.com wrote: > > > From: Changbin Du > > > > > > This patch introduced 4 new interfaces to allocate a prepared transparent > > > huge page. T

Re: [PATCH v3 14/14] nubus: Add support for the driver model

2017-12-10 Thread Finn Thain
On Thu, 7 Dec 2017, Philippe Ombredanne wrote: > > diff --git a/drivers/nubus/bus.c b/drivers/nubus/bus.c > > new file mode 100644 > > index ..b5045e639b4d > > --- /dev/null > > +++ b/drivers/nubus/bus.c > > @@ -0,0 +1,119 @@ > > +/* > > + * Bus implementation for the NuBus subsystem.

Re: [PATCH 3/3] x86/PCI: limit the size of the 64bit BAR to 256GB

2017-12-10 Thread Christian König
Am 08.12.2017 um 18:56 schrieb Bjorn Helgaas: On Wed, Dec 06, 2017 at 01:51:18PM -0600, Bjorn Helgaas wrote: On Wed, Nov 29, 2017 at 03:12:29PM +0100, Christian König wrote: This avoids problems with Xen which hides some memory resources from the OS and potentially also allows memory hotplug wh

Re: [PATCH v2] fs: handle shrinker registration failure in sget_userns

2017-12-10 Thread Michal Hocko
On Sun 10-12-17 11:33:18, Tetsuo Handa wrote: > Al Viro wrote: > > On Sat, Dec 09, 2017 at 08:59:22PM +, Al Viro wrote: > > > On Wed, Nov 29, 2017 at 12:55:15PM +0100, Michal Hocko wrote: > > > > On Thu 23-11-17 14:55:40, Al Viro wrote: > > > > > On Thu, Nov 23, 2017 at 03:35:37PM +0100, Michal

Re: [PATCH v2] mm: terminate shrink_slab loop if signal is pending

2017-12-10 Thread Michal Hocko
On Fri 08-12-17 10:06:26, Suren Baghdasaryan wrote: > On Fri, Dec 8, 2017 at 6:03 AM, Tetsuo Handa > wrote: > > Michal Hocko wrote: > >> On Fri 08-12-17 20:36:16, Tetsuo Handa wrote: > >> > On 2017/12/08 17:22, Michal Hocko wrote: > >> > > On Thu 07-12-17 17:23:05, Suren Baghdasaryan wrote: > >> >

Re: [PATCH v2] mm: terminate shrink_slab loop if signal is pending

2017-12-10 Thread Michal Hocko
On Sat 09-12-17 17:08:42, Tetsuo Handa wrote: > Suren Baghdasaryan wrote: > > On Fri, Dec 8, 2017 at 6:03 AM, Tetsuo Handa > > wrote: > > >> > >> This change checks for pending > > >> > >> fatal signals inside shrink_slab loop and if one is detected > > >> > >> terminates this loop early. > > >> >

Re: [PATCH v4] mmap.2: MAP_FIXED updated documentation

2017-12-10 Thread Michal Hocko
On Tue 05-12-17 19:14:34, john.hubb...@gmail.com wrote: > From: John Hubbard > > Previously, MAP_FIXED was "discouraged", due to portability > issues with the fixed address. In fact, there are other, more > serious issues. Also, alignment requirements were a bit vague. > So: > > -- Expand th

Re: [PATCH] mm/slab: make calculate_alignment() function static

2017-12-10 Thread Michal Hocko
On Sun 10-12-17 17:01:32, Byongho Lee wrote: > calculate_alignment() function is only used inside 'slab_common.c'. > So make it static and let compiler do more optimizations. > > After this patch there's small improvements in 'text' and 'data' size. > > $ gcc --version > gcc (GCC) 7.2.1 2017112

Re: [PATCH] Documentation: Better document the hardlockup_panic sysctl

2017-12-10 Thread Christoph von Recklinghausen
Ack Sent from my iPhone > On Dec 10, 2017, at 2:48 AM, Scott Wood wrote: > > Commit ac1f591249d95372f ("kernel/watchdog.c: add sysctl knob > hardlockup_panic") added the hardlockup_panic sysctl, but did not add it > to Documentation/sysctl/kernel.txt. Add this, and reference it from the > corr

Re: [PATCH v2] mm: terminate shrink_slab loop if signal is pending

2017-12-10 Thread Tetsuo Handa
Michal Hocko wrote: > > > I agree that making waits/loops killable is generally good. But be sure > > > to be > > > prepared for the worst case. For example, start __GFP_KILLABLE from "best > > > effort" > > > basis (i.e. no guarantee that the allocating thread will leave the page > > > allocato

[PATCH] arm: imx: suspend/resume: use outer_disable/resume

2017-12-10 Thread Peng Fan
Use outer_disable/resume for suspend/resume. With the two APIs used, code could be simplified and easy to extend to introduce l2c_write_sec for i.MX platforms when moving Linux Kernel runs in non-secure world. Signed-off-by: Peng Fan Cc: Shawn Guo Cc: Sascha Hauer Cc: Fabio Estevam Cc: Russell

Re: [PATCH v3 14/14] nubus: Add support for the driver model

2017-12-10 Thread Philippe Ombredanne
Finn, On Sun, Dec 10, 2017 at 11:03 AM, Finn Thain wrote: > On Thu, 7 Dec 2017, Philippe Ombredanne wrote: > >> > diff --git a/drivers/nubus/bus.c b/drivers/nubus/bus.c >> > new file mode 100644 >> > index ..b5045e639b4d >> > --- /dev/null >> > +++ b/drivers/nubus/bus.c >> > @@ -0,0 +

[PATCH] ext4: delayed inode update for the consistency of file size after a crash

2017-12-10 Thread seongbaeSon
For one write() system call, the file size in the ext4_inode can be updated twice; at the write() system call and when dirty pages of the file are written to storage (i.g. fsync() or kworker thread). The write() system call that appends data to a file but does not need to allocate a block updates t

Re: iio/…: Use common error handling code

2017-12-10 Thread SF Markus Elfring
> Again, a general kernel development rule is to float only one > patch of a given type until you have had feedback on it. A few weeks later … > As I said in one of the other patches, I always reply to all > patches I am rejecting so that anyone coming across them later > on their own from an ar

Re: [PATCH 4.14 00/75] 4.14.5-stable review

2017-12-10 Thread Greg Kroah-Hartman
On Sat, Dec 09, 2017 at 07:32:38PM +0200, Thomas Backlund wrote: > Den 09.12.2017 kl. 19:13, skrev Greg Kroah-Hartman: > > On Sat, Dec 09, 2017 at 07:56:40AM +, Ivan Kozik wrote: > > > On Sat, Dec 9, 2017 at 7:45 AM, Greg Kroah-Hartman > > > wrote: > > > > On Sat, Dec 09, 2017 at 03:34:24AM +0

Re: v4.15-rc2 on thinkpad x60: ethernet stopped working

2017-12-10 Thread Gabriel C
On 10.12.2017 09:39, Pavel Machek wrote: Hi! Hi, In v4.15-rc2+, network manager can not see my ethernet card, and manual attempts to ifconfig it up did not really help, either. Card is: 02:00.0 Ethernet controller: Intel Corporation 82573L Gigabit Ethernet Controller Dmesg says: dmesg

Re: [PATCH v3 1/2] eeprom: at24: fix coding style issues

2017-12-10 Thread Bartosz Golaszewski
2017-12-10 2:31 GMT+01:00 Christoph Böhmwalder : > On Thu, Dec 07, 2017 at 02:39:14PM +0100, Bartosz Golaszewski wrote: >> -#define AT24_DEVICE_MAGIC(_len, _flags) \ >> - ((1 << AT24_SIZE_FLAGS | (_flags)) \ >> +#define AT24_DEVICE_MAGIC(_len, _flags)

Re: [PATCH tip/core/rcu 02/20] torture: Prepare scripting for shift from %p to %pK

2017-12-10 Thread Andy Shevchenko
On Mon, Dec 4, 2017 at 6:11 PM, Paul E. McKenney wrote: > On Mon, Dec 04, 2017 at 02:13:38PM +, David Laight wrote: >> From: Paul E. McKenney >> > Sent: 04 December 2017 13:42 >> > On Mon, Dec 04, 2017 at 12:32:30PM +, David Laight wrote: >> > > From: Paul E. McKenney >> > > > Sent: 01 Dec

Re: [PATCH v3 1/2] eeprom: at24: fix coding style issues

2017-12-10 Thread Andy Shevchenko
On Thu, Dec 7, 2017 at 3:39 PM, Bartosz Golaszewski wrote: > Fix issues reported by checkpatch for at24.c. > +module_param(io_limit, uint, ); > +module_param(write_timeout, uint, ); 0 is a pretty much octal number as . So, I would prefer not to blindly follow the stupid advise from

Re: [PATCH] regmap: allow to disable all locking mechanisms

2017-12-10 Thread Andy Shevchenko
On Wed, Dec 6, 2017 at 4:26 PM, Bartosz Golaszewski wrote: > We have a use case in the at24 EEPROM driver (recently converted to > using regmap instead of raw i2c/smbus calls) where we read from/write > to the regmap in a loop, while protecting the entire loop with > a mutex. > > Currently this im

[PATCH] checkpatch: don't require octal permissions for "0"

2017-12-10 Thread Bartosz Golaszewski
If the permission value is 0, don't raise the NON_OCTAL_PERMISSIONS error. There's no possibility of an error if there are no permissions. Suggested-by: Andy Shevchenko Signed-off-by: Bartosz Golaszewski --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] tty: nozomi: Delete an error message for a failed memory allocation in nozomi_card_init()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 15:37:26 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/tty/nozomi.c | 1 - 1 file changed, 1 deletion(-) diff --git

Re: [PATCH v2 1/2] x86/mce/AMD: Define function to get SMCA bank type

2017-12-10 Thread Borislav Petkov
On Thu, Dec 07, 2017 at 02:39:54PM -0600, Yazen Ghannam wrote: > diff --git a/arch/x86/kernel/cpu/mcheck/mce_amd.c > b/arch/x86/kernel/cpu/mcheck/mce_amd.c > index a38ab1fa53a2..219d5115f4d4 100644 > --- a/arch/x86/kernel/cpu/mcheck/mce_amd.c > +++ b/arch/x86/kernel/cpu/mcheck/mce_amd.c > @@ -110,

Re: [PATCH] iio: dummy_evgen: modify the return value check for irq_sim_init()

2017-12-10 Thread Jonathan Cameron
On Sat, 2 Dec 2017 16:33:59 +0100 Bartosz Golaszewski wrote: > 2017-12-02 12:43 GMT+01:00 Jonathan Cameron : > > On Mon, 27 Nov 2017 11:54:49 +0100 > > Bartosz Golaszewski wrote: > > > >> As discussed with Marc Zyngier: irq_sim_init() and its devres variant > >> should return the base of the a

Re: [PATCH] iio: common: ssp_sensors: use ktime_get_real_ns() timestamps

2017-12-10 Thread Jonathan Cameron
On Sat, 2 Dec 2017 11:49:49 + Jonathan Cameron wrote: > On Mon, 27 Nov 2017 12:51:48 +0100 > Arnd Bergmann wrote: > > > getnstimeofday() suffers from the overflow in y2038 on 32-bit > > architectures and requires a conversion into the nanosecond format that > > we want here. > > > > This c

Re: [PATCH v2] iio: mma8452: replace license description with SPDX specifier

2017-12-10 Thread Jonathan Cameron
On Sat, 18 Nov 2017 23:23:57 -0500 harinath Nampally wrote: > > This replaces the custom license information text with the appropriate > > SPDX identifier. While the information here stays the same, it is easier > > to read. > > Signed-off-by: Martin Kepplinger > > Acked-by: Peter Meerwald-Stadl

Re: [PATCH v1 1/6] ARM: dts: imx7d-sbc-iot: add initial iot gateway dts

2017-12-10 Thread Fabio Estevam
On Tue, Dec 5, 2017 at 8:27 PM, wrote: > +&usdhc1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_usdhc1>; > + cd-gpios = <&gpio5 0 GPIO_ACTIVE_LOW>; > + wp-gpios = <&gpio5 1 GPIO_ACTIVE_HIGH>; > + enable-sdio-wakeup; This is a deprecated property. Plea

Re: [PATCH] regmap: allow to disable all locking mechanisms

2017-12-10 Thread Bartosz Golaszewski
2017-12-10 14:10 GMT+01:00 Andy Shevchenko : > On Wed, Dec 6, 2017 at 4:26 PM, Bartosz Golaszewski wrote: >> We have a use case in the at24 EEPROM driver (recently converted to >> using regmap instead of raw i2c/smbus calls) where we read from/write >> to the regmap in a loop, while protecting the

Re: [PATCH v1 2/6] ARM: dts: imx7d-cl-som: add nodes for usbh, and usbotg2

2017-12-10 Thread Fabio Estevam
On Tue, Dec 5, 2017 at 8:27 PM, wrote: > From: Tyler Baker > > Add device tree nodes for the USB hub, and USB OTG. i2c2 on this > platform supports low state retention power state so lets use it. Looks like the I2C2 fix should be a separate patch.

Re: [PATCH v1 3/6] ARM: dts: imx7s: add dma support

2017-12-10 Thread Fabio Estevam
On Tue, Dec 5, 2017 at 8:27 PM, wrote: > From: Tyler Baker > > Enable dma on all SPI interfaces. > > Signed-off-by: Tyler Baker Reviewed-by: Fabio Estevam

Re: [PATCH v1 4/6] ARM: dts: imx7s: add node and supplies for vdd1p2

2017-12-10 Thread Fabio Estevam
On Tue, Dec 5, 2017 at 8:27 PM, wrote: > From: Tyler Baker > > Add the regulator nodes and supplies for vdd1p2. This regulator is > used to power the GPC and USB HSIC PHY. > > Signed-off-by: Tyler Baker > --- > arch/arm/boot/dts/imx7s.dtsi | 17 - > 1 file changed, 16 insertion

[PATCH] mtd: mchp23k256: propagate return value of spi_sync()

2017-12-10 Thread Antonio Borneo
The call to spi_sync() can fail. Check the return value and propagate it. Signed-off-by: Antonio Borneo --- To: David Woodhouse To: Brian Norris To: Boris Brezillon To: Marek Vasut To: Richard Weinberger To: Cyrille Pitchen To: Andrew Lunn To: Chris Packham To: Rob Herring To: linux-...@

Re: [PATCH v1 5/6] ARM: dts: imx7d-sbc-iot: enable PCIe peripheral

2017-12-10 Thread Fabio Estevam
On Tue, Dec 5, 2017 at 8:27 PM, wrote: > From: Tyler Baker > > Add a PCIe device tree node to enable PCIe support. > > Signed-off-by: Tyler Baker Reviewed-by: Fabio Estevam

Re: [PATCH v2] fs: handle shrinker registration failure in sget_userns

2017-12-10 Thread Al Viro
On Sun, Dec 10, 2017 at 11:05:15AM +0100, Michal Hocko wrote: > > My unregister_shrinker() fortification patch > > ( > > http://lkml.kernel.org/r/1511523385-6433-1-git-send-email-penguin-ker...@i-love.sakura.ne.jp > > ) > > is not yet in the mmotm tree due to disagreement between Michal and I, b

Re: [PATCH v1 6/6] ARM: dts: imx7s: add usb hsic phy domain

2017-12-10 Thread Fabio Estevam
On Tue, Dec 5, 2017 at 8:27 PM, wrote: > From: Tyler Baker > > The GPCv2 driver should control the MIPI, PCIe, > and USB HSIC PHY regulators. Add the USB HSIC > power domain to the GPC node. > > Signed-off-by: Tyler Baker Looks good. I am assuming you tested USB HSIC operation on the mx7 compu

[PATCH 0/2] tty-MOXA: Adjustments for moxa_init_board()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 16:25:43 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Return directly after a failed kcalloc() drivers/tty/moxa.c | 8 ++-- 1

[PATCH net-next 1/3] net:tracepoint: replace tcp_set_state tracepoint with sock_set_state tracepoint

2017-12-10 Thread Yafang Shao
As sk_state is a common field for struct sock, so the state transition should not be a TCP specific feature. So I rename tcp_set_state tracepoint to sock_set_state tracepoint with some minor changes and move it into file trace/events/sock.h. The minor changes against on the original tcp_set_state

[PATCH 1/2] tty: moxa: Delete an error message for a failed memory allocation in moxa_init_board()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 16:03:21 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/tty/moxa.c | 1 - 1 file changed, 1 deletion(-) diff --git a/

[PATCH net-next 0/3] replace tcp_set_state tracepoint with sock_set_state tracepoint

2017-12-10 Thread Yafang Shao
Hi, According to the discussion in the mail thread https://patchwork.kernel.org/patch/10099243/, tcp_set_state tracepoint is renamed to sock_set_state tracepoint and is moved to include/trace/events/sock.h. Using this new tracepoint to trace TCP/DCCP/SCTP state transition. Yafang Shao (3): net

[PATCH net-next 2/3] net:tracepoint: using sock_set_state tracepoint to trace DCCP state transition

2017-12-10 Thread Yafang Shao
With changes in inet_ files, DCCP state transitions are traced with sock_set_state tracepoint. Signed-off-by: Yafang Shao --- net/dccp/proto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/dccp/proto.c b/net/dccp/proto.c index b68168f..92b9613 100644 --- a/net/dccp/prot

[PATCH net-next 3/3] net:tracepoint: using sock_set_state tracepoint to trace SCTP state transition

2017-12-10 Thread Yafang Shao
With changes in inet_ files, SCTP state transitions are traced with sockt_set_state tracepoint. Signed-off-by: Yafang Shao --- net/sctp/endpointola.c | 2 +- net/sctp/sm_sideeffect.c | 4 ++-- net/sctp/socket.c| 14 +++--- 3 files changed, 10 insertions(+), 10 deletions(-) d

[PATCH 2/2] tty: moxa: Return directly after a failed kcalloc() in moxa_init_board()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 16:15:41 +0100 * Return directly after a call of the function "kcalloc" failed at the beginning. * Delete the jump label "err" which became unnecessary with this refactoring. Signed-off-by: Markus Elfring --- drivers/tty/moxa.c | 7 ++- 1 fi

[PATCH] tty: isicom: Delete an error message for a failed memory allocation in load_firmware()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 17:00:05 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/tty/isicom.c | 6 ++ 1 file changed, 2 insertions(+), 4 de

Re: iio/…: Use common error handling code

2017-12-10 Thread Jonathan Cameron
On Sun, 10 Dec 2017 13:33:01 +0100 SF Markus Elfring wrote: > > Again, a general kernel development rule is to float only one > > patch of a given type until you have had feedback on it. > > A few weeks later … > > > > As I said in one of the other patches, I always reply to all > > patches

Re: [PATCH] iio: accel: bmc150: Add OF device ID table

2017-12-10 Thread Jonathan Cameron
On Mon, 4 Dec 2017 11:24:40 +0100 Javier Martinez Canillas wrote: > Hello Jonathan, > > On 12/04/2017 10:44 AM, Jonathan Cameron wrote: > > On Mon, 4 Dec 2017 09:29:38 +0100 > > Hans de Goede wrote: > > > >> Hi, > >> > >> On 01-12-17 12:10, Javier Martinez Canillas wrote: > >>> The driver

Re: [PATCH v2 2/2] x86/mce: Report only DRAM ECC as memory errors on AMD systems

2017-12-10 Thread Borislav Petkov
On Thu, Dec 07, 2017 at 02:39:55PM -0600, Yazen Ghannam wrote: > From: Yazen Ghannam > > The MCA_STATUS[ErrorCodeExt] field is very bank type specific. We currently > check if the ErrorCodeExt value is 0x0 or 0x8 in mce_is_memory_error(), but > we don't check the bank. This means that we could fl

Re: [PATCH][next] iio: light: make a couple of config structures static

2017-12-10 Thread Jonathan Cameron
On Mon, 4 Dec 2017 12:04:06 + Colin King wrote: > From: Colin Ian King > > Structures st_uvis25_i2c_regmap_config and st_uvis25_spi_regmap_config are > local to the source and do not need to be in global scope, so make them > both static. > > Cleans up sparse warnings: > warning: symbol '

Re: Linux 4.15-rc2: Regression in resume from ACPI S3

2017-12-10 Thread Pavel Machek
On Sat 2017-12-09 14:47:41, Linus Torvalds wrote: > On Dec 9, 2017 14:01, "Pavel Machek" wrote: > > > Strange. I was at 4.15-rc1, and suspend worked there (thinkpad x60, > 32-bit). It was broken in -next. I updated to current mainline ( > 4ded3bec65a07343258ed8fd9d46483f032d866f ) and suspend is

[PATCH v4] leds: trigger: Introduce a NETDEV trigger

2017-12-10 Thread Ben Whitten
This commit introduces a NETDEV trigger for named device activity. Available triggers are link, rx, and tx. Signed-off-by: Ben Whitten --- Changes in v4: Adopt SPDX licence header Changes in v3: Cancel the software blink prior to a oneshot re-queue Changes in v2: Sort includes and redate documen

[PATCH] tty: cyclades: Delete an error message for a failed memory allocation in cy_init_card()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 17:21:28 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/tty/cyclades.c | 4 +--- 1 file changed, 1 insertion(+), 3 del

Re: [linux-sunxi] [PATCH 1/8] iio: adc: axp20x_adc: put ADC rate setting in a per-variant function

2017-12-10 Thread Jonathan Cameron
On Tue, 5 Dec 2017 11:35:49 +0800 Chen-Yu Tsai wrote: > On Mon, Dec 4, 2017 at 10:12 PM, Quentin Schulz > wrote: > > To prepare for a new comer that set a different register with different > > values, move rate setting in a function that is specific to each AXP > > variant. > > > > Signed-off-by

Re: [PATCH 2/8] iio: adc: axp20x_adc: add support for AXP813 ADC

2017-12-10 Thread Jonathan Cameron
On Mon, 4 Dec 2017 15:12:48 +0100 Quentin Schulz wrote: > The X-Powers AXP813 PMIC is really close to what is already done for > AXP20X/AXP22X. > > There are two pairs of bits to set the rate (one for Voltage and Current > measurements and one for TS/GPIO0 voltage measurements) instead of one.

Re: Linux 4.15-rc2: Regression in resume from ACPI S3

2017-12-10 Thread Linus Torvalds
On Sun, Dec 10, 2017 at 8:23 AM, Pavel Machek wrote: > > Can you do something about html emails? Quoting them doesn't work too well. Yeah, and they don't show up onlkml either because of rules. I try to avoid them, but have been more on mobile for various reasons lately than usual. That should be

Re: [PATCH 3/8] mfd: axp20x: probe axp20x_adc driver for AXP813

2017-12-10 Thread Jonathan Cameron
On Thu, 7 Dec 2017 17:14:30 +0800 Chen-Yu Tsai wrote: > On Thu, Dec 7, 2017 at 5:03 PM, Quentin Schulz > wrote: > > Hi Chen-Yu, > > > > On 07/12/2017 09:54, Chen-Yu Tsai wrote: > >> On Thu, Dec 7, 2017 at 4:51 PM, Quentin Schulz > >> wrote: > >>> Hi Maxime, > >>> > >>> On 05/12/2017 09:08,

Re: iio/…: Use common error handling code

2017-12-10 Thread SF Markus Elfring
> Hi Markus, I've accepted the ones that I think made an improvement > outweighing the inherent small costs of making any change. Does such a kind of feedback mean that you reconsidered any places where you expressed a rejection initially? > We also need to avoid code constructs that are unusual

Re: [PATCH 4/8] dt-bindings: power: supply: axp20x: add AXP813 battery DT binding

2017-12-10 Thread Jonathan Cameron
On Mon, 4 Dec 2017 15:12:50 +0100 Quentin Schulz wrote: > The AXP813 can have a battery as power supply, so let's add it to the > list of compatibles. > > Signed-off-by: Quentin Schulz > --- > Documentation/devicetree/bindings/power/supply/axp20x_battery.txt | 8 +++ > 1 file changed, 4 i

Re: [PATCH 5/8] power: supply: axp20x_battery: add support for AXP813

2017-12-10 Thread Jonathan Cameron
On Mon, 4 Dec 2017 15:12:51 +0100 Quentin Schulz wrote: > The X-Powers AXP813 PMIC has got some slight differences from > AXP20X/AXP22X PMICs: > - the maximum voltage supplied by the PMIC is 4.35 instead of 4.36/4.24 > for AXP20X/AXP22X, > - the constant charge current formula is different, >

Re: [PATCH v2 5/7] drm/rockchip: dw_hdmi: add hclk_vio

2017-12-10 Thread Heiko Stuebner
Hi Algea, Am Samstag, 30. September 2017, 09:45:12 CET schrieb Algea Cao: > Add clk hclk_vio and enable it when hdmi bind. Could you explain what the hclk_vio reference is needed for please? Because from from what I tracked down in the TRM and code, this hclk_vio is defined wrong in the clock-dr

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Jonathan Cameron
On Sun, 10 Dec 2017 10:08:04 +0100 (CET) Peter Meerwald-Stadler wrote: > > The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > comments below A few additions from me, but Peter got most stuff. You could make the driver safe for i2c DMA but it's borderline on whether any of th

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Andreas Färber
Hi, Am 10.12.2017 um 10:08 schrieb Peter Meerwald-Stadler: >> The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > comments below > >> It is found among others on the Infineon 3D Magnetic Sensor 2Go Kit, >> which allows to detach the sensor as a breakout board. >> >> Cc: Marius

Re: [PATCH] ext4: delayed inode update for the consistency of file size after a crash

2017-12-10 Thread Theodore Ts'o
On Sun, Dec 10, 2017 at 09:12:57PM +0900, seongbaeSon wrote: > 1. Current file offset of fileA is 14 KB. An application appends 2 KB data to > fileA by executing a write() system call. At this time, the file size in > the ext4_inode of fileA is updated to 16 KB by ext4_da_write_end(). > 2. Current

Re: [PATCH v1 3/7] iio: adc: ina2xx: Remove unneeded dummy read to clear CNVR flag

2017-12-10 Thread Jonathan Cameron
On Fri, 8 Dec 2017 18:41:48 +0100 Stefan Brüns wrote: > Although the datasheet states the CNVR flag is cleared by reading the > BUS_VOLTAGE register, it is actually cleared by reading any of the > voltage/current/power registers. > > The behaviour has been confirmed by TI support: > http://e2e.t

Re: [PATCH v1 5/7] iio: adc: ina2xx: Use a monotonic clock for delay calculation

2017-12-10 Thread Jonathan Cameron
On Fri, 8 Dec 2017 18:41:50 +0100 Stefan Brüns wrote: > The iio timestamp clock is user selectable and may be non-monotonic. Also, > only part of the acquisition time is measured, thus the delay was longer > than intended. > > Signed-off-by: Stefan Brüns > --- > > drivers/iio/adc/ina2xx-adc.c

Re: [PATCH 09/45] drivers: media: remove duplicate includes

2017-12-10 Thread Pravin Shedge
On Thu, Dec 7, 2017 at 7:05 PM, Sakari Ailus wrote: > Hi Pravin, > > On Wed, Dec 06, 2017 at 10:22:02PM +0530, Pravin Shedge wrote: >> These duplicate includes have been found with scripts/checkincludes.pl but >> they have been removed manually to avoid removing false positives. >> >> Signed-off-b

Re: [PATCH v1 0/7] iio: adc: ina2xx: Rework CNVR alignment, fix busy loops

2017-12-10 Thread Jonathan Cameron
On Fri, 8 Dec 2017 18:41:45 +0100 Stefan Brüns wrote: > Currently, the INA2xx driver may end up causing 100% load on a single core > and fully loading the I2C bus, which is caused by two different issues: > > The code uses a udelay to bridge the gab between two subsequent samples. > As the sampl

Re: [PATCH 2/2] iio: magnetometer: Add Infineon TLV493D-A1B6

2017-12-10 Thread Jonathan Cameron
On Sun, 10 Dec 2017 18:14:44 +0100 Andreas Färber wrote: > Hi, Some quick notes inline. Jonathan > > Am 10.12.2017 um 10:08 schrieb Peter Meerwald-Stadler: > >> The Infineon TLV493D is a 3D magnetic sensor, A1B6 is I2C based. > > > > comments below > > > >> It is found among others on t

[PATCH] TC: Delete an error message for a failed memory allocation in tc_bus_add_devices()

2017-12-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 10 Dec 2017 18:42:42 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/tc/tc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletion

Re: [PATCH 27/45] net: remove duplicate includes

2017-12-10 Thread Pravin Shedge
On Thu, Dec 7, 2017 at 2:28 AM, David Miller wrote: > From: Pravin Shedge > Date: Wed, 6 Dec 2017 23:02:58 +0530 > >> These duplicate includes have been found with scripts/checkincludes.pl but >> they have been removed manually to avoid removing false positives. >> >> Signed-off-by: Pravin Shedg

[PATCH 2/3 v5] battery: Add the ThinkPad "Not Charging" quirk

2017-12-10 Thread Ognjen Galic
The EC/ACPI firmware on Lenovo ThinkPads used to report a status of "Unknown" when the battery is between the charge start and charge stop thresholds. On Windows, it reports "Not Charging" so the quirk has been added to also report correctly. Now the "status" attribute returns "Not Charging" when

[PATCH 34/45] arch/blackfin: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/blackfin/kernel/irqchip.c| 1 - arch/blackfin/mach-common/ints-priority.c | 1 - 2 files changed, 2 dele

Re: [PATCH v3 1/2] iio: hx711: add triggered buffer support

2017-12-10 Thread Jonathan Cameron
On Wed, 6 Dec 2017 09:51:56 +0100 Andreas Klinger wrote: > Add buffer to device data struct and add trigger function > > Data format is quite simple: > voltage - channel 0 32 Bit > voltage - channel 1 32 Bit > timestamp 64 Bit > > Using both channels at the sam

[PATCH 35/45] arch/mips: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/mips/mm/init.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/mips/mm/init.c b/arch/mips/mm/init.c index

Re: [PATCH v2] iio: accel: bmc150: Check for a second ACPI device for BOSC0200

2017-12-10 Thread Jonathan Cameron
On Wed, 6 Dec 2017 17:52:34 + Jeremy Cline wrote: > Some BOSC0200 acpi_device-s describe two accelerometers in a single ACPI > device. Check for a companion device and handle a second i2c_client > if it is present. > > Signed-off-by: Jeremy Cline The requirement for this is still horrible,

[PATCH 36/45] arch/nios2: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/nios2/mm/fault.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/nios2/mm/fault.c b/arch/nios2/mm/fault.c in

[PATCH 37/45] arch/parisc: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/parisc/kernel/unwind.c | 1 - arch/parisc/lib/delay.c | 2 -- 2 files changed, 3 deletions(-) diff --git a/arch

[PATCH 39/45] arch/s390: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/s390/kernel/kprobes.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/s390/kernel/kprobes.c b/arch/s390/kern

Re: [PATCH v3 1/2] iio: hx711: add triggered buffer support

2017-12-10 Thread Jonathan Cameron
On Sun, 10 Dec 2017 18:20:39 + Jonathan Cameron wrote: > On Wed, 6 Dec 2017 09:51:56 +0100 > Andreas Klinger wrote: Sorry, hit send prematurely on this one. Few more points line, but looking much better. > > > Add buffer to device data struct and add trigger function > > > > Data format i

[PATCH 40/45] arch/tile: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/tile/kernel/single_step.c | 1 - arch/tile/kernel/unaligned.c | 1 - 2 files changed, 2 deletions(-) diff --git a

Re: [PATCH v3 2/2] iio: hx711: fix bug in reset functionality

2017-12-10 Thread Jonathan Cameron
On Wed, 6 Dec 2017 09:52:20 +0100 Andreas Klinger wrote: > Return value in hx711_reset() should indicate status of dout otherwise the > calling function is reporting an error as false positive > > If there are two reads too close to each other, then the second one will > never succeed. This happ

[PATCH 42/45] sample/sockmap: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- samples/sockmap/sockmap_user.c | 1 - 1 file changed, 1 deletion(-) diff --git a/samples/sockmap/sockmap_user.c b/sample

Re: [PATCH v4] leds: trigger: Introduce a NETDEV trigger

2017-12-10 Thread Jacek Anaszewski
Hi Ben, Thanks for the update. I have one doubt about comment style at the top of the file. Please refer below. On 12/10/2017 05:24 PM, Ben Whitten wrote: > This commit introduces a NETDEV trigger for named device > activity. Available triggers are link, rx, and tx. > > Signed-off-by: Ben Whitte

[PATCH 43/45] sound/soc/zte: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- sound/soc/zte/zx-i2s.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/sound/soc/zte/zx-i2s.c b/sound/soc/zte/zx-i2

[PATCH 44/45] arch/sh: remove duplicate includes

2017-12-10 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- arch/sh/boards/mach-dreamcast/irq.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/sh/boards/mach-dreamcast/irq.

objtool: Use value of intended enum to compile with clang

2017-12-10 Thread Lukas Bulwahn
Hi, Please include this patch in tools/objtool to properly compile with clang-5.0. I successfully compiled this patch (make defconfig && make) on top of tag next-20171208 from linux-next with clang-5.0.0 and gcc (Debian 7.2.0-16) 7.2.0, but I did not test anything further than that. Exact compil

[PATCH] objtool: Use value of intended enum to compile with clang

2017-12-10 Thread Lukas Bulwahn
Just use the right value and avoid an implicit conversion between the two enumeration types that just happened to work in this case. It seems that this must have just been overlooked in the new implementation of objtool in commit baa41469a7b9 ("objtool: Implement stack validation 2.0"). I found th

Re: [PATCH v3 1/2] eeprom: at24: fix coding style issues

2017-12-10 Thread Bartosz Golaszewski
2017-12-10 13:57 GMT+01:00 Andy Shevchenko : > On Thu, Dec 7, 2017 at 3:39 PM, Bartosz Golaszewski wrote: >> Fix issues reported by checkpatch for at24.c. > >> +module_param(io_limit, uint, ); > >> +module_param(write_timeout, uint, ); > > > 0 is a pretty much octal number as . > So, I

Re: [PATCH tip/core/rcu 02/20] torture: Prepare scripting for shift from %p to %pK

2017-12-10 Thread Paul E. McKenney
On Sun, Dec 10, 2017 at 02:52:49PM +0200, Andy Shevchenko wrote: > On Mon, Dec 4, 2017 at 6:11 PM, Paul E. McKenney > wrote: > > On Mon, Dec 04, 2017 at 02:13:38PM +, David Laight wrote: > >> From: Paul E. McKenney > >> > Sent: 04 December 2017 13:42 > >> > On Mon, Dec 04, 2017 at 12:32:30PM +

Re: Linux 4.15-rc2: Regression in resume from ACPI S3

2017-12-10 Thread Pavel Machek
On Sun 2017-12-10 08:37:56, Linus Torvalds wrote: > On Sun, Dec 10, 2017 at 8:23 AM, Pavel Machek wrote: > > > > Can you do something about html emails? Quoting them doesn't work too well. > > Yeah, and they don't show up onlkml either because of rules. I try to > avoid them, but have been more o

Re: [PATCH v4 3/5] staging: Introduce NVIDIA Tegra video decoder driver

2017-12-10 Thread Dmitry Osipenko
On 05.12.2017 16:03, Hans Verkuil wrote: > On 12/05/17 13:17, Dmitry Osipenko wrote: >> Hi Hans, >> >> On 04.12.2017 17:04, Hans Verkuil wrote: >>> Hi Dmitry, >>> >>> As you already mention in the TODO, this should become a v4l2 codec driver. >>> >>> Good existing examples are the coda, qcom/venus

Re: [PATCH] eeprom: at24: don't use the default regmap locking

2017-12-10 Thread Bartosz Golaszewski
2017-12-03 22:47 GMT+01:00 Bartosz Golaszewski : > We need to always wait for the last write to complete and we want the > regmap to be protected from concurrent access during that wait. That > makes using the regmap's default locking impossible and we're > protecting it with an external mutex. > >

Re: [PATCH v2] iio: mma8452: replace license description with SPDX specifier

2017-12-10 Thread Philippe Ombredanne
On Sun, Dec 10, 2017 at 4:08 PM, Jonathan Cameron wrote: > On Sat, 18 Nov 2017 23:23:57 -0500 > harinath Nampally wrote: > >> > This replaces the custom license information text with the appropriate >> > SPDX identifier. While the information here stays the same, it is easier >> > to read. >> > S

Re: [PATCH] iio: stm32: Adopt SPDX identifier

2017-12-10 Thread Philippe Ombredanne
On Sun, Dec 10, 2017 at 7:31 PM, Jonathan Cameron wrote: > On Thu, 7 Dec 2017 11:10:44 +0100 > Philippe Ombredanne wrote: > >> On Tue, Dec 5, 2017 at 3:55 PM, Benjamin Gaignard >> wrote: >> > Add SPDX identifier in stm32's files in IIO directory >> > >> > Signed-off-by: Benjamin Gaignard > Appl

  1   2   3   >