Re: [PATCH v7 0/8]/[PULL REQUEST] Trim unused exported kernel symbols

2016-04-20 Thread Michal Marek
Dne 19.4.2016 v 05:32 Nicolas Pitre napsal(a): > Ping ! > > /me only hears back empty cave echoes ... Pulled now, sorry for the delay. Michal

Re: [Cocci] coccinelle: bool if (foo) return true; else return false;

2016-04-20 Thread Michael Stefaniuc
On 04/19/2016 09:15 PM, Julia Lawall wrote: > > > On Tue, 19 Apr 2016, Joe Perches wrote: > >> There's ~150 of these in the kernel. >> >> Maybe there's use for this conversion to be added >> to scripts/coccinelle/misc/boolreturn.cocci or in >> a separate file. >> >> $ cat booltruefalse.cocci >>

[PATCH 1/2] Documentation: DT: vdma: Add clock support for vdma

2016-04-20 Thread Kedareswara rao Appana
This patch updates the binding doc with clock description for vdma. Signed-off-by: Kedareswara rao Appana --- Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt | 6 ++ 1 file changed, 6 insertions(+) diff --git a/Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt

[PATCH 2/2] dmaengine: vdma: Add clock support

2016-04-20 Thread Kedareswara rao Appana
Added basic clock support. The clocks are requested at probe and released at remove. Signed-off-by: Kedareswara rao Appana --- drivers/dma/xilinx/xilinx_vdma.c | 56 1 file changed, 56 insertions(+) diff --git a/drivers/dma/xilinx/xilinx_vdma.c

Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

2016-04-20 Thread Olliver Schinagl
Hey Ricardo, On 19-04-16 15:42, Ricardo Ribalda Delgado wrote: Hi again On Tue, Apr 19, 2016 at 3:27 PM, Olliver Schinagl wrote: Hey Ricardo, Without actually looking at the code right now, but the driver does a read/modify/write on the register, and a register is shared among several leds.

Re: [PATCH V2 1/2] scripts: genksyms: fix resource leak

2016-04-20 Thread Michal Marek
On Tue, Apr 12, 2016 at 11:54:59PM +0300, Maxim Zhukov wrote: > This commit fixed resource leak at func main > > Signed-off-by: Maxim Zhukov Applied to kbuild.git#kbuild. Although the leak is harmless. Michal

Re: [Cocci] coccinelle: bool if (foo) return true; else return false;

2016-04-20 Thread Joe Perches
On Wed, 2016-04-20 at 09:19 +0200, Michael Stefaniuc wrote: > On 04/19/2016 09:15 PM, Julia Lawall wrote: > > On Tue, 19 Apr 2016, Joe Perches wrote: > > > There's ~150 of these in the kernel. > > > > > > Maybe there's use for this conversion to be added > > > to

Re: [PATCH] ACPI / amba: Remove CLK_IS_ROOT

2016-04-20 Thread G Gregory
On 20 April 2016 at 02:28, Stephen Boyd wrote: > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate > CLK_IS_ROOT", 2016-02-02) so remove it. > > Cc: Graeme Gregory > Cc: Aleksey Makarov > Signed-off-by: Stephen Boyd > --- > drivers/acpi/acpi_amba.c | 3 +-- > 1 file changed, 1

Re: [PATCH v2] pinctrl-exynos5440: Use off-stack memory for pinctrl_gpio_range

2016-04-20 Thread Andrew Jeffery
On Wed, 2016-04-20 at 08:56 +0200, Krzysztof Kozlowski wrote: > When you resubmit patches (unless there is a significant change in the > patch) please always put accumulated tags after existing tags. Okay, I wasn't sure about that. Thanks for the clarification and I'll do so in the future.

Re: [PATCH v2 1/4] scatterlist: Introduce some helper functions

2016-04-20 Thread Baolin Wang
Hi Robert, On 5 April 2016 at 15:10, Baolin Wang wrote: > Hi Robert, > > Sorry for the late reply. > > On 2 April 2016 at 23:00, Robert Jarzmik wrote: >> Baolin Wang writes: >> >>> +/** >>> + * sg_is_contiguous - Check if the scatterlists are contiguous >>> + * @sga: SG entry >>> + * @sgb: SG

Re: [PATCH] ARM: mv78xx0: Remove CLK_IS_ROOT

2016-04-20 Thread Arnd Bergmann
On Tuesday 19 April 2016 18:40:04 Stephen Boyd wrote: > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate > CLK_IS_ROOT", 2016-02-02) so remove it. > > Cc: Andrew Lunn > Cc: Jason Cooper Signed-off-by: Stephen Boyd > Looks good, like all other instances in arch/arm. If you

Re: [PATCH v2] mmc: sdhci-pci: add Support of Synopsys DWC_MSHC IP

2016-04-20 Thread Jaehoon Chung
Hi Pradu, On 04/20/2016 04:00 PM, Prabu Thangamuthu wrote: > Hi Jaehoon Chung, > > Thank you for detailed review comments. > > On 04/19/2016 04:31 PM, Jaehoon Chung wrote: >> On 04/19/2016 04:18 PM, Prabu Thangamuthu wrote: >>> Synopsys DWC_MSHC is compliant with SD Host Specifications. This

Re: [PATCH] kernel/futex: handle the case where we got a "late" waiter

2016-04-20 Thread Thomas Gleixner
On Tue, 19 Apr 2016, Davidlohr Bueso wrote: > On Fri, 15 Apr 2016, Sebastian Andrzej Siewior wrote: > > > futex_unlock_pi() gets uval before taking the hb lock. Now imagine > > someone in futex_lock_pi() took the lock. While futex_unlock_pi() waits > > for the hb lock, the LOCK_PI sets

[PATCH] parport: parport_pc: PCI SIO access should also depend on SIO option

2016-04-20 Thread Sudip Mukherjee
From: "Maciej S. Szmigiero" CONFIG_PARPORT_PC_SUPERIO toggles Super IO chip support in parport_pc code, however only code accessing SIO chip via ISA (or LPC) bus was conditional on it. This patch makes SIO chip accesses via PCI bus also dependent on this config option. It should be noted that

Re: [PATCH 0/8] STMPE fixes/rework and add STMPE1600 support

2016-04-20 Thread Patrice Chotard
On 04/19/2016 05:53 PM, Stephen Warren wrote: On 04/19/2016 06:18 AM, patrice.chot...@st.com wrote: From: Patrice Chotard This series cleans and fixes some bugs in MFD/GPIO STMPE drivers and prepare the ground to add new STMPE1600 support. STMPE1600 datasheet is available here :

Re: [BUG linux-next] Kernel panic found with linux-next-20160414

2016-04-20 Thread Vladimir Murzin
CC LAKML in case somebody hit the same panic there. Vladimir On 20/04/16 00:43, Shi, Yang wrote: > Hi folks, > > When I ran ltp on linux-next-20160414 on my ARM64 machine, I got the > below kernel panic: > > Unable to handle kernel paging request at virtual address ffc007846000 > pgd =

Re: [PATCH v8 6/7] power: cros_usbpd-charger: Add EC-based USB PD charger driver

2016-04-20 Thread Tomeu Vizoso
Hi Sebastian, is there any chance that you could review this patch? Thanks, Tomeu On 12 April 2016 at 14:32, Tomeu Vizoso wrote: > From: Sameer Nanda > > This driver exposes the charger functionality in the PD EC to userspace. > > Signed-off-by: Tomeu Vizoso > Cc: Sameer Nanda > Cc: Benson

Re: [PATCH v5 12/14] arm64, acpi, numa: NUMA support based on SRAT and SLIT

2016-04-20 Thread Dennis Chen
On 20 April 2016 at 09:40, David Daney wrote: > From: Hanjun Guo > > Introduce a new file to hold ACPI based NUMA information parsing from > SRAT and SLIT. > > SRAT includes the CPU ACPI ID to Proximity Domain mappings and memory > ranges to Proximity Domain mapping. SLIT has the information of

Re: next-20160401+: ARM: DRA7: linux-next regression: mm/slab: clean-up kmem_cache_node setup

2016-04-20 Thread Joonsoo Kim
Ccing Stephen. On Wed, Apr 20, 2016 at 08:11:41AM +0100, Jon Hunter wrote: > Hi Joonsoo, > > On 11/04/16 12:44, Jon Hunter wrote: > > On 11/04/16 03:02, Joonsoo Kim wrote: > >> On Fri, Apr 08, 2016 at 03:39:20PM -0500, Nishanth Menon wrote: > >>> Hi, > >>> > >>>

Re: [PATCH v5 1/6] perf tools: Derive trigger class from auxtrace_snapshot

2016-04-20 Thread Jiri Olsa
On Mon, Apr 18, 2016 at 02:55:27PM +, Wang Nan wrote: SNIP > -static volatile int auxtrace_snapshot_err; > static volatile int auxtrace_record__snapshot_started; > +static DEFINE_TRIGGER(auxtrace_snapshot); > > static void sig_handler(int sig) > { > @@ -282,11 +247,12 @@ static void

Re: [PATCH v7 7/8] genirq/msi: map/unmap the MSI doorbells on msi_domain_alloc/free_irqs

2016-04-20 Thread Eric Auger
o the wrong git tree, please drop us a note to > help improving the system] > > url: > https://github.com/0day-ci/linux/commits/Eric-Auger/KVM-PCIe-MSI-passthrough-on-ARM-ARM64-kernel-part-2-3-msi-changes/20160420-011957 > config: x86_64-lkp (attached as .config) > reprodu

[PATCH v1] IB/ipoib: Add readout of statistics using ethtool

2016-04-20 Thread Hans Westgaard Ry
IPoIB collects statistics of traffic including number of packets sent/received, number of bytes transferred, and certain errors. This patch makes these statistics available to be queried by ethtool. Signed-off-by: Hans Westgaard Ry Reviewed-by: Yuval Shaia Reviewed-by: Santosh Shilimkar

[PATCH] SCSI: LIBSCSI: Fixed codeing style errors

2016-04-20 Thread Bob Stlt
Fixed codeing style formatting errors. Signed-off-by: Bob Stlt --- drivers/scsi/libiscsi.c | 90 - 1 file changed, 45 insertions(+), 45 deletions(-) diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c index 6bffd91..41be9d3 100644 ---

Re: [PATCH] Coccinelle: setup_timer: Add space in front of parentheses

2016-04-20 Thread Michal Marek
Dne 20.3.2016 v 13:25 Julia Lawall napsal(a): > > > On Sun, 20 Mar 2016, Vaishali Thakkar wrote: > >> Add space in front of the offending parentheses to silent the >> parse error for older Coccinelle versions. This makes the rule >> usable with all Coccinelle versions. >> >> Reported-by:

Re: [PATCH v5 2/6] perf record: Split output into multiple files via '--switch-output'

2016-04-20 Thread Jiri Olsa
On Mon, Apr 18, 2016 at 02:55:28PM +, Wang Nan wrote: SNIP > "Record build-id of all DSOs regardless of hits"), > OPT_BOOLEAN(0, "timestamp-filename", _filename, > "append timestamp to output filename"), > + OPT_BOOLEAN(0, "switch-output",

Re: sound: use-after-free in snd_timer_interrupt

2016-04-20 Thread Dmitry Vyukov
On Sun, Apr 3, 2016 at 8:33 AM, Takashi Iwai wrote: >> >> It is not easily reproducible. I've hit several times while running >> >> fuzzer for a week. Here is one of the logs for the record: >> >>

Re: [PATCH] scripts: coccinelle: remove check to move constants to right

2016-04-20 Thread Michal Marek
On Sat, Mar 19, 2016 at 06:43:08PM +0100, Julia Lawall wrote: > On Sat, 19 Mar 2016, Wolfram Sang wrote: > > > The header mentions this check depends on personal taste. I agree. > > Running coccicheck on patches before I apply them, this SmPL produced > > enough false positives for me that I'd

Re: [PATCH 3/3] RTC: Update seconds time programming logic

2016-04-20 Thread Alexandre Belloni
On 20/04/2016 at 07:10:22 +, Anurag Kumar Vulisha wrote : > The reason for me keeping this logic is, our RTC controller updates the read > register after 1 sec > delay, so when read , it gives 1 sec delay(correct time - 1 sec). So to avoid > that we are programming > load time + 1sec into

Re: [PATCH v1] IB/ipoib: Add readout of statistics using ethtool

2016-04-20 Thread Joe Perches
On Wed, 2016-04-20 at 09:50 +0200, Hans Westgaard Ry wrote: > IPoIB collects statistics of traffic including number of packets > sent/received, number of bytes transferred, and certain errors. This > patch makes these statistics available to be queried by ethtool. trivial notes: > diff --git

Re: [PATCH v2] carl9170: Clarify kconfig text

2016-04-20 Thread Kalle Valo
Christian Lamparter writes: > On Monday, April 18, 2016 07:42:05 PM Kalle Valo wrote: >> Christian Lamparter writes: >> >> > On Monday, April 18, 2016 06:45:09 PM Kalle Valo wrote: >> > >> >> Why even mention anything about a "special firmware" as the firmware is >> >> already available from

Re: [PATCH] rtc: pcf8563: Remove CLK_IS_ROOT

2016-04-20 Thread Alexandre Belloni
On 19/04/2016 at 18:12:19 -0700, Stephen Boyd wrote : > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate > CLK_IS_ROOT", 2016-02-02) so remove it. > > Cc: Heiko Schocher > Signed-off-by: Stephen Boyd > --- > drivers/rtc/rtc-pcf8563.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH 1/2] Documentation: DT: vdma: Add clock support for vdma

2016-04-20 Thread Shubhrajyoti Datta
On Wed, Apr 20, 2016 at 12:49 PM, Kedareswara rao Appana wrote: > This patch updates the binding doc with clock description > for vdma. > > Signed-off-by: Kedareswara rao Appana > --- > Documentation/devicetree/bindings/dma/xilinx/xilinx_vdma.txt | 6 ++ > 1 file changed, 6 insertions(+) >

Re: [PATCH 2/3] sched: Correctly handle nohz ticks cpu load accounting

2016-04-20 Thread Wanpeng Li
Hi Frederic, 2016-04-13 21:56 GMT+08:00 Frederic Weisbecker : > Ticks can happen while the CPU is in dynticks-idle or dynticks-singletask > mode. In fact "nohz" or "dynticks" only mean that we exit the periodic > mode and we try to minimize the ticks as much as possible. The nohz > subsystem uses

Re: [PATCH] rtc: ds1307: Remove CLK_IS_ROOT

2016-04-20 Thread Alexandre Belloni
On 19/04/2016 at 18:15:48 -0700, Stephen Boyd wrote : > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate > CLK_IS_ROOT", 2016-02-02) so remove it. > > Cc: Akinobu Mita > Cc: Michael Tatarinov > Signed-off-by: Stephen Boyd > --- > drivers/rtc/rtc-ds1307.c | 2 -- > 1 file

Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

2016-04-20 Thread Ricardo Ribalda Delgado
Hi Ollivier On Wed, Apr 20, 2016 at 9:21 AM, Olliver Schinagl wrote: What I am propossing is at probe(): replace: if (pdata) { /* Configure output: open-drain or totem pole (push-pull) */ if (pdata->outdrv == PCA963X_OPEN_DRAIN) i2c_smbus_write_byte_data(client, PCA963X_MODE2, 0x01); else

Re: [PATCH v5 3/6] perf record: Force enable --timestamp-filename when --switch-output is provided

2016-04-20 Thread Jiri Olsa
On Mon, Apr 18, 2016 at 02:55:29PM +, Wang Nan wrote: > Without this patch, the last output doesn't have timestamp appended if > --timestamp-filename is not explicitly provided. For example: > > # perf record -a --switch-output & > [1] 11224 > # kill -s SIGUSR2 11224 > [ perf record:

Re: [PATCH] rtc: hym8563: Remove CLK_IS_ROOT

2016-04-20 Thread Alexandre Belloni
On 19/04/2016 at 18:13:58 -0700, Stephen Boyd wrote : > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate > CLK_IS_ROOT", 2016-02-02) so remove it. > > Cc: Heiko Stuebner > Signed-off-by: Stephen Boyd > --- > drivers/rtc/rtc-hym8563.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [BUG linux-next] Kernel panic found with linux-next-20160414

2016-04-20 Thread Hugh Dickins
On Tue, 19 Apr 2016, Shi, Yang wrote: > Hi folks, > > When I ran ltp on linux-next-20160414 on my ARM64 machine, I got the below > kernel panic: > > Unable to handle kernel paging request at virtual address ffc007846000 > pgd = ffc01e21d000 > [ffc007846000] *pgd=,

[PATCH] arm: dts: mt2701: Use real clock for UARTs

2016-04-20 Thread Erin Lo
We used to use a fixed rate clock for the UARTs. Now that we have clock support we can associate the correct clocks to the UARTs and drop the 26MHz fixed rate UART clock. Signed-off-by: Erin Lo --- This patch dependents on "Add clock support for Mediatek MT2701"[1]. If we have clock support, we

Re: [PATCH] kbuild/mkspec: support 'update-bootloader'-based systems

2016-04-20 Thread Michal Marek
On Thu, Apr 07, 2016 at 03:00:54PM +0200, Jiri Kosina wrote: > From: Jiri Kosina > > When uninstalling kernel RPM, we're unconditionally calling > "new-kernel-pkg --remove". There is of course a test whether the script exists. > This is useless on systems which are based on >

Re: [PATCH v2] mm: SLAB freelist randomization

2016-04-20 Thread Joonsoo Kim
On Tue, Apr 19, 2016 at 09:44:54AM -0700, Thomas Garnier wrote: > On Tue, Apr 19, 2016 at 12:15 AM, Joonsoo Kim wrote: > > On Mon, Apr 18, 2016 at 10:14:39AM -0700, Thomas Garnier wrote: > >> Provides an optional config (CONFIG_FREELIST_RANDOM) to randomize the > >> SLAB freelist. The list is

[char-misc-next 1/2] mei: don't clean control queues on notify request timeout

2016-04-20 Thread Tomas Winkler
From: Alexander Usyskin Timeout on notify request is not a fatal condition, and actually cleaning control queues will disrupt other control flows of the same client. Signed-off-by: Alexander Usyskin Signed-off-by: Tomas Winkler --- drivers/misc/mei/client.c | 8 ++-- 1 file changed, 2

[char-misc-next 2/2] mei: fix waiting for wr_ctrl for corner cases.

2016-04-20 Thread Tomas Winkler
From: Alexander Usyskin A control message reply may not be received if either a link reset has occurred or disconnection is initiated by the FW. In the both cases the client state will be set straight to DISCONNECTED and the driver will wait till timeout. Adding DISCONNECTED state in the waiting

[PATCH 2/2] regulator: fan53555: Add support for FAN53555UC13X type

2016-04-20 Thread Wadim Egorov
IC type options 00, 13 and 23 are sharing the same DIE_ID 0. Let's differentiate between these revisions. FAN53555UC13X has the ID 0 and REV 0xf, starts at 800mV and increments in 10mV steps. Signed-off-by: Wadim Egorov --- drivers/regulator/fan53555.c | 22 ++ 1 file

[PATCH 1/2] regulator: fan53555: Add support for FAN53555BUC18X type

2016-04-20 Thread Wadim Egorov
FAN53555BUC18X has the DIE_ID 8, starts at 600mV and increments in 10mV steps. Signed-off-by: Wadim Egorov --- drivers/regulator/fan53555.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/regulator/fan53555.c b/drivers/regulator/fan53555.c index 2cb5cc3..d4bfa93 100644 ---

Re: sound: use-after-free in snd_timer_interrupt

2016-04-20 Thread Takashi Iwai
On Wed, 20 Apr 2016 09:56:04 +0200, Dmitry Vyukov wrote: > > On Sun, Apr 3, 2016 at 8:33 AM, Takashi Iwai wrote: > >> >> It is not easily reproducible. I've hit several times while running > >> >> fuzzer for a week. Here is one of the logs for the record: > >> >>

Re: linux-next crash during very early boot

2016-04-20 Thread Joonsoo Kim
On Fri, Apr 15, 2016 at 10:10:33AM -0400, valdis.kletni...@vt.edu wrote: > On Thu, 14 Apr 2016 10:35:47 +0900, Joonsoo Kim said: > > On Wed, Apr 13, 2016 at 08:29:46PM -0400, Valdis Kletnieks wrote: > > > I'm seeing my laptop crash/wedge up/something during very early > > > boot - before it can

Re: [PATCH v6 06/12] usb: otg: get rid of CONFIG_USB_OTG_FSM in favour of CONFIG_USB_OTG

2016-04-20 Thread Roger Quadros
On 18/04/16 11:05, Peter Chen wrote: > On Tue, Apr 05, 2016 at 05:05:11PM +0300, Roger Quadros wrote: >> Let's use CONFIG_USB_OTG as a single config option to enable >> USB OTG and the OTG FSM. This makes things a lot less confusing. >> >> Update all users of CONFIG_USB_OTG_FSM to CONFIG_USB_OTG.

Re: [PATCH v6 01/12] usb: hcd: Initialize hcd->flags to 0

2016-04-20 Thread Roger Quadros
On 19/04/16 04:56, Peter Chen wrote: > On Mon, Apr 18, 2016 at 10:11:29AM -0400, Alan Stern wrote: >> On Mon, 18 Apr 2016, Peter Chen wrote: >> >>> On Wed, Apr 06, 2016 at 09:32:22AM +0300, Roger Quadros wrote: On 06/04/16 09:09, Felipe Balbi wrote: > > Hi, > > Roger Quadros

[PATCH v1 1/1] powerpc/86xx: Mode pci1 definition to the include file

2016-04-20 Thread Alessio Igor Bogani
Signed-off-by: Alessio Igor Bogani --- Please note that this patch replaces completely https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-April/141785.html arch/powerpc/boot/dts/fsl/gef_ppc9a.dts| 4 arch/powerpc/boot/dts/fsl/gef_sbc310.dts | 22 -

[PATCH] ARM: dts: AM57xx/DRA7: Update SoC voltage rail limits to match data sheet

2016-04-20 Thread Nishanth Menon
As per the data sheet starting from SPRUHQ0H (Nov 2015 - Latest[1]), VDD_CORE can vary from 0.85v to 1.15v for AVS class0. VDD GPU/DSP et.al. can range from 0.85v to 1.25V with AVS class0 Since dynamic voltage scaling is disabled for DRA7/AM57xx SoCs for all SoC rails other than MPU, the

Re: [PATCH v5 3/6] perf record: Force enable --timestamp-filename when --switch-output is provided

2016-04-20 Thread Wangnan (F)
On 2016/4/20 15:59, Jiri Olsa wrote: On Mon, Apr 18, 2016 at 02:55:29PM +, Wang Nan wrote: Without this patch, the last output doesn't have timestamp appended if --timestamp-filename is not explicitly provided. For example: # perf record -a --switch-output & [1] 11224 # kill -s

Re: [PATCH] kbuild: drop FORCE from PHONY targets

2016-04-20 Thread Michal Marek
Dne 13.3.2016 v 01:13 Masahiro Yamada napsal(a): > These targets are marked as PHONY. No need to add FORCE to their > dependency. > > Signed-off-by: Masahiro Yamada > --- > > Makefile | 8 > arch/arm/vdso/Makefile | 2 +- > arch/ia64/Makefile | 4

Re: [PATCH] kbuild: specify modules(_install) as PHONY rather than FORCE

2016-04-20 Thread Michal Marek
On Sun, Mar 13, 2016 at 09:39:22AM +0900, Masahiro Yamada wrote: > As in other places, PHONY is a better fit for "modules" and > "modules_install". > > Signed-off-by: Masahiro Yamada Applied to kbuild.git#kbuild. Michal

Re: [PATCH 2/2 v6] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver

2016-04-20 Thread Mark Brown
On Wed, Apr 20, 2016 at 02:40:47PM +0800, PC Liao wrote: > +Optional properties: > +- capture-dai: audio codec dai on capture path > + < 0> : Default setting. Connect rt5650 I2S1 for capture. (dai_name > = rt5645-aif1) > + < 1> : Connect rt5650 I2S2 for capture. (dai_name = rt5645-aif2) I

Re: [PATCH] kbuild: mark help target as PHONY

2016-04-20 Thread Michal Marek
On Sun, Mar 13, 2016 at 09:39:55AM +0900, Masahiro Yamada wrote: > Obviously, the "help" should be a PHONY target. > > Signed-off-by: Masahiro Yamada Applied to kbuild.git#kbuild. Michal

Re: [PATCH v5 12/14] arm64, acpi, numa: NUMA support based on SRAT and SLIT

2016-04-20 Thread Ganapatrao Kulkarni
On Wed, Apr 20, 2016 at 1:11 PM, Dennis Chen wrote: > On 20 April 2016 at 09:40, David Daney wrote: >> From: Hanjun Guo >> >> Introduce a new file to hold ACPI based NUMA information parsing from >> SRAT and SLIT. >> >> SRAT includes the CPU ACPI ID to Proximity Domain mappings and memory >>

Re: [PATCHv7 00/29] THP-enabled tmpfs/shmem using compound pages

2016-04-20 Thread Hugh Dickins
On Mon, 18 Apr 2016, Shi, Yang wrote: > Hi Kirill, > > Finally, I got some time to look into and try yours and Hugh's patches, got Thank you. > two problems. > > 1. A quick boot up test on my ARM64 machine with your v7 tree shows some > unexpected error: > > systemd-journald[285]: Failed to

Re: [PATCH v2] pinctrl-exynos5440: Use off-stack memory for pinctrl_gpio_range

2016-04-20 Thread Tomasz Figa
2016-04-20 15:56 GMT+09:00 Krzysztof Kozlowski : > On 04/20/2016 03:54 AM, Andrew Jeffery wrote: >> The range is registered into a linked list which can be referenced >> throughout the lifetime of the driver. Ensure the range's memory is useful >> for the same lifetime by adding it to the driver's

[PATCH] lib: Always NUL terminate ucs2_as_utf8

2016-04-20 Thread Chris Wilson
If the caller, in this case efivarfs_callback(), only provides sufficent room for the expanded utf8 and not enough to include the terminating NUL byte, that NUL byte is skipped. When the caller then interprets it as a string, it may then read from past its allocated memory: [ 170.605647]

Re: [PATCH] kbuild: delete unnecessary "@:"

2016-04-20 Thread Michal Marek
On Thu, Mar 31, 2016 at 08:14:16PM +0900, Masahiro Yamada wrote: > Since commit 2aedcd098a94 ('kbuild: suppress annoying "... is up to > date." message'), $(call if_changed,...) is evaluated to "@:" > when there is nothing to do. > > We no longer need to add "@:" after $(call if_changed,...) to >

RE: [PATCH 1/2] Documentation: DT: vdma: Add clock support for vdma

2016-04-20 Thread Appana Durga Kedareswara Rao
Hi Shubhrajyoti, > -Original Message- > From: Shubhrajyoti Datta [mailto:shubhrajyoti.da...@gmail.com] > Sent: Wednesday, April 20, 2016 1:30 PM > To: Appana Durga Kedareswara Rao > Cc: Rob Herring ; Pawel Moll ; > Mark Rutland ; Ian Campbell > ; Kumar Gala ; Michal > Simek ; Soren

Re: [PATCH 4/4] drivers: firmware: psci: use const and __initconst for psci_cpuidle_ops

2016-04-20 Thread Lorenzo Pieralisi
On Tue, Mar 22, 2016 at 10:42:43PM +0800, Jisheng Zhang wrote: > The psci_cpuidle_ops structures is not over-written, so add "const" > qualifier and replace __initdata with __initconst. > > Signed-off-by: Jisheng Zhang > --- > drivers/firmware/psci.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH v2 1/4] scatterlist: Introduce some helper functions

2016-04-20 Thread Herbert Xu
On Wed, Apr 20, 2016 at 03:34:47PM +0800, Baolin Wang wrote: > It seems there are no more comments about this patchset? We really > want to these APIs to coalesce scatterlists to improve the crypto > engine efficiency. Thanks. Your patch-set makes no sense. If the data can be coalesced then it

Re: [PATCH v2 1/2] clk: Add Oxford Semiconductor OXNAS Standard Clocks

2016-04-20 Thread Neil Armstrong
Hi Stephen, On 04/19/2016 11:29 PM, Stephen Boyd wrote: > On 04/18, Neil Armstrong wrote: >> + >> +static int oxnas_stdclk_probe(struct platform_device *pdev) >> +{ >> +struct device_node *np = pdev->dev.of_node; >> +struct clk_oxnas_data *clk_oxnas; >> +struct regmap *regmap; >> +

Re: [PATCH] kbuild: drop redundant "PHONY += FORCE"

2016-04-20 Thread Michal Marek
On Fri, Apr 08, 2016 at 11:16:10AM +0900, Masahiro Yamada wrote: > "PHONY += FORCE" is already cared by scripts/Makefile.build, > which these files are included from. arch/ia64/Makefile is in fact included via the main Makefile. But this one also has the PHONY += FORCE statement. Maybe we should

Re: [PATCH] phy: rockhip-usb: Remove CLK_IS_ROOT

2016-04-20 Thread Heiko Stübner
Am Dienstag, 19. April 2016, 18:17:26 schrieb Stephen Boyd: > This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate > CLK_IS_ROOT", 2016-02-02) so remove it. > > Cc: Heiko Stuebner > Signed-off-by: Stephen Boyd obviously correct, but for what it's worth Reviewed-by: Heiko Stuebner

Re: [PATCH 1/2] kbuild: rename cmd_cc_i_c to cmd_cpp_i_c

2016-04-20 Thread Michal Marek
On Fri, Apr 08, 2016 at 12:00:57PM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Apr 08, 2016 at 11:24:47AM +0900, Masahiro Yamada escreveu: > > This command just preprocesses .c files into .i files, so cmd_cpp_i_c > > seems more suitable. > > Thanks for doing it to the tools/build/ copy,

RE: [PATCH v2 0/1] drivers: net: cpsw: Fix NULL pointer dereference with two slave PHYs

2016-04-20 Thread Andrew Goodbody
> -Original Message- > From: Andrew Goodbody > Date: Mon, 18 Apr 2016 14:53:25 +0100 > > > This is a fix for a NULL pointer dereference from cpsw which is > > triggered by having two slave PHYs attached to a cpsw network device. > > The problem is due to only maintaining a single

Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

2016-04-20 Thread Olliver Schinagl
Hey Ricardo, On 20-04-16 10:01, Ricardo Ribalda Delgado wrote: Hi Ollivier On Wed, Apr 20, 2016 at 9:21 AM, Olliver Schinagl wrote: What I am propossing is at probe(): replace: if (pdata) { /* Configure output: open-drain or totem pole (push-pull) */ if (pdata->outdrv ==

[PATCH] kbuild: Get rid of KBUILD_STR

2016-04-20 Thread Michal Marek
The compiler can accept -DKBUILD_MODNAME="foo", it's just a matter of quoting. That way, we reduce the gcc command line a bit. Signed-off-by: Michal Marek --- scripts/Makefile.lib | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/Makefile.lib

Re: [PATCH] vexpress/spc: Remove CLK_IS_ROOT

2016-04-20 Thread Sudeep Holla
On 20/04/16 02:36, Stephen Boyd wrote: This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate CLK_IS_ROOT", 2016-02-02) so remove it. Signed-off-by: Stephen Boyd FWIW: Acked-by: Sudeep Holla -- Regards, Sudeep

Re: [Letux-kernel] [PATCH] drivers: led: is31fl319x: 6/9-channel light effect led driver

2016-04-20 Thread H. Nikolaus Schaller
Hi Jacek, >>> +static void is31fl319x_brightness_set(struct led_classdev *led_cdev, >>> + enum led_brightness brightness) >>> +{ >>> + struct is31fl319x_led *led = container_of(led_cdev, >>> + struct is31fl319x_led, >>> +

[PATCH 1/2] net: ethernet: davinci_emac: Fix Unbalanced pm_runtime_enable

2016-04-20 Thread Neil Armstrong
In order to avoid an Unbalanced pm_runtime_enable in the DaVinci emac driver when the device is removed and re-probed, and a pm_runtime_disable() call in davinci_emac_remove(). Actually, using unbind/bind on a TI DM8168 SoC gives : $ echo 4a12.ethernet >

[PATCH 2/2] net: ethernet: davinci_emac: Fix platform_data overwrite

2016-04-20 Thread Neil Armstrong
When the DaVinci emac driver is removed and re-probed, the actual pdev->dev.platform_data is populated with an unwanted valid pointer saved by the previous davinci_emac_of_get_pdata() call, causing a kernel crash when calling priv->int_disable() in emac_int_disable(). Unable to handle kernel

Re: [PATCH] kbuild: drop redundant "PHONY += FORCE"

2016-04-20 Thread Masahiro Yamada
Hi Michal, 2016-04-20 17:47 GMT+09:00 Michal Marek : > On Fri, Apr 08, 2016 at 11:16:10AM +0900, Masahiro Yamada wrote: >> "PHONY += FORCE" is already cared by scripts/Makefile.build, >> which these files are included from. > > arch/ia64/Makefile is in fact included via the main Makefile. But

Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

2016-04-20 Thread Ricardo Ribalda Delgado
Hi On Wed, Apr 20, 2016 at 10:51 AM, Olliver Schinagl wrote: >> As I said before, the reason for this proposal is that the code NEVER >> clears PCA963X_MODE2_DMBLNK, only sets it. >> Unfortunately I do not have the HW to test this change. > > The code never clears it, but the hardware does. So

Re: [PATCH] mcb: Fixed bar number assignment for the gdd

2016-04-20 Thread Johannes Thumshirn
On Wed, Apr 20, 2016 at 08:35:47AM +0200, Andreas Werner wrote: > The bar number is found in reg2 within the gdd. Therefore > we need to change the assigment from reg1 to reg2 which > is the correct location. > > Signed-off-by: Andreas Werner > --- > drivers/mcb/mcb-parse.c | 2 +- > 1 file

Re: [PATCH v6 00/17] memory: omap-gpmc: mtd: nand: Support GPMC NAND on non-OMAP platforms

2016-04-20 Thread Roger Quadros
On 19/04/16 23:11, Boris Brezillon wrote: > On Tue, 19 Apr 2016 15:46:19 +0300 > Roger Quadros wrote: > >> >> After all the changes we discussed in [1] I was able to test >> nand/next-with-gpmc-rework >> and it worked fine. >> >> [1] - >>

Re: [PATCH 1/2] i2c: s3c2410: Print errno code in error logs

2016-04-20 Thread Krzysztof Kozlowski
On 04/19/2016 10:10 PM, Javier Martinez Canillas wrote: > The driver not always prints the error code in case of a failure but this > information can be very useful for debugging. So let's print if available. > > Signed-off-by: Javier Martinez Canillas > > --- > > Hello, > > This patch and

Re: [PATCH 2/2] i2c: s3c2410: Check clk_prepare_enable() return value

2016-04-20 Thread Krzysztof Kozlowski
On 04/19/2016 10:10 PM, Javier Martinez Canillas wrote: > The clk_prepare_enable() function can fail so check the return > value and propagate the error in case of a failure. > > Signed-off-by: Javier Martinez Canillas > > --- > > drivers/i2c/busses/i2c-s3c2410.c | 7 ++- > 1 file

Re: [PATCH 1/5] input: twl6040-vibra: fix DT node memory management

2016-04-20 Thread H. Nikolaus Schaller
> Am 19.04.2016 um 19:06 schrieb Dmitry Torokhov : > > On Tue, Apr 19, 2016 at 09:43:08AM +0200, H. Nikolaus Schaller wrote: >> >>> Am 18.04.2016 um 23:22 schrieb Dmitry Torokhov : >>> >>> On Mon, Apr 18, 2016 at 09:55:37PM +0200, H. Nikolaus Schaller wrote: commit e7ec014a47e4 ("Input:

Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

2016-04-20 Thread Olliver Schinagl
On 20-04-16 10:56, Ricardo Ribalda Delgado wrote: Hi On Wed, Apr 20, 2016 at 10:51 AM, Olliver Schinagl wrote: As I said before, the reason for this proposal is that the code NEVER clears PCA963X_MODE2_DMBLNK, only sets it. Unfortunately I do not have the HW to test this change. The code

Re: Warning when unloading network block device

2016-04-20 Thread Markus Pargmann
Hi, On Friday 15 April 2016 18:30:32 Mikulas Patocka wrote: > Hi > > The patch 23272a6754b81ff6503e09c743bb4ceeeab39997 (nbd: Remove signal > usage) triggers a warning in nbd when the network block device is unloaded > - WARN_ON_ONCE(in_irq() || irqs_disabled()) in the function >

[PATCH V2 2/3] arm: dts: imx6sl: add clocks property for ocotp node

2016-04-20 Thread Peng Fan
Add clocks property for ocotp node. Signed-off-by: Peng Fan Cc: Shawn Guo Cc: Sascha Hauer Cc: Rob Herring --- V2: none arch/arm/boot/dts/imx6sl.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi index d12b250..b37da94

[PATCH V2 1/3] arm: dts: imx6qdl: add clocks property for ocotp node

2016-04-20 Thread Peng Fan
Add clocks property for ocotp node. Signed-off-by: Peng Fan Cc: Shawn Guo Cc: Sascha Hauer Cc: Rob Herring --- V2: none arch/arm/boot/dts/imx6qdl.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index b42822a..6b2ef6c

Re: [PATCH 4/4] drm/i915: Move ioremap_wc tracking onto VMA

2016-04-20 Thread Luis R. Rodriguez
On Tue, Apr 19, 2016 at 01:33:58PM +0100, Chris Wilson wrote: > diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c > index 6ce2c31b9a81..9ef47329e8ae 100644 > --- a/drivers/gpu/drm/i915/i915_gem.c > +++ b/drivers/gpu/drm/i915/i915_gem.c > @@ -3346,6 +3346,15 @@ static

[PATCH V2 3/3] nvmem: imx-ocotp: handling clock

2016-04-20 Thread Peng Fan
Before access ocotp nvmem area, the clock should be enabled. Or, `hexdump nvmem` will hang the system. So, use such flow: " 1. clock_enable_prepare 2. read nvmem ocotp area 3. clock_disable_unprepare " Signed-off-by: Peng Fan Cc: Srinivas Kandagatla Cc: Maxime Ripard Cc: Shawn Guo ---

Re: [PATCH 2/5] input: twl6040-vibra: add handler to unregister input if module is removed

2016-04-20 Thread H. Nikolaus Schaller
Hi Dmitry, > Am 19.04.2016 um 18:53 schrieb Dmitry Torokhov : > > On Tue, Apr 19, 2016 at 10:05:08AM +0200, H. Nikolaus Schaller wrote: >> >>> Am 19.04.2016 um 09:57 schrieb Dmitry Torokhov : >>> >>> On Tue, Apr 19, 2016 at 09:33:10AM +0200, H. Nikolaus Schaller wrote: > Am

Re: [PATCH v7 5/8] genirq/irq: introduce msi_doorbell's structs and related callback

2016-04-20 Thread Marc Zyngier
On 19/04/16 18:13, Eric Auger wrote: > The purpose is to be able to retrieve the MSI doorbells of an irqchip. > This is now needed since on some platforms those doorbells must be > iommu mapped (in case the MSIs transit through an IOMMU that do not > bypass those transactions). > > The assumption

[PATCH 1/2] eeprom: at25: Fix SPI device leak

2016-04-20 Thread Mark Brown
The at25 driver is using spi_dev_get() apparently just to take a copy of the SPI device used to instantiate it but never calls spi_dev_put() to free it. Since the device is guaranteed to exist between probe() and remove() there should be no need for the driver to take an extra reference to it so

[PATCH 2/2] eeprom: 93xx46: Fix SPI device leak

2016-04-20 Thread Mark Brown
The 93xx46 driver is using spi_dev_get() apparently just to take a copy of the SPI device used to instantiate it but never calls spi_dev_put() to free it. Since the device is guaranteed to exist between probe() and remove() there should be no need for the driver to take an extra reference to it

Re: [PATCH] arm64: spin-table: add missing of_node_put()

2016-04-20 Thread Will Deacon
On Wed, Apr 20, 2016 at 10:23:31AM +0900, Masahiro Yamada wrote: > Since of_get_cpu_node() increments refcount, the node should be put. Urgh, I really hate the dt refcounting stuff. > Signed-off-by: Masahiro Yamada > --- > > arch/arm64/kernel/smp_spin_table.c | 11 ++- > 1 file

Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

2016-04-20 Thread Ricardo Ribalda Delgado
Hello again On Wed, Apr 20, 2016 at 11:06 AM, Olliver Schinagl wrote: > The devil is in the details :) :) >> >> Saving mode2 sounds like a good compromise then. >> >> But I still believe that we should limit the lock to ledout. No matter >> what we do, we cannot have two leds blinking at

Re: [Letux-kernel] [PATCH 5/5] input: twl6040-vibra: remove mutex

2016-04-20 Thread H. Nikolaus Schaller
> Am 19.04.2016 um 10:08 schrieb H. Nikolaus Schaller : > > >> Am 19.04.2016 um 10:01 schrieb Dmitry Torokhov : >> >> On Tue, Apr 19, 2016 at 09:49:01AM +0200, H. Nikolaus Schaller wrote: >>> Am 18.04.2016 um 23:20 schrieb Dmitry Torokhov : On Mon, Apr 18, 2016 at 09:55:41PM

[PATCH 1/3] i2c: s3c2410: Add missing clock unprepare on probe() error path

2016-04-20 Thread Krzysztof Kozlowski
If during probe() the s3c24xx_i2c_init() failed, the clock was left in disabled but prepared state. Signed-off-by: Krzysztof Kozlowski --- drivers/i2c/busses/i2c-s3c2410.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c

[PATCH 2/3] i2c: s3c2410: Minor function-level comment cleanup

2016-04-20 Thread Krzysztof Kozlowski
Cleanup the weird function-level comments and remove obvious documentatoin for probe/remove. Signed-off-by: Krzysztof Kozlowski --- drivers/i2c/busses/i2c-s3c2410.c | 110 --- 1 file changed, 32 insertions(+), 78 deletions(-) diff --git

[PATCH 3/3] i2c: s3c2410: Cleanup indentation and comment style

2016-04-20 Thread Krzysztof Kozlowski
Improve the readability by: - fixing indentation, - switching to proper block comments, - removing spurious blank lines, - checkpatch: void function return statements are not generally useful - checkpatch: braces {} are not necessary for any arm of this statement - checkpatch: missing a

Re: [PATCH v7 6/8] irqchip/gicv2m: implement msi_doorbell_info callback

2016-04-20 Thread Marc Zyngier
On 19/04/16 18:13, Eric Auger wrote: > This patch implements the msi_doorbell_info callback in the > gicv2m driver. > > The driver now is able to return its doorbell characteristics > (base, size, prot). A single doorbell is exposed. > > This will allow the msi layer to iommu_map this doorbell

Re: [PATCH] asm-generic: wire up preadv2/pwritev2

2016-04-20 Thread Will Deacon
On Tue, Apr 19, 2016 at 10:59:14AM -0400, Christoph Hellwig wrote: > Signed-off-by: Christoph Hellwig > --- > include/uapi/asm-generic/unistd.h | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/include/uapi/asm-generic/unistd.h > b/include/uapi/asm-generic/unistd.h

<    5   6   7   8   9   10   11   12   13   14   >