Re: linux-next: manual merge of the net-next tree with the rockchip tree

Hi all, On Tue, 22 Aug 2017 11:24:14 +1000 Stephen Rothwell wrote: > > Today's linux-next merge of the net-next tree got a conflict in: > > arch/arm64/boot/dts/rockchip/rk3328-evb.dts > > between commits: > > ab78718bda79 ("arm64: dts: rockchip: Enable tsadc module

Re: printk: what is going on with additional newlines?

Hello, I'll answer to both Linus and Joe, just to keep it all one place. On (09/01/17 13:21), Linus Torvalds wrote: > On Fri, Sep 1, 2017 at 10:32 AM, Joe Perches wrote: > > > > Yes, it's a poor name. At least keep using a pr_ prefix. > > I'd suggest perhaps just

Re: printk: what is going on with additional newlines?

On (09/04/17 13:30), Sergey Senozhatsky wrote: > On (09/01/17 10:32), Joe Perches wrote: > [..] > > > +static inlin __printf(2, 3) __cold > > > > uncompiled > > > > > +static int __prbuf_write(struct seq_buf *s, const char *fmt, ...) > > > > inline > > > > thanks. > > there is always a

Re: linux-next: manual merge of the tip tree with the arm64 tree

Hi all, On Tue, 22 Aug 2017 13:38:02 +1000 Stephen Rothwell wrote: > > Today's linux-next merge of the tip tree got a conflict in: > > drivers/firmware/efi/libstub/arm64-stub.c > > between commit: > > 170976bcab07 ("efi/arm64: add EFI_KIMG_ALIGN") > > from the

linux-next: manual merge of the usb tree with the mips tree

Hi Greg, Today's linux-next merge of the usb tree got a conflict in: drivers/phy/Makefile between commit: 0ab3aa747f26 ("phy: Add an USB PHY driver for the Lantiq SoCs using the RCU module") from the mips tree and commit: cd4ec4b03dc1 ("phy: phy-mt65xx-usb3: add mediatek directory and

Re: Applied "regulator: pbias: Select voltage table based on max-voltage" to the regulator tree

Hi Mark, On Friday 01 September 2017 09:53 PM, Mark Brown wrote: > On Fri, Sep 01, 2017 at 03:18:03PM +0200, Ulf Hansson wrote: >> On 31 August 2017 at 15:50, Mark Brown wrote: >>> On Thu, Aug 31, 2017 at 05:37:34PM +0530, Kishon Vijay Abraham I wrote: > This patch

Re: linux-next: manual merge of the v4l-dvb tree with the arm-soc tree

Hi all, On Tue, 22 Aug 2017 10:55:34 +1000 Stephen Rothwell wrote: > > Today's linux-next merge of the v4l-dvb tree got a conflict in: > > arch/arm/configs/imx_v6_v7_defconfig > > between commit: > > b834bc1c52b8 ("ARM: imx_v6_v7_defconfig: Enable staging

[PATCH] binder: fix "cast to pointer from integer of different size" warning

The binder driver now could cause warnings as below on 32bit platforms if ANDROID_BINDER_IPC_32BIT is unselected: drivers/android/binder.c:1550:15: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] This patch fix all of them. Signed-off-by: Jisheng Zhang

Re: linux-next: manual merge of the tip tree with the iommu tree

Hi all, On Tue, 22 Aug 2017 13:50:57 +1000 Stephen Rothwell wrote: > > Hi all, > > Today's linux-next merge of the tip tree got conflicts in: > > drivers/iommu/amd_iommu.c > drivers/iommu/amd_iommu_init.c > drivers/iommu/amd_iommu_proto.h >

Re: linux-next: build warning after merge of the dma-mapping tree

Thanks Stephen, I'll fix it.

Re: printk: what is going on with additional newlines?

On Mon, 2017-09-04 at 14:22 +0900, Sergey Senozhatsky wrote: > there is only way to serialize printks against other printks -- take > the logbuf lock. If that's really necessary, instead make that logbuf_lock a public interface and keep the rest of the code simple. I think it's more important to

[PATCH] ALSA: hda: Fix regression of hdmi eld control created based on invalid pcm

Commit fb087eaaef72 ("ALSA: hda - hdmi eld control created based on pcm") forget to filter out invalid pcm numbers, if there is only one invalid pcm number, then this issue causes we create eld control for invalid pcm silently, but when there are more than one invalid pcm numbers, then this issue

[PATCH 2/2] [media] meye: Adjust two function calls together with a variable assignment

From: Markus Elfring Date: Sun, 3 Sep 2017 15:41:53 +0200 The script "checkpatch.pl" pointed information out like the following. ERROR: do not use assignment in if condition Thus fix the affected source code places. Signed-off-by: Markus Elfring

[PATCH v2] kaslr: get ACPI SRAT table to avoid movable memory

KASLR should choose the memory region of immovable node to extract kernel. So get ACPI SRAT table and store the memory region of movable node which kaslr shold avoid. Signed-off-by: Chao Fan --- v2: Add the support for bios --- arch/x86/boot/compressed/kaslr.c | 346

Re: [PATCH] ALSA: hda: Fix regression of hdmi eld control created based on invalid pcm

On Sun, 03 Sep 2017 15:18:49 +0200, Wang YanQing wrote: > > Commit fb087eaaef72 ("ALSA: hda - hdmi eld control created based on pcm") > forget to filter out invalid pcm numbers, if there is only one invalid pcm > number, then this issue causes we create eld control for invalid pcm silently, > but

[PATCH] dt-binding: phy: don't confuse with Ethernet phy properties

The generic PHY 'phys' property sometime appears in the same node with the Ethernet PHY 'phy' or 'phy-handle' properties. Add a warning in phy-bindings.txt to reduce confusion. Signed-off-by: Baruch Siach --- Documentation/devicetree/bindings/phy/phy-bindings.txt | 4 +++- 1

Re: [PATCH] ALSA: hda: Fix forget to free resource in error handling code path in hda_codec_driver_probe

On Sun, 03 Sep 2017 16:11:22 +0200, Wang YanQing wrote: > > When hda_codec_driver_probe meet error and return failure, we need > to free resource with patch_ops.free, or we will get resource leak. No, the codec driver is responsible to free resources in its error path. thanks, Takashi

[PATCH 2/3] ARM: dts: sun7i: Use defines for clock and reset indices

We can now use defines for clock/reset indices defined in the devicetree binding include files. Signed-off-by: Priit Laes --- arch/arm/boot/dts/sun7i-a20.dtsi | 146 1 file changed, 74 insertions(+), 72 deletions(-) diff --git

[PATCH 1/3] ARM: dts: sun4i: Use defines for clock and reset indices

We can now use defines for clock/reset indices defined in the dt-bindings includes. Signed-off-by: Priit Laes --- arch/arm/boot/dts/sun4i-a10.dtsi | 121 1 file changed, 61 insertions(+), 60 deletions(-) diff --git

[PATCH 0/3] ARM: dts: sunxi: Use defines for ccu clock indices

This is a follow-up commit for sun{4,7}i ccu conversion. Now that all the relevant pieces have been merged we can use dt-binding headers again. Also included is additional patch to add i2s0 block to sun4i dtsi header. Priit Laes (3): ARM: dts: sun4i: Use defines for clock and reset indices

[PATCH 0/2] [media] Motion Eye: Adjustments for meye_probe()

From: Markus Elfring Date: Sun, 3 Sep 2017 15:54:45 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete three error messages for a failed memory allocation Adjust two function calls together with a

[PATCH 1/2] [media] meye: Delete three error messages for a failed memory allocation in meye_probe()

From: Markus Elfring Date: Sun, 3 Sep 2017 15:30:04 +0200 Omit extra messages for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] ALSA: hda: Fix forget to free resource in error handling code path in hda_codec_driver_probe

When hda_codec_driver_probe meet error and return failure, we need to free resource with patch_ops.free, or we will get resource leak. Signed-off-by: Wang YanQing --- sound/pci/hda/hda_bind.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/pci/hda/hda_bind.c

Re: [PATCH] ALSA: hda: Fix assign invalid fixed slots pcm numbers to HDA_PCM_TYPE_HDMI in get_empty_pcm_device

On Sun, 03 Sep 2017 15:52:31 +0200, Wang YanQing wrote: > > The length of pcm_dev_bits in hda_bus is SNDRV_PCM_DEVICES, > and the definiton of SNDRV_PCM_DEVICES is: > " > #if defined(CONFIG_SND_DYNAMIC_MINORS) > #define SNDRV_PCM_DEVICES (SNDRV_OS_MINORS-2) > #else > #define

[PATCH v2 7/8] staging: ccree: replace noop macro with inline

Replace noop macro with a noop inline function Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_driver.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ccree/ssi_driver.h b/drivers/staging/ccree/ssi_driver.h index

[PATCH v2 8/8] staging: ccree: remove BUG macro usage

Replace BUG() macro usage that crash the kernel with alternatives that signal error and/or try to recover. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_buffer_mgr.c | 14 ++ drivers/staging/ccree/ssi_cipher.c | 1 -

[PATCH v2 6/8] staging: ccree: move over to BIT macro for bit defines

Use BIT macro for bit definitions where needed. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_cipher.h | 10 +- drivers/staging/ccree/ssi_driver.c | 3 ++- drivers/staging/ccree/ssi_driver.h | 6 +++--- 3 files changed, 10 insertions(+), 9

[PATCH v2 3/8] staging: ccree: Use platform_get_irq and devm_request_irq

From: Suniel Mahesh It is recommended to use managed function devm_request_irq(), which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace platform_get_resource(), request_irq() and corresponding error handling with

[PATCH 0/8] staging: ccree: more cleanup work for 4.14

More cleanup work from Sunil and myself. I've previously sent some of these as part of a larger patch set. I've decided to split the patch set to smaller chunks to make it more manageable. This patch set applies on top of commit 28eb51f7468a ("staging:rtl8188eu:core Fix remove unneccessary else

[PATCH v2 4/8] staging: ccree: simplify resource release on error

The resource release on probe/init error was being handled in an awkward manner and possibly leaking memory on certain (unlikely) error path. Fix it by simplifying the error resource release and making it easier to track. Reported-by: Dan Carpenter Signed-off-by: Gilad

[PATCH v2 2/8] staging: ccree: Convert to devm_ioremap_resource for map, unmap

From: Suniel Mahesh It is recommended to use managed function devm_ioremap_resource(), which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace request_mem_region(), ioremap() and corresponding error handling with

[PATCH v2 5/8] staging: ccree: remove unused completion

icache_setup_completion is no longer used. Remove it. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_driver.c | 2 -- drivers/staging/ccree/ssi_driver.h | 1 - 2 files changed, 3 deletions(-) diff --git a/drivers/staging/ccree/ssi_driver.c

[PATCH v2 1/8] staging: ccree: Replace kzalloc with devm_kzalloc

From: Suniel Mahesh It is recommended to use managed function devm_kzalloc, which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace kzalloc with devm_kzalloc. (b) drop kfree(), because memory allocated with devm_kzalloc() is

Re: [PATCH 4.9 01/12] p54: memset(0) whole array

On Sun, 2017-09-03 at 14:32 +0200, Christian Lamparter wrote: > On Saturday, September 2, 2017 8:51:01 AM CEST Joe Perches wrote: > > On Thu, 2017-08-31 at 09:40 -0700, Joe Perches wrote: > > > On Thu, 2017-08-31 at 17:44 +0200, Greg Kroah-Hartman wrote: > > > > 4.9-stable review patch. If anyone

Re: [PATCH] sound: soc: fsl: Do not set DAI sysclk when it is equal to system freq

On Sun, Sep 3, 2017 at 11:40 AM, Łukasz Majewski wrote: > This is the part of fsl_ssi_set_bclk() function which is called after > fsl_ssi_set_dai_sysclk() (which sets ssi_private->bitclk_freq = freq;). > > Before the aforementioned check we do have: > > if

Re: [alsa-devel] [PATCH v6 01/12] ALSA: ac97: split out the generic ac97 registers

On Sat, 02 Sep 2017 21:54:03 +0200, Robert Jarzmik wrote: > > Split out from the ac97_codec.h the ac97 generic registers, which can be > used by a codec, typically a generic ac97 codec, and by the ac97 bus, to > scan an ac97 AC-Link. > > This split encompasses all the AC97 standard registers,

Re: [PATCH 7/10] net/mlx4_core: Use ARRAY_SIZE macro

On 03/09/2017 3:19 PM, Thomas Meyer wrote: Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and

[PATCH][thermal-next] thermal: brcmstb: remove two redundant integer range checks

From: Colin Ian King The comparisons for integer low on low > INT_MAX and also integer high > INT_MAX are never going to be true since an int type cannot be greater than INT_MAX. Remove these redundant checks. Detected by: CoverityScan CID#1455245, 1455248 ("Operands

[PATCH v2] media: net.h: add kernel-doc and use it at Documentation/

As we did with frontend.h, ca.h and dmx.h, move the struct definition to net.h. That should help to keep it updated, as more stuff gets added there. Signed-off-by: Mauro Carvalho Chehab --- v2: - Version 1 was broken: it was missing net-types.rst file and if_num was

[PATCH] mmc: dw_mmc: make const arrays mszs static

From: Colin Ian King Don't populate the const arrays mszs on the stack, instead make them static. Makes the object code smaller by over 310 bytes: Before: textdata bss dec hex filename 475278528 320 56375dc37 drivers/mmc/host/dw_mmc.o

[PATCH 3/3] ARM: dts: sun4i: Add i2s0 block to dtsi

sun4i-a10.dtsi was missing i2s0 block. Add it. Signed-off-by: Priit Laes --- arch/arm/boot/dts/sun4i-a10.dtsi | 13 + 1 file changed, 13 insertions(+) diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi index c5efd53..9899ecd 100644 ---

[PATCH] ALSA: hda: Fix assign invalid fixed slots pcm numbers to HDA_PCM_TYPE_HDMI in get_empty_pcm_device

The length of pcm_dev_bits in hda_bus is SNDRV_PCM_DEVICES, and the definiton of SNDRV_PCM_DEVICES is: " #if defined(CONFIG_SND_DYNAMIC_MINORS) #define SNDRV_PCM_DEVICES (SNDRV_OS_MINORS-2) #else #define SNDRV_PCM_DEVICES 8 #endif " So the max valid fixed slots pcm number is 7,

[PATCH] ASoC: fsl_spdif: make const arrays rate static

From: Colin Ian King Don't populate the const arrays rate on the stack, instead make them static. Makes the object code smaller by 220 bytes: Before: textdata bss dec hex filename 243859776 128 3428985f1 sound/soc/fsl/fsl_spdif.o

Re: [PATCH] sound: soc: fsl: Do not set DAI sysclk when it is equal to system freq

Hi Fabio, [Sorry for the top-posting] The driver currently has: /* * Hardware limitation: The bclk rate must be * never greater than 1/5 IPG clock rate */ if (freq * 5 > clk_get_rate(ssi_private->clk)) { dev_err(cpu_dai->dev, "bitclk > ipgclk/5\n"); return -EINVAL; } Unfortunately not.

Re: RFC: Revert move default dialect from CIFS to to SMB3

Hi! Just a quick feedback from my side. After reading Andrew explanation in this thread about the "movement against SMB1" I kind of think "maybe the proposed revert for 4.13 and doing it properly in 4.14 would really have been a good fit". But whatever, doesn't bother me much any more: Steve

[PATCH] ALSA: hda: Fix resource leak issue in snd_hda_codec_build_controls and snd_hda_codec_parse_pcms

When patch_ops.init, patch_ops.build_pcms and patch_ops.build_controls return failure, we need to free resource with patch_ops.free, or we will get resource leak. Signed-off-by: Wang YanQing --- sound/pci/hda/hda_codec.c | 7 ++- 1 file changed, 6 insertions(+), 1

Re: [PATCH 0/10] Use ARRAY_SIZE macro - v4.13-rc7

On Sun, 2017-09-03 at 14:19 +0200, Thomas Meyer wrote: > Use ARRAY_SIZE macro, rather than explicitly coding some variant of it > yourself. > > Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e > 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) >

Re: [PATCH v7 00/19] simplify crypto wait for async op

On Thu, Aug 31, 2017 at 3:31 PM, Harsh Jain wrote: > HI Gilad, > > I think we need an update in ESP also. Now EBUSY return means driver > has accepted, Packet should not be dropped in > > esp_output_tail() function. Good catch. You are right and the same holds true for

hotplug lockdep splat (tip)

[ 126.626908] Unregister pv shared memory for cpu 1 [ 126.631041] [ 126.631269] == [ 126.632442] WARNING: possible circular locking dependency detected [ 126.633599] 4.13.0.g06260ca-tip-lockdep #2 Tainted: GE [ 126.634380]

Re: [PATCH][next] IB/core: fix spelling mistake: "aceess" -> "access"

On Fri, Sep 01, 2017 at 09:22:14AM +0100, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake in WARN message > > Signed-off-by: Colin Ian King > --- > drivers/infiniband/core/uverbs_ioctl_merge.c | 2 +- > 1 file

drivers/clocksource/timer-of.h:35:28: error: field 'clkevt' has incomplete type

Hi Daniel, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d0d6ab53c9abd7dd1070f43a0455328874127ba8 commit: dc11bae78529526605c5c45c369c9512fd012093 clocksource/drivers: Add timer-of common init routine date: 3

Re: [alsa-devel] [PATCH v6 02/12] ALSA: ac97: add an ac97 bus

On Sat, 02 Sep 2017 21:54:04 +0200, Robert Jarzmik wrote: > > AC97 is a bus for sound usage. It enables for a AC97 AC-Link to link one > controller to 0 to 4 AC97 codecs. > > The goal of this new implementation is to implement a device/driver > model for AC97, with an automatic scan of the bus

Re: [PATCH][next] net/mlx4_core: fix memory leaks on error exit path

On 31/08/2017 7:30 PM, Colin King wrote: From: Colin Ian King The structures hca_param and func_cap are not being kfree'd on an error exit path causing two memory leaks. Fix this by jumping to the existing free memory error exit path. Detected by CoverityScan,

RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> > > > Thanks for this v2. > > Introducing the 'efi_switch_mm() ' helper instead of manually > > twiddling with %cr3 seems more cleaner. > > > > I have tested this patchset on a x86_64 machine with the following > > configurations: > > > > 1. Primary kernel boot with efi=old_map 2. Primary kernel

[PATCH 1/1] RAS/CEC: use the right length of "cec_disable"

parse_cec_param() compares a string with "cec_disable" using only 7 characters of the 11-character-long string. Fix the length. Fixes: 011d82611172 ("RAS: Add a Corrected Errors Collector") Signed-off-by: Nicolas Iooss --- drivers/ras/cec.c | 2 +- 1 file changed, 1

[PATCH 1/2] tracing: irqsoff: Move code under tracer config options

In preparation for adding critical sections events such that they don't depend on enabling of tracers, move tracer specific code to their config options and common code that would be needed by a subsequent patch into a common area. This patch is only code movement patch. Cc: Steven Rostedt

[PATCH 5/8] staging: ccree: remove unused completion

icache_setup_completion is no longer used. Remove it. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_driver.c | 2 -- drivers/staging/ccree/ssi_driver.h | 1 - 2 files changed, 3 deletions(-) diff --git a/drivers/staging/ccree/ssi_driver.c

[PATCH 8/8] staging: ccree: remove BUG macro usage

Replace BUG() macro usage that crash the kernel with alternatives that signal error and/or try to recover. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_buffer_mgr.c | 14 ++ drivers/staging/ccree/ssi_cipher.c | 1 -

[PATCH 2/8] staging: ccree: Convert to devm_ioremap_resource for map, unmap

From: Suniel Mahesh It is recommended to use managed function devm_ioremap_resource(), which simplifies driver cleanup paths and driver code. This patch does the following: (a) replace request_mem_region(), ioremap() and corresponding error handling with

[PATCH 0/8] staging: ccree: more cleanup work for 4.14

More cleanup work from Sunil and myself. I've previously sent some of these as part of a larger patch set. I've decided to split the patch set to smaller chunks to make it more manageable. This patch set applies on top of commit 28eb51f7468a ("staging:rtl8188eu:core Fix remove unneccessary else

[PATCH 6/8] staging: ccree: move over to BIT macro for bit defines

Use BIT macro for bit definitions where needed. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_cipher.h | 10 +- drivers/staging/ccree/ssi_driver.c | 3 ++- drivers/staging/ccree/ssi_driver.h | 6 +++--- 3 files changed, 10 insertions(+), 9

[GIT pull] x86 fixes for 4.13

Linus, please pull the latest x86-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86-urgent-for-linus This update contains: - Expand the space for uncompressing as the LZ4 worst case does not fit into the currently reserved space. - Validate

Re: [PATCH] iio staging: tsl2x7x: clean up limit checks

On Mon, 21 Aug 2017 13:11:03 +0300 Dan Carpenter wrote: > The second part of this patch is probably the most interesting. We > use "TSL2X7X_MAX_LUX_TABLE_SIZE * 3" as the limit instead of just > "TSL2X7X_MAX_LUX_TABLE_SIZE". It creates a static checker warning that >

Re: [PATCH] iio: multiplexer: add NULL check on devm_kzalloc() and devm_kmemdup() return values

On Sat, 26 Aug 2017 08:09:43 +0200 Peter Rosin wrote: > On 2017-07-09 19:10, Jonathan Cameron wrote: > > On Fri, 7 Jul 2017 11:26:35 +0200 > > Peter Rosin wrote: > > > >> On 2017-07-07 06:57, Peter Rosin wrote: > >>> On 2017-07-07 06:53, Gustavo A. R.

[PATCH 2/10] drm/amdgpu: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

[PATCH 0/10] Use ARRAY_SIZE macro - v4.13-rc7

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Run against

[PATCH 5/10] [media] lgdt3306a: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

Linux 4.13: Reported regressions as of Sunday, 2017-09-03

Hi! Find below my fifth regression report for Linux 4.13. It lists 4 regressions I'm currently aware of. There are no new ones; 2 got fixed since the last report. You can also find the report at http://bit.ly/lnxregrep413 where I try to update it every now and then. As always: Are you aware of

[PATCH 9/10] [SCSI] bfa: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

Re: [PATCH 4.9 01/12] p54: memset(0) whole array

On Saturday, September 2, 2017 8:51:01 AM CEST Joe Perches wrote: > On Thu, 2017-08-31 at 09:40 -0700, Joe Perches wrote: > > On Thu, 2017-08-31 at 17:44 +0200, Greg Kroah-Hartman wrote: > > > 4.9-stable review patch. If anyone has any objections, please let me > > > know. > > > > > >

[PATCH 4/10] drm/nouveau/bios/init: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

[PATCH 3/10] drm/i915/gvt: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

[PATCH 6/10] ixgbe: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

Re: [RFC v2 1/6] platform/x86: intel_pmc_ipc: Use devm_* calls in driver probe function

On Sun, Sep 3, 2017 at 4:24 AM, wrote: > From: Kuppuswamy Sathyanarayanan > > This patch cleans up unnecessary free/alloc calls in ipc_plat_probe(), > ipc_pci_probe() and ipc_plat_get_res() functions by using

Re: fanotify hangs with AUFS

Ariel Zelivansky: > I just opened bug 196815 on the kernel bugzilla > (https://bugzilla.kernel.org/show_bug.cgi?id=196815). Let me know if > there's anything else I can do It seems that the problem exists in aufs. I will dive into it. Mainline people should not worry about it. Ariel, Aufs is not

[PATCH] rpmsg: glink: initialize ret to zero to ensure error status check is correct

From: Colin Ian King The new switch cases for RPM_CMD_RX_DONE, RPM_CMD_RX_DONE_W_REUSE, RPM_CMD_RX_INTENT_REQ_ACK, RPM_CMD_INTENT and RPM_CMD_RX_INTENT_REQ from 4 recent commits are not setting ret and so a later non-zero check on ret is testing on a garbage value in

Re: [PATCH][next] net/mlx4_core: fix incorrect size allocation for dev->caps.spec_qps

On 31/08/2017 8:07 PM, Colin King wrote: From: Colin Ian King The current allocation for dev->caps.spec_qps is for the size of the pointer and not the size of the actual mlx4_spec_qps structure. Fix this by using the correct size. Also splint allocation over a

Re: Converting struct timer_list callback argument to struct timer_list *

On Fri, Sep 01, 2017 at 09:24:22AM -0700, Kees Cook wrote: > #define list_entry(ptr, type, member) container_of(ptr, type, member) > #define rb_entry(ptr, type, member) container_of(ptr, type, member) > > The use of a "timer_entry()" at the start of callbacks repeats the > struct name, which I

[PATCH 1/1] dax: initialize variable pfn before using it

Function dax_pmd_insert_mapping() contains the following code: pfn_t pfn; if (bdev_dax_pgoff(bdev, sector, size, ) != 0) goto fallback; /* ... */ fallback: trace_dax_pmd_insert_mapping_fallback(inode, vmf, length, pfn, ret); When the condition in the

[PATCH 1/1] x86/xen: fix section of xen_init_time_ops() in header

Commit d162809f85b4 ("xen/x86: Do not call xen_init_time_ops() until shared_info is initialized") moved xen_init_time_ops() from __init to __ref without updating xen-ops.h accordingly. Fix this. Fixes: d162809f85b4 ("xen/x86: Do not call xen_init_time_ops() until shared_info is initialized")

[PATCH 0/2] tracing: Add critical section event support

These patches add critical section event tracing support. Changes since RFC [1] are: - Make critical events independent of the preempt/irqsoff tracer. - Store pointers instead of symbol names to do less in the fast path. - Other changes based on Steven's comments. Here's an example of how

[PATCH 2/2] tracing: Add support for critical section events

Critical section trace events can be used for tracing the start and end of a critical section which can be used by a trace viewer like systrace to graphically view the start and end of a critical section and correlate them with latencies and scheduling issues. Reason for starting critical

[GIT pull] timer fix for 4.13

Linus, please pull the latest timers-urgent-for-linus git tree from: git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers-urgent-for-linus A single fix for a thinko in the raw timekeeper update which causes clock MONOTONIC_RAW to run with erratically increased frequency.

[PATCH 1/1] drm/amd/powerplay: initialize a variable before using it

Function vega10_apply_state_adjust_rules() only initializes stable_pstate_sclk_dpm_percentage when data->registry_data.stable_pstate_sclk_dpm_percentage is not between 1 and 100. The variable is then used to compute stable_pstate_sclk, which therefore uses an uninitialized value. Fix this by

[PATCH 7/10] net/mlx4_core: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

[PATCH 10/10] staging/atomisp: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

[PATCH 8/10] ath9k: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed-off-by:

[PATCH] ASoC: samsung: i2s: Fix error handling path in i2s_set_sysclk()

If 'clk_prepare_enable()' fails, we must 'put' the corresponding clock. Othewise, there is a resource leak. Fixes: f5c97c7b0438 ("ASoC: samsung: i2s: Handle return value of clk_prepare_enable.") Signed-off-by: Christophe JAILLET --- I'm not 100% confident in this

[PATCH v4 3/3] platform/x86: intel_cht_int33fe: Update fusb302 type string, add properties

The fusb302 driver as merged in staging uses "typec_fusb302" as i2c-id rather then just "fusb302" and needs us to set a number of device- properties, adjust the intel_cht_int33fe driver accordingly. One of the properties set is max-snk-mv which makes the fusb302 driver negotiate up to 12V

[PATCH v4 2/3] i2c-cht-wc: Add device-properties for fusb302 integration

Add device-properties to make the bq24292i charger connected to the bus get its input-current-limit from the fusb302 Type-C port controller which is used on boards with the cht-wc PMIC, as well as regulator_init_data for the 5V boost converter on the bq24292i. Since this means we now hook-up the

[PATCH v4 0/3] i2c: Hookup typec power-negotation to the PMIC and charger

Hi Wolfram, Almost all patches from my patch series for hooking up typec power-negotation to the PMIC and charger drivers have been queued for merging into 4.14, leaving only the 3 patches of the v4 of this series. The first 2 patches are i2c patches, if you could review and merge these

[PATCH v4 1/3] i2c: Allow overriding dev_name through board_info

For devices not instantiated through ACPI the i2c-client's device-name gets set to - by default, e.g. "0-0022" this means that the device-name is dependent on the order in which the i2c-busses are enumerated. In some cases having a predictable constant device-name is desirable, for example on non

Re: [PATCH] iio: imu: inv_mpu6050: fix missing break in switch

On Wed, 23 Aug 2017 08:41:43 +0200 (CEST) Peter Meerwald-Stadler wrote: > > Add missing break statement to prevent the code for case > > IIO_CHAN_INFO_CALIBBIAS falling through to the default case. > > > > Also, add a break to the default case for the switch within > > case

Re: [PATCH] ALSA: hda: Fix resource leak issue in snd_hda_codec_build_controls and snd_hda_codec_parse_pcms

On Sun, 03 Sep 2017 18:10:53 +0200, Wang YanQing wrote: > > On Sun, Sep 03, 2017 at 11:27:31PM +0800, Wang YanQing wrote: > > When patch_ops.init, patch_ops.build_pcms and patch_ops.build_controls > > return failure, we need to free resource with patch_ops.free, or we will > > get resource leak.

Re: Linux 4.13: Reported regressions as of Sunday, 2017-09-03

On Sun, Sep 3, 2017 at 2:36 AM, Thorsten Leemhuis wrote: > > [x86/mm/gup] e585513b76: will-it-scale.per_thread_ops -6.9% regression > Status: Asked on the list, but looks like issue gets ignored by everyone > Note: I'm a bit unsure if adding this issue to this list was

Re: [PATCH 0/3] iio: Introduce the generic counter interface

On Mon, 28 Aug 2017 10:16:33 -0400 William Breathitt Gray wrote: > On Sun, Aug 20, 2017 at 01:00:16PM +0100, Jonathan Cameron wrote: > >On Mon, 31 Jul 2017 12:02:45 -0400 > >William Breathitt Gray wrote: > > > >> Summary > >> === > > >

Re: [PATCH 3/3] iio: 104-quad-8: Add IIO generic counter interface support

On Mon, 28 Aug 2017 12:23:45 -0400 William Breathitt Gray wrote: > On Sun, Aug 20, 2017 at 01:11:18PM +0100, Jonathan Cameron wrote: > >On Mon, 31 Jul 2017 12:03:46 -0400 > >William Breathitt Gray wrote: > > > >> This patch adds support for the

Re: [PATCH 4/4][RFC v2] x86/apic: Spread the vectors by choosing the idlest CPU

On Fri, 1 Sep 2017, Chen Yu wrote: > This is the major logic to spread the vectors on different CPUs. > The main idea is to choose the 'idlest' CPU which has assigned > the least number of vectors as the candidate/hint for the vector > allocation domain, in the hope that the vector allocation

[PATCH RFC v3 3/4] samples/bpf: Fix pt_regs issues when cross-compiling

BPF samples fail to build when cross-compiling for ARM64 because of incorrect pt_regs param selection. This is because clang defines __x86_64__ and bpf_headers thinks we're building for x86. Since clang is building for the BPF target, it shouldn't make assumptions about what target the BPF program

Re: [PATCH 3/3] iio: 104-quad-8: Add IIO generic counter interface support

On Sun, 3 Sep 2017 18:50:01 +0100 Jonathan Cameron wrote: > On Mon, 28 Aug 2017 12:23:45 -0400 > William Breathitt Gray wrote: > > > On Sun, Aug 20, 2017 at 01:11:18PM +0100, Jonathan Cameron wrote: > > >On Mon, 31 Jul 2017 12:03:46 -0400 > >

  1   2   3   4   5   >