Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output

2015-09-22 Thread Peter Rosin
On 2015-09-22 14:48, Daniel Baluta wrote: > On Tue, Sep 22, 2015 at 3:39 PM, Peter Rosin wrote: >> From: Peter Rosin >> >> Signed-off-by: Peter Rosin >> --- >> Documentation/ABI/testing/sysfs-bus-iio |2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/Documentation/ABI/testing/sysf

[PATCH v4 05/11] intel_th: Add driver infrastructure for Intel(R) Trace Hub devices

2015-09-22 Thread Alexander Shishkin
Intel(R) Trace Hub (TH) is a set of hardware blocks (subdevices) that produce, switch and output trace data from multiple hardware and software sources over several types of trace output ports encoded in System Trace Protocol (MIPI STPv2) and is intended to perform full system debugging. For these

[PATCH v4 08/11] intel_th: Add Software Trace Hub driver

2015-09-22 Thread Alexander Shishkin
Software Trace Hub (STH) is a trace source device in the Intel TH architecture, it generates data that then goes through the switch into one or several output ports. STH collects data from software sources using the stm device class abstraction. Signed-off-by: Alexander Shishkin --- drivers/hwt

[PATCH v4 06/11] intel_th: Add pci glue layer for Intel(R) Trace Hub

2015-09-22 Thread Alexander Shishkin
This patch adds basic support for PCI-based Intel TH devices. It requests 2 bars (configuration registers for the subdevices and STH channel MMIO region) and calls into Intel TH core code to create the bus with subdevices etc. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/intel_th/Kcon

Re: [linux-sunxi] Re: [PATCH 5/5] ARM: sun5i: Add C.H.I.P DTS

2015-09-22 Thread Chen-Yu Tsai
On Tue, Sep 22, 2015 at 8:47 PM, Maxime Ripard wrote: > On Sat, Sep 19, 2015 at 12:41:07AM +0800, Chen-Yu Tsai wrote: >> On Fri, Sep 18, 2015 at 4:48 PM, Maxime Ripard >> wrote: >> > The C.H.I.P. is a small SBC with an Allwinner R8, 8GB of NAND, 512MB of >> > RAM, USB host and OTG, a wifi / bluet

Re: [PATCH] Coccinelle: remove incorrect -include option transformation

2015-09-22 Thread Julia Lawall
On Tue, 22 Sep 2015, Andrzej Hajda wrote: > kbuild/gcc uses -include option to include files and -I to provide paths for > #include <> directive. The same is true for spatch. > > Signed-off-by: Andrzej Hajda > --- > scripts/coccicheck | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/scr

[PATCH v4 09/11] intel_th: Add Memory Storage Unit driver

2015-09-22 Thread Alexander Shishkin
Memory Storage Unit (MSU) is a trace output device that collects trace data to system memory. It consists of 2 independent Memory Storage Controllers (MSCs). This driver provides userspace interfaces to configure in-memory tracing parameters, such as contiguous (high-order allocation) buffer or mu

[PATCH v4 11/11] MAINTAINERS: add an entry for Intel(R) Trace Hub

2015-09-22 Thread Alexander Shishkin
Add myself as a maintainer for the Intel(R) Trace Hub framework and drivers. Signed-off-by: Alexander Shishkin --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index ebb137d9b9..2bfa7da0f9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5510,6 +5

[PATCH v4 10/11] intel_th: Add PTI output driver

2015-09-22 Thread Alexander Shishkin
Parallel Trace Interface (PTI) unit is a trace output device that sends data over a PTI port. The driver provides interfaces to configure bus width, bus clock divider and mode. Tracing is enabled via output device's "active" attribute. Signed-off-by: Alexander Shishkin --- .../ABI/testing/sysfs

[PATCH v4 07/11] intel_th: Add Global Trace Hub driver

2015-09-22 Thread Alexander Shishkin
Global Trace Hub (GTH) is the central component of Intel TH architecture; it carries out switching between the trace sources and trace outputs, can enable/disable tracing, perform STP encoding, internal buffering, control backpressure from outputs to sources and so on. This property is also reflec

Re: [PATCH v3 1/2] iio: resistance: Document that resistance can be output

2015-09-22 Thread Daniel Baluta
On Tue, Sep 22, 2015 at 3:55 PM, Peter Rosin wrote: > On 2015-09-22 14:48, Daniel Baluta wrote: >> On Tue, Sep 22, 2015 at 3:39 PM, Peter Rosin wrote: >>> From: Peter Rosin >>> >>> Signed-off-by: Peter Rosin >>> --- >>> Documentation/ABI/testing/sysfs-bus-iio |2 ++ >>> 1 file changed, 2 i

Re: [PATCH] Coccinelle: remove incorrect -include option transformation

2015-09-22 Thread Julia Lawall
On Tue, 22 Sep 2015, Andrzej Hajda wrote: > kbuild/gcc uses -include option to include files and -I to provide paths for > #include <> directive. The same is true for spatch. Nicolas, if you think this is OK, please ack it. thanks, julia > Signed-off-by: Andrzej Hajda > --- > scripts/coccic

RE: [PATCH 1/3] ASoC: codecs: Add da7219 codec driver

2015-09-22 Thread Opensource [Adam Thomson]
On September 21, 2015 18:11, Mark Brown wrote: > > In a system scenario the likelihood of that happening is small as you > > cannot use the mic or headphones until they've been inserted. The system is > > only likely to act after the jack insertion events have occurred. However, > > it > > This

Re: [PATCH v3 2/2] iio: mcp4531: Driver for Microchip digital potentiometers

2015-09-22 Thread Peter Meerwald
> Add support for Microchip digital potentiometers and rheostats > MCP4531, MCP4532, MCP4551, MCP4552 > MCP4631, MCP4632, MCP4651, MCP4652 > > These are either single (45xx) or dual (46xx) wipers with either > 129 (4x3x) or 257 (4x5x) steps, and configured either as > potentiometers (

Re: [linux-sunxi] Re: [PATCH 5/5] ARM: sun5i: Add C.H.I.P DTS

2015-09-22 Thread Hans de Goede
Hi, On 22-09-15 14:57, Chen-Yu Tsai wrote: On Tue, Sep 22, 2015 at 8:47 PM, Maxime Ripard wrote: +&usb_otg { + dr_mode = "otg"; + status = "okay"; +}; + +&usbphy { + pinctrl-names = "default"; + pinctrl-0 = <&chip_id_det_pin>; + status = "okay"; + + usb0

Re: [PATCH 5/8] tools build: Allow setting the feature detection user

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 08:51:00AM +0200, Jiri Olsa escreveu: > On Mon, Sep 21, 2015 at 06:23:18PM -0300, Arnaldo Carvalho de Melo wrote: > > From: Arnaldo Carvalho de Melo > > > > We will use the tools/build/ autodetection in the eBPF patchkit > > and it is currently sharing the output directory

Re: [PATCH] btrfs: cleanup btrfs_balance profile validity checks

2015-09-22 Thread David Sterba
On Sun, Aug 30, 2015 at 09:45:49PM +, Alexandru Moise wrote: > Improve readability by generalizing the profile validity checks, > I had to read through those if statements half a dozen times on my > first try just to get an idea of what's happening there. > > Signed-off-by: Alexandru Moise <00

Re: [PATCH] staging: wicl1000: fix dereference after free in wilc_wlan_cleanup()

2015-09-22 Thread Javier Martinez Canillas
Hello Sudip, Thanks a lot for your feedback. On 09/22/2015 02:16 PM, Sudip Mukherjee wrote: > On Tue, Sep 22, 2015 at 12:24:50PM +0200, Javier Martinez Canillas wrote: >> The wilc_wlan_cleanup() function iterates over the list of transmission >> buffers freeing all of them and then iterates over

RE: [PATCH 1/3] f2fs: introduce __try_update_largest_extent

2015-09-22 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Tuesday, September 22, 2015 2:40 AM > To: Chao Yu > Cc: linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 1/3] f2fs: introduce __try_update_largest_extent > >

Re: [PATCH] btrfs: use a single if() statement for one outcome in get_block_rsv()

2015-09-22 Thread David Sterba
On Wed, Sep 09, 2015 at 12:18:50AM +, Alexandru Moise wrote: > Rather than have three separate if() statements for the same outcome > we should just OR them together in the same if() statement. > > Signed-off-by: Alexandru Moise <00moses.alexande...@gmail.com> Reviewed-by: David Sterba -- To

Re: [PATCH 5/8] tools build: Allow setting the feature detection user

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 10:02:42AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Sep 22, 2015 at 08:51:00AM +0200, Jiri Olsa escreveu: > > On Mon, Sep 21, 2015 at 06:23:18PM -0300, Arnaldo Carvalho de Melo wrote: > > one nit ;-) > > Good, at first I thought the mistake was more embarassing,

Re: [PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-22 Thread David Sterba
On Mon, Sep 07, 2015 at 05:24:37PM +0300, Alexandru Moise wrote: > Use memset() to null out the btrfs_delayed_ref_root of > btrfs_transaction instead of setting all the members to 0 by hand. > > Signed-off-by: Alexandru Moise <00moses.alexande...@gmail.com> Reviewed-by: David Sterba Please use

Re: [PATCH 3/3] regulator: vexpress: Fix module autoload for OF platform driver

2015-09-22 Thread Liviu Dudau
On Fri, Sep 18, 2015 at 06:09:51PM +0100, Luis de Bethencourt wrote: > This platform driver has a OF device ID table but the OF module > alias information is not created so module autoloading won't work. > > Signed-off-by: Luis de Bethencourt Acked-by: Liviu Dudau Thanks, Liviu > --- > drive

Re: [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer

2015-09-22 Thread Dan Carpenter
On Sat, Sep 05, 2015 at 09:21:58PM +0200, Anders Fridlund wrote: > I resent it since no one responded to it for 10 days. From what I came > to understand that is the common practice. If that is not the case I'm > sorry (and please let me know so I don't do it again). > 10 days is too short. Espe

Re: [PATCHv2 3/4] power: Add Qualcomm SMBB driver

2015-09-22 Thread Sebastian Reichel
Hi, On Tue, Sep 15, 2015 at 06:33:00AM -0700, Bjorn Andersson wrote: > On Tue 15 Sep 05:43 PDT 2015, Sebastian Reichel wrote: > > > Hi, > > > > On Thu, Jul 30, 2015 at 10:53:56AM -0700, Bjorn Andersson wrote: > > > From: Courtney Cavin > > > > > > Add the Qualcomm Switch-Mode Battery Charger a

[PATCH v2] USB: serial: add Moxa UPORT 11x0 driver

2015-09-22 Thread Mathieu OTHACEHE
Add a driver which supports : - UPort 1110 : 1 port RS-232 USB to Serial Hub. - UPort 1130 : 1 port RS-422/485 USB to Serial Hub. - UPort 1130I : 1 port RS-422/485 USB to Serial Hub with Isolation. - UPort 1150 : 1 port RS-232/422/485 USB to Serial Hub. - UPort 1150I : 1 port RS-232/422/485 USB

[PATCH v2 1/3] f2fs: introduce __try_update_largest_extent

2015-09-22 Thread Chao Yu
This patch adds a new helper __try_update_largest_extent for cleanup. Signed-off-by: Chao Yu --- v2 o cleanup parameter passed. fs/f2fs/extent_cache.c | 14 +- fs/f2fs/f2fs.h | 7 +++ 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/fs/f2fs/extent_cache.c

Re: [PATCH 1/2] btrfs: avoid passing int param to start_transaction which takes u64

2015-09-22 Thread David Sterba
On Sun, Sep 13, 2015 at 06:47:20PM +, Alexandru Moise wrote: > Signed-off-by: Alexandru Moise <00moses.alexande...@gmail.com> > --- > v2: Forgot to add transaction.h when I made the commit, many thanks > Holger for pointing it out. > > fs/btrfs/transaction.c | 2 +- > fs/btrfs/transaction.h |

Re: [PATCH] Coccinelle: remove incorrect -include option transformation

2015-09-22 Thread Andrzej Hajda
On 09/22/2015 02:58 PM, Julia Lawall wrote: > > On Tue, 22 Sep 2015, Andrzej Hajda wrote: > >> kbuild/gcc uses -include option to include files and -I to provide paths for >> #include <> directive. The same is true for spatch. >> >> Signed-off-by: Andrzej Hajda >> --- >> scripts/coccicheck | 1 -

Re: [PATCH 8/8] perf tools: Make perf depend on libbpf

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 09:02:59AM +0200, Jiri Olsa escreveu: > On Mon, Sep 21, 2015 at 06:23:21PM -0300, Arnaldo Carvalho de Melo wrote: > > SNIP > > > + NO_LIBBPF := 1 > > else > >ifneq ($(filter s% -static%,$(LDFLAGS),),) > > msg := $(error No static glibc found, ple

Re: [PATCH 1/2] power:lp8727_charger: use the private data instead of updating I2C device platform data

2015-09-22 Thread Sebastian Reichel
Hi, On Tue, Aug 25, 2015 at 03:07:07PM +0900, Milo Kim wrote: > Currently, lp8727 charger driver parses the DT and copies values into the > 'cl->dev.platform_data' if 'of_node' exists. > This may have architectural issue. Platform data is configurable through > the DT or I2C board info inside the

Re: [PATCH v2] power: Remove unnecessary MODULE_ALIAS() for I2C drivers

2015-09-22 Thread Sebastian Reichel
Hi, On Mon, Aug 31, 2015 at 10:52:00AM +0200, Javier Martinez Canillas wrote: > These drivers already have an I2C device id table that is used to create > module aliases and the used MODULE_ALIAS() was either already in the I2C > table so it was redundant or wasn't a valid I2C id so it was never u

[PATCH v2] Coccinelle: fix incorrect -include option transformation

2015-09-22 Thread Andrzej Hajda
kbuild/gcc uses -include option to include files and -I to provide paths for #include <> directive. In case of spatch latter option should be prefixed with two -. Signed-off-by: Andrzej Hajda --- scripts/coccicheck | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/cocci

Re: [PATCH] 88pm860x_battery: Convert to using managed resources

2015-09-22 Thread Sebastian Reichel
Hi, On Thu, Sep 17, 2015 at 06:03:40PM +0530, Vaishali Thakkar wrote: > Use managed resource functions devm_request_threaded_irq and > devm_power_supply_register to simplify error handling. > > To be compatible with the change, various gotos are replaced > with direct returns and unneeded labels

Re: [PATCH 8/8] perf tools: Make perf depend on libbpf

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 09:03:40AM +0200, Jiri Olsa escreveu: > missing doc hunk in Makefile.perf Ok, adding this. Thanks! So, with that comment about FEATURE_DUMP fixed: https://git.kernel.org/cgit/linux/kernel/git/acme/linux.git/commit/?h=perf/ebpf&id=c1500a11e9b3ec5546233013b7c4b64af103c617 C

Re: [PATCH 2/3] PCI: Add quirks for devices found on Cavium ThunderX SoCs.

2015-09-22 Thread Bjorn Helgaas
Hi David, On Fri, Sep 18, 2015 at 06:00:28PM -0700, David Daney wrote: > On 09/18/2015 12:45 PM, Arnd Bergmann wrote: > >On Friday 18 September 2015 10:00:32 David Daney wrote: > >>On 09/18/2015 12:19 AM, Arnd Bergmann wrote: > >>>On Thursday 17 September 2015 15:41:33 David Daney wrote: > Fro

[PATCH] f2fs: fix to correct freed section number during gc

2015-09-22 Thread Chao Yu
We pass 'nfree' to has_not_enough_free_secs to check whether there is enough free section, but 'nfree' indicates the number of segment gced, should alter the value to section number. Signed-off-by: Chao Yu --- fs/f2fs/gc.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/

[PATCH v5] tools lib traceevent: str addresses in heterogeneous arch environments

2015-09-22 Thread Kapileshwar Singh
When a trace recorded on a 32-bit device is processed with a 64-bit binary, the higher 32-bits of the address need to ignored The lack of this results in the output of the 64-bit pointer value to the trace as the 32-bit address lookup fails in find_printk. Before: burn-1778 [003] 548.600305: b

Re: [PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-22 Thread Alexandru Moise
On Tue, Sep 22, 2015 at 03:04:54PM +0200, David Sterba wrote: > On Mon, Sep 07, 2015 at 05:24:37PM +0300, Alexandru Moise wrote: > > Use memset() to null out the btrfs_delayed_ref_root of > > btrfs_transaction instead of setting all the members to 0 by hand. > > > > Signed-off-by: Alexandru Moise

Re: [PATCH] Coccinelle: remove incorrect -include option transformation

2015-09-22 Thread Julia Lawall
On Tue, 22 Sep 2015, Andrzej Hajda wrote: > On 09/22/2015 02:58 PM, Julia Lawall wrote: > > > > On Tue, 22 Sep 2015, Andrzej Hajda wrote: > > > >> kbuild/gcc uses -include option to include files and -I to provide paths > >> for > >> #include <> directive. The same is true for spatch. > >> > >>

[PATCH v2 00/12] usb: gadget: amd5536udc: fix memory leaks

2015-09-22 Thread Sudip Mukherjee
Proper error handling was done in the probe and some unwanted comparison was removed. Also some forward declaration was removed. regards sudip Sudip Mukherjee (12): usb: gadget: amd5536udc: rewrite init_dma_pools usb: gadget: amd5536udc: fix error path usb: gadget: amd5536udc: use WARN_ON

[PATCH v2 11/12] usb: gadget: amd5536udc: remove forward declaration of udc_basic_init

2015-09-22 Thread Sudip Mukherjee
Rearrange the udc_basic_init function to remove the forward declaration. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 55 ++--- 1 file changed, 27 insertions(+), 28 deletions(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drive

[PATCH v2 03/12] usb: gadget: amd5536udc: use WARN_ON

2015-09-22 Thread Sudip Mukherjee
Use WARN_ON() instead of halting the kernel with BUG_ON() and also fix the checkpatch warning. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drivers/usb/gadget/ud

[PATCH v2 10/12] usb: gadget: amd5536udc: remove forward declaration of udc_pci_*

2015-09-22 Thread Sudip Mukherjee
Remove the forward declarations of udc_pci_probe and udc_pci_remove. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drivers/usb/gadget/udc/amd5536udc.c index 6d64129..00ae069 10064

Re: [PATCH v2] Coccinelle: fix incorrect -include option transformation

2015-09-22 Thread Julia Lawall
On Tue, 22 Sep 2015, Andrzej Hajda wrote: > kbuild/gcc uses -include option to include files and -I to provide paths for > #include <> directive. In case of spatch latter option should be prefixed with > two -. Acked-by: Julia Lawall > Signed-off-by: Andrzej Hajda > --- > scripts/coccicheck |

[PATCH v2 05/12] usb: gadget: amd5536udc: remove unnecessary conditions

2015-09-22 Thread Sudip Mukherjee
The condition checking for irq_registered, regs, mem_region and active are not required as this is the remove function. And we are in the remove means that probe was successful and they can never be NULL at this point of code. It was required in the original code as the remove function was part of

Re: [PATCH 4/4] sound/core/seq: make prioq_match return bool

2015-09-22 Thread Yaowei Bai
On Mon, Sep 21, 2015 at 04:11:00PM +0200, Johannes Berg wrote: > > > -static inline int prioq_match(struct snd_seq_event_cell *cell, > > +static inline bool prioq_match(struct snd_seq_event_cell *cell, > > int client, int timestamp) > > I think you should fix up the inde

Re: [PATCH 1/3] mm/vmscan: make inactive_anon_is_low_global return directly

2015-09-22 Thread Yaowei Bai
On Mon, Sep 21, 2015 at 06:18:07PM +0200, Michal Hocko wrote: > On Wed 16-09-15 19:59:58, Yaowei Bai wrote: > > Delete unnecessary if to let inactive_anon_is_low_global return > > directly. > > > > No functional changes. > > Is this really an improvement? I am not so sure. If anything the > funct

Re: [PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-22 Thread David Sterba
On Tue, Sep 22, 2015 at 04:19:36PM +0300, Alexandru Moise wrote: > Alright, do you want me to resend it with the different subject tag > or shall I just keep that in mind for future patches? No need to resend. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body o

[PATCH v2 12/12] usb: gadget: amd5536udc: NULL comparison

2015-09-22 Thread Sudip Mukherjee
A NULL comparison can be written as if (var) or if (!var). Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drivers/usb/gadget/udc/amd5536udc.c index 157bff1..

[PATCH v2 09/12] usb: gadget: amd5536udc: remove forward declaration of udc_free_dma_chain

2015-09-22 Thread Sudip Mukherjee
Rearrange udc_free_dma_chain to remove the forward declaration. While at it fixed all the relevant checkpatch warnings. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 51 + 1 file changed, 24 insertions(+), 27 deletions(-) diff --git

[PATCH v2 06/12] usb: gadget: amd5536udc: remove forward declaration of udc_probe

2015-09-22 Thread Sudip Mukherjee
Rearrange the udc_probe function to remove the forward declarations. While rearranging also fixed the relevant checkpatch warnings. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 133 ++-- 1 file changed, 66 insertions(+), 67 deletions(-)

Re: [PATCH] perf: Fix perf-targz-src-pkg building failure

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 07:32:57AM +, Wang Nan escreveu: > Commit 01ca9fd41d6f2ad796a6b109b5253e06b6ae6dc7 ("tools: Add err.h with > ERR_PTR PTR_ERR interface") introduces 'linux/err.h', but forgot to insert > it into MANIFEST, which causes failure when calling tests/perf-targz-src-pkg > to cre

Re: [PATCH 0/6] mac80211: Deinline large functions in driver-ops.h

2015-09-22 Thread Johannes Berg
On Tue, 2015-09-22 at 15:28 +0200, Johannes Berg wrote: > On Fri, 2015-09-18 at 15:19 +0200, Denys Vlasenko wrote: > > After measuring size of inlines, these functions were found to be > > largest. > > These patches reduce code size by about 21 kbytes. > > > > Only patches 1, 2 and 5 applied to

Re: [PATCH 0/6] mac80211: Deinline large functions in driver-ops.h

2015-09-22 Thread Johannes Berg
On Fri, 2015-09-18 at 15:19 +0200, Denys Vlasenko wrote: > After measuring size of inlines, these functions were found to be > largest. > These patches reduce code size by about 21 kbytes. > Only patches 1, 2 and 5 applied to mac80211-next. johannes -- To unsubscribe from this list: send the li

Re: [PATCH 4/4] sound/core/seq: make prioq_match return bool

2015-09-22 Thread Johannes Berg
On Tue, 2015-09-22 at 21:22 +0800, Yaowei Bai wrote: > On Mon, Sep 21, 2015 at 04:11:00PM +0200, Johannes Berg wrote: > > > > > -static inline int prioq_match(struct snd_seq_event_cell *cell, > > > +static inline bool prioq_match(struct snd_seq_event_cell *cell, > > > int c

Re: [PATCH] dma-mapping: Deinline dma_[un]map_single_attrs()

2015-09-22 Thread Denys Vlasenko
On 09/21/2015 09:42 PM, Thomas Gleixner wrote: > On Sun, 20 Sep 2015, Denys Vlasenko wrote: >> --- >> include/asm-generic/dma-mapping-common.h | 40 ++- >> kernel/Makefile | 3 +- >> kernel/dma-mapping-common.c | 47 >> +++

Re: [PATCH -tip 2/3] sched/wake_q: Relax to acquire semantics

2015-09-22 Thread Boqun Feng
On Tue, Sep 22, 2015 at 02:51:36PM +0200, Martin Schwidefsky wrote: > On Tue, 22 Sep 2015 20:23:26 +0800 > Boqun Feng wrote: > > > Hi Martin, > > > > On Tue, Sep 22, 2015 at 12:27:35PM +0200, Martin Schwidefsky wrote: > > > On Mon, 21 Sep 2015 11:22:52 +0200 > > > Martin Schwidefsky wrote: > >

Re: [PATCH v4 5/5] power: wm831x_power: Support USB charger current limit management

2015-09-22 Thread Sebastian Reichel
Hi, On Fri, Aug 21, 2015 at 01:34:52PM +0800, Baolin Wang wrote: > Integrate with the newly added USB charger interface to limit the current > we draw from the USB input based on the input device configuration > identified by the USB stack, allowing us to charge more quickly from high > current in

[PATCH v2 08/12] usb: gadget: amd5536udc: remove forward declaration of udc_create_dma_chain

2015-09-22 Thread Sudip Mukherjee
Rearrange udc_create_dma_chain to remove the forward declaration. While rearranging fixed the relevant checkpatch warnings. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 238 ++-- 1 file changed, 117 insertions(+), 121 deletions(-) diff

Re: crisv32 runtime failure in -next due to 'page-flags: define behavior SL*B-related flags on compound pages'

2015-09-22 Thread Kirill A. Shutemov
On Tue, Sep 22, 2015 at 02:50:19PM +0200, Hans-Peter Nilsson wrote: > > From: Mikael Starvik > > Date: Tue, 22 Sep 2015 14:19:38 +0200 > > > For cris it is completely valid to do that. > > Correct, just as it's completely valid for any system to specify > an ABI that says that structures are lai

[PATCH v2 07/12] usb: gadget: amd5536udc: remove forward declaration of udc_remote_wakeup

2015-09-22 Thread Sudip Mukherjee
Rearrange the udc_remote_wakeup function to remove the forward declaration. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 41 ++--- 1 file changed, 20 insertions(+), 21 deletions(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/dr

[PATCH v2] dma-mapping: Deinline dma_[un]map_single_attrs()

2015-09-22 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining these functions have sizes and callsite counts as follows: dma_map_single_attrs: 214 bytes, 587 calls dma_unmap_single_attrs: 115 bytes, 771 calls This patch reduces kernel size by 152310 bytes. text dat

[PATCH v2 01/12] usb: gadget: amd5536udc: rewrite init_dma_pools

2015-09-22 Thread Sudip Mukherjee
A rewrite of init_dma_pools() with proper error handling. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drivers/usb/gadget/udc/amd5536udc.c ind

Re: [PATCH v2] Add virt directory to the top Makefile

2015-09-22 Thread Michal Marek
Dne 22.9.2015 v 14:23 Feng Wu napsal(a): > We need to build files in virt/lib/, so add virt directory to > the top Makefile. > > Signed-off-by: Feng Wu > --- > v2: > - Remove the explicit 'virt' entry in the KBUILD_ALLDIRS assignment > - Remove './' in virt/Makefile Acked-by: Michal Marek Mich

Re: [linux-sunxi] Re: [PATCH 5/5] ARM: sun5i: Add C.H.I.P DTS

2015-09-22 Thread Hans de Goede
Hi, On 22-09-15 15:01, Hans de Goede wrote: Hi, On 22-09-15 14:57, Chen-Yu Tsai wrote: On Tue, Sep 22, 2015 at 8:47 PM, Maxime Ripard wrote: +&usb_otg { + dr_mode = "otg"; + status = "okay"; +}; + +&usbphy { + pinctrl-names = "default"; + pinctrl-0 = <&chip_id_det

[PATCH v2 02/12] usb: gadget: amd5536udc: fix error path

2015-09-22 Thread Sudip Mukherjee
Handle the error properly instead of calling the pci remove function. Signed-off-by: Sudip Mukherjee --- drivers/usb/gadget/udc/amd5536udc.c | 29 ++--- 1 file changed, 22 insertions(+), 7 deletions(-) diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drivers/usb/gadge

[PATCH v2 04/12] usb: gadget: amd5536udc: use free_dma_pools

2015-09-22 Thread Sudip Mukherjee
We have the function free_dma_pools() which frees all the dma pools. Use it instead of calling all the functions separately. The if conditions for data_requests and stp_requests are also not required here as this is the remove function and we are here means probe has succeeded and dma has been succ

Re: [PATCH] perf probe: Fix module probing with shortname

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 03:34:32AM +, Wang Nan escreveu: > After commit 3d39ac538629e4f00a6e1c38d46346f1b8e69505 ("perf machine: > No need to have two DSOs lists"), perf probe with module short name doesn't > work again. For example: > > # lsmod | grep e1000e > e1000e233472

Re: [PATCH 8/8] perf tools: Make perf depend on libbpf

2015-09-22 Thread Jiri Olsa
On Tue, Sep 22, 2015 at 10:18:07AM -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Sep 22, 2015 at 09:03:40AM +0200, Jiri Olsa escreveu: > > missing doc hunk in Makefile.perf > > Ok, adding this. Thanks! So, with that comment about FEATURE_DUMP fixed: > > https://git.kernel.org/cgit/linux/kernel

[PATCH v2] staging: wicl1000: fix dereference after free in wilc_wlan_cleanup()

2015-09-22 Thread Javier Martinez Canillas
The wilc_wlan_cleanup() function iterates over the list of transmission buffers freeing all of them and then iterates over the receive buffers list to free all of them as well. But on the receive loop a pointer to struct txq_entry_t is dereferenced instead of the pointer to a struct rxq_entry_t. T

Re: [PATCH 1/2] btrfs: avoid passing int param to start_transaction which takes u64

2015-09-22 Thread Alexandru Moise
On Tue, Sep 22, 2015 at 03:09:33PM +0200, David Sterba wrote: > On Sun, Sep 13, 2015 at 06:47:20PM +, Alexandru Moise wrote: > > Signed-off-by: Alexandru Moise <00moses.alexande...@gmail.com> > > --- > > v2: Forgot to add transaction.h when I made the commit, many thanks > > Holger for pointing

Re: [RESEND PATCH] media: vb2: Fix vb2_dc_prepare do not correct sync data to device

2015-09-22 Thread tiffany lin
Hi Sakari, On Tue, 2015-09-22 at 15:07 +0300, Sakari Ailus wrote: > Hi Tiffany, > > On Tue, Sep 22, 2015 at 06:19:25PM +0800, tiffany lin wrote: > > Hi Hans, > > > > On Mon, 2015-09-21 at 15:13 +0200, Hans Verkuil wrote: > > > Hi Tiffany! > > > > > > On 21-09-15 14:26, Tiffany Lin wrote: > > >

Re: [PATCH 8/8] perf tools: Make perf depend on libbpf

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 03:31:36PM +0200, Jiri Olsa escreveu: > On Tue, Sep 22, 2015 at 10:18:07AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Tue, Sep 22, 2015 at 09:03:40AM +0200, Jiri Olsa escreveu: > > > missing doc hunk in Makefile.perf > > > > Ok, adding this. Thanks! So, with that comment

Re: [linux-sunxi] [PATCH 0/2] ASoC: Add support for the Allwinner A10 codec

2015-09-22 Thread Priit Laes
On Sat, 2015-09-12 at 15:26 +0200, Maxime Ripard wrote: > Hi everyone, > > This patch set adds the support for what Allwinner calls the codec on > their SoCs. > > This codec is actually a combination of a codec and DAI, tied > together > in a single memory-mapped IP. It is completely standalone,

Re: [PATCH] powercap / RAPL : remove dependency on iosf_mbi

2015-09-22 Thread Austin S Hemmelgarn
On 2015-09-21 17:36, Jacob Pan wrote: On Mon, 21 Sep 2015 11:48:14 +0800 Pengyu Ma wrote: On 09/18/2015 11:43 PM, Jacob Pan wrote: On Fri, 18 Sep 2015 02:09:55 +0200 "Rafael J. Wysocki" wrote: On Thursday, September 17, 2015 03:31:41 PM Pengyu Ma wrote: iosf_mbi is supported on Quark, B

[PATCH] kernel/watchdog: is_hardlockup can be boolean

2015-09-22 Thread Yaowei Bai
This patch makes is_hardlockup return bool to improve readability due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- kernel/watchdog.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ke

Re: [PATCH 1/2] btrfs: avoid passing int param to start_transaction which takes u64

2015-09-22 Thread David Sterba
On Tue, Sep 22, 2015 at 04:33:07PM +0300, Alexandru Moise wrote: > > I think it's better to do it the other way around, ie. let all the > > equivalents of 'num_items' be an int. We know that this is a small > > number, using u64 for that is an overkill. As the number is always > > positive, please

Re: [PATCH 2/3] power_supply: Add support for tps65217-charger.

2015-09-22 Thread Sebastian Reichel
Hi, On Tue, Sep 08, 2015 at 10:09:38AM +0200, Enric Balletbo i Serra wrote: > This patch adds support for the tps65217 charger driver. This driver is > responsible for controlling the charger aspect of the tps65217 mfd. > Currently, this mainly consists of turning on and off the charger, but > som

[PATCH 2/2] kernel/trace_probe: is_good_name can be boolean

2015-09-22 Thread Yaowei Bai
This patch makes is_good_name return bool to improve readability due to this particular function only using either one or zero as its return value. No functional change. Signed-off-by: Yaowei Bai --- kernel/trace/trace_probe.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

Re: [PATCH 1/3] power: reset: at91-reset/trivial: driver applies to SAMA5 family as well

2015-09-22 Thread Sebastian Reichel
Hi, On Thu, Sep 10, 2015 at 12:35:13PM +0200, Nicolas Ferre wrote: > This diver doesn't applies only on SAM9 SoC families but on SAMA5 families > as well. Thanks, queued. -- Sebastian signature.asc Description: Digital signature

Re: [PATCH v5] tools lib traceevent: str addresses in heterogeneous arch environments

2015-09-22 Thread Steven Rostedt
On Tue, 22 Sep 2015 14:22:03 +0100 Kapileshwar Singh wrote: > if (!(field->flags & FIELD_IS_ARRAY) && > field->size == pevent->long_size) { > - addr = *(unsigned long *)(data + field->offset); > + > + /* Handle heterogeneous

[PATCH 1/2] kernel/trace: call helper function to check global_trace

2015-09-22 Thread Yaowei Bai
Use helper function tracing_is_enabled() to check if global_trace has been disabled. No functional change. Signed-off-by: Yaowei Bai --- kernel/trace/trace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 6e79408..b0bd322a 1

[tip:timers/core] timers: Fix data race in timer_stats_account_timer()

2015-09-22 Thread tip-bot for Dmitry Vyukov
Commit-ID: 3ed769bdb2a2484fd7f9f7f3047413053aacbe21 Gitweb: http://git.kernel.org/tip/3ed769bdb2a2484fd7f9f7f3047413053aacbe21 Author: Dmitry Vyukov AuthorDate: Fri, 18 Sep 2015 15:54:23 +0200 Committer: Thomas Gleixner CommitDate: Tue, 22 Sep 2015 15:43:18 +0200 timers: Fix data race

[tip:timers/core] time: Fix spelling in comments

2015-09-22 Thread tip-bot for Zhen Lei
Commit-ID: 571af55a31d3652ac1f758f116835a76d0335661 Gitweb: http://git.kernel.org/tip/571af55a31d3652ac1f758f116835a76d0335661 Author: Zhen Lei AuthorDate: Tue, 25 Aug 2015 14:42:53 +0800 Committer: Thomas Gleixner CommitDate: Tue, 22 Sep 2015 12:54:23 +0200 time: Fix spelling in comme

Re: [PATCH 2/2] selftests/userfaultfd: improve syscall number definition

2015-09-22 Thread Shuah Khan
On 09/22/2015 04:45 AM, Andre Przywara wrote: > At the moment the userfaultfd test program only supports x86 and an > architecture called "powewrpc" ;-) > Fix that typo and add the syscall numbers for other architectures as > well. > Also as in an ideal world a syscall number should come from the s

[PATCH v2] x86: hyperv: fix build in !CONFIG_KEXEC_CORE case

2015-09-22 Thread Vitaly Kuznetsov
Recent changes in Hyper-V driver ("Drivers: hv: vmbus: add special crash handler") broke the build when CONFIG_KEXEC_CORE is not set: arch/x86/built-in.o: In function `hv_machine_crash_shutdown': arch/x86/kernel/cpu/mshyperv.c:112: undefined reference to `native_machine_crash_shutdown' Decorate

Re: [Patch v4] Do not reserve crashkernel high memory if crashkernel low memory reserving failed

2015-09-22 Thread Baoquan He
Add Jerry to CC list. On 09/22/15 at 07:48pm, Baoquan He wrote: > People reported that when allocating crashkernel memory using > ",high" and ",low" syntax, there were cases where the reservation > of the "high" portion succeeds, but the reservation of the "low" > portion fails. Then kexec can lo

Re: [PATCH 1/3] irqchip: atmel-aic5: fix bug with mask/unmask

2015-09-22 Thread Thomas Gleixner
On Tue, 22 Sep 2015, Boris Brezillon wrote: > On Tue, 22 Sep 2015 12:27:08 +0200 (CEST) > Thomas Gleixner wrote: > > Why is this locking dgc->gc[0] and fiddling with some other generic > > chip? > > Actually, we always access the same set of registers for all irqs of the > domain, and thus need t

Re: [tip:perf/core] tools lib api fs: Remove debugfs, tracefs and findfs objects

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Mon, Sep 21, 2015 at 05:20:03PM -0700, Vinson Lee escreveu: > On Mon, Sep 14, 2015 at 11:59 PM, tip-bot for Jiri Olsa > wrote: > > Commit-ID: 60a1133a5b39738671eff1e4d77bedc1ee3fa528 > > Gitweb: > > http://git.kernel.org/tip/60a1133a5b39738671eff1e4d77bedc1ee3fa528 > > Author: Jiri Ol

[PATCH] staging: rtl8723au: Mark type casts to __le32 as intentional

2015-09-22 Thread Lars Svensson
Fixing Sparse warnings about intentional type casts in rtw_security.c as below. CHECK drivers/staging/rtl8723au/core/rtw_security.c drivers/staging/rtl8723au/core/rtw_security.c:248:22: \ warning: cast to restricted __le32 drivers/staging/rtl8723au/core/rtw_security.c:249:24: \ warning: cast t

Re: crisv32 runtime failure in -next due to 'page-flags: define behavior SL*B-related flags on compound pages'

2015-09-22 Thread Hans-Peter Nilsson
> From: "Kirill A. Shutemov" > Date: Tue, 22 Sep 2015 15:27:51 +0200 > On Tue, Sep 22, 2015 at 02:50:19PM +0200, Hans-Peter Nilsson wrote: > > That element (the struct) needs *explicit* padding or alignment > > to the required multiplicity of bytes for anyone to portably be > > able to imply some

Re: [PATCH 1/2] kernel/trace: call helper function to check global_trace

2015-09-22 Thread Steven Rostedt
On Tue, 22 Sep 2015 21:43:12 +0800 Yaowei Bai wrote: > Use helper function tracing_is_enabled() to check if global_trace > has been disabled. Actually, I prefer not to. > > No functional change. > > Signed-off-by: Yaowei Bai > --- > kernel/trace/trace.c | 2 +- > 1 file changed, 1 insertion

Re: [PATCH 3/3] power: reset: at91-shdwc: add new shutdown controller driver

2015-09-22 Thread Sebastian Reichel
Hi, On Thu, Sep 10, 2015 at 12:39:18PM +0200, Nicolas Ferre wrote: > Sama5d2 SoC has a completely new shutdown controller with new > features and register layout. It thus makes sense to add a new > driver for this new peripheral. > > This driver is Device Tree only and handles events from the > w

Re: [PATCH 2/2] kernel/trace_probe: is_good_name can be boolean

2015-09-22 Thread Steven Rostedt
On Tue, 22 Sep 2015 21:43:13 +0800 Yaowei Bai wrote: > This patch makes is_good_name return bool to improve readability > due to this particular function only using either one or zero as its > return value. This patch makes sense. I'll take it. Thanks! -- Steve > > No functional change. > >

Re: [PATCH v1 1/3] clocksource: rockchip: Make the driver more readability and compatible

2015-09-22 Thread Heiko Stübner
Hi Caesar, Am Freitag, 18. September 2015, 16:51:09 schrieb Caesar Wang: > Build the arm64 SoCs (e.g.: RK3368) on Rockchip platform, > There are some failure with build up on timer driver for rockchip. > > logs: > ... > drivers/clocksource/rockchip_timer.c:156:13: error: 'NO_IRQ' undeclared > /tm

[tip:irq/urgent] irqchip/atmel-aic5: Use per chip mask caches in mask/unmask()

2015-09-22 Thread tip-bot for Ludovic Desroches
Commit-ID: d32dc9aa10c739363c775baf4499416b2e0dc11f Gitweb: http://git.kernel.org/tip/d32dc9aa10c739363c775baf4499416b2e0dc11f Author: Ludovic Desroches AuthorDate: Mon, 21 Sep 2015 15:46:04 +0200 Committer: Thomas Gleixner CommitDate: Tue, 22 Sep 2015 15:55:51 +0200 irqchip/atmel-aic5

Re: [PATCH 8/8] perf tools: Make perf depend on libbpf

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 03:31:36PM +0200, Jiri Olsa escreveu: > On Tue, Sep 22, 2015 at 10:18:07AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Tue, Sep 22, 2015 at 09:03:40AM +0200, Jiri Olsa escreveu: > > > missing doc hunk in Makefile.perf > > > > Ok, adding this. Thanks! So, with that comment

Re: [linux-sunxi] Re: [PATCH 5/5] ARM: sun5i: Add C.H.I.P DTS

2015-09-22 Thread Maxime Ripard
On Tue, Sep 22, 2015 at 03:01:58PM +0200, Hans de Goede wrote: > >In short, this is not about "power supply" but VBUS detection. IIRC, > >if no VBUS detection method is provided, the phy driver just waits a > >period of time after an ID pin change and then considers VBUS invalid. > > Right, but th

Re: [PATCH v5] tools lib traceevent: str addresses in heterogeneous arch environments

2015-09-22 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 22, 2015 at 09:46:18AM -0400, Steven Rostedt escreveu: > On Tue, 22 Sep 2015 14:22:03 +0100 > Kapileshwar Singh wrote: > > > if (!(field->flags & FIELD_IS_ARRAY) && > > field->size == pevent->long_size) { > > - addr = *(unsigned long *)(da

<    1   2   3   4   5   6   7   8   9   10   >