[PATCH v4 0/3] pinctrl: Ingenic: Add support for SSI and I2S pins.

2020-09-12 Thread Zhou Yanjie
1.Add SSI pins support for JZ4770 and JZ4780. 2.Correct the pullup and pulldown parameters of JZ4780. 3.Add I2S pins support for JZ4780, X1000, X1500, and X1830. v2->v3: 1.Add Paul Cercueil's Reviewed-by. 2.Fix bug about PE15's pull-up parameter. v3->v4: 1.Add Paul Cercueil's Reviewed-by. 2.Fix t

[PATCH] tulip: windbond-840: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Christophe JAILLET
The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'alloc_ringdesc()' GFP_KERNEL can be used because it

Re: [PATCH] net: dl2k: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Leon Romanovsky
On Sun, Sep 13, 2020 at 08:14:17AM +0200, Christophe JAILLET wrote: > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > >

Re: [RESEND PATCH] staging: rtl8188eu: Fix else after return WARNING (checkpatch)

2020-09-12 Thread Greg Kroah-Hartman
On Sun, Sep 13, 2020 at 12:19:50PM +0530, Sohom Datta wrote: > >From 4c8c8f3ff7f4d711daea4ac3bb987fcecc7ef1ed Mon Sep 17 00:00:00 2001 > From: Sohom > Date: Sat, 12 Sep 2020 18:04:56 +0530 > Subject: [RESEND PATCH] staging: rtl8188eu: Fix else after return WARNING > (checkpatch) Why is this in t

compat_alloc_user_space removal, was Re: [PATCH 3/3] scsi: megaraid_sas: simplify compat_ioctl handling

2020-09-12 Thread Christoph Hellwig
On Sat, Sep 12, 2020 at 02:49:05PM +0200, Arnd Bergmann wrote: > fs/quota/compat.c: dqblk = compat_alloc_user_space(sizeof(struct if_dqblk)); > fs/quota/compat.c: dqblk = compat_alloc_user_space(sizeof(struct if_dqblk)); > fs/quota/compat.c: fsqstat = compat_alloc_user_space(sizeof(struct > fs_quot

[RESEND PATCH] staging: rtl8188eu: Fix else after return WARNING (checkpatch)

2020-09-12 Thread Sohom Datta
>From 4c8c8f3ff7f4d711daea4ac3bb987fcecc7ef1ed Mon Sep 17 00:00:00 2001 From: Sohom Date: Sat, 12 Sep 2020 18:04:56 +0530 Subject: [RESEND PATCH] staging: rtl8188eu: Fix else after return WARNING (checkpatch) Fixed: WARNING: else is not generally useful after a break or return 1636: FILE: ./rtw_

[PATCH v3 1/2] USB: PHY: JZ4770: Remove unnecessary function calls.

2020-09-12 Thread Zhou Yanjie
Remove unnecessary "of_match_ptr()", because Ingenic SoCs all depend on Device Tree. Suggested-by: Paul Cercueil Signed-off-by: 周琰杰 (Zhou Yanjie) --- Notes: v3: New patch. drivers/usb/phy/phy-jz4770.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/

[PATCH v3 0/2] Use the generic PHY framework for Ingenic USB PHY.

2020-09-12 Thread Zhou Yanjie
v2->v3: 1.Remove unnecessary "of_match_ptr()" before the move to the generic PHY. 2.Change "depends on (MACH_INGENIC && MIPS) || COMPILE_TEST" to "depends on MIPS || COMPILE_TEST". 3.Keep the adjustments of "ingenic_usb_phy_init()" and "ingenic_usb_phu_exit()" positions in v2 to make them consi

[PATCH v3 2/2] USB: PHY: JZ4770: Use the generic PHY framework.

2020-09-12 Thread Zhou Yanjie
Used the generic PHY framework API to create the PHY, and move the driver to driver/phy/ingenic. And adjust the position of some codes to make it consistent with the style of other USB PHY drivers. Tested-by: 周正 (Zhou Zheng) Co-developed-by: 漆鹏振 (Qi Pengzhen) Signed-off-by: 漆鹏振 (Qi Pengzhen) Si

Re: [Linux-kernel-mentees] [PATCH] net: fix uninit value error in __sys_sendmmsg

2020-09-12 Thread Anant Thazhemadam
On 13/09/20 11:43 am, Greg KH wrote: > On Sun, Sep 13, 2020 at 11:26:39AM +0530, Anant Thazhemadam wrote: >> The crash report showed that there was a local variable; >> >> iovstack.i@__sys_sendmmsg created at: >> ___sys_sendmsg net/socket.c:2388 [inline] >> __sys_sendmmsg+0x6db/0xc90 net/so

Re: [PATCH v2] coccinelle: misc: add flexible_array.cocci script

2020-09-12 Thread Denis Efremov
Hi, On 9/12/20 5:43 PM, Julia Lawall wrote: > > > On Mon, 10 Aug 2020, Denis Efremov wrote: > >> Commit 68e4cd17e218 ("docs: deprecated.rst: Add zero-length and one-element >> arrays") marks one-element and zero-length arrays as deprecated. Kernel >> code should always use "flexible array membe

Re: [Linux-kernel-mentees] [PATCH] net: fix uninit value error in __sys_sendmmsg

2020-09-12 Thread Greg KH
On Sun, Sep 13, 2020 at 11:26:39AM +0530, Anant Thazhemadam wrote: > The crash report showed that there was a local variable; > > iovstack.i@__sys_sendmmsg created at: > ___sys_sendmsg net/socket.c:2388 [inline] > __sys_sendmmsg+0x6db/0xc90 net/socket.c:2480 > > that was left uninitialize

[PATCH] net: dl2k: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Christophe JAILLET
The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'rio_probe1()' GFP_KERNEL can be used because it is

[PATCH] net: fix uninit value error in __sys_sendmmsg

2020-09-12 Thread Anant Thazhemadam
The crash report showed that there was a local variable; iovstack.i@__sys_sendmmsg created at: ___sys_sendmsg net/socket.c:2388 [inline] __sys_sendmmsg+0x6db/0xc90 net/socket.c:2480 that was left uninitialized. The contents of iovstack are of interest, since the respective pointer is pas

[PATCH V2] natsemi: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Christophe JAILLET
The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'alloc_ring()' (natsemi.c) GFP_KERNEL can be used be

[PATCH v2] arm64: dts: layerscape: Add label to pcie nodes

2020-09-12 Thread Wasim Khan
From: Wasim Khan Add label to pcie nodes so that they are easy to refer. Signed-off-by: Wasim Khan --- Changes in v2: - clubbed separate patches in single patch arch/arm64/boot/dts/freescale/fsl-ls1012a-oxalis.dts | 2 +- arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi | 5 +++-- arch/

RE: [PATCH 2/3] phy: cadence: salvo: Constify cdns_nxp_sequence_pair

2020-09-12 Thread Peter Chen
> > cdns_nxp_sequence_pair[] are never modified and can be made const to allow > the compiler to put them in read-only memory. > > Signed-off-by: Rikard Falkeborn Reviewed-by: Peter Chen Peter > --- > drivers/phy/cadence/phy-cadence-salvo.c | 6 +++--- > 1 file changed, 3 insertions(+), 3

Re: [PATCH v2 3/3] ARM: dts: add Plymovent M2M board

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 08:03:12AM +0200, Marco Felsch wrote: > On 20-09-11 07:09, Oleksij Rempel wrote: > > Plymovent M2M is a control interface produced for the Plymovent filter > > systems. > > > > Signed-off-by: David Jander > > Signed-off-by: Oleksij Rempel > > --- > > arch/arm/boot/dts/Ma

[PATCH] tee/tee_shm.c: Fix error handling path

2020-09-12 Thread Souptick Joarder
When shm->num_pages <= 0, we should avoid calling release_registered_pages() in error handling path. Signed-off-by: Souptick Joarder Cc: John Hubbard --- drivers/tee/tee_shm.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/tee/tee_shm.c b/drivers/tee/tee_shm.

[ANNOUNCE] 4.9.236-rt154

2020-09-12 Thread Clark Williams
Hello RT-list! I'm pleased to announce the 4.9.236-rt154 stable release. You can get this release via the git tree at: git://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-stable-rt.git branch: v4.9-rt Head SHA1: d81b19f462cdf108afa79e7f7717190a280a299e Or to build 4.9.236-rt154 direct

Re: [PATCH] driver/pci: reduce the single block time in pci_read_config

2020-09-12 Thread Jiang Biao
Hi, Bjorn On Thu, 10 Sep 2020 at 09:59, Bjorn Helgaas wrote: > > On Thu, Sep 10, 2020 at 09:54:02AM +0800, Jiang Biao wrote: > > Hi, > > > > On Thu, 10 Sep 2020 at 09:25, Bjorn Helgaas wrote: > > > > > > On Mon, Aug 24, 2020 at 01:20:25PM +0800, Jiang Biao wrote: > > > > From: Jiang Biao > > >

Re: [PATCH] checkpatch: Allow not using -f with files that are in git

2020-09-12 Thread Joe Perches
On Mon, 2020-08-24 at 17:09 -0700, Joe Perches wrote: > If a file exists in git and checkpatch is used without the -f > flag for scanning a file, then checkpatch will scan the file > assuming it's a patch and emit: > > ERROR: Does not appear to be a unified-diff format patch > > Change the behavi

Re: [PATCH] clk: qcom: gcc-msm8939: remove defined but not used variables

2020-09-12 Thread Bjorn Andersson
On Thu 10 Sep 20:37 CDT 2020, Jason Yan wrote: > This addresses the following gcc warning with "make W=1": > > drivers/clk/qcom/gcc-msm8939.c:610:32: warning: > ‘gcc_xo_gpll6_gpll0a_map’ defined but not used > [-Wunused-const-variable=] > static const struct parent_map gcc_xo_gpll6_gpll0a_map[]

Re: [RFC PATCH v1 1/1] sched/fair: select idle cpu from idle cpumask in sched domain

2020-09-12 Thread Jiang Biao
Hi, Aubrey On Fri, 11 Sep 2020 at 23:48, Aubrey Li wrote: > > Added idle cpumask to track idle cpus in sched domain. When a CPU > enters idle, its corresponding bit in the idle cpumask will be set, > and when the CPU exits idle, its bit will be cleared. > > When a task wakes up to select an idle

Re: [PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread kernel test robot
Hi Douglas, I love your patch! Yet something to improve: [auto build test ERROR on spi/for-next] [also build test ERROR on v5.9-rc4 next-20200911] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https:/

Re: [PATCH v2 1/2] PM / Domains: Add GENPD_FLAG_NO_SUSPEND/RESUME flags

2020-09-12 Thread Bjorn Andersson
On Thu 10 Sep 03:18 CDT 2020, Ulf Hansson wrote: > On Thu, 10 Sep 2020 at 09:23, Sibi Sankar wrote: > > > > On 2020-08-25 23:23, Bjorn Andersson wrote: > > > On Tue 25 Aug 02:20 CDT 2020, Stephen Boyd wrote: > > >> Quoting Bjorn Andersson (2020-08-24 09:42:12) > > >> > On Fri 21 Aug 14:41 PDT 202

Re: [PATCH v3 6/8] iommu/arm-smmu: Add impl hook for inherit boot mappings

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 12:13 CDT 2020, Robin Murphy wrote: > On 2020-09-04 16:55, Bjorn Andersson wrote: > > Add a new operation to allow platform implementations to inherit any > > stream mappings from the boot loader. > > Is there a reason we need an explicit step for this? The aim of the > cfg_probe ho

Re: [PATCH] drm/msm/adreno: fix probe without iommu

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 11:08 CDT 2020, Luca Weiss wrote: > The function iommu_domain_alloc returns NULL on platforms without IOMMU > such as msm8974. This resulted in PTR_ERR(-ENODEV) being assigned to > gpu->aspace so the correct code path wasn't taken. > > Fixes: ccac7ce373c1 ("drm/msm: Refactor address

Re: [PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 20:11 CDT 2020, Doug Anderson wrote: > Hi, > > On Sat, Sep 12, 2020 at 3:53 PM Bjorn Andersson > wrote: > > > > On Sat 12 Sep 16:07 CDT 2020, Douglas Anderson wrote: > > > > > In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I > > > explained that the maximum si

Re: [PATCH] ARM: imx7ulp: enable cpufreq

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 11:31:44AM +0800, peng@nxp.com wrote: > From: Peng Fan > > Enable cpufreq for i.MX7ULP when imx cpufreq dt driver enabled. > > Signed-off-by: Peng Fan Applied, thanks.

Re: [PATCH 1/4] dt-bindings: fsl: add i.MX7ULP PMC binding doc

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 07:19:59AM +, Aisheng Dong wrote: > > From: Peng Fan > > Sent: Friday, September 11, 2020 11:31 AM > > > > Add i.MX7ULP Power Management Controller binding doc > > > > Signed-off-by: Peng Fan > > --- > > .../bindings/arm/freescale/imx7ulp-pmc.yaml | 33 +++

Re: [PATCH net-next v2 7/7] net: ipa: do not enable GSI interrupt for wakeup

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > We now trigger a system resume when we receive an IPA SUSPEND > interrupt. We should *not* wake up on GSI interrupts. > Reviewed-by: Bjorn Andersson > Signed-off-by: Alex Elder > --- > drivers/net/ipa/gsi.c | 17 - > drivers/

Re: [PATCH 1/4] dt-bindings: fsl: add i.MX7ULP PMC binding doc

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 07:45:20AM +, Peng Fan wrote: > > b/Documentation/devicetree/bindings/arm/freescale/imx7ulp-pmc.yaml > > > @@ -0,0 +1,33 @@ > > > +# SPDX-License-Identifier: GPL-2.0 > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/arm/freescale/imx7ulp-pmc.yaml# > >

Re: [PATCH net-next v2 6/7] net: ipa: enable wakeup on IPA interrupt

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > Now that we handle wakeup interrupts properly, arrange for the IPA > interrupt to be treated as a wakeup interrupt. > Reviewed-by: Bjorn Andersson > Signed-off-by: Alex Elder > --- > drivers/net/ipa/ipa_interrupt.c | 14 ++ > 1 fi

Re: [PATCH 1/6] of: Add basic infrastructure to create early probe arrays

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > We currently probe interrupt controller and timers that need > to be available very early using an infratstructure that creates s/infratstructure/infrastructure/ > struct of_device_id instances in a special section. These are > individual struc

Re: [PATCH 8/8] arm64: dts: imx8mm-var-som-symphony: Drop unused gpioledgrp

2020-09-12 Thread Shawn Guo
On Tue, Sep 08, 2020 at 05:02:41PM +0200, Krzysztof Kozlowski wrote: > The gpioledgrp in iomux is not used, so it can be dropped. > > Signed-off-by: Krzysztof Kozlowski Applied, thanks.

Re: [PATCH 1/8] ARM: dts: imx6qdl: Correct interrupt flags in examples

2020-09-12 Thread Shawn Guo
Add Tim who is the board owner. On Tue, Sep 08, 2020 at 05:02:34PM +0200, Krzysztof Kozlowski wrote: > GPIO_ACTIVE_x flags are not correct in the context of interrupt flags. > These are simple defines so they could be used in DTS but they will not > have the same meaning: > 1. GPIO_ACTIVE_HIGH = 0

Re: [PATCH net-next v2 4/7] net: ipa: manage endpoints separate from clock

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > Currently, when (before) the last IPA clock reference is dropped, > all endpoints are suspended. And whenever the first IPA clock > reference is taken, all endpoints are resumed (or started). > > In most cases there's no need to start endpoints w

Re: [PATCH net-next v2 5/7] net: ipa: use device_init_wakeup()

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > The call to wakeup_source_register() in ipa_probe() does not do what > it was intended to do. Call device_init_wakeup() in ipa_setup() > instead, to set the IPA device as wakeup-capable and to initially > enable wakeup capability. > > When we rec

Re: [PATCH net-next v2 3/7] net: ipa: verify reference flag values

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > We take a single IPA clock reference to keep the clock running until > we get a system suspend operation, and maintain a flag indicating > whether that reference has been taken. When a suspend request > arrives, we drop that reference and clear th

[ANNOUNCE] 4.14.198-rt96

2020-09-12 Thread Clark Williams
Hello RT-list! I'm pleased to announce the 4.14.198-rt96 stable release. You can get this release via the git tree at: git://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-stable-rt.git branch: v4.14-rt Head SHA1: 8c4828cbd4220fc1c97c0db534fce850b86aa8d4 Or to build 4.14.198-rt96 direc

BUG: Bad page state in process dirtyc0w_child

2020-09-12 Thread Qian Cai
Occasionally, running this LTP test will trigger an error below on s390: https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/security/dirtyc0w/dirtyc0w.c https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/security/dirtyc0w/dirtyc0w_child.c this .config: https

Re: [PATCH 0/6] arch: arm64: Add label to pcie nodes

2020-09-12 Thread Shawn Guo
On Wed, Sep 09, 2020 at 07:14:30PM +0530, Wasim Khan wrote: > From: Wasim Khan > > This patch series adds label to pcie nodes > so that they are easy to refer. > > Wasim Khan (6): > arm64: dts: lx2160a: Add label to pcie nodes > arm64: dts: ls1012a: Add label to pcie nodes > arm64: dts: ls

Re: [PATCH v5 05/10] powerpc/smp: Dont assume l2-cache to be superset of sibling

2020-09-12 Thread Michael Ellerman
Srikar Dronamraju writes: > * Michael Ellerman [2020-09-11 21:55:23]: > >> Srikar Dronamraju writes: >> > Current code assumes that cpumask of cpus sharing a l2-cache mask will >> > always be a superset of cpu_sibling_mask. >> > >> > Lets stop that assumption. cpu_l2_cache_mask is a superset of

Re: [PATCH] firmware: imx: scu-pd: ignore power domain not owned

2020-09-12 Thread Shawn Guo
On Tue, Sep 08, 2020 at 06:07:01PM +0800, peng@nxp.com wrote: > From: Peng Fan > > Should not register power domain that not owned by current > partition. > > Alought power domains will not be registered when power on failure, > we have to let CPU waste more cycles. > > Whether power on or

Re: [PATCH] Revert "net: linkwatch: add check for netdevice being present to linkwatch_do_dev"

2020-09-12 Thread David Miller
From: Geert Uytterhoeven Date: Sat, 12 Sep 2020 14:33:59 +0200 > "dev" is not the bridge device, but the physical Ethernet interface, which > may already be suspended during s2ram. Hmmm, ok. Looking more deeply NETDEV_CHANGE causes br_port_carrier_check() to run which exits early if netif_runni

Re: [PATCH v3 1/3] ARM: dts: imx6qdl: tqma6: fix indentation

2020-09-12 Thread Shawn Guo
On Mon, Sep 07, 2020 at 10:04:27AM +0200, Matthias Schiffer wrote: > The PMIC configuration is indented one level too deep. > > Fixes: cac849e9bbc8 ("ARM: dts: imx6qdl: add TQMa6{S,Q,QP} SoM") > Signed-off-by: Matthias Schiffer I change the subject prefix to 'ARM: dts: imx6qdl-tqma6:' and applie

Re: [RESEND PATCH v2 2/2] drm/mediatek: mtk_dpi: Convert to bridge driver

2020-09-12 Thread Chun-Kuang Hu
Hi, Daniel Vetter 於 2020年9月10日 週四 下午1:48寫道: > > On Thu, Sep 10, 2020 at 06:35:21AM +0800, Chun-Kuang Hu wrote: > > Hi, Andrzej & Neil: > > > > Enric Balletbo i Serra 於 2020年8月26日 週三 > > 下午4:53寫道: > > > > > > > > Convert mtk_dpi to a bridge driver with built-in encoder support for > > > compatib

Re: [PATCH] tlan: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 09:43:58 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > > W

Re: [PATCH] net: tehuti: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 16:12:32 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > > W

Re: [PATCH] rocker: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 13:44:18 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > > W

Re: [PATCH] sc92031: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 13:28:58 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > > W

Re: [PATCH net-next] net: ethernet: mlx4: Avoid assigning a value to ring_cons but not used it anymore in mlx4_en_xmit()

2020-09-12 Thread David Miller
From: Luo Jiaxing Date: Sat, 12 Sep 2020 16:08:15 +0800 > We found a set but not used variable 'ring_cons' in mlx4_en_xmit(), it will > cause a warning when build the kernel. And after checking the commit record > of this function, we found that it was introduced by a previous patch. > > So, We

Re: [Linux-kernel-mentees] [PATCH net] tipc: Fix memory leak in tipc_group_create_member()

2020-09-12 Thread David Miller
From: Peilin Ye Date: Sat, 12 Sep 2020 06:22:30 -0400 > @@ -291,10 +291,11 @@ static void tipc_group_add_to_tree(struct tipc_group > *grp, > else if (key > nkey) > n = &(*n)->rb_right; > else > - return; > +

Re: [PATCH v2] clk: imx: gate2: Fix a few typos

2020-09-12 Thread Shawn Guo
On Sat, Sep 05, 2020 at 05:10:16PM +0200, Jonathan Neuschäfer wrote: > A few words were misspelled in this comment. > > Signed-off-by: Jonathan Neuschäfer Applied, thanks.

Re: [PATCH 08/13] arm64: dts: imx8mm-var-som-symphony: Use newer interrupts property

2020-09-12 Thread Shawn Guo
On Fri, Sep 04, 2020 at 04:53:07PM +0200, Krzysztof Kozlowski wrote: > The int-gpios was deprecated in favor of generic interrupts property. > > Signed-off-by: Krzysztof Kozlowski Applied patch #8 ~ #13, thanks.

Re: [PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Doug Anderson
Hi, On Sat, Sep 12, 2020 at 3:53 PM Bjorn Andersson wrote: > > On Sat 12 Sep 16:07 CDT 2020, Douglas Anderson wrote: > > > In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I > > explained that the maximum size we could program the FIFO was > > "mas->tx_fifo_depth - 3" but that

Re: [PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread Doug Anderson
Hi, On Sat, Sep 12, 2020 at 3:54 PM Bjorn Andersson wrote: > > On Sat 12 Sep 16:08 CDT 2020, Douglas Anderson wrote: > > > When setting up a bidirectional transfer we need to program both the > > TX and RX lengths. We don't need a memory barrier between those two > > writes. Factor out the __io

Re: [PATCH] mm: memcontrol: Fix out-of-bounds on the buf returned by memory_stat_format

2020-09-12 Thread Andrew Morton
On Sat, 12 Sep 2020 23:51:00 +0800 Muchun Song wrote: > The memory_stat_format() returns a format string, but the return buf > may not including the trailing '\0'. So the users may read the buf > out of bounds. That sounds serious. Is a cc:stable appropriate?

Re: [PATCH 6/6] irqchip/qcom-pdc: Allow modular build

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Switch the driver to a "hybrid probe" model which preserves the > built-in behaviour while allowing the driver to be optionnally > built as a module for development purposes. > Reviewed-by: Bjorn Andersson > Signed-off-by: Marc Zyngier > ---

Re: [PATCH 5/6] irqchip/mtk-sysirq: Allow modular build

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Switch the driver to a "hybrid probe" model which preserves the > built-in behaviour while allowing the driver to be optionnally > built as a module for development purposes. > Reviewed-by: Bjorn Andersson > Signed-off-by: Marc Zyngier > ---

Re: [PATCH 4/6] irqchip/mtk-cirq: Allow modular build

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Switch the driver to a "hybrid probe" model which preserves the > built-in behaviour while allowing the driver to be optionnally > built as a module for development purposes. > Reviewed-by: Bjorn Andersson > Signed-off-by: Marc Zyngier > ---

Re: [PATCH 3/6] irqchip: Introduce IRQCHIP_HYBRID_DRIVER_{BEGIN,END} macros

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Although we are trying to move to a world where a large number > of irqchip drivers can safely be built as platform drivers > the reality is that most endpoint drivers are not ready for that, > and will fail to probe as they expect their interrup

Re: [PATCH 2/6] irqchip: Make IRQCHIP_MATCH() type safe

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > IRQCHIP_DECLARE() is backed by macros that perform some elementary > type checking on the probe function. Unfortunately, IRQCHIP_MATCH() > doesn't, risking difficult debugging sessions... > > Rewrite IRQCHIP_MATCH() in terms of _OF_DECLARE_ELMT(

Re: [PATCH 1/6] of: Add basic infrastructure to create early probe arrays

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > We currently probe interrupt controller and timers that need > to be available very early using an infratstructure that creates > struct of_device_id instances in a special section. These are > individual structures that are ultimately collated b

Re: [PATCH] spi: spi-geni-qcom: Don't wait to start 1st transfer if transmitting

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 13:17 CDT 2020, Douglas Anderson wrote: > If we're sending bytes over SPI, we know the FIFO is empty at the > start of the transfer. There's no reason to wait for the interrupt > telling us to start--we can just start right away. Then if we > transmit everything in one swell foop w

Re: [PATCH 2/3] spi: spi-geni-qcom: Don't program CS_TOGGLE again and again

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 16:08 CDT 2020, Douglas Anderson wrote: > We always toggle the chip select manually in spi-geni-qcom so that we > can properly implement the Linux API. There's no reason to program > this to the hardware on every transfer. Program it once at init and > be done with it. > > This sa

Re: [PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 16:08 CDT 2020, Douglas Anderson wrote: > When setting up a bidirectional transfer we need to program both the > TX and RX lengths. We don't need a memory barrier between those two > writes. Factor out the __iowmb() and use writel_relaxed(). This > saves a fraction of a microsecon

Re: [PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 16:07 CDT 2020, Douglas Anderson wrote: > In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I > explained that the maximum size we could program the FIFO was > "mas->tx_fifo_depth - 3" but that I chose "mas->tx_fifo_depth()" > because I was worried about decreased

Re: [PATCH v12 0/5] Add support for DisplayPort driver on SnapDragon

2020-09-12 Thread abhinavk
Thanks Rob for pushing the changes. I confirm that the versions which you have pushed are the latest ones. I have rebased audio changes on top of the latest DP changes and posted them here https://patchwork.freedesktop.org/series/81612/. The other DP change which is missing is the video pattern

[PATCH 2/3] spi: spi-geni-qcom: Don't program CS_TOGGLE again and again

2020-09-12 Thread Douglas Anderson
We always toggle the chip select manually in spi-geni-qcom so that we can properly implement the Linux API. There's no reason to program this to the hardware on every transfer. Program it once at init and be done with it. This saves some part of a microsecond of overhead on each transfer. While

[PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread Douglas Anderson
When setting up a bidirectional transfer we need to program both the TX and RX lengths. We don't need a memory barrier between those two writes. Factor out the __iowmb() and use writel_relaxed(). This saves a fraction of a microsecond of setup overhead on bidirectional transfers. Signed-off-by:

[PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Douglas Anderson
In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I explained that the maximum size we could program the FIFO was "mas->tx_fifo_depth - 3" but that I chose "mas->tx_fifo_depth()" because I was worried about decreased bandwidth. Since that time: * All the interconnect patches hav

RE: [PATCH 01/24] inet_net_pton.3: Use 'PRIx32' rather than "%x" when printing 'uint32_t' values

2020-09-12 Thread David Laight
From: Michael Kerrisk > Sent: 11 September 2020 10:31 > > Hi Alex, > > On 9/10/20 11:13 PM, Alejandro Colomar wrote: > > Signed-off-by: Alejandro Colomar > > --- > > man3/inet_net_pton.3 | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/man3/inet_net_pton.3 b/man3

Re: [PATCH v3 0/10] libperf and arm64 userspace counter access support

2020-09-12 Thread Jiri Olsa
On Fri, Sep 11, 2020 at 03:51:08PM -0600, Rob Herring wrote: > This is resurrecting Raphael's series[1] to enable userspace counter > access on arm64. My previous versions are here[2][3]. > > Changes in v3: > - Dropped removing x86 rdpmc test until libperf tests can run via 'perf test' > - Added

Re: [PATCH] perf bench: Fix 2 memory sanitizer warnings

2020-09-12 Thread Jiri Olsa
On Fri, Sep 11, 2020 at 10:37:25PM -0700, Ian Rogers wrote: > Memory sanitizer warns if a write is performed where the memory > being read for the write is uninitialized. Avoid this warning by > initializing the memory. > > Signed-off-by: Ian Rogers Acked-by: Jiri Olsa thanks, jirka > --- >

Re: [PATCH v3 0/4] Fixes for setting event freq/periods

2020-09-12 Thread Jiri Olsa
On Fri, Sep 11, 2020 at 07:56:51PM -0700, Ian Rogers wrote: > Some fixes that address issues for regular and pfm4 events with 2 > additional perf_event_attr tests. Various authors, David Sharp isn't > currently at Google. > > v3. moved a loop into a helper following Adrian Hunter's suggestion. >

[PATCH 2/3] phy: cadence: salvo: Constify cdns_nxp_sequence_pair

2020-09-12 Thread Rikard Falkeborn
cdns_nxp_sequence_pair[] are never modified and can be made const to allow the compiler to put them in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/cadence/phy-cadence-salvo.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/phy/cadence/phy-c

Re: [RESEND][RFC PATCH 0/6] Fork brute force attack mitigation (fbfam)

2020-09-12 Thread Ondrej Mosnacek
On Sat, Sep 12, 2020 at 4:51 PM Mel Gorman wrote: > On Sat, Sep 12, 2020 at 11:36:52AM +0200, John Wood wrote: > > On Sat, Sep 12, 2020 at 12:56:18AM -0700, Kees Cook wrote: > > > On Sat, Sep 12, 2020 at 10:03:23AM +1000, James Morris wrote: > > > > On Thu, 10 Sep 2020, Kees Cook wrote: > > > > >

[PATCH 3/3] phy: cadence: torrent: Constify regmap_config structs

2020-09-12 Thread Rikard Falkeborn
The regmap_config structs are never modified and can be made const to allow the compiler to put them in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/cadence/phy-cadence-torrent.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/phy/cad

[PATCH 1/3] phy: cadence: Sierra: Constify static structs

2020-09-12 Thread Rikard Falkeborn
The static cdns_reg_pairs and regmap_config structs are not modified and can be made const. This allows the compiler to put them in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/cadence/phy-cadence-sierra.c | 24 1 file changed, 12 insertions(+), 12 d

[PATCH 0/3] drivers/phy/cadence: Constify static structs

2020-09-12 Thread Rikard Falkeborn
Constify a number of static structs that are never changed to allow the compiler to put them in read-only memory. Rikard Falkeborn (3): phy: cadence: Sierra: Constify static structs phy: cadence: salvo: Constify cdns_nxp_sequence_pair phy: cadence: torrent: Constify regmap_config structs d

Re: [PATCH v3 3/7] ASoC: sun4i-i2s: Add support for H6 I2S

2020-09-12 Thread Samuel Holland
Maxime, On 9/10/20 9:33 AM, Maxime Ripard wrote: > On Thu, Sep 03, 2020 at 09:54:39PM -0500, Samuel Holland wrote: >> On 9/3/20 3:58 PM, Maxime Ripard wrote: >>> On Thu, Sep 03, 2020 at 10:02:31PM +0200, Clément Péron wrote: Hi Maxime, On Wed, 29 Jul 2020 at 17:16, Mark Brown wrote

Re: [PATCH] Using a pointer and kzalloc in place of a struct directly

2020-09-12 Thread Anant Thazhemadam
On 12/09/20 8:25 pm, Greg KH wrote: > On Sat, Sep 12, 2020 at 05:43:38PM +0530, Anant Thazhemadam wrote: >> On 12/09/20 5:17 pm, Greg KH wrote: >>> Note, your "To:" line seemed corrupted, and why not cc: the bpf mailing >>> list as well? >> Oh, I'm sorry about that. I pulled the emails of all the

Re: [GIT PULL] Btrfs fixes for 5.9-rc5

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Sat, 12 Sep 2020 19:01:18 +0200: > git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux.git for-5.9-rc4-tag has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/edf6b0e1e4ddb12e022ce0c17829bad6d4161ea7 Thank you! -- Deet-doot-dot, I am a

Re: [GIT PULL] SMB3 DFS Fix

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Sat, 12 Sep 2020 11:56:45 -0500: > git://git.samba.org/sfrench/cifs-2.6.git tags/5.9-rc4-smb3-fix has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/5a3c558a9f05f4664f569b06f04d6b217785fd21 Thank you! -- Deet-doot-dot, I am a bot. https://ko

Re: [GIT PULL] seccomp fixes for v5.9-rc5

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Fri, 11 Sep 2020 17:20:02 -0700: > https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git > tags/seccomp-v5.9-rc5 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/ef2e9a563b0cd7965e2a1263125dcbb1c86aa6cc Thank you! -- Deet-doot-do

Re: [GIT PULL] libnvdimm fix for v5.9-rc5

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Fri, 11 Sep 2020 22:19:57 +: > git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm.git > tags/libnvdimm-fix-v5.9-rc5 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/4f8b0a5b3f7e5f03b188de9025b60c15559790f9 Thank you! -- Deet-

RE: [PATCH v3 11/11] scsi: storvsc: Support PAGE_SIZE larger than 4K

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Hyper-V always use 4k page size (HV_HYP_PAGE_SIZE), so when > communicating with Hyper-V, a guest should always use HV_HYP_PAGE_SIZE > as the unit for page related data. For storvsc, the data is > vmbus_packet_mpb_array. And since i

Re: [PATCH] media: vidtv: fix 32-bit warnings

2020-09-12 Thread Joe Perches
On Sat, 2020-09-12 at 11:22 +0200, Mauro Carvalho Chehab wrote: > There are several warnings produced when the driver is built > for 32-bit archs. Solve them. [] > diff --git a/drivers/media/test-drivers/vidtv/vidtv_common.c > b/drivers/media/test-drivers/vidtv/vidtv_common.c [] > diff --git a/dri

[PATCH] natsemi: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Christophe JAILLET
The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'alloc_ring()' (natsemi.c) GFP_KERNEL can be used be

RE: [PATCH v3 10/11] Driver: hv: util: Make ringbuffer at least take two pages

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at > least 2 * PAGE_SIZE: one page for the header and at least one page of > the data part (because of the alignment requirement for double mapping). > > So make

RE: [PATCH v3 09/11] HID: hyperv: Make ringbuffer at least take two pages

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at > least 2 * PAGE_SIZE: one page for the header and at least one page of > the data part (because of the alignment requirement for double mapping). > > So make

RE: [PATCH v3 08/11] Input: hyperv-keyboard: Make ringbuffer at least take two pages

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at > least 2 * PAGE_SIZE: one page for the header and at least one page of > the data part (because of the alignment requirement for double mapping). > > So make

RE: [PATCH v3 07/11] hv_netvsc: Use HV_HYP_PAGE_SIZE for Hyper-V communication

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When communicating with Hyper-V, HV_HYP_PAGE_SIZE should be used since > that's the page size used by Hyper-V and Hyper-V expects all > page-related data using the unit of HY_HYP_PAGE_SIZE, for example, the > "pfn" in hv_page_buffer

RE: [PATCH v3 06/11] hv: hyperv.h: Introduce some hvpfn helper functions

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When a guest communicate with the hypervisor, it must use HV_HYP_PAGE to > calculate PFN, so introduce a few hvpfn helper functions as the > counterpart of the page helper functions. This is the preparation for > supporting guest wh

RE: [PATCH v3 05/11] Drivers: hv: vmbus: Move virt_to_hvpfn() to hyperv header

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > There will be more places other than vmbus where we need to calculate > the Hyper-V page PFN from a virtual address, so move virt_to_hvpfn() to > hyperv generic header. > > Signed-off-by: Boqun Feng > --- > drivers/hv/channel.c

RE: [PATCH v3 04/11] Drivers: hv: Use HV_HYP_PAGE in hv_synic_enable_regs()

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Both the base_*_gpa should use the guest page number in Hyper-V page, so > use HV_HYP_PAGE instead of PAGE. > > Signed-off-by: Boqun Feng > --- > drivers/hv/hv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Rev

RE: [PATCH v3 03/11] Drivers: hv: vmbus: Introduce types of GPADL

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > This patch introduces two types of GPADL: HV_GPADL_{BUFFER, RING}. The > types of GPADL are purely the concept in the guest, IOW the hypervisor > treat them as the same. > > The reason of introducing the types for GPADL is to supp

RE: [PATCH v3 02/11] Drivers: hv: vmbus: Move __vmbus_open()

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Pure function movement, no functional changes. The move is made, because > in a later change, __vmbus_open() will rely on some static functions > afterwards, so we separate the move and the modification of > __vmbus_open() in two pa

  1   2   3   4   >