Re: [PATCH v4 1/4] fpga: dfl: change data type of feature id to u16

2020-08-28 Thread Xu Yilun
On Wed, Aug 19, 2020 at 09:14:31PM -0700, Moritz Fischer wrote: > On Thu, Aug 13, 2020 at 05:04:09PM +0800, Xu Yilun wrote: > > On Thu, Aug 13, 2020 at 08:28:05AM +, David Laight wrote: > > > From: Xu Yilun > > > > Sent: 13 August 2020 08:59 > > > > On Wed, Aug 12, 2020 at 08:52:39AM +,

Re: [PATCH 1/4] drivers: hv: dxgkrnl: core code

2020-08-28 Thread Greg KH
On Thu, Aug 27, 2020 at 04:29:59PM -0700, Iouri Tarassov wrote: > On 8/14/2020 5:55 AM, Greg KH wrote: > > On Fri, Aug 14, 2020 at 08:38:53AM -0400, Sasha Levin wrote: > > > Add support for a Hyper-V based vGPU implementation that exposes the > > > DirectX API to Linux userspace. > > > >

Re: drivers/net/wireless/realtek/rtw88/pci.c:1477:5: warning: no previous prototype for 'rtw_pci_probe'

2020-08-28 Thread Kalle Valo
Tony Chuang writes: >> + linux-wireless >> >> kernel test robot writes: >> >> > Hi Zong-Zhe, >> > >> > FYI, the error/warning still remains. >> > >> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git >> master >> > head: 23ee3e4e5bd27bdbc0f1785eef7209ce872794c7 >>

Re: [PATCH 06/23] rxrpc: use ASSERT_FAIL()/ASSERT_WARN() to cleanup some code

2020-08-28 Thread kernel test robot
Hi Chunguang, Thank you for the patch! Yet something to improve: [auto build test ERROR on mkp-scsi/for-next] [also build test ERROR on scsi/for-next block/for-next linus/master asm-generic/master v5.9-rc2 next-20200827] [If your patch is applied to the wrong git tree, kindly drop us a note.

[PATCH V2 0/8] opp: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
Hello, This cleans up some of the user code around calls to dev_pm_opp_of_remove_table(). All the patches can be picked by respective maintainers directly except for the last patch, which needs the previous two to get merged first. These are based for 5.9-rc1. Rajendra, Since most of these

[PATCH V2 3/8] drm/msm: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. While at it, also create a label to put clkname. Signed-off-by: Viresh

[PATCH V2 2/8] drm/lima: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. Reviewed-by: Qiang Yu Signed-off-by: Viresh Kumar --- V2: Applied

[PATCH V2 4/8] mmc: sdhci-msm: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. Signed-off-by: Viresh Kumar --- V2: - Compare with -ENODEV only for

[PATCH V2 6/8] spi: spi-qcom-qspi: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. While at it, create a new label and put clkname on errors. Signed-off-by:

[PATCH V2 7/8] tty: serial: qcom_geni_serial: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. While at it, create a new label to put clkname on errors. Signed-off-by:

[PATCH V2 8/8] qcom-geni-se: remove has_opp_table

2020-08-28 Thread Viresh Kumar
has_opp_table isn't used anymore, remove it. Signed-off-by: Viresh Kumar --- V2: No changes. --- include/linux/qcom-geni-se.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/include/linux/qcom-geni-se.h b/include/linux/qcom-geni-se.h index 8f385fbe5a0e..02d1417c8ecf 100644 ---

[PATCH V2 5/8] spi: spi-geni-qcom: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. While at it, create a new label and put clkname on errors. Signed-off-by:

[PATCH V2 1/8] cpufreq: imx6q: Unconditionally call dev_pm_opp_of_remove_table()

2020-08-28 Thread Viresh Kumar
dev_pm_opp_of_remove_table() doesn't report any errors when it fails to find the OPP table with error -ENODEV (i.e. OPP table not present for the device). And we can call dev_pm_opp_of_remove_table() unconditionally here. Signed-off-by: Viresh Kumar --- V2: No changes. ---

Re: [PATCH 1/4] drivers: hv: dxgkrnl: core code

2020-08-28 Thread Greg KH
On Thu, Aug 27, 2020 at 05:05:44PM -0700, Iouri Tarassov wrote: > > On 8/14/2020 6:04 AM, Greg KH wrote: > > On Fri, Aug 14, 2020 at 08:38:53AM -0400, Sasha Levin wrote: > > > Add support for a Hyper-V based vGPU implementation that exposes the > > > DirectX API to Linux userspace. > > > >

Re: [PATCH 1/4] drivers: hv: dxgkrnl: core code

2020-08-28 Thread Greg KH
On Thu, Aug 27, 2020 at 04:45:55PM -0700, Iouri Tarassov wrote: > > On 8/14/2020 5:57 AM, Greg KH wrote: > > On Fri, Aug 14, 2020 at 08:38:53AM -0400, Sasha Levin wrote: > > > Add support for a Hyper-V based vGPU implementation that exposes the > > > DirectX API to Linux userspace. > > > > Api

[PATCH] aio: make aio wait path to account iowait time

2020-08-28 Thread Xianting Tian
As the normal aio wait path(read_events() -> wait_event_interruptible_hrtimeout()) doesn't account iowait time, so use this patch to make it to account iowait time, which can truely reflect the system io situation when using a tool like 'top'. The test result as below. Test environment:

Re: [PATCH 1/4] drivers: hv: dxgkrnl: core code

2020-08-28 Thread Greg KH
On Thu, Aug 27, 2020 at 05:25:23PM -0700, Iouri Tarassov wrote: > > > +{ > > > + struct dxgprocess_adapter *adapter_info = dxgmem_alloc(process, > > > + > > > DXGMEM_PROCESS_ADAPTER, > > > +

Re: [PATCH 1/4] drivers: hv: dxgkrnl: core code

2020-08-28 Thread Greg KH
On Thu, Aug 27, 2020 at 05:25:23PM -0700, Iouri Tarassov wrote: > > > +bool dxghwqueue_acquire_reference(struct dxghwqueue *hwqueue) > > > +{ > > > + return refcount_inc_not_zero(>refcount); > > > +} > > > > Midlayers are evil. > I strongly agree in general, but think that in our case the layers

Re: Some questions about the patching process

2020-08-28 Thread Greg Kroah-Hartman
On Thu, Aug 27, 2020 at 02:17:20PM -0500, Qiushi Wu wrote: > Hi Greg, > Thanks for your response! You responded in html format which got rejected by the public list, please resend in text-only and I will be glad to reply. thanks, greg k-h

Re: [PATCH v11 25/25] x86/cet/shstk: Add arch_prctl functions for shadow stack

2020-08-28 Thread Florian Weimer
* H. J. Lu: > Can you think of ANY issues of passing more arguments to arch_prctl? On x32, the glibc arch_prctl system call wrapper only passes two arguments to the kernel, and applications have no way of detecting that. musl only passes two arguments on all architectures. It happens to work

Re: [PATCH v2 09/18] iio: afe: iio-rescale: Simplify with dev_err_probe()

2020-08-28 Thread Krzysztof Kozlowski
On Thu, Aug 27, 2020 at 11:46:40PM +0200, Peter Rosin wrote: > On 2020-08-27 21:26, Krzysztof Kozlowski wrote: > > Common pattern of handling deferred probe can be simplified with > > dev_err_probe(). Less code and also it prints the error value. > > > > Signed-off-by: Krzysztof Kozlowski > >

Re: [PATCH 3/3] drm/vc4: hdmi: Add pixel bvb clock control

2020-08-28 Thread Hoegeun Kwon
On 8/27/20 6:49 PM, Stefan Wahren wrote: > Am 27.08.20 um 06:35 schrieb Hoegeun Kwon: >> Hi Stefan, >> >> Thank you for your review. >> >> >> On 8/26/20 7:04 PM, Stefan Wahren wrote: >>> Hi Hoeguen, >>> >>> Am 21.08.20 um 09:10 schrieb Hoegeun Kwon: There is a problem that the output does not

Re: [PATCH 2/3] clk: samsung: exynos5420/5250: Add IDs to the CPU parent clk definitions

2020-08-28 Thread Krzysztof Kozlowski
On Wed, Aug 26, 2020 at 07:15:28PM +0200, Sylwester Nawrocki wrote: > Use non-zero clock IDs in definitions of the CPU parent clocks > for exynos5420, exynos5250 SoCs. This will allow us to reference > the parent clocks directly in the driver by cached struct clk_hw > pointers, rather than doing

Re: [PATCH 1/3] clk: samsung: Add clk ID definitions for the CPU parent clocks

2020-08-28 Thread Krzysztof Kozlowski
On Wed, Aug 26, 2020 at 07:15:27PM +0200, Sylwester Nawrocki wrote: > Add clock ID definitions for the CPU parent clocks for SoCs > which don't have such definitions yet. This will allow us to > reference the parent clocks directly by cached struct clk_hw > pointers in the clock provider, rather

[PATCH 0/2] fix memleak when using pci_iounmap()

2020-08-28 Thread Yang Yingliang
config GENERIC_IOMAP is not selected on some arch, pci_iounmap() don't implement, when we using pci_iomap/pci_iounmap, it will lead to memory leak. This patch set moves the implemention of pci_iounmap() to lib/pci_iomap.c to fix this. Yang Yingliang (2): iomap: move some definitions to

[PATCH 1/2] iomap: move some definitions to include/linux/io.h

2020-08-28 Thread Yang Yingliang
Move some IO macros and bad_io_access() to include/linux/io.h This prepares for moving pci_iounmap() to lib/pci_iomap.c. Signed-off-by: Yang Yingliang --- include/linux/io.h | 36 lib/iomap.c| 36 2 files changed,

Re: [PATCH v2] opp: Power on (virtual) power domains managed by the OPP core

2020-08-28 Thread Viresh Kumar
On 27-08-20, 13:44, Stephan Gerhold wrote: > Hmm. Actually I was using this parameter for initial testing, and forced > on the power domains from the qcom-cpufreq-nvmem driver. For my v1 patch > I wanted to enable the power domains in dev_pm_opp_set_rate(), so there > using the virt_devs parameter

[PATCH 2/2] pci: fix memleak when calling pci_iomap/unmap()

2020-08-28 Thread Yang Yingliang
config GENERIC_IOMAP is disabled on some archs(e.g. arm64), so pci_iounmap() does nothing, when we using pci_iomap/pci_iounmap(), it will lead to memory leak. Move pci_iounmap() to lib/pci_map.c to fix this. Signed-off-by: Yang Yingliang --- include/asm-generic/pci_iomap.h | 2 ++ lib/iomap.c

Re: [PATCH v2 3/5] fpga manager: xilinx-spi: rework write_complete loop implementation

2020-08-28 Thread Luca Ceresoli
Hi Tom, On 27/08/20 21:34, Tom Rix wrote: > > On 8/27/20 12:26 PM, Luca Ceresoli wrote: >> Hi Tom, >> >> thanks for the prompt feedback! >> >> On 27/08/20 20:59, Tom Rix wrote: >>> On 8/27/20 7:32 AM, Luca Ceresoli wrote: In preparation to add error checking for gpiod_get_value(), rework

[PATCH] In the device tree file openbmc-flash-layout.dtsi, rofs partition offset is defined as 0x4c0000, while its node name is "rofs@c0000" which is a typo. It should be "rofs@4c0000".

2020-08-28 Thread Kun Zhao
ARM: dts: openbmc-flash-layout: Fix a typo of rofs offset Signed-off-by: Kun Zhao --- arch/arm/boot/dts/openbmc-flash-layout.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/openbmc-flash-layout.dtsi b/arch/arm/boot/dts/openbmc-flash-layout.dtsi index

Re: [PATCH v2] x86: Use xorl %0,%0 in __get_user_asm

2020-08-28 Thread Uros Bizjak
On Thu, Aug 27, 2020 at 10:14 PM Al Viro wrote: > > On Thu, Aug 27, 2020 at 08:09:04PM +0200, Uros Bizjak wrote: > > xorl %0,%0 is equivalent to xorq %0,%0 as both will zero the > > entire register. Use xorl %0,%0 for all operand sizes to avoid > > REX prefix byte when legacy registers are used

Re: [PATCH 3/3] clk: samsung: Use cached clk_hws instead of __clk_lookup() calls

2020-08-28 Thread Krzysztof Kozlowski
On Wed, Aug 26, 2020 at 07:15:29PM +0200, Sylwester Nawrocki wrote: > For the CPU clock registration two parent clocks are required, these > are now being passed as struct clk_hw pointers, rather than by the > global scope names. That allows us to avoid __clk_lookup() calls > and simplifies a bit

Re: [PATCH 1/2] ASoC: wm8994: Skip setting of the WM8994_MICBIAS register for WM1811

2020-08-28 Thread Krzysztof Kozlowski
On Thu, Aug 27, 2020 at 07:33:56PM +0200, Sylwester Nawrocki wrote: > The WM8994_MICBIAS register is not available in the WM1811 CODEC so skip > initialization of that register for that device. > This suppresses an error during boot: > "wm8994-codec: ASoC: error at snd_soc_component_update_bits on

Re: [PATCH 2/2] ASoC: wm8994: Ensure the device is resumed in wm89xx_mic_detect functions

2020-08-28 Thread Krzysztof Kozlowski
On Thu, Aug 27, 2020 at 07:33:57PM +0200, Sylwester Nawrocki wrote: > When the wm8958_mic_detect, wm8994_mic_detect functions get called from > the machine driver, e.g. from the card's late_probe() callback, the CODEC > device may be PM runtime suspended and any regmap writes have no effect. > Add

Re: [PATCH] sched/numa: use runnable_avg to classify node

2020-08-28 Thread Vincent Guittot
On Thu, 27 Aug 2020 at 20:22, Mel Gorman wrote: > > On Thu, Aug 27, 2020 at 05:43:11PM +0200, Vincent Guittot wrote: > > > The testing was a mixed bag of wins and losses but wins more than it > > > loses. Biggest loss was a 9.04% regression on nas-SP using openmp for > > > parallelisation on

Re: [PATCH v5 03/18] crypto: sun8i-ss: support hash algorithms

2020-08-28 Thread Herbert Xu
On Fri, Aug 21, 2020 at 01:43:20PM +, Corentin Labbe wrote: > > + err = pm_runtime_get_sync(op->ss->dev); > + if (err < 0) > + goto error_pm; > + return 0; You need to handle the error case like this: https://patchwork.kernel.org/patch/11728595/ Cheers, -- Email:

Re: [PATCH v5 02/18] crypto: sun8i-ss: Add support for the PRNG

2020-08-28 Thread Herbert Xu
On Fri, Aug 21, 2020 at 01:43:19PM +, Corentin Labbe wrote: > > + err = pm_runtime_get_sync(ss->dev); > + if (err < 0) > + goto err_pm; > + err = 0; The error case needs to do this: https://patchwork.kernel.org/patch/11728595/ Cheers, -- Email: Herbert Xu Home

Re: [PATCH 1/7] soundwire: bus: use property to set interrupt masks

2020-08-28 Thread Vinod Koul
Hi Mark, On 18-08-20, 22:06, Bard Liao wrote: > From: Pierre-Louis Bossart > > Add a slave-level property and program the SCP_INT1_MASK as desired by > the codec driver. Since there is no DisCo property this has to be an > implementation-specific firmware property or hard-coded in the driver. >

[PATCH] clk: imx: fix i.MX7D peripheral clk mux flags

2020-08-28 Thread peng . fan
From: Peng Fan According to RM, Page 574, Chapter 5.2.6.4.3 Peripheral clock slice, "IP clock slices must be stopped to change the clock source.". So we must have CLK_SET_PARENT_GATE flag to avoid glitch. Signed-off-by: Peng Fan --- drivers/clk/imx/clk-imx7d.c | 131

Re: [PATCH v3] mm: Fix kthread_use_mm() vs TLB invalidate

2020-08-28 Thread Nicholas Piggin
Excerpts from Nicholas Piggin's message of August 28, 2020 1:26 pm: > Excerpts from pet...@infradead.org's message of August 21, 2020 11:04 pm: >> On Fri, Aug 21, 2020 at 11:09:51AM +0530, Aneesh Kumar K.V wrote: >>> Peter Zijlstra writes: >>> >>> > For SMP systems using IPI based TLB

Re: [PATCH] Remove unneeded variable t1

2020-08-28 Thread Herbert Xu
Xu Wang wrote: > Remove unneeded variable t1 seed_encrypt() and > seed_decrypt(). > > Signed-off-by: Xu Wang > --- > crypto/seed.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/crypto/seed.c b/crypto/seed.c > index 5e3bef3a617d..69b3058d6a32 100644 > ---

Re: [PATCH v2 09/18] iio: afe: iio-rescale: Simplify with dev_err_probe()

2020-08-28 Thread Peter Rosin
On 2020-08-28 08:24, Krzysztof Kozlowski wrote: > On Thu, Aug 27, 2020 at 11:46:40PM +0200, Peter Rosin wrote: >> On 2020-08-27 21:26, Krzysztof Kozlowski wrote: >>> Common pattern of handling deferred probe can be simplified with >>> dev_err_probe(). Less code and also it prints the error

Re: [PATCH] leds: pwm: Allow automatic labels for DT based devices

2020-08-28 Thread Alexander Dahl
Hello Jacek, Am Donnerstag, 27. August 2020, 23:28:45 CEST schrieb Jacek Anaszewski: > On 8/26/20 11:37 AM, Alexander Dahl wrote: > > From: Alexander Dahl > > > > If LEDs are configured through device tree and the property 'label' is > > omitted, the label is supposed to be generated from the

Re: [PATCH v2 09/18] iio: afe: iio-rescale: Simplify with dev_err_probe()

2020-08-28 Thread Krzysztof Kozlowski
On Fri, 28 Aug 2020 at 08:58, Peter Rosin wrote: > >> I'm not a huge fan of adding *one* odd line breaking the 80 column > >> recommendation to any file. I like to be able to fit multiple > >> windows side by side in a meaningful way. Also, I don't like having > >> a shitload of emptiness on my

Re: [PATCH v2] ARM: STM32: Replace HTTP links with HTTPS ones

2020-08-28 Thread Alexandre Torgue
Hi Alexander On 7/22/20 9:20 PM, Alexander A. Klimov wrote: Rationale: Reduces attack surface on kernel devs opening the links for MITM as HTTPS traffic is much harder to manipulate. Deterministic algorithm: For each file: If not .svg: For each line: If doesn't contain

Re: [PATCH] linux/kernel.h: add container_from()

2020-08-28 Thread Allen
> > I don't see that kind of redundancy being a _problem_, though. "So > much redundancy" is just over-stating the issue completely. > > In fact, we often encourage people to split declaration from > initialization exactly because it results in simpler expressions and > more legible code, even if

Re: [PATCH v2] null_blk: add support for max open/active zone limit for zoned devices

2020-08-28 Thread Damien Le Moal
On 2020/08/27 22:50, Niklas Cassel wrote: > Add support for user space to set a max open zone and a max active zone > limit via configfs. By default, the default values are 0 == no limit. > > Call the block layer API functions used for exposing the configured > limits to sysfs. > > Add

Re: [PATCH v2] i2c: i2c-rcar: Auto select RESET_CONTROLLER

2020-08-28 Thread Wolfram Sang
On Thu, Aug 27, 2020 at 11:23:30AM +0200, Eugeniu Rosca wrote: > From: Dirk Behme > > The i2c-rcar driver utilizes the Generic Reset Controller kernel > feature, so select the RESET_CONTROLLER option when the I2C_RCAR > option is selected. > > Fixes: 2b16fd63059ab9 ("i2c: rcar: handle RXDMA HW

[PATCH net] drivers/net/wan/hdlc_cisco: Add hard_header_len

2020-08-28 Thread Xie He
This driver didn't set hard_header_len. This patch sets hard_header_len for it according to its header_ops->create function. This driver's header_ops->create function (cisco_hard_header) creates a header of (struct hdlc_header), so hard_header_len should be set to sizeof(struct hdlc_header). Cc:

[PATCH v3 1/3] riscv: Set more data to cacheinfo

2020-08-28 Thread Zong Li
Set cacheinfo.{size,sets,line_size} for each cache node, then we can get these information from userland through auxiliary vector. Signed-off-by: Zong Li --- arch/riscv/kernel/cacheinfo.c | 59 ++- 1 file changed, 44 insertions(+), 15 deletions(-) diff --git

Re: [PATCH 12/19] md: use __register_blkdev to allocate devices on demand

2020-08-28 Thread Song Liu
On Tue, Aug 25, 2020 at 11:53 PM Christoph Hellwig wrote: > > Use the simpler mechanism attached to major_name to allocate a brd device > when a currently unregistered minor is accessed. > > Signed-off-by: Christoph Hellwig Acked-by: Song Liu

[PATCH v3 0/3] Get cache information from userland

2020-08-28 Thread Zong Li
There are no standard CSR registers to provide cache information, the way for RISC-V is to get this information from DT. Currently, AT_L1I_X, AT_L1D_X and AT_L2_X are present in glibc header, and sysconf syscall could use them to get information of cache through AUX vector. We exploit 'struct

[PATCH v3 3/3] riscv: Add cache information in AUX vector

2020-08-28 Thread Zong Li
There are no standard CSR registers to provide cache information, the way for RISC-V is to get this information from DT. Currently, AT_L1I_X, AT_L1D_X and AT_L2_X are present in glibc header, and sysconf syscall could use them to get information of cache through AUX vector. The result of 'getconf

[PATCH v3 2/3] riscv: Define AT_VECTOR_SIZE_ARCH for ARCH_DLINFO

2020-08-28 Thread Zong Li
AT_VECTOR_SIZE_ARCH should be defined with the maximum number of NEW_AUX_ENT entries that ARCH_DLINFO can contain, but it wasn't defined for RISC-V at all even though ARCH_DLINFO will contain one NEW_AUX_ENT for the VDSO address. Signed-off-by: Zong Li Reviewed-by: Palmer Dabbelt ---

[PATCH] MAINTAINERS: orphan sections with qlogic.com group alias

2020-08-28 Thread Lukas Bulwahn
--- applies cleanly on current master and next-20200828 James, Martin, please pick this minor non-urgent patch. Anil, Sudarsana, if these drivers are still maintained by qlogic, please provide actual names of people that maintain these drivers. MAINTAINERS | 9 +++-- 1 file changed, 3

Re: [PATCH] linux/kernel.h: add container_from()

2020-08-28 Thread Allen
> > > You really have to pick some pretty excessive type names (or variable > > > names) to get close to 80 characters. Again, to pick an example: > > > > > > struct timer_group_priv *priv = container_of(handle, > > > struct timer_group_priv, timer[handle->num]); >

Re: [PATCH v2 13/18] iio: imu: inv_mpu6050: Simplify with dev_err_probe()

2020-08-28 Thread Jean-Baptiste Maneyrol
Hello, thanks for the patch. Best regards, JB Reviewed-by: Jean-Baptiste Maneyrol From: linux-iio-ow...@vger.kernel.org on behalf of Krzysztof Kozlowski Sent: Thursday, August 27, 2020 21:26 To: Jonathan Cameron; Hartmut Knaack; Lars-Peter Clausen;

[PATCH] kernel.h: Silence sparse warning in lower_32_bits

2020-08-28 Thread Herbert Xu
I keep getting sparse warnings in crypto such as: CHECK ../drivers/crypto/ccree/cc_hash.c ../drivers/crypto/ccree/cc_hash.c:49:9: warning: cast truncates bits from constant value (47b5481dbefa4fa4 becomes befa4fa4) ../drivers/crypto/ccree/cc_hash.c:49:26: warning: cast truncates bits from

[PATCH v9 0/4] scsi: ufs: Add Host Performance Booster Support

2020-08-28 Thread Daejun Park
Changelog: v8 -> v9 1. Change sysfs initialization. 2. Change reading descriptor during HPB initialization 3. Fix problems commentted in Bart's review. 4. Change base commit from 5.9/scsi-queue to 5.10/scsi-queue. v7 -> v8 Remove wrongly added tags. v6 -> v7 1. Remove UFS feature layer. 2.

Re: [PATCH -next] crypto: sa2ul - Fix pointer-to-int-cast warning

2020-08-28 Thread Herbert Xu
On Tue, Aug 18, 2020 at 10:00:01PM +0800, YueHaibing wrote: > drivers/crypto/sa2ul.c: In function ‘sa_sha_init’: > drivers/crypto/sa2ul.c:1486:33: warning: cast from pointer to integer of > different size [-Wpointer-to-int-cast] >crypto_ahash_digestsize(tfm), (u64)rctx); >

Re: [PATCH] crypto: ccree - fix runtime PM imbalance on error

2020-08-28 Thread Herbert Xu
On Fri, Aug 21, 2020 at 04:15:13PM +0800, dinghao@zju.edu.cn wrote: > pm_runtime_get_sync() increments the runtime PM usage counter > even when it returns an error code. However, users of cc_pm_get(), > a direct wrapper of pm_runtime_get_sync(), assume that PM usage > counter will not change

Re: [PATCH] crypto: cavium/nitrox: add an error message to explain the failure of pci_request_mem_regions

2020-08-28 Thread Herbert Xu
On Thu, Aug 20, 2020 at 10:12:08PM -0500, George Acosta wrote: > Provide an error message for users when pci_request_mem_regions failed. > > Signed-off-by: George Acosta > --- > drivers/crypto/cavium/nitrox/nitrox_main.c | 1 + > 1 file changed, 1 insertion(+) Patch applied. Thanks. --

[PATCH v3] hwmon (pmbus/max20730): add device monitoring via debugfs

2020-08-28 Thread Ugur Usug
Add debugfs interface support for accessing device specific registers (MFR_VOUT_MIN, MFR_DEVSET1 and MFR_DEVSET2) and others including OPERATION, ON_OFF_CONFIG, SMB_ALERT_MASK, VOUT_MODE, VOUT_COMMAND and VOUT_MAX. Signed-off-by: Ugur Usug --- drivers/hwmon/pmbus/max20730.c | 365

Re: [PATCH v2 5/7][next] printk: ringbuffer: add finalization/extension support

2020-08-28 Thread Petr Mladek
On Thu 2020-08-27 12:04:58, John Ogness wrote: > On 2020-08-26, Petr Mladek wrote: > >> This series makes a very naive assumption that the previous > >> descriptor is either in the reserved or committed queried states. The > >> fact is, it can be in any of the 4 queried states. Adding support for

Re: [PATCH 4/4] regmap: sdw: add support for SoundWire 1.2 MBQ

2020-08-28 Thread Vinod Koul
On 26-08-20, 11:57, Pierre-Louis Bossart wrote: > > > > +#include > > > +#include > > > +#include > > > > Curious why do you need this header? > > I'll return the question back to you, since you added this header for > regmap-sdw.c: > > 7c22ce6e21840 (Vinod Koul 2018-01-08

Re: [PATCH 0/1] block io layer filters api

2020-08-28 Thread Damien Le Moal
On 2020/08/28 4:14, Sergei Shtepa wrote: > Hello everyone! Requesting for your comments and suggestions. > > We propose new kernel API that should be beneficial for out-of-tree > kernel modules of multiple backup vendors: block layer filter API. > > Functionality: > * Provide callback to

Re: [PATCH 05/11] soundwire: bus: update multi-link definition with hw sync details

2020-08-28 Thread Vinod Koul
On 26-08-20, 09:09, Pierre-Louis Bossart wrote: > > > > > + * @hw_sync_min_links: Number of links used by a stream above which > > > + * hardware-based synchronization is required. This value is only > > > + * meaningful if multi_link is set. If set to 1, hardware-based > > > + * synchronization

[PATCH v2] dt-bindings: gpio: renesas,rcar-gpio: Add r8a774e1 support

2020-08-28 Thread Lad Prabhakar
Document Renesas RZ/G2H (R8A774E1) GPIO blocks compatibility within the relevant dt-bindings. R8A774E1 GPIO module is identical to R-Car Gen3 family. No driver change is needed due to the fallback compatible value "renesas,rcar-gen3-gpio". Signed-off-by: Lad Prabhakar Reviewed-by: Geert

Re: [PATCH v2] null_blk: add support for max open/active zone limit for zoned devices

2020-08-28 Thread Klaus Jensen
On Aug 28 07:06, Damien Le Moal wrote: > On 2020/08/27 22:50, Niklas Cassel wrote: > > +static blk_status_t null_finish_zone(struct nullb_device *dev, struct > > blk_zone *zone) > > +{ > > + if (zone->type == BLK_ZONE_TYPE_CONVENTIONAL) > > + return BLK_STS_IOERR; > > + > > + switch

Re: [PATCH 7/7] [NOT FOR MERGE] ARM: dts: sun8i: a83t: bananapi-m3: Enable OV8865 camera

2020-08-28 Thread Kévin L'hôpital
Hello, Le Thu, 27 Aug 2020 17:38:43 +0200, Maxime Ripard a écrit : > Hi, > > On Wed, Aug 26, 2020 at 10:58:34AM +0200, Kévin L'hôpital wrote: > > > > + { > > > > + assigned-clocks = < CLK_CSI_MCLK>; > > > > + assigned-clock-parents = <>; > > > > + assigned-clock-rates =

Re: [PATCH v2 1/2] iommu/iova: Retry from last rb tree node if iova search fails

2020-08-28 Thread Vijayanand Jitta
On 8/20/2020 6:19 PM, vji...@codeaurora.org wrote: > From: Vijayanand Jitta > > When ever a new iova alloc request comes iova is always searched > from the cached node and the nodes which are previous to cached > node. So, even if there is free iova space available in the nodes > which are

Re: [PATCH 5/7] media: sunxi: sun6i-csi: Add support of MIPI CSI-2 for A83T

2020-08-28 Thread Kévin L'hôpital
Hello, Le Thu, 27 Aug 2020 17:41:19 +0200, Maxime Ripard a écrit : > On Wed, Aug 26, 2020 at 11:17:28AM +0200, Kévin L'hôpital wrote: > > > > + mdelay(10); > > > > > > Why do you need an mdelay here? > > > > yes a msleep could be more correct here. > > My question was more about

Re: [PATCH 05/14 v5] usb: typec: tcpci: Add set_vbus tcpci callback

2020-08-28 Thread Greg Kroah-Hartman
On Mon, Aug 24, 2020 at 09:22:01PM -0700, Badhri Jagan Sridharan wrote: > set_vbus callback allows TCPC which are TCPCI based, however, > does not support turning on sink and source mode through > Command.SinkVbus and Command.SourceVbusDefaultVoltage. > > Signed-off-by: Badhri Jagan Sridharan >

Re: [PATCH v2] null_blk: add support for max open/active zone limit for zoned devices

2020-08-28 Thread Damien Le Moal
On 2020/08/28 16:23, Klaus Jensen wrote: > On Aug 28 07:06, Damien Le Moal wrote: >> On 2020/08/27 22:50, Niklas Cassel wrote: >>> +static blk_status_t null_finish_zone(struct nullb_device *dev, struct >>> blk_zone *zone) >>> +{ >>> + if (zone->type == BLK_ZONE_TYPE_CONVENTIONAL) >>> +

Re: [ANNOUNCE] v5.9-rc2-rt1

2020-08-28 Thread Daniel Wagner
On Thu, Aug 27, 2020 at 05:50:24PM +0200, Daniel Vacek wrote: > Maybe the threshold is 33554432, eg. 32 megs... I've rearranged the load addresses for the kernel and the dtb and now the board is booting again. Starting with a full test run now. The size list was mainly to see if the v5.9 tree is

Re: [PATCH 2/3] opp: Allow opp-supported-hw to contain multiple versions

2020-08-28 Thread Dmitry Osipenko
26.08.2020 14:50, Viresh Kumar пишет: > The bindings allow multiple versions to be passed to "opp-supported-hw" > property, either of which can result in enabling of the OPP. > > Update code to allow that. > > Signed-off-by: Viresh Kumar > --- > drivers/opp/of.c | 47

Re: linux-next: Tree for Aug 26

2020-08-28 Thread Anders Roxell
On Wed, 26 Aug 2020 at 21:39, Paul E. McKenney wrote: > > On Wed, Aug 26, 2020 at 08:19:01PM +0200, Anders Roxell wrote: > > On Wed, 26 Aug 2020 at 08:33, Stephen Rothwell > > wrote: > > [ . . . ] > > > I've built and run an arm64 allmodconfig kernel where I use the > > defconfig as the base, I

Re: [PATCH 3/3] ARM: tegra: Pass multiple versions in opp-supported-hw property

2020-08-28 Thread Dmitry Osipenko
26.08.2020 14:50, Viresh Kumar пишет: > We can now pass multiple versions in "opp-supported-hw" property, lets > do that and simplify the tables a bit. > > Signed-off-by: Viresh Kumar > > --- > Dmitry, I think there is further scope of simplifying stuff here by > using the opp-microvolt-

RE: [Cocci] [PATCH] usb: atm: don't use snprintf() for sysfs attrs

2020-08-28 Thread David Laight
From: Kees Cook > Sent: 27 August 2020 23:21 ... > > Agreed. This just makes me cringe. If the API design declares that when > a show() callback starts, buf has been allocated with PAGE_SIZE bytes, > then that's how the logic should proceed, and it should be using > scnprintf... > > show(...) {

Re: [PATCH] usb/c67x00/c67x00-drv: Fix Data Race bug

2020-08-28 Thread Greg KH
On Wed, Aug 26, 2020 at 08:14:59PM +0530, madhuparnabhowmi...@gmail.com wrote: > From: Madhuparna Bhowmik > > Currently in c67x00_drv_probe() IRQ is requested before calling > c67x00_probe_sie() and hence if interrupt happens the reading of certain > variables in the handler can race with

Re: [PATCH] Revert "wlcore: Adding suppoprt for IGTK key in wlcore driver"

2020-08-28 Thread Kalle Valo
Steve deRosier writes: > On Tue, Aug 25, 2020 at 10:49 PM Mauro Carvalho Chehab > wrote: >> >> This patch causes a regression betwen Kernel 5.7 and 5.8 at wlcore: >> with it applied, WiFi stops working, and the Kernel starts printing >> this message every second: >> >>wlcore: PHY firmware

[PATCH] drm/bridge: Fix the dsi remote end-points

2020-08-28 Thread Vinod Koul
DSI end-points are supposed to be at node 0 and node 1 as per binding. So fix this and use node 0 and node 1 for dsi. Reported-by: Dmitry Baryshkov Fixes: 23278bf54afe ("drm/bridge: Introduce LT9611 DSI to HDMI bridge") Signed-off-by: Vinod Koul --- drivers/gpu/drm/bridge/lontium-lt9611.c | 4

[PATCH] arm64: dts: qcom: sdm845-db845c: Fix hdmi nodes

2020-08-28 Thread Vinod Koul
As per binding documentation, we should have dsi as node 0 and hdmi audio as node 1, so fix it Reported-by: Dmitry Baryshkov Fixes: aef9a119dfb9 ("arm64: dts: qcom: sdm845-db845c: Add hdmi bridge nodes") Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 12 ++--

Re: [PATCH v2 1/3] platform/x86: Add Intel Input Output Manager (IOM) driver

2020-08-28 Thread Greg Kroah-Hartman
On Fri, Aug 21, 2020 at 09:05:06PM -0700, Rajmohan Mani wrote: > Input Output Manager (IOM) is part of the Tiger Lake SoC that > configures the Type-C Sub System (TCSS). IOM is a micro controller > that handles Type-C topology, configuration and PM functions of > various Type-C devices connected

Re: [PATCH] Revert "wlcore: Adding suppoprt for IGTK key in wlcore driver"

2020-08-28 Thread Mauro Carvalho Chehab
Em Thu, 27 Aug 2020 13:36:28 -0700 Steve deRosier escreveu: > > > And let's revisit the discussion of having a kernel splat because an > > > unrelated piece of code fails yet the driver does exactly what it is > > > supposed to do. We shouldn't be dumping registers and stack-trace when > > > the

Re: [PATCH 07/11] soundwire: intel: Only call sdw stream APIs for the first cpu_dai

2020-08-28 Thread Vinod Koul
On 28-08-20, 01:47, Liao, Bard wrote: > > snd_pcm_substream *substream, > > > goto err; > > > } > > > > > > - ret = sdw_prepare_stream(dma->stream); > > > + /* > > > + * All cpu dais belong to a stream. To ensure sdw_prepare_stream > > > + * is called once per stream, we

Re: [PATCHv1 0/2] power: supply: smb347-charger: Cleanups

2020-08-28 Thread Dmitry Osipenko
26.08.2020 17:41, Sebastian Reichel пишет: > Hi Dmitry, > > Please check the following cleanup patches for smb347 driver > do not break functionality on your Nexus 7. > > -- Sebastian > > Sebastian Reichel (2): > power: supply: smb347-charger: Drop pdata support > power: supply:

Re: [PATCHv1 1/2] power: supply: smb347-charger: Drop pdata support

2020-08-28 Thread Dmitry Osipenko
26.08.2020 17:41, Sebastian Reichel пишет: > There are no platforms using the pdata support, so let's > drop it to simplify the driver. > > Signed-off-by: Sebastian Reichel > --- > drivers/power/supply/smb347-charger.c | 287 ++ > include/linux/power/smb347-charger.h |

Re: [PATCHv1 2/2] power: supply: smb347-charger: Use generic property framework

2020-08-28 Thread Dmitry Osipenko
26.08.2020 17:41, Sebastian Reichel пишет: > Simplify the driver and remove the DT specific code by > using the generic device property framework. > > Signed-off-by: Sebastian Reichel > --- > drivers/power/supply/smb347-charger.c | 40 +-- > 1 file changed, 19

Re: [PATCH v2] null_blk: add support for max open/active zone limit for zoned devices

2020-08-28 Thread Klaus Jensen
On Aug 28 07:36, Damien Le Moal wrote: > On 2020/08/28 16:23, Klaus Jensen wrote: > > On Aug 28 07:06, Damien Le Moal wrote: > >> On 2020/08/27 22:50, Niklas Cassel wrote: > >>> +static blk_status_t null_finish_zone(struct nullb_device *dev, struct > >>> blk_zone *zone) > >>> +{ > >>> + if

Re: [PATCH] USB: integrate macro definitions into include/linux/usb.h

2020-08-28 Thread Greg KH
On Tue, Aug 25, 2020 at 11:44:21PM +0800, yanfei...@windriver.com wrote: > From: Yanfei Xu > > include/linux/usb.h also contains 'Hard limit' and 'Arbitrary limit' > macro definitions in it, hence we can integrate these from config.c > into include/linux/usb.h Why? No one uses these values

[PATCH] spi: spi-cadence-quadspi: Fix mapping of buffers for DMA reads

2020-08-28 Thread Vignesh Raghavendra
Buffers need to mapped to DMA channel's device pointer instead of SPI controller's device pointer as its system DMA that actually does data transfer. Data inconsistencies have been reported when reading from flash without this fix. Fixes: 31fb632b5d43c ("spi: Move cadence-quadspi driver to

Re: [PATCH 07/11] soundwire: intel: Only call sdw stream APIs for the first cpu_dai

2020-08-28 Thread Vinod Koul
On 26-08-20, 09:35, Pierre-Louis Bossart wrote: > > > > - ret = sdw_prepare_stream(dma->stream); > > > + /* > > > + * All cpu dais belong to a stream. To ensure sdw_prepare_stream > > > + * is called once per stream, we should call it only when > > > + * dai = first_cpu_dai. > > > + */ > > >

[PATCH 0/1] Refine mtk wdt driver init flow

2020-08-28 Thread Freddy Hsin
Add mtk_wdt_init() functin in probe flow to determine enable/disable the hw watchdog by its original setting Freddy Hsin (1): driver: watchdog: Add mtk_wdt_init for mediatek watchdog hw init drivers/watchdog/mtk_wdt.c | 22 +- 1 file changed, 21 insertions(+), 1

[PATCH v1 1/1] driver: watchdog: Add mtk_wdt_init for mediatek watchdog hw init

2020-08-28 Thread Freddy Hsin
1. add a hw initialization function 2. enable/disable the watchdog depends on the original hw setting 3. set WDOD_HW_RUNNING in start function in order to start kicker after driver probe and clear the bit in stop function Change-Id: I25aa797f3b88288f26984455e499e599e27f09fa Signed-off-by:

Re: [PATCH 09/11] soundwire: intel: add dynamic debug trace for clock-stop invalid configs

2020-08-28 Thread Vinod Koul
On 26-08-20, 09:38, Pierre-Louis Bossart wrote: > > > On 8/26/20 4:48 AM, Vinod Koul wrote: > > On 18-08-20, 10:41, Bard Liao wrote: > > > From: Pierre-Louis Bossart > > > > > > Detect cases where the clock is assumed to be stopped but the IP is > > > not in the relevant state, and add a

[PATCH v2] dt-bindings: usb: renesas,usb-xhci: Document r8a774e1 support

2020-08-28 Thread Lad Prabhakar
Document r8a774e1 xhci support. The driver will use the fallback compatible string "renesas,rcar-gen3-xhci", therefore no driver change is needed. Signed-off-by: Lad Prabhakar Reviewed-by: Marian-Cristian Rotariu Acked-by: Rob Herring Reviewed-by: Geert Uytterhoeven --- Hi All, This patch is

Re: [PATCH 08/20] dt-bindings: usb: usb-xhci: Document r8a774e1 support

2020-08-28 Thread Lad, Prabhakar
Hi Shimoda-san On Fri, Aug 28, 2020 at 2:11 AM Yoshihiro Shimoda wrote: > > Hi Lad-san, > > > From: Lad, Prabhakar, Sent: Friday, August 28, 2020 2:15 AM > > > > Hi Greg, > > > > On Thu, Jul 16, 2020 at 6:19 PM Lad Prabhakar > > wrote: > > > > > > Document r8a774e1 xhci support. The driver will

[PATCH v9 1/4] scsi: ufs: Add HPB feature related parameters

2020-08-28 Thread Daejun Park
This is a patch for parameters to be used for HPB feature. Acked-by: Avri Altman Reviewed-by: Can Guo Tested-by: Bean Huo Signed-off-by: Daejun Park --- drivers/scsi/ufs/ufs.h | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/scsi/ufs/ufs.h b/drivers/scsi/ufs/ufs.h

Re: [for-next][PATCH 2/2] tracing: Use temp buffer when filtering events

2020-08-28 Thread Wen Gong
On 2016-05-04 21:52, Steven Rostedt wrote: From: "Steven Rostedt (Red Hat)" Filtering of events requires the data to be written to the ring buffer before it can be decided to filter or not. This is because the parameters of the filter are based on the result that is written to the ring buffer

  1   2   3   4   5   6   7   8   9   10   >