Re: [PATCH 4/6] PCI: keystone: Convert to using hierarchy domain for legacy interrupts

2021-03-25 Thread Krzysztof Wilczyński
Hi Kishon, Thank you for sending the series over! A few small nitpick, so feel free to ignore it. [...] > + ret = irq_domain_alloc_irqs_parent(domain, virq, 1, &parent_fwspec); > + if (ret < 0) { > + pr_err("Failed to allocate parent irq %u: %d\n", > +pare

Re: [PATCH V2] arm64: dts: qcom: sc7280: Add nodes for eMMC and SD card

2021-03-25 Thread sbhanu
On 2021-03-25 09:28, Veerabhadrarao Badiganti wrote: On 3/23/2021 9:41 PM, Doug Anderson wrote: Hi, On Sat, Mar 20, 2021 at 11:18 AM Shaik Sajida Bhanu wrote: Add nodes for eMMC and SD card on sc7280. Signed-off-by: Shaik Sajida Bhanu --- This change is depends on the below patch series: h

Re: [PATCH V3] ALSA: pcm: Fix couple of typos

2021-03-25 Thread Bhaskar Chowdhury
On 17:37 Thu 25 Mar 2021, Bhaskar Chowdhury wrote: On 11:47 Thu 25 Mar 2021, Takashi Iwai wrote: On Thu, 25 Mar 2021 10:56:39 +0100, Bhaskar Chowdhury wrote: On 10:37 Thu 25 Mar 2021, Takashi Iwai wrote: >On Thu, 25 Mar 2021 10:06:09 +0100, >Bhaskar Chowdhury wrote: >> >> s/unconditonally/unco

Re: [PATCH v2 6/7] cmdline: Gives architectures opportunity to use generically defined boot cmdline manipulation

2021-03-25 Thread Christophe Leroy
Le 03/03/2021 à 18:57, Will Deacon a écrit : On Tue, Mar 02, 2021 at 05:25:22PM +, Christophe Leroy wrote: Most architectures have similar boot command line manipulation options. This patchs adds the definition in init/Kconfig, gated by CONFIG_HAVE_CMDLINE that the architectures can selec

[PATCH V4] ALSA: pcm: Fix couple of typos

2021-03-25 Thread Bhaskar Chowdhury
s/unconditonally/unconditionally/ s/succesful/successful/ Signed-off-by: Bhaskar Chowdhury --- Changes from V3: Yet another try to make it work sound/core/pcm_native.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c

[PATCH v5 7/9] soundwire: export sdw_compare_devid, sdw_extract_slave_id and sdw_slave_add

2021-03-25 Thread Srinivas Kandagatla
Exporting these three functions makes sense as it can be used by other controllers like Qualcomm during auto-enumeration! Reported-by: kernel test robot Signed-off-by: Srinivas Kandagatla --- drivers/soundwire/bus.c | 4 +++- drivers/soundwire/slave.c | 1 + include/linux/soundwire/sd

[PATCH v9 2/3] arm64: dts: ti: k3-j7200-common-proc-board: Disable unused gpio modules

2021-03-25 Thread Aswath Govindraju
From: Faiz Abbas There are 6 gpio instances inside SoC with 2 groups as show below: Group one: wkup_gpio0, wkup_gpio1 Group two: main_gpio0, main_gpio2, main_gpio4, main_gpio6 Only one instance from each group can be used at a time. So use main_gpio0 and wkup_gpio0 in current linux conte

[PATCH v9 1/3] arm64: dts: ti: k3-j7200: Add gpio nodes

2021-03-25 Thread Aswath Govindraju
From: Faiz Abbas There are 4 instances of gpio modules in main domain: gpio0, gpio2, gpio4 and gpio6 Groups are created to provide protection between different processor virtual worlds. Each of these modules I/O pins are muxed within the group. Exactly one module can be selected to contr

[PATCH v5 4/9] soundwire: qcom: start the clock during initialization

2021-03-25 Thread Srinivas Kandagatla
Start the clock during initialization, doing this explicitly will add more clarity when we are adding clock stop feature. Signed-off-by: Srinivas Kandagatla Reviewed-by: Pierre-Louis Bossart --- drivers/soundwire/qcom.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/soundwire/qc

[PATCH v5 9/9] soundwire: qcom: wait for enumeration to be complete in probe

2021-03-25 Thread Srinivas Kandagatla
Signed-off-by: Srinivas Kandagatla Reviewed-by: Pierre-Louis Bossart --- drivers/soundwire/qcom.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/soundwire/qcom.c b/drivers/soundwire/qcom.c index c6c923329b15..706d44200a36 100644 --- a/drivers/soundwire/qcom.c +++ b/drivers/soun

[PATCH v5 6/9] soundwire: qcom: add support to new interrupts

2021-03-25 Thread Srinivas Kandagatla
Add support to new interrupts which includes reporting some of the error interrupts and adding support to SLAVE pending interrupt! This patch also changes the interrupt handler behaviour on handling any pending interrupts by checking it before returning out of irq handler. Signed-off-by: Srinivas

[PATCH v5 8/9] soundwire: qcom: add auto enumeration support

2021-03-25 Thread Srinivas Kandagatla
Qualcomm SoundWire controller supports Auto Enumeration of the devices within the IP. This patch enables support for this feature. Signed-off-by: Srinivas Kandagatla Reviewed-by: Pierre-Louis Bossart --- drivers/soundwire/qcom.c | 86 +--- 1 file changed, 81

[PATCH v9 3/3] arm64: dts: ti: k3-j7200: Add support for higher speed modes and update delay select values for MMCSD subsystems

2021-03-25 Thread Aswath Govindraju
The following speed modes are now supported in J7200 SoC, - HS200 and HS400 modes at 1.8 V card voltage, in MMCSD0 subsystem [1]. - UHS-I speed modes in MMCSD1 subsystem [1]. Add support for UHS-I modes by adding voltage regulator device tree nodes and corresponding pinmux details, to power cycle

[PATCH v9 0/3] J7200: Add support for GPIO and higher speed modes in MMCSD subsystems

2021-03-25 Thread Aswath Govindraju
The following series of patches - Add support for GPIO subsystem in main and wakeup domains. - Add voltage regulator device tree nodes and their corresponding pinmux to support power cycle and voltage switch required for UHS-I modes - sets respective tags in sdhci0 node to support higher speeds -

[PATCH v5 5/9] soundwire: qcom: update register read/write routine

2021-03-25 Thread Srinivas Kandagatla
In the existing code every soundwire register read and register write are kinda blocked. Each of these are using a special command id that generates interrupt after it successfully finishes. This is really overhead, limiting and not really necessary unless we are doing something special. We can si

[PATCH v5 3/9] soundwire: qcom: set continue execution flag for ignored commands

2021-03-25 Thread Srinivas Kandagatla
version 1.5.1 and higher IPs of this controller required to set continue execution on ignored command flag. This patch sets this flag. Signed-off-by: Srinivas Kandagatla Reviewed-by: Pierre-Louis Bossart --- drivers/soundwire/qcom.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion

[PATCH v5 2/9] soundwire: qcom: add support to missing transport params

2021-03-25 Thread Srinivas Kandagatla
Some of the transport parameters derived from device tree are not fully parsed by the driver. This patch adds support to parse those missing parameters. Signed-off-by: Srinivas Kandagatla Reviewed-by: Pierre-Louis Bossart --- drivers/soundwire/qcom.c | 107 ++---

[PATCH v5 0/9] soundwire: qcom: various improvements

2021-03-25 Thread Srinivas Kandagatla
Thanks for reviewing v4 of this patchset! During testing SoundWire controller on SM8250 MTP, we found few issues like all the interrupts are not handled, all transport parameters are not read from device tree. Patch to add Auto Enumeration supported by the controller is also part of this series.

[PATCH v5 1/9] dt-bindings: soundwire: qcom: clarify data port bus parameters

2021-03-25 Thread Srinivas Kandagatla
Some of the parameters for data ports are not applicable or not implemented in IP. So mark them as invalid/not applicable in DT so that controller is aware of this. Add comment to these bindings to provide more clarity on the values! Signed-off-by: Srinivas Kandagatla Acked-by: Rob Herring Revi

Re: [PATCH] PCI: ACPI: PM: Fix debug message in acpi_pci_set_power_state()

2021-03-25 Thread Krzysztof Wilczyński
Hi, [...] > To address this issue, modify the debug message in question to print > the current power state of the target PCI device's ACPI companion > instead of printing the target power state which may not reflect > the real final power state of the device. [...] Thank you! Reviewed-by: Krzysz

Re: [PATCH 4/4] PCI/sysfs: Allow userspace to query and set device reset mechanism

2021-03-25 Thread Leon Romanovsky
On Thu, Mar 25, 2021 at 11:53:24AM -0600, Alex Williamson wrote: > On Thu, 25 Mar 2021 18:09:58 +0200 > Leon Romanovsky wrote: > > > On Thu, Mar 25, 2021 at 08:55:04AM -0600, Alex Williamson wrote: > > > On Thu, 25 Mar 2021 10:37:54 +0200 > > > Leon Romanovsky wrote: > > > > > > > On Wed, Mar

[PATCH -next] drivers/soc/litex: remove duplicated include from litex_soc_ctrl.c

2021-03-25 Thread Zheng Yongjun
Remove duplicated include. Reported-by: Hulk Robot Signed-off-by: Zheng Yongjun --- drivers/soc/litex/litex_soc_ctrl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/soc/litex/litex_soc_ctrl.c b/drivers/soc/litex/litex_soc_ctrl.c index 6268bfa7f0d6..c3e379a990f2 100644 --- a/driver

[PATCH -next] bus: bt1-apb: Remove duplicated include from bt1-apb.c

2021-03-25 Thread Zheng Yongjun
Remove duplicated include. Reported-by: Hulk Robot Signed-off-by: Zheng Yongjun --- drivers/bus/bt1-apb.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/bus/bt1-apb.c b/drivers/bus/bt1-apb.c index b25ff941e7c7..74b1b712ef3a 100644 --- a/drivers/bus/bt1-apb.c +++ b/drivers/bus/bt1-ap

[PATCH -next] ARM: sa11x0: remove duplicated include from hackkit.c

2021-03-25 Thread Zheng Yongjun
Remove duplicated include. Reported-by: Hulk Robot Signed-off-by: Zheng Yongjun --- arch/arm/mach-sa1100/hackkit.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-sa1100/hackkit.c b/arch/arm/mach-sa1100/hackkit.c index 3085f1c2e586..3fe34ee7c0ab 100644 --- a/arch/arm/mach-sa110

[PATCH -next] drivers/soc/litex: remove duplicated include from litex_soc_ctrl.c

2021-03-25 Thread Zheng Yongjun
Remove duplicated include. Reported-by: Hulk Robot Signed-off-by: Zheng Yongjun --- drivers/soc/litex/litex_soc_ctrl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/soc/litex/litex_soc_ctrl.c b/drivers/soc/litex/litex_soc_ctrl.c index 6268bfa7f0d6..c3e379a990f2 100644 --- a/driver

[PATCH v2] media: venus : hfi: add venus image info into smem

2021-03-25 Thread Dikshita Agarwal
Fill fw version info into smem to be printed as part of soc info. Signed-off-by: Dikshita Agarwal Changes since v1: adressed comments from stephen. removed unwanted code. --- drivers/media/platform/qcom/venus/hfi_msgs.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions

Re: [PATCH] livepatch: klp_send_signal should treat PF_IO_WORKER like PF_KTHREAD

2021-03-25 Thread dongkai (H)
On 2021/3/25 17:26, Miroslav Benes wrote: On Thu, 25 Mar 2021, Dong Kai wrote: commit 15b2219facad ("kernel: freezer should treat PF_IO_WORKER like PF_KTHREAD for freezing") is to fix the freezeing issue of IO threads by making the freezer not send them fake signals. Here live patching consist

Re: [PATCH v2 6/8] block: keyslot-manager: introduce blk_ksm_restrict_dus_to_queue_limits()

2021-03-25 Thread kernel test robot
Hi Satya, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on block/for-next] [also build test WARNING on dm/for-next mkp-scsi/for-next scsi/for-next linux/master linus/master v5.12-rc4 next-20210325] [If your patch is applied to the wrong git tree, kindly drop us

[PATCH 8/8] iommu/vt-d: fix a couple of spelling mistakes

2021-03-25 Thread Zhen Lei
There are several spelling mistakes, as follows: guarentees ==> guarantees resgister ==> register insufficent ==> insufficient creats ==> creates tabke ==> take Signed-off-by: Zhen Lei --- drivers/iommu/intel/dmar.c | 6 +++--- drivers/iommu/intel/iommu.c | 2 +- drivers/iommu/i

[PATCH 2/8] iommu/omap: Fix spelling mistake "alignement" -> "alignment"

2021-03-25 Thread Zhen Lei
There is a spelling mistake in a comment, fix it. Signed-off-by: Zhen Lei --- drivers/iommu/omap-iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iommu/omap-iommu.c b/drivers/iommu/omap-iommu.c index 71f29c0927fc710..b2a6ab700ec43d1 100644 --- a/drivers/iommu/o

[PATCH 6/8] iommu/amd: fix a couple of spelling mistakes

2021-03-25 Thread Zhen Lei
There are several spelling mistakes, as follows: alignement ==> alignment programing ==> programming implemtation ==> implementation assignement ==> assignment By the way, both "programing" and "programming" are acceptable, but the latter seems more formal. Signed-off-by: Zhen Lei --- drivers/i

[PATCH 3/8] iommu/mediatek: Fix spelling mistake "phyiscal" -> "physical"

2021-03-25 Thread Zhen Lei
There is a spelling mistake in a comment, fix it. Signed-off-by: Zhen Lei --- drivers/iommu/mtk_iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iommu/mtk_iommu.c b/drivers/iommu/mtk_iommu.c index 6ecc007f07cd52e..c8c9bf1d70b29dc 100644 --- a/drivers/iommu/mtk_

[PATCH 5/8] iommu: fix a couple of spelling mistakes

2021-03-25 Thread Zhen Lei
There are several spelling mistakes, as follows: funcions ==> functions distiguish ==> distinguish detroyed ==> destroyed Signed-off-by: Zhen Lei --- drivers/iommu/iommu.c | 4 ++-- drivers/iommu/iova.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/iommu/iommu.c

[PATCH 7/8] iommu/arm-smmu: Fix spelling mistake "initally" -> "initially"

2021-03-25 Thread Zhen Lei
There is a spelling mistake in a comment, fix it. Signed-off-by: Zhen Lei --- drivers/iommu/arm/arm-smmu/arm-smmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu.c b/drivers/iommu/arm/arm-smmu/arm-smmu.c index d8c6bfde6a61587..8e4e8fea10

[PATCH 4/8] iommu/sun50i: Fix spelling mistake "consits" -> "consists"

2021-03-25 Thread Zhen Lei
There is a spelling mistake in a comment, fix it. Signed-off-by: Zhen Lei --- drivers/iommu/sun50i-iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iommu/sun50i-iommu.c b/drivers/iommu/sun50i-iommu.c index ea6db1341916524..7685b96b2d445a7 100644 --- a/drivers/i

[PATCH 0/8] iommu: fix a couple of spelling mistakes detected by codespell tool

2021-03-25 Thread Zhen Lei
This detection and correction covers the entire driver/iommu directory. Zhen Lei (8): iommu/pamu: fix a couple of spelling mistakes iommu/omap: Fix spelling mistake "alignement" -> "alignment" iommu/mediatek: Fix spelling mistake "phyiscal" -> "physical" iommu/sun50i: Fix spelling mistake

[PATCH 1/8] iommu/pamu: fix a couple of spelling mistakes

2021-03-25 Thread Zhen Lei
There are several spelling mistakes, as follows: Returs ==> Returns defaul ==> default assocaited ==> associated Signed-off-by: Zhen Lei --- drivers/iommu/fsl_pamu.c| 2 +- drivers/iommu/fsl_pamu_domain.c | 2 +- drivers/iommu/fsl_pamu_domain.h | 2 +- 3 files changed, 3 insertions(+),

Re: [PATCH] clk: bcm: rpi: Don't register as OF provider if !dev->np

2021-03-25 Thread Marek Szyprowski
On 25.03.2021 19:57, Nicolas Saenz Julienne wrote: > There are two ways clk-raspberrypi might be registered: through > device-tree or through an explicit platform device registration. The > latter happens after firmware/raspberrypi's probe, and it's limited to > RPi3s, which solely use the ARM cloc

Re: [RFT PATCH v3 13/27] arm64: Add Apple vendor-specific system registers

2021-03-25 Thread Hector Martin
On 25/03/2021 04.04, Will Deacon wrote: On Wed, Mar 24, 2021 at 06:59:21PM +, Mark Rutland wrote: So far we've kept arch/arm64/ largely devoid of IMP-DEF bits, and it seems a shame to add something with the sole purpose of collating that, especially given arch code shouldn't need to touch th

Re: [Ksummit-discuss] [1/5] reporting-issues: header and TLDR

2021-03-25 Thread Guenter Roeck
On 3/25/21 11:15 PM, Thorsten Leemhuis wrote: > On 26.03.21 07:13, Thorsten Leemhuis wrote: >> > mention if backporting is planed or considered too complex. If backporting was planned

Re: [RFC] mm: activate access-more-than-once page via NUMA balancing

2021-03-25 Thread Huang, Ying
Mel Gorman writes: > On Thu, Mar 25, 2021 at 12:33:45PM +0800, Huang, Ying wrote: >> > I caution against this patch. >> > >> > It's non-deterministic for a number of reasons. As it requires NUMA >> > balancing to be enabled, the pageout behaviour of a system changes when >> > NUMA balancing is ac

[5/5] reporting-issues: addendum

2021-03-25 Thread Thorsten Leemhuis
On 26.03.21 07:13, Thorsten Leemhuis wrote: > > Lo! Since a few months mainline in > Documentation/admin-guide/reporting-issues.rst contains a text written > to obsolete the good old reporting-bugs text. For now, the new document > still contains a warning at the top that basically says "this is W

[4/5] reporting-issues: reference section, stable and longterm sub-processes

2021-03-25 Thread Thorsten Leemhuis
On 26.03.21 07:13, Thorsten Leemhuis wrote: > Lo! Since a few months mainline in > Documentation/admin-guide/reporting-issues.rst contains a text written > to obsolete the good old reporting-bugs text. For now, the new document > still contains a warning at the top that basically says "this is WIP"

[PATCH] pkcs7: Use octal permissions '0444'

2021-03-25 Thread Meng Yu
Fixed following checkpatch warning: Symbolic permissions 'S_IWUSR | S_IRUGO' are not preferred. Consider using octal permissions '0644'. Signed-off-by: Meng Yu --- crypto/asymmetric_keys/pkcs7_key_type.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crypto/asymmetric_keys/p

Re: [PATCH v2 6/7] cmdline: Gives architectures opportunity to use generically defined boot cmdline manipulation

2021-03-25 Thread Christophe Leroy
Le 25/03/2021 à 20:32, Will Deacon a écrit : On Thu, Mar 25, 2021 at 12:18:38PM +0100, Christophe Leroy wrote: - For ARM it means to append the bootloader arguments to the CONFIG_CMDLINE - For Powerpc it means to append the CONFIG_CMDLINE to the bootloader arguments - For SH it means to app

[3/5] reporting-issues: reference section, main guide

2021-03-25 Thread Thorsten Leemhuis
On 26.03.21 07:13, Thorsten Leemhuis wrote: > > Lo! Since a few months mainline in > Documentation/admin-guide/reporting-issues.rst contains a text written > to obsolete the good old reporting-bugs text. For now, the new document > still contains a warning at the top that basically says "this is W

[tip:irq/core] BUILD SUCCESS 6e457914935a3161eeb74e319abf9fd511aa1e4d

2021-03-25 Thread kernel test robot
allyesconfig powerpc allmodconfig powerpc allnoconfig x86_64 randconfig-a002-20210325 x86_64 randconfig-a003-20210325 x86_64 randconfig-a006-20210325 x86_64 randconfig-a001

[2/5] reporting-issues: step-by-step-guide: main and two sub-processes for stable/longterm

2021-03-25 Thread Thorsten Leemhuis
On 26.03.21 07:13, Thorsten Leemhuis wrote: > Lo! Since a few months mainline in > Documentation/admin-guide/reporting-issues.rst contains a text written > to obsolete the good old reporting-bugs text. For now, the new document > still contains a warning at the top that basically says "this is WIP"

[PATCH] crypto: hisilicon/hpre - rsa key should not be empty

2021-03-25 Thread Meng Yu
We should ensure key is not empty before we set key. Signed-off-by: Meng Yu --- drivers/crypto/hisilicon/hpre/hpre_crypto.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/crypto/hisilicon/hpre/hpre_crypto.c b/drivers/crypto/hisilicon/hpre/hpre_crypto.c index 53068d2..7cf7d80

[1/5] reporting-issues: header and TLDR

2021-03-25 Thread Thorsten Leemhuis
On 26.03.21 07:13, Thorsten Leemhuis wrote: > > Lo! Since a few months mainline in > Documentation/admin-guide/reporting-issues.rst contains a text written > to obsolete the good old reporting-bugs text. For now, the new document > still contains a warning at the top that basically says "this is W

[PATCH 2/2] powerpc/powernv: remove the nvlink support

2021-03-25 Thread Christoph Hellwig
This code was only used by the vfio-nvlink2 code, which itself had no proper use. Drop this huge chunk of code build into every powernv or generic build. Signed-off-by: Christoph Hellwig Acked-by: Greg Kroah-Hartman --- arch/powerpc/include/asm/opal.h| 3 - arch/powerpc/include/a

[PATCH 1/2] vfio/pci: remove vfio_pci_nvlink2

2021-03-25 Thread Christoph Hellwig
This driver never had any open userspace (which for VFIO would include VM kernel drivers) that use it, and thus should never have been added by our normal userspace ABI rules. Signed-off-by: Christoph Hellwig Acked-by: Greg Kroah-Hartman --- drivers/vfio/pci/Kconfig| 6 - drivers/

[PATCH v3 2/2] leds: rt4505: Add support for Richtek RT4505 flash LED controller

2021-03-25 Thread cy_huang
From: ChiYuan Huang Add support for RT4505 flash LED controller. It can support up to 1.5A flash current with hardware timeout and low input voltage protection. Signed-off-by: ChiYuan Huang Acked-by: Jacek Anaszewski --- drivers/leds/flash/Kconfig | 11 + drivers/leds/flash/Makefile

remove the nvlink2 pci_vfio subdriver v2

2021-03-25 Thread Christoph Hellwig
Hi all, the nvlink2 vfio subdriver is a weird beast. It supports a hardware feature without any open source component - what would normally be the normal open source userspace that we require for kernel drivers, although in this particular case user space could of course be a kernel driver in a V

[PATCH v3 1/2] leds: rt4505: Add DT binding document for Richtek RT4505

2021-03-25 Thread cy_huang
From: ChiYuan Huang Add DT binding document for Richtek RT4505 flash LED controller. Signed-off-by: ChiYuan Huang Reviewed-by: Rob Herring --- Changes since v3 - Port this patch to led for-next tree. - Merge Acks in the commit context. - Reorder the patch series from the docs first. Changes s

FYI & RFC: obsoleting reporting-bugs and making reporting-issues official

2021-03-25 Thread Thorsten Leemhuis
Lo! Since a few months mainline in Documentation/admin-guide/reporting-issues.rst contains a text written to obsolete the good old reporting-bugs text. For now, the new document still contains a warning at the top that basically says "this is WIP". But I'd like to remove that warning and delete r

Re: [PATCH] powerpc/iommu/debug: Remove redundant NULL check

2021-03-25 Thread Daniel Axtens
Daniel Axtens writes: It looks like the kernel test robot also reported this: "[PATCH] powerpc/iommu/debug: fix ifnullfree.cocci warnings" Weirdly I don't see it in patchwork. I'm not sure which one mpe will want to take but either would do. >> Fix the following coccicheck warnings: >> >> ./fs/

[PATCH] __scsi_remove_device: fix comments minor error

2021-03-25 Thread dudengke
From: dudengke Signed-off-by: dudengke --- drivers/scsi/scsi_sysfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/scsi_sysfs.c b/drivers/scsi/scsi_sysfs.c index b6378c8ca783..86e8d0bf821d 100644 --- a/drivers/scsi/scsi_sysfs.c +++ b/drivers/scsi/scsi_sysfs.c

Re: [PATCH v2 2/2] leds: leds-multi-gpio: Add multiple GPIOs LED driver

2021-03-25 Thread Hermes Zhang
On 3/26/21 1:56 PM, Alexander Dahl wrote: >> + >> +module_platform_driver(multi_gpio_led_driver); >> + >> +MODULE_AUTHOR("Hermes Zhang "); >> +MODULE_DESCRIPTION("Multiple GPIOs LED driver"); >> +MODULE_LICENSE("GPL v2"); >> +MODULE_ALIAS("platform:leds-multi-gpio"); > I did not review thouroughly,

[PATCH] kconfig: streamline_config.pl: Couple of typo fixes

2021-03-25 Thread Bhaskar Chowdhury
s/configuraton/configuration/ s/orignal/original/ Signed-off-by: Bhaskar Chowdhury --- scripts/kconfig/streamline_config.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/kconfig/streamline_config.pl b/scripts/kconfig/streamline_config.pl index 1c78ba49ca99..9

Re: [PATCH] crypto: vmx: fix incorrect kernel-doc comment syntax in files

2021-03-25 Thread Daniel Axtens
Hi Aditya, Thanks for your patch! > The opening comment mark '/**' is used for highlighting the beginning of > kernel-doc comments. > There are certain files in drivers/crypto/vmx, which follow this syntax, > but the content inside does not comply with kernel-doc. > Such lines were probably not m

Re: [PATCH v8 3/3] arm64: dts: ti: k3-j7200: Add support for higher speed modes and update delay select values for MMCSD subsystems

2021-03-25 Thread Aswath Govindraju
Hi Kishon, On 25/03/21 9:44 pm, Kishon Vijay Abraham I wrote: > Hi Aswath, > > On 24/03/21 12:07 pm, Aswath Govindraju wrote: >> The following speed modes are now supported in J7200 SoC, >> - HS200 and HS400 modes at 1.8 V card voltage, in MMCSD0 subsystem [1]. >> - UHS-I speed modes in MMCSD1 su

[PATCH -next] ASoC: cs42l42: Remove unused including

2021-03-25 Thread Zheng Yongjun
Remove including that don't need it. Reported-by: Hulk Robot Signed-off-by: Zheng Yongjun --- sound/soc/codecs/cs42l42.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/cs42l42.c b/sound/soc/codecs/cs42l42.c index bf982e145e94..bd043f5d5d90 100644 --- a/sound/soc/codecs/cs42

[PATCH -next] ASoC: cs35l35: Remove unused including

2021-03-25 Thread Zheng Yongjun
Remove including that don't need it. Reported-by: Hulk Robot Signed-off-by: Zheng Yongjun --- sound/soc/codecs/cs35l35.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/cs35l35.c b/sound/soc/codecs/cs35l35.c index 55d529aa0011..5d361c74e803 100644 --- a/sound/soc/codecs/cs35

Re: [PATCH v2 2/2] leds: leds-multi-gpio: Add multiple GPIOs LED driver

2021-03-25 Thread Alexander Dahl
Hello Hermes, great to see your improved series. See below for my remarks. On Fri, Mar 26, 2021 at 01:28:01PM +0800, Hermes Zhang wrote: > From: Hermes Zhang > > Introduce a new multiple GPIOs LED driver. This LED will made of > multiple GPIOs (up to 8) and will map different brightness to diff

[PATCH] scripts: modpost.c: Fix a few typos

2021-03-25 Thread Bhaskar Chowdhury
s/agorithm/algorithm/ s/criterias/criteria/ s/targetting/targeting/ two different places. Signed-off-by: Bhaskar Chowdhury --- scripts/mod/modpost.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 24725e50c7b

Re: [PATCH] powerpc/iommu/debug: Remove redundant NULL check

2021-03-25 Thread Daniel Axtens
Hi Jiapeng Chong, writes: > Fix the following coccicheck warnings: > > ./fs/io_uring.c:5989:4-9: WARNING: NULL check before some freeing > functions is not needed. This looks correct to me, and matches the description of debugfs_remove in Documentation/filesystems/debugfs.rst. If you have a num

[PATCH] drivers: staging: netlogic: fix unmet dependency for PHYLIB

2021-03-25 Thread Julian Braha
When NETLOGIC_XLR_NET is enabled, and NETDEVICES is disabled, Kbuild gives the following warning: WARNING: unmet direct dependencies detected for PHYLIB Depends on [n]: NETDEVICES [=n] Selected by [y]: - NETLOGIC_XLR_NET [=y] && STAGING [=y] && CPU_XLR [=y] This is because NETLOGIC_XLR_NET

Re: [PATCH] [v2] tools: testing: Remove duplicate includes

2021-03-25 Thread Daniel Axtens
Wan Jiabing writes: > sched.h has been included at line 33, so remove the > duplicate one at line 36. > pthread.h has been included at line 17,so remove the > duplicate one at line 20. I can see that both of these are correct from the diff. > inttypes.h has been included at line 19, so remov

Re: Linux 4.14.227

2021-03-25 Thread Samuel Zou
On 2021/3/24 18:23, Greg Kroah-Hartman wrote: I'm announcing the release of the 4.14.227 kernel. All users of the 4.14 kernel series must upgrade. The updated 4.14.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.14.y and can

Re: [PATCH] [v2] arch: powerpc: Remove duplicate includes

2021-03-25 Thread Daniel Axtens
Wan Jiabing writes: > mmu-hash.h: asm/bug.h has been included at line 12, so remove > the duplicate one at line 21. Looking at the file I had wondered if this was due to a #ifdef being removed, but no, the second one was just added in commit 891121e6c02c ("powerpc/mm: Differentiate between huge

Re: kinldy see the V2 ..just sent [PATCH] fddi: skfp: Rudimentary spello fixes

2021-03-25 Thread Bhaskar Chowdhury
On 17:09 Thu 25 Mar 2021, David Miller wrote: From: Bhaskar Chowdhury Date: Thu, 25 Mar 2021 12:38:35 +0530 s/autohorized/authorized/ s/recsource/resource/ s/measuered/measured/ sauthoriziation/authorization/ Signed-off-by: Bhaskar Chowdhury Does not apply cleanly to net-next please respi

[PATCH V2] fddi: skfp: Rudimentary spello fixes

2021-03-25 Thread Bhaskar Chowdhury
s/autohorized/authorized/ s/recsource/resource/ s/measuered/measured/ s/authoriziation/authorization/ Signed-off-by: Bhaskar Chowdhury --- Changes from V1: David said the patch didn't applied cleanly,so recreate it. Randy found out a glitch in changelog text, fixed it. drivers/net/fddi/skf

[PATCH] arch: mips: fix unmet dependency for MTD_COMPLEX_MAPPINGS

2021-03-25 Thread Julian Braha
When CAVIUM_OCTEON_SOC is enabled, and MTD is disabled, Kbuild gives the following warning: WARNING: unmet direct dependencies detected for MTD_COMPLEX_MAPPINGS Depends on [n]: MTD [=n] && HAS_IOMEM [=y] Selected by [y]: - CAVIUM_OCTEON_SOC [=y] && This is because CAVIUM_OCTEON_SOC selects

[PATCH v2 2/2] leds: leds-multi-gpio: Add multiple GPIOs LED driver

2021-03-25 Thread Hermes Zhang
From: Hermes Zhang Introduce a new multiple GPIOs LED driver. This LED will made of multiple GPIOs (up to 8) and will map different brightness to different GPIOs states which defined in dts file. Signed-off-by: Hermes Zhang --- drivers/leds/Kconfig | 3 + drivers/leds/Makefi

[PATCH v2 1/2] dt-binding: leds: Document leds-multi-gpio bindings

2021-03-25 Thread Hermes Zhang
From: Hermes Zhang This binding represents LED devices which are controller with multiple GPIO lines in order to achieve more than two brightness states. Signed-off-by: Hermes Zhang --- .../bindings/leds/leds-multi-gpio.yaml| 50 +++ 1 file changed, 50 insertions(+) cr

[PATCH v2 0/2] New multiple GPIOs LED driver

2021-03-25 Thread Hermes Zhang
From: Hermes Zhang changes v2: - use max_brightness(=2^gpio number) instead of LED_FULL - alloc priv at once - move driver code to new simple folder - update commit message for dt-binding commit - move dt-binding commit before driver code Hermes Zhang (2): dt-binding: leds: Document leds-multi

Re: [PATCH 2/2] dt-bindings: cpufreq: update cpu type and clock name for MT8173 SoC

2021-03-25 Thread Viresh Kumar
On 26-03-21, 11:12, Seiya Wang wrote: > Update the cpu type of cpu2 and cpu3 since MT8173 used Cortex-a72. > > Signed-off-by: Seiya Wang > --- > Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/Docume

[PATCH] arch: mips: fix unmet dependency for DEBUG_INFO

2021-03-25 Thread Julian Braha
When SB1XXX_CORELIS is enabled, COMPILE_TEST is disabled, and DEBUG_KERNEL is disabled, Kbuild gives the following warning: WARNING: unmet direct dependencies detected for DEBUG_INFO Depends on [n]: DEBUG_KERNEL [=n] && !COMPILE_TEST [=n] Selected by [y]: - SB1XXX_CORELIS [=y] && SIBYTE_SB1x

Re: [PATCH v2] kernel/resource: Fix locking in request_free_mem_region

2021-03-25 Thread Balbir Singh
On Fri, Mar 26, 2021 at 12:20:35PM +1100, Alistair Popple wrote: > request_free_mem_region() is used to find an empty range of physical > addresses for hotplugging ZONE_DEVICE memory. It does this by iterating > over the range of possible addresses using region_intersects() to see if > the range is

Re: [PATCH -next] powerpc/eeh: Remove unused inline function eeh_dev_phb_init_dynamic()

2021-03-25 Thread Daniel Axtens
Hi, > commit 475028efc708 ("powerpc/eeh: Remove eeh_dev_phb_init_dynamic()") > left behind this, so can remove it. I had a look: the inline that you are removing here is for the !CONFIG_EEH case, which explains why it was missed the first time. This looks like a good change. Out of interest, wha

Re: [PATCH -next] powerpc/smp: Remove unused inline functions

2021-03-25 Thread Daniel Axtens
Hi, > commit 441c19c8a290 ("powerpc/kvm/book3s_hv: Rework the secondary inhibit > code") > left behind this, so can remove it. > Interesting: that commit removed some instances of (un)inhibit_secondary_onlining, but it seems to have missed the ones for the uni-processor case, which your patch re

Re: [PATCH v7 5/6] x86/signal: Detect and prevent an alternate signal stack overflow

2021-03-25 Thread Andy Lutomirski
I forgot to mention why I cc'd all you fine Xen folk: On Thu, Mar 25, 2021 at 11:13 AM Andy Lutomirski wrote: > > > } else if (IS_ENABLED(CONFIG_X86_32) && > >!onsigstack && > >regs->ss != __USER_DS && This bit here seems really dubious on Xen PV.

Re: [PATCH v7 5/6] x86/signal: Detect and prevent an alternate signal stack overflow

2021-03-25 Thread Andy Lutomirski
On Thu, Mar 25, 2021 at 11:54 AM Borislav Petkov wrote: > > On Thu, Mar 25, 2021 at 11:13:12AM -0700, Andy Lutomirski wrote: > > diff --git a/arch/x86/kernel/signal.c b/arch/x86/kernel/signal.c > > index ea794a083c44..53781324a2d3 100644 > > --- a/arch/x86/kernel/signal.c > > +++ b/arch/x86/kernel

[rcu:master] BUILD SUCCESS dd44ee94db05b80fef1469155c374caad58d8a2e

2021-03-25 Thread kernel test robot
ig powerpc allyesconfig powerpc allmodconfig powerpc allnoconfig x86_64 randconfig-a002-20210325 x86_64 randconfig-a003-20210325 x86_64 randconfig-a006-20210325 x86_64 randc

[rcu:rcu/next] BUILD SUCCESS 1a0dfc099c1e61e22045705265a1323ac294bea6

2021-03-25 Thread kernel test robot
allmodconfig powerpc allnoconfig x86_64 randconfig-a002-20210325 x86_64 randconfig-a003-20210325 x86_64 randconfig-a006-20210325 x86_64 randconfig-a001-20210325 x86_64 randconfig-a005

Re: [PATCH] selftests: powerpc: unmark non-kernel-doc comments

2021-03-25 Thread Daniel Axtens
Randy Dunlap writes: > Drop the 'beginning of kernel-doc' notation markers (/**) > in places that are not in kernel-doc format. This looks good to me. Arguably we don't need the comments at all, but it doesn't seem to hurt to keep them. checkpatch is OK with the entire file, so there's nothing

Re: [PATCH bpf] bpf: link: refuse non-zero file_flags in BPF_OBJ_GET

2021-03-25 Thread Andrii Nakryiko
On Thu, Mar 25, 2021 at 8:22 AM Lorenz Bauer wrote: > > Invoking BPF_OBJ_GET on a pinned bpf_link checks the path access > permissions based on file_flags, but the returned fd ignores flags. > This means that any user can acquire a "read-write" fd for a pinned > link with mode 0664 by invoking BPF

Why does glibc use AVX-512?

2021-03-25 Thread Andy Lutomirski
Hi all- glibc appears to use AVX512F for memcpy by default. (Unless Prefer_ERMS is default-on, but I genuinely can't tell if this is the case. I did some searching.) The commit adding it refers to a 2016 email saying that it's 30% on KNL. Unfortunately, AVX-512 is now available in normal hardw

Re: [PATCH v3 09/17] treewide: Change list_sort to use const pointers

2021-03-25 Thread Kees Cook
On Tue, Mar 23, 2021 at 01:39:38PM -0700, Sami Tolvanen wrote: > list_sort() internally casts the comparison function passed to it > to a different type with constant struct list_head pointers, and > uses this pointer to call the functions, which trips indirect call > Control-Flow Integrity (CFI) c

Re: [PATCH v3 05/17] workqueue: use WARN_ON_FUNCTION_MISMATCH

2021-03-25 Thread Kees Cook
On Tue, Mar 23, 2021 at 01:39:34PM -0700, Sami Tolvanen wrote: > With CONFIG_CFI_CLANG, a callback function passed to > __queue_delayed_work from a module points to a jump table entry > defined in the module instead of the one used in the core kernel, > which breaks function address equality in thi

Re: [PATCH v3 06/17] kthread: use WARN_ON_FUNCTION_MISMATCH

2021-03-25 Thread Kees Cook
On Tue, Mar 23, 2021 at 01:39:35PM -0700, Sami Tolvanen wrote: > With CONFIG_CFI_CLANG, a callback function passed to > __kthread_queue_delayed_work from a module points to a jump table > entry defined in the module instead of the one used in the core > kernel, which breaks function address equalit

Re: [PATCH v3 04/17] module: ensure __cfi_check alignment

2021-03-25 Thread Kees Cook
On Tue, Mar 23, 2021 at 01:39:33PM -0700, Sami Tolvanen wrote: > CONFIG_CFI_CLANG_SHADOW assumes the __cfi_check() function is page > aligned and at the beginning of the .text section. While Clang would > normally align the function correctly, it fails to do so for modules > with no executable code

Re: [PATCH bpf-next] libbpf: fix bail out from 'ringbuf_process_ring()' on error

2021-03-25 Thread Andrii Nakryiko
On Thu, Mar 25, 2021 at 8:02 AM Pedro Tammela wrote: > > The current code bails out with negative and positive returns. > If the callback returns a positive return code, 'ring_buffer__consume()' > and 'ring_buffer__poll()' will return a spurious number of records > consumed, but mostly important w

Re: [PATCH v31 10/12] selftests/landlock: Add user space tests

2021-03-25 Thread Kees Cook
On Wed, Mar 24, 2021 at 08:15:18PM +0100, Mickaël Salaün wrote: > From: Mickaël Salaün > > Test all Landlock system calls, ptrace hooks semantic and filesystem > access-control with multiple layouts. > > Test coverage for security/landlock/ is 93.6% of lines. The code not > covered only deals w

Re: [PATCH v31 12/12] landlock: Add user and kernel documentation

2021-03-25 Thread Kees Cook
On Wed, Mar 24, 2021 at 08:15:20PM +0100, Mickaël Salaün wrote: > From: Mickaël Salaün > > Add a first document describing userspace API: how to define and enforce > a Landlock security policy. This is explained with a simple example. > The Landlock system calls are described with their expected

Re: [PATCH][next] scsi: aacraid: Replace one-element array with flexible-array member

2021-03-25 Thread Gustavo A. R. Silva
Hi Martin, On 3/25/21 22:34, Martin K. Petersen wrote: > > Gustavo, > >> Precisely this sort of confusion is one of the things we want to avoid >> by using flexible-array members instead of one-element arrays. > > Ah, you're right! > > Now that I look at it again I also don't think that was t

Re: [PATCH v7 00/38] SCMI vendor protocols and modularization

2021-03-25 Thread Florian Fainelli
On 3/16/2021 5:48 AM, Cristian Marussi wrote: > Hi all, > > The current SCMI implementation does not provide an interface to easily > develop and include a custom vendor protocol implementation as prescribed > by the SCMI standard, also because, there is not currently any custom > protocol in t

Re: [PATCH] mm: page_alloc: fix memcg accounting leak in speculative cache lookup

2021-03-25 Thread Hugh Dickins
On Fri, 26 Mar 2021, Matthew Wilcox wrote: > On Thu, Mar 25, 2021 at 06:55:42PM -0700, Hugh Dickins wrote: > > The first reason occurred to me this morning. I thought I had been > > clever to spot the PageHead race which you fix here. But now I just feel > > very stupid not to have spotted the ve

arch/arm64/kvm/perf.c:58:36: error: implicit declaration of function 'perf_num_counters'

2021-03-25 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: db24726bfefa68c606947a86132591568a06bfb4 commit: 6b5b368fccd7109b052e45af8ba1464c8d140a49 KVM: arm64: Turn kvm_arm_support_pmu_v3() into a static key date: 3 weeks ago config: arm64-randconfig-r005-2021032

  1   2   3   4   5   6   7   8   9   10   >