Re: [PATCH v7 2/2] ARM: ftrace: Add MODULE_PLTS support

2021-03-07 Thread Alexander Sverdlin
Hi! On 07/03/2021 18:26, Qais Yousef wrote: > I tried on 5.12-rc2 and 5.11 but couldn't reproduce the problem using your > instructions on the other email. But most likely because I'm hitting another > problem that could be masking it. I'm not sure it is related or just randomly > happened to hit

Re: [PATCH] staging: rtl8192e: Fix possible buffer overflow in _rtl92e_wx_set_scan

2021-03-07 Thread Dan Carpenter
On Fri, Mar 05, 2021 at 03:00:14PM +, Lee wrote: > > Hi Dan, > > Do you think any of these could be potential issues: > > driver/staging/ > > rtl8192e/rtllib_rx.c:2442 memcpy(dst->ssid, src->ssid, src->ssid_len); Smatch says that at this point we know "src->ssid_len" is in the

linux-next: error fetching the arm-soc-fixes tree

2021-03-07 Thread Stephen Rothwell
Hi all, Fetching the arm-soc-fixes tree (git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git#arm/fixes) produces this error: fatal: couldn't find remote ref refs/heads/arm/fixes -- Cheers, Stephen Rothwell pgpD1vPOkV3z3.pgp Description: OpenPGP digital signature

[PATCH] docs/kokr: make sections on bug reporting match practice

2021-03-07 Thread SeongJae Park
From: SeongJae Park Translate this commit to Korean: cf6d6fc27936 ("docs: process/howto.rst: make sections on bug reporting match practice") Signed-off-by: SeongJae Park --- Documentation/translations/ko_KR/howto.rst | 18 +- 1 file changed, 9 insertions(+), 9

Re: [PATCH 1/1] RISC-V: correct enum sbi_ext_rfence_fid

2021-03-07 Thread Atish Patra
On Sat, Mar 6, 2021 at 4:12 AM Anup Patel wrote: > > On Sat, Mar 6, 2021 at 11:19 AM Heinrich Schuchardt > wrote: > > > > The constants in enum sbi_ext_rfence_fid should match the SBI > > specification. See > >

Re: [PATCH v3 1/3] KVM: nVMX: Sync L2 guest CET states between L1/L2

2021-03-07 Thread Yang Weijiang
On Thu, Mar 04, 2021 at 08:46:45AM -0800, Sean Christopherson wrote: > On Thu, Mar 04, 2021, Yang Weijiang wrote: > > @@ -3375,6 +3391,12 @@ enum nvmx_vmentry_status > > nested_vmx_enter_non_root_mode(struct kvm_vcpu *vcpu, > > if (kvm_mpx_supported() && > >

RE: [PATCH] usb: cdnsp: Fixes incorrect value in ISOC TRB

2021-03-07 Thread Pawel Laszczak
You have right. It's the operator priority issue. I've made this condition as separate "if" statement as suggested by Greg. V2 has been posted. Pawel > > >On 21-03-05 06:10:59, Pawel Laszczak wrote: >> From: Pawel Laszczak >> >> The value "start_cycle ? 0 : 1" in assignment caused >>

Re: [PATCH v2] usb: cdnsp: Fixes incorrect value in ISOC TRB

2021-03-07 Thread Greg KH
On Mon, Mar 08, 2021 at 08:31:03AM +0100, Pawel Laszczak wrote: > From: Pawel Laszczak > > Fixes issue with priority of operator. Operator "|" priority is > higher then "? :". > To improve the readability the operator "? :" has been replaced with > "if ()" statement. > > Fixes: 3d82904559f4

Re: [PATCH 4/6] usbip: fix stub_dev usbip_sockfd_store() races leading to gpf

2021-03-07 Thread Tetsuo Handa
On 2021/03/08 12:53, Shuah Khan wrote: > Fix the above problems: > - Stop using kthread_get_run() macro to create/start threads. > - Create threads and get task struct reference. > - Add kthread_create() failure handling and bail out. > - Hold usbip_device lock to update local and shared states

Re: [PATCH v6 36/37] firmware: arm_scmi: add protocol modularization support

2021-03-07 Thread Sudeep Holla
On Tue, Feb 02, 2021 at 10:15:54PM +, Cristian Marussi wrote: > Extend SCMI protocols accounting mechanism to address possible module > usage and add the support to possibly define new protocols as loadable > modules. > > Keep Standard protocols built into the SCMI core. > The changes look

[PATCH v2] usb: cdnsp: Fixes incorrect value in ISOC TRB

2021-03-07 Thread Pawel Laszczak
From: Pawel Laszczak Fixes issue with priority of operator. Operator "|" priority is higher then "? :". To improve the readability the operator "? :" has been replaced with "if ()" statement. Fixes: 3d82904559f4 ("usb: cdnsp: cdns3 Add main part of Cadence USBSSP DRD Driver") Signed-off-by:

[PATCH] coccinelle: misc: update uninitialized_var.cocci documentation

2021-03-07 Thread Denis Efremov
Remove the documentation link from the warning message because commit 3942ea7a10c9 ("deprecated.rst: Remove now removed uninitialized_var") removed the section from documentation. Update the rule documentation accordingly. Signed-off-by: Denis Efremov ---

Re: [PATCH v2] iio: buffer: fix use-after-free for attached_buffers array

2021-03-07 Thread Alexandru Ardelean
On Sun, Mar 7, 2021 at 8:55 PM Alexandru Ardelean wrote: > > Thanks to Lars for finding this. > The free of the 'attached_buffers' array should be done as late as > possible. This change moves it to iio_buffers_put(), which looks like > the best place for it, since it takes place right before the

Re: [PATCH v3] amba: Remove deferred device addition

2021-03-07 Thread Marek Szyprowski
Hi Saravana, On 05.03.2021 19:02, Saravana Kannan wrote: > On Fri, Mar 5, 2021 at 3:45 AM Marek Szyprowski > wrote: >> On 04.03.2021 20:51, Saravana Kannan wrote: >>> The uevents generated for an amba device need PID and CID information >>> that's available only when the amba device is powered

[PATCH] ASoC: bcm: add missing call to of_node_put()

2021-03-07 Thread Yang Li
In one of the error paths of the for_each_child_of_node() loop, add missing call to of_node_put(). Fix the following coccicheck warning: ./sound/soc/bcm/cygnus-ssp.c:1346:1-33: WARNING: Function "for_each_available_child_of_node" should have of_node_put() before return around line 1352.

Re: [PATCH v3 3/3] gpio: xilinx: Utilize generic bitmap_get_value and _set_value

2021-03-07 Thread Michal Simek
On 3/6/21 3:06 PM, Syed Nayyar Waris wrote: > This patch reimplements the xgpio_set_multiple() function in > drivers/gpio/gpio-xilinx.c to use the new generic functions: > bitmap_get_value() and bitmap_set_value(). The code is now simpler > to read and understand. Moreover, instead of looping

[PATCH] arm64: dts: zynqmp: Add compatible strings for si5328

2021-03-07 Thread quanyang . wang
From: Quanyang Wang The function of_i2c_get_board_info will call of_modalias_node to check if a device_node contains "compatible" string. So let's assign the proper string "silabs,si5328" to clock-generator@69's compatible property to eliminate the error info as below: i2c i2c-10: of_i2c:

Re: [PATCH] x86/perf: Fix guest_get_msrs static call if there is no PMU

2021-03-07 Thread Dmitry Vyukov
On Mon, Mar 8, 2021 at 3:26 AM Xu, Like wrote: > > On 2021/3/6 6:33, Sean Christopherson wrote: > > Handle a NULL x86_pmu.guest_get_msrs at invocation instead of patching > > in perf_guest_get_msrs_nop() during setup. If there is no PMU, setup > > "If there is no PMU" ... > > How to set up this

drivers/usb/gadget/function/f_fs.c:3829:8-15: WARNING opportunity for memdup_user

2021-03-07 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 144c79ef33536b4ecb4951e07dbc1f2b7fa99d32 commit: 8704fd73bf5658bf4b827643f7f526481082d83f USB: gadget: f_fs: remove likely/unlikely date: 3 months ago config: x86_64-randconfig-c002-20210307 (attached

[PATCH] USB: gadget: f_fs: fix memdup_user.cocci warnings

2021-03-07 Thread kernel test robot
From: kernel test robot drivers/usb/gadget/function/f_fs.c:3829:8-15: WARNING opportunity for memdup_user Use memdup_user rather than duplicating its implementation This is a little bit restricted to reduce false positives Generated by: scripts/coccinelle/api/memdup_user.cocci Fixes:

[PATCH] phy: hisilicon: add missing call to of_node_put()

2021-03-07 Thread Yang Li
In one of the error paths of the for_each_child_of_node() loop, add missing call to of_node_put(). Fix the following coccicheck warning: ./drivers/phy/hisilicon/phy-hisi-inno-usb2.c:138:1-23: WARNING: Function "for_each_child_of_node" should have of_node_put() before break around line 158.

Re: [block] 52f019d43c: ndctl.test-libndctl.fail

2021-03-07 Thread h...@lst.de
On Sat, Mar 06, 2021 at 08:33:04PM +, Williams, Dan J wrote: > Yes, it looks like my unit test checks for exactly the behavior you > changed. It was convenient to test that the device could be switched > back to rw via BLKROSET, but I don't require that. The new behaviour of > letting the

[PATCH] arm64: dts: mt8173: fix property typo of 'phys' in dsi node

2021-03-07 Thread Chunfeng Yun
Use 'phys' instead of 'phy'. Fixes: 81ad4dbaf7af ("arm64: dts: mt8173: Add display subsystem related nodes") Cc: stable Signed-off-by: Chunfeng Yun --- arch/arm64/boot/dts/mediatek/mt8173.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v4] coccinelle: misc: add minmax script

2021-03-07 Thread Denis Efremov
Check for opencoded min(), max() implementations. Signed-off-by: Denis Efremov --- Changes in v2: - <... ...> instead of ... when any - org mode reports fixed - patch rule to drop excessive () Changes in v3: - "depends on patch && (pmax || pmaxif || pmin || pminif)" fixed Changes in v4: -

RE: [EXT] Re: vfio-pci: protect remap_pfn_range() from simultaneous calls

2021-03-07 Thread Bharat Bhushan
> -Original Message- > From: Ankur Arora > Sent: Monday, March 8, 2021 12:29 PM > To: Bharat Bhushan > Cc: alex.william...@redhat.com; linux-kernel@vger.kernel.org; Sunil Kovvuri > Goutham ; termi...@gmail.com > Subject: Re: [EXT] Re: vfio-pci: protect remap_pfn_range() from

Re: [RFC 1/1] s390/pci: expose a PCI device's UID as its index

2021-03-07 Thread Niklas Schnelle
On 3/7/21 9:46 PM, Krzysztof Wilczyński wrote: > Hi Niklas, > > [...] >> +static ssize_t index_show(struct device *dev, >> + struct device_attribute *attr, char *buf) >> +{ >> +struct zpci_dev *zdev = to_zpci(to_pci_dev(dev)); >> +u32 index = ~0; >> + >> +if

Re: [EXT] Re: vfio-pci: protect remap_pfn_range() from simultaneous calls

2021-03-07 Thread Ankur Arora
On 2021-03-02 4:47 a.m., Bharat Bhushan wrote: Hi Ankur, -Original Message- From: Ankur Arora Sent: Friday, February 26, 2021 6:24 AM To: Bharat Bhushan Cc: alex.william...@redhat.com; ankur.a.ar...@oracle.com; linux- ker...@vger.kernel.org; Sunil Kovvuri Goutham ; termi...@gmail.com

Re: [PATCH v9 2/9] x509: Detect sm2 keys by their parameters OID

2021-03-07 Thread Tianjia Zhang
Hi, On 3/5/21 11:04 PM, Stefan Berger wrote: On 3/5/21 2:37 AM, Tianjia Zhang wrote: Hi, On 3/4/21 7:46 AM, Stefan Berger wrote: Tianjia,     can you say whether SM2 support works for you before and after applying this patch? I cannot verify it with an sm2 key I have created using a

Re: linux-kernel janitorial RFP: Mark static arrays as const

2021-03-07 Thread Julia Lawall
On Sun, 7 Mar 2021, Joe Perches wrote: > On Sun, 2021-03-07 at 20:14 +0100, Julia Lawall wrote: > > > > On Wed, 3 Mar 2021, Joe Perches wrote: > > > > > On Wed, 2021-03-03 at 10:41 +0100, Rasmus Villemoes wrote: > > > > On 02/03/2021 18.42, Joe Perches wrote: > > > > > Here is a possible

Re: [v8 PATCH 07/13] mm: vmscan: add shrinker_info_protected() helper

2021-03-07 Thread Shakeel Butt
On Tue, Feb 16, 2021 at 4:13 PM Yang Shi wrote: > > The shrinker_info is dereferenced in a couple of places via > rcu_dereference_protected > with different calling conventions, for example, using mem_cgroup_nodeinfo > helper > or dereferencing memcg->nodeinfo[nid]->shrinker_info. And the

Re: [PATCH] sound: soc: codecs: Fix a spello in the file wm8955.c

2021-03-07 Thread Randy Dunlap
On 3/6/21 3:51 AM, Bhaskar Chowdhury wrote: > > s/sortd/sorted/ > > Signed-off-by: Bhaskar Chowdhury Acked-by: Randy Dunlap > --- > sound/soc/codecs/wm8955.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/soc/codecs/wm8955.c b/sound/soc/codecs/wm8955.c >

Re: [v8 PATCH 06/13] mm: memcontrol: rename shrinker_map to shrinker_info

2021-03-07 Thread Shakeel Butt
On Tue, Feb 16, 2021 at 4:13 PM Yang Shi wrote: > > The following patch is going to add nr_deferred into shrinker_map, the change > will > make shrinker_map not only include map anymore, so rename it to > "memcg_shrinker_info". > And this should make the patch adding nr_deferred cleaner and

Re: [v8 PATCH 04/13] mm: vmscan: remove memcg_shrinker_map_size

2021-03-07 Thread Shakeel Butt
On Tue, Feb 16, 2021 at 4:13 PM Yang Shi wrote: > > Both memcg_shrinker_map_size and shrinker_nr_max is maintained, but actually > the > map size can be calculated via shrinker_nr_max, so it seems unnecessary to > keep both. > Remove memcg_shrinker_map_size since shrinker_nr_max is also used by

[PATCH] MIPS: Enable some missed configs in loongson3_defconfig to support bpftrace

2021-03-07 Thread Tiezhu Yang
bpftrace is a high-level tracing language for Linux enhanced Berkeley Packet Filter (eBPF) available in recent Linux kernels (4.x). bpftrace uses LLVM as a backend to compile scripts to BPF-bytecode and makes use of BCC for interacting with the Linux BPF system, as well as existing Linux tracing

Re: [PATCH] mm,hwpoison: return -EBUSY when page already poisoned

2021-03-07 Thread 堀口 直也
On Fri, Mar 05, 2021 at 02:11:43PM -0800, Luck, Tony wrote: > This whole page table walking patch is trying to work around the > races caused by multiple calls to memory_failure() for the same > page. > > Maybe better to just avoid the races. The comment right above > memory_failure says: > >

Re: [PATCH v1 1/1] mtd: spi-nor: intel-spi: Move platform data header to x85 subfolder

2021-03-07 Thread Tudor Ambarus
On Thu, 4 Mar 2021 16:08:20 +0200, Andy Shevchenko wrote: > In order to group x86 related platform data move intel-spi.h to x85 folder. > > While at it, remove duplicate inclusion in C file. s/x85/x86 in subject and commit description and applied to spi-nor/next. Thanks! [1/1] mtd: spi-nor:

[PATCH] docs: gpu: fix typo

2021-03-07 Thread Jianhui Zhao
Signed-off-by: Jianhui Zhao --- Documentation/gpu/todo.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 40ccac61137e..b7f1acb355f5 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst @@

[PATCH 2/2] arm64: imx8mp: imx8mp-phycore-som enable spi nor

2021-03-07 Thread Heiko Schocher
enable the mt25qu256aba spi nor on the imx8mp-phycore-som. Signed-off-by: Heiko Schocher --- .../dts/freescale/imx8mp-phycore-som.dtsi | 27 +++ 1 file changed, 27 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mp-phycore-som.dtsi

[PATCH 4/6] mm: Drop redundant ARCH_ENABLE_[HUGEPAGE|THP]_MIGRATION

2021-03-07 Thread Anshuman Khandual
ARCH_ENABLE_[HUGEPAGE|THP]_MIGRATION configs have duplicate definitions on platforms that subscribe them. Drop these reduntant definitions and instead just select them appropriately. Cc: Catalin Marinas Cc: Will Deacon Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc:

[PATCH 3/6] mm: Generalize ARCH_ENABLE_MEMORY_[HOTPLUG|HOTREMOVE]

2021-03-07 Thread Anshuman Khandual
ARCH_ENABLE_MEMORY_[HOTPLUG|HOTREMOVE] configs have duplicate definitions on platforms that subscribe them. Instead, just make them generic options which can be selected on applicable platforms. Cc: Catalin Marinas Cc: Will Deacon Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul

[PATCH 1/2] arm64: dts: imx8mp: add flexspi node

2021-03-07 Thread Heiko Schocher
add node for the flexspi modul on imx8mp. Signed-off-by: Heiko Schocher --- arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 1 file changed, 16 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi index

[PATCH 0/6] mm: some config cleanups

2021-03-07 Thread Anshuman Khandual
This series contains config cleanup patches which reduces code duplication across platforms and also improves maintainability. There is no functional change intended with this series. This has been boot tested on arm64 but only build tested on some other platforms. This applies on 5.12-rc2 Cc:

[PATCH 0/2] enable flexspi support on imx8mp

2021-03-07 Thread Heiko Schocher
This series enables support for the SPI NOR on the imx8mp based phyboard-pollux-rdk board. Heiko Schocher (2): arm64: dts: imx8mp: add flexspi node arm64: imx8mp: imx8mp-phycore-som enable spi nor .../dts/freescale/imx8mp-phycore-som.dtsi | 27 +++

Re: [v8 PATCH 03/13] mm: vmscan: use shrinker_rwsem to protect shrinker_maps allocation

2021-03-07 Thread Shakeel Butt
On Tue, Feb 16, 2021 at 4:13 PM Yang Shi wrote: > > Since memcg_shrinker_map_size just can be changed under holding shrinker_rwsem > exclusively, the read side can be protected by holding read lock, so it sounds > superfluous to have a dedicated mutex. > > Kirill Tkhai suggested use write lock

[PATCH v4 3/4] dt-bindings: PCI: ti,j721e: Add endpoint mode dt-bindings for TI's AM64 SoC

2021-03-07 Thread Kishon Vijay Abraham I
Add endpoint mode dt-bindings for TI's AM64 SoC. This is the same IP used in J7200, however AM64 is a non-coherent architecture. Signed-off-by: Kishon Vijay Abraham I Reviewed-by: Rob Herring --- .../devicetree/bindings/pci/ti,j721e-pci-ep.yaml | 9 + 1 file changed, 5

[PATCH v4 0/4] AM64: Add PCIe bindings and driver support

2021-03-07 Thread Kishon Vijay Abraham I
AM64 uses the same PCIe controller as in J7200, however AM642 EVM doesn't have a clock generator (unlike J7200 base board). Here the clock from the SERDES has to be routed to the PCIE connector. This series provides an option for the pci-j721e.c driver to drive reference clock output to the

[PATCH v4 4/4] PCI: j721e: Add support to provide refclk to PCIe connector

2021-03-07 Thread Kishon Vijay Abraham I
Add support to provide refclk to PCIe connector. Signed-off-by: Kishon Vijay Abraham I --- drivers/pci/controller/cadence/pci-j721e.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/cadence/pci-j721e.c

[PATCH v4 1/4] dt-bindings: PCI: ti,j721e: Add binding to represent refclk to the connector

2021-03-07 Thread Kishon Vijay Abraham I
Add binding to represent refclk to the PCIe connector. Signed-off-by: Kishon Vijay Abraham I --- .../devicetree/bindings/pci/ti,j721e-pci-host.yaml | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml

[PATCH v4 2/4] dt-bindings: PCI: ti,j721e: Add host mode dt-bindings for TI's AM64 SoC

2021-03-07 Thread Kishon Vijay Abraham I
Add host mode dt-bindings for TI's AM64 SoC. This is the same IP used in J7200, however AM64 is a non-coherent architecture. Signed-off-by: Kishon Vijay Abraham I Reviewed-by: Rob Herring --- .../devicetree/bindings/pci/ti,j721e-pci-host.yaml| 11 +++ 1 file changed, 7

[PATCH v2] usb: typec: tcpm: turn tcpm_ams_finish into void function

2021-03-07 Thread Yang Li
This function always return '0' and no callers use the return value. So make it a void function. This eliminates the following coccicheck warning: ./drivers/usb/typec/tcpm/tcpm.c:778:5-8: Unneeded variable: "ret". Return "0" on line 794 Reported-by: Abaci Robot Signed-off-by: Yang Li ---

Re: [PATCH] usb: typec: tcpm: turn tcpm_ams_finish into void function

2021-03-07 Thread Guenter Roeck
On 3/7/21 10:21 PM, Yang Li wrote: > This function always return '0' and no callers use the return value. > So make it a void function. > > This eliminates the following coccicheck warning: > ./drivers/usb/typec/tcpm/tcpm.c:778:5-8: Unneeded variable: "ret". > Return "0" on line 794 > >

Re: [PATCH v2 5/5] mtd: spi-nor: swp: Drop 'else' after 'return'

2021-03-07 Thread Pratyush Yadav
On 06/03/21 11:50AM, Tudor Ambarus wrote: > else is not generally useful after a break or return. > > Signed-off-by: Tudor Ambarus Reviewed-by: Pratyush Yadav -- Regards, Pratyush Yadav Texas Instruments Inc.

[PATCH] kbuild: dummy-tools, support MPROFILE_KERNEL checks for ppc

2021-03-07 Thread Jiri Slaby
ppc64le checks for -mprofile-kernel to define MPROFILE_KERNEL Kconfig. Kconfig calls arch/powerpc/tools/gcc-check-mprofile-kernel.sh for that purpose. This script performs two checks: 1) build with -mprofile-kernel should contain "_mcount" 2) build with -mprofile-kernel with a function marked as

Re: [PATCH v2 3/5] mtd: spi-nor: Get rid of duplicated argument in spi_nor_parse_sfdp()

2021-03-07 Thread Pratyush Yadav
On 06/03/21 11:50AM, Tudor Ambarus wrote: > spi_nor_parse_sfdp(nor, nor->params); > passes for the second argument a member within the first argument. > Drop the second argument and obtain it directly from the first, > and do it across all the children functions. This is a follow up for > 'commit

Re: [PATCH v1 1/1] mtd: spi-nor: intel-spi: Move platform data header to x85 subfolder

2021-03-07 Thread Vignesh Raghavendra
On 3/4/21 7:38 PM, Andy Shevchenko wrote: > In order to group x86 related platform data move intel-spi.h to x85 folder. > > While at it, remove duplicate inclusion in C file. > > Signed-off-by: Andy Shevchenko > --- Nit, typo in $subject and commit message: s/x85/x86. Tudor may be able to

Re: [PATCH v2 2/5] mtd: spi-nor: core: Add vdbg msg for spi_nor_erase_multi_sectors()

2021-03-07 Thread Pratyush Yadav
On 06/03/21 11:49AM, Tudor Ambarus wrote: > Useful when debugging non-uniform erase. > > Signed-off-by: Tudor Ambarus > --- > v2: > - s/dev_dbg/dev_vdb > - move vdbg message the first thing in the while > > drivers/mtd/spi-nor/core.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git

[PATCH] usb: typec: tcpm: turn tcpm_ams_finish into void function

2021-03-07 Thread Yang Li
This function always return '0' and no callers use the return value. So make it a void function. This eliminates the following coccicheck warning: ./drivers/usb/typec/tcpm/tcpm.c:778:5-8: Unneeded variable: "ret". Return "0" on line 794 Reported-by: Abaci Robot Signed-off-by: Yang Li ---

Re: [PATCH] mtd: spi-nor: use is_power_of_2()

2021-03-07 Thread Tudor Ambarus
On Sat, 6 Mar 2021 00:45:52 +0100, Michael Walle wrote: > There is already a function to check if an integer is a power of 2. Use > it. Applied to spi-nor/next, thanks! [1/1] mtd: spi-nor: use is_power_of_2() https://git.kernel.org/mtd/c/04fc298c7d08 Best regards, -- Tudor Ambarus

Re: [v8 PATCH 05/13] mm: vmscan: use kvfree_rcu instead of call_rcu

2021-03-07 Thread Shakeel Butt
On Tue, Feb 16, 2021 at 4:13 PM Yang Shi wrote: > > Using kvfree_rcu() to free the old shrinker_maps instead of call_rcu(). > We don't have to define a dedicated callback for call_rcu() anymore. > > Signed-off-by: Yang Shi > --- > mm/vmscan.c | 7 +-- > 1 file changed, 1 insertion(+), 6

Re: [PATCH v6 35/37] firmware: arm_scmi: make notify_priv really private

2021-03-07 Thread Sudeep Holla
On Tue, Feb 02, 2021 at 10:15:53PM +, Cristian Marussi wrote: > Notification private data is currently accessible via handle->notify_priv; > this data was indeed meant to be private to the notification core support > and not to be accessible by SCMI drivers: make it private hiding it inside >

Re: [PATCH] mtd: spi-nor: gigadevice: Add support for gd25lb256e

2021-03-07 Thread Tudor.Ambarus
On 2/9/21 1:11 PM, Md Sadre Alam wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > Add support for gd25lb256e. This device tested on IPQ5018 > platform with dd from/to the flash for read/write respectly, typo: respectly > and mtd erase for

[PATCH 18/18] arm64: qcom: ipq6018: drop '0x' from unit address

2021-03-07 Thread Vinod Koul
Nodes need not contain '0x' for the unit address. Drop it to fix the below warning: arch/arm64/boot/dts/qcom/ipq6018-cp01-c1.dt.yaml: reserved-memory: 'memory@0x6' does not match any of the regexes Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +- 1 file changed,

[PATCH 16/18] arm64: qcom: ipq8074: fix pci node reg property

2021-03-07 Thread Vinod Koul
reg property should be array of values, here it is a single array, leading to below warning: arch/arm64/boot/dts/qcom/ipq8074-hk01.dt.yaml: soc: pci@1000:reg:0: [268435456, 3869, 268439328, 168, 557056, 8192, 269484032, 4096] is too long arch/arm64/boot/dts/qcom/ipq8074-hk01.dt.yaml: soc:

[PATCH 17/18] arm64: qcom: sdm660: use reg value for memory node

2021-03-07 Thread Vinod Koul
memory node like other node should be node@reg, which is missing in this case, so fix it up arch/arm64/boot/dts/qcom/ipq8074-hk01.dt.yaml: /: memory: False schema does not allow {'device_type': ['memory'], 'reg': [[0, 1073741824, 0, 536870912]]} Signed-off-by: Vinod Koul ---

[PATCH 15/18] arm64: qcom: sdm660: don't use empty memory node

2021-03-07 Thread Vinod Koul
We expect bootloader to full memory details but passing empty values can give warning, so add a default value Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/sdm660.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sdm660.dtsi

[PATCH 14/18] arm64: qcom: sdm630: don't use empty memory node

2021-03-07 Thread Vinod Koul
We expect bootloader to full memory details but passing empty values can give warning, so add a default value Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/sdm630.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi

[PATCH 11/18] arm64: qcom: sdm660: don't use underscore in node name

2021-03-07 Thread Vinod Koul
We have underscore (_) in node name so fix that up as well. Fix this by changing node name to use dash (-) Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/sdm660.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sdm660.dtsi

[PATCH 13/18] arm64: qcom: msm8998: don't use empty memory node

2021-03-07 Thread Vinod Koul
We expect bootloader to full memory details but passing empty values can give warning, so add a default value Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/msm8998.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi

[PATCH 10/18] arm64: qcom: sdm630: don't use underscore in node name

2021-03-07 Thread Vinod Koul
We have underscore (_) in node name so fix that up as well. Fix this by changing node name to use dash (-) Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/sdm630.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi

[PATCH 09/18] arm64: qcom: msm8996: don't use underscore in node name

2021-03-07 Thread Vinod Koul
We have underscore (_) in node name leading to warning: arch/arm64/boot/dts/qcom/apq8096-db820c.dt.yaml: clocks: $nodename:0: 'clocks' does not match '^([a-z][a-z0-9\\-]+-bus|bus|soc|axi|ahb|apb)(@[0-9a-f]+)?$' arch/arm64/boot/dts/qcom/apq8096-db820c.dt.yaml: clocks: xo_board: {'type':

[PATCH 12/18] arm64: qcom: msm8996: don't use empty memory node

2021-03-07 Thread Vinod Koul
We expect bootloader to full memory details but passing empty values can give warning, so add a default value Signed-off-by: Vinod Koul --- arch/arm64/boot/dts/qcom/msm8996.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi

[PATCH 08/18] arm64: qcom: msm8994: don't use underscore in node name

2021-03-07 Thread Vinod Koul
We have underscore (_) in node name leading to warning: arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dt.yaml: clocks: xo_board: {'type': 'object'} is not allowed for {'compatible': ['fixed-clock'], '#clock-cells': [[0]], 'clock-frequency': [[1920]], 'phandle': [[26]]}

[PATCH 07/18] dt-bindings: arm: qcom: Drop qcom,mtp

2021-03-07 Thread Vinod Koul
qcom,mtp is used msm8916-mtp.dts but not documented, it is a generic mtp compatible and we have specfifc ones for each mtp so drop this arch/arm64/boot/dts/qcom/msm8916-mtp.dt.yaml: /: compatible: 'oneOf' conditional failed, one must be fixed: ['qcom,msm8916-mtp', 'qcom,msm8916-mtp/1',

[PATCH 06/18] dt-bindings: arm: qcom: Document alcatel,idol347 board

2021-03-07 Thread Vinod Koul
Document the alcatel,idol347 board. It was missing leading to warning: arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dt.yaml: /: compatible: 'oneOf' conditional failed, one must be fixed: Additional items are not allowed ('qcom,msm8916' was unexpected) Signed-off-by: Vinod Koul ---

[PATCH 05/18] dt-bindings: arm: qcom: Document sony boards for apq8094

2021-03-07 Thread Vinod Koul
Document the various sony boards for apq8094. These are used in various sony dts files but not documented Signed-off-by: Vinod Koul --- Documentation/devicetree/bindings/arm/qcom.yaml | 10 ++ 1 file changed, 10 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml

[PATCH 02/18] arm64: qcom: msm8916: don't use empty memory node

2021-03-07 Thread Vinod Koul
We expect bootloader to full memory details but passing empty values gives warning, so add a default value arch/arm64/boot/dts/qcom/apq8016-sbc.dt.yaml: /: memory: False schema does not allow {'device_type': ['memory'], 'reg': [[0, 0, 0, 0]]} Signed-off-by: Vinod Koul ---

[PATCH 04/18] arm64: qcom: msm8994: don't use empty memory node

2021-03-07 Thread Vinod Koul
We expect bootloader to full memory details but passing empty values gives warning, so add a default value arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dt.yaml: /: memory: False schema does not allow {'device_type': ['memory'], 'reg': [[0, 0, 0, 0]]} Signed-off-by:

[PATCH 01/18] arm64: qcom: apq8016-sbc: drop qcom,sbc

2021-03-07 Thread Vinod Koul
apq8016-sbc is one of the compaitibles for this board, but is not documented, so drop it. This fixes these two warns: arch/arm64/boot/dts/qcom/apq8016-sbc.dt.yaml: /: compatible: ['qcom,apq8016-sbc', 'qcom,apq8016', 'qcom,sbc'] is not valid under any of the given schemas (Possible causes of the

[PATCH 03/18] dt-bindings: arm: qcom: Document ipq6018-cp01 board

2021-03-07 Thread Vinod Koul
Document the ipq6018-cp01 board. It was missing leading to warning: arch/arm64/boot/dts/qcom/ipq6018-cp01-c1.dt.yaml: /: compatible: 'oneOf' conditional failed, one must be fixed: ['qcom,ipq6018-cp01', 'qcom,ipq6018'] is too short ['qcom,ipq6018-cp01', 'qcom,ipq6018'] is too long

[PATCH 00/18] arm64: qcom: fix dtbs_check warning

2021-03-07 Thread Vinod Koul
qcom dts folder shows a bunch of dtbs_check warning, this is an attempt to fix some of them. Many are due to txt binding which should be fixed when we have the yaml binding documents. Few of them are fixed as below with missing nodes names, not properly using nodes. Vinod Koul (18): arm64:

linux-next: Fixes tag needs some work in the drivers-x86 tree

2021-03-07 Thread Stephen Rothwell
Hi all, In commit b5b5ff84fd93 ("platform/surface: aggregator: Make SSAM_DEFINE_SYNC_REQUEST_x define static functions") Fixes tag Fixes: 510c8114fc74 ("platform/surface: Add platform profile driver") has these problem(s): - Target SHA1 does not exist Maybe you meant Fixes:

[PATCH v2] irqchip/irq-mst: Support polarity configuration

2021-03-07 Thread Mark-PK Tsai
Support irq polarity configuration and save and restore the config when system suspend and resume. Signed-off-by: Mark-PK Tsai --- drivers/irqchip/irq-mst-intc.c | 90 -- 1 file changed, 87 insertions(+), 3 deletions(-) diff --git

linux-next: Signed-off-by missing for commit in the rcu tree

2021-03-07 Thread Stephen Rothwell
Hi all, Commit 3e90d423e754 ("EXP net: phy: make mdio_bus_phy_suspend/resume as __maybe_unused") is missing a Signed-off-by from its committer. -- Cheers, Stephen Rothwell pgpqnFoVCNSEx.pgp Description: OpenPGP digital signature

Re: [PATCH v6 02/37] firmware: arm_scmi: introduce protocol handle definitions

2021-03-07 Thread Sudeep Holla
On Tue, Feb 02, 2021 at 10:15:20PM +, Cristian Marussi wrote: > Add basic protocol handles definitions and private data helpers support. > > A protocol handle identifies a protocol instance initialized against a > specific handle; it embeds all the references to the core SCMI xfer methods >

Re: [PATCH v2] mtd: spi-nor: winbond: Add support for w25q512jvq

2021-03-07 Thread Tudor Ambarus
On Mon, 8 Feb 2021 15:53:03 +0800, Shuhao Mai wrote: > Add support for w25q512jvq. This is of the same series chip with > w25q256jv, which is already supported, but with size doubled and > different JEDEC ID. > > Tested on Intel whitley platform with dd from/to the flash for > read/write

[PATCH v4 3/3] mtd: rawnand: qcom: Add support for secure regions in NAND memory

2021-03-07 Thread Manivannan Sadhasivam
On a typical end product, a vendor may choose to secure some regions in the NAND memory which are supposed to stay intact between FW upgrades. The access to those regions will be blocked by a secure element like Trustzone. So the normal world software like Linux kernel should not touch these

[PATCH v4 2/3] dt-bindings: mtd: Add a property to declare secure regions in NAND chips

2021-03-07 Thread Manivannan Sadhasivam
On a typical end product, a vendor may choose to secure some regions in the NAND memory which are supposed to stay intact between FW upgrades. The access to those regions will be blocked by a secure element like Trustzone. So the normal world software like Linux kernel should not touch these

[PATCH v4 0/3] Add support for secure regions in Qcom NANDc driver

2021-03-07 Thread Manivannan Sadhasivam
On a typical end product, a vendor may choose to secure some regions in the NAND memory which are supposed to stay intact between FW upgrades. The access to those regions will be blocked by a secure element like Trustzone. So the normal world software like Linux kernel should not touch these

[PATCH v4 1/3] dt-bindings: mtd: Convert Qcom NANDc binding to YAML

2021-03-07 Thread Manivannan Sadhasivam
Convert Qcom NANDc devicetree binding to YAML. Signed-off-by: Manivannan Sadhasivam Reviewed-by: Rob Herring --- .../devicetree/bindings/mtd/qcom,nandc.yaml | 196 ++ .../devicetree/bindings/mtd/qcom_nandc.txt| 142 - 2 files changed, 196 insertions(+), 142

Re: [Intel-wired-lan] [PATCH] e1000e: Fix error handling in e1000_set_d0_lplu_state_82571

2021-03-07 Thread Neftin, Sasha
On 2/28/2021 11:44, Dinghao Liu wrote: There is one e1e_wphy() call in e1000_set_d0_lplu_state_82571 that we have caught its return value but lack further handling. Check and terminate the execution flow just like other e1e_wphy() in this function. Signed-off-by: Dinghao Liu ---

[PATCH] crypto: aegis128 - Move simd prototypes into aegis.h

2021-03-07 Thread Herbert Xu
On Sun, Feb 28, 2021 at 03:14:55AM +0800, kernel test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 5695e51619745d4fe3ec2506a2f0cd982c5e27a4 > commit: a4397635afea5d127548d64e0055ed471ef2d5be crypto: aegis128 - provide a > SIMD

Re: linux-kernel janitorial RFP: Mark static arrays as const

2021-03-07 Thread Joe Perches
On Sun, 2021-03-07 at 20:14 +0100, Julia Lawall wrote: > > On Wed, 3 Mar 2021, Joe Perches wrote: > > > On Wed, 2021-03-03 at 10:41 +0100, Rasmus Villemoes wrote: > > > On 02/03/2021 18.42, Joe Perches wrote: > > > > Here is a possible opportunity to reduce data usage in the kernel. > > > > > >

[PATCH v4 06/12] arm64: dts: mediatek: mt2712: harmonize node names

2021-03-07 Thread Chunfeng Yun
This is used to fix dtbs_check warning. Signed-off-by: Chunfeng Yun --- v2~v4: no changes --- arch/arm64/boot/dts/mediatek/mt2712e.dtsi | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt2712e.dtsi

[PATCH v4 12/12] arm: dts: mt2701: harmonize node names and compatibles

2021-03-07 Thread Chunfeng Yun
This is used to fix dtbs_check warning Signed-off-by: Chunfeng Yun --- v2~v4: no changes --- arch/arm/boot/dts/mt2701.dtsi | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi index

[PATCH v4 09/12] arm64: dts: mediatek: mt8183: fix dtbs_check warning

2021-03-07 Thread Chunfeng Yun
Harmonize node names, compatibles and properties. Signed-off-by: Chunfeng Yun --- v4: no changes v3: remove property clock-names suggested by CK v2: no changes --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git

[PATCH v4 11/12] arm: dts: mt7623: harmonize node names and compatibles

2021-03-07 Thread Chunfeng Yun
This is used to fix dtbs_check warning Signed-off-by: Chunfeng Yun --- v2~v4: no changes --- arch/arm/boot/dts/mt7623.dtsi | 26 ++ arch/arm/boot/dts/mt7623n.dtsi | 4 ++-- 2 files changed, 16 insertions(+), 14 deletions(-) diff --git a/arch/arm/boot/dts/mt7623.dtsi

[PATCH v4 05/12] arm64: dts: mediatek: mt8173: fix dtbs_check warning

2021-03-07 Thread Chunfeng Yun
Harmonize nodes names, compatibles and remove unused property. Signed-off-by: Chunfeng Yun --- v2~v4: no changes --- arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 4 +--- arch/arm64/boot/dts/mediatek/mt8173.dtsi| 13 +++-- 2 files changed, 8 insertions(+), 9 deletions(-) diff

[PATCH v4 10/12] arm: dts: mt7629: harmonize node names and compatibles

2021-03-07 Thread Chunfeng Yun
This is used to fix dtbs_check warning Signed-off-by: Chunfeng Yun --- v2~v4: no changes --- arch/arm/boot/dts/mt7629.dtsi | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/arch/arm/boot/dts/mt7629.dtsi b/arch/arm/boot/dts/mt7629.dtsi index

[PATCH v4 08/12] arm64: dts: mediatek: mt7622: harmonize node names and compatibles

2021-03-07 Thread Chunfeng Yun
This is used to fix dtbs_check warning Signed-off-by: Chunfeng Yun --- v2~v4: no changes --- arch/arm64/boot/dts/mediatek/mt7622.dtsi | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt7622.dtsi b/arch/arm64/boot/dts/mediatek/mt7622.dtsi

[PATCH v4 02/12] dt-bindings: phy: mediatek: dsi-phy: modify compatible dependence

2021-03-07 Thread Chunfeng Yun
mt7623-mipi-tx is compatible to mt2701-mipi-tx, and use "mediatek,mt2701-mipi-tx" instead on MT7623, so modify the compatible items to make dependence clear. Cc: Chun-Kuang Hu Cc: Philipp Zabel Acked-by: Chun-Kuang Hu Reviewed-by: Rob Herring Signed-off-by: Chunfeng Yun --- v4: add acked-by

  1   2   3   4   5   6   7   >