Re: [PATCH] Kbuild: enable -Wfallthrough for clang

2020-11-07 Thread Miguel Ojeda
On Sat, Nov 7, 2020 at 8:08 AM Nick Desaulniers wrote: > > Partial revert of commit e2079e93f562 ("kbuild: Do not enable > -Wimplicit-fallthrough for clang for now") > > This has been fixed up over time thanks to the addition of "fallthrough" > pseudo-keyword in > commit 294f69e662d1

Re: [PATCH 19/41] ath: regd: Provide description for ath_reg_apply_ir_flags's 'reg' param

2020-11-07 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/regd.c:378: warning: Function parameter or member > 'reg' not described in 'ath_reg_apply_ir_flags' > > Cc: Kalle Valo > Cc: "David S. Miller" > Cc: Jakub Kicinski > Cc:

Re: [PATCH 10/41] ath9k: ar9330_1p1_initvals: Remove unused const variable 'ar9331_common_tx_gain_offset1_1'

2020-11-07 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/ath9k/ar9330_1p1_initvals.h:1013:18: warning: > ‘ar9331_common_tx_gain_offset1_1’ defined but not used > [-Wunused-const-variable=] > > Cc: QCA ath9k Development > Cc: Kalle Valo > Cc: "David

Re: [PATCH net-next 09/11] ath6kl: fix enum-conversion warning

2020-11-07 Thread Kalle Valo
Arnd Bergmann wrote: > gcc -Wextra points out a type mismatch > > drivers/net/wireless/ath/ath6kl/wmi.c: In function 'ath6kl_wmi_cmd_send': > drivers/net/wireless/ath/ath6kl/wmi.c:1825:19: warning: implicit conversion > from 'enum ' to 'enum wmi_data_hdr_data_type' [-Wenum-conversion] > 1825

Re: [PATCH 01/41] wil6210: wmi: Correct misnamed function parameter 'ptr_'

2020-11-07 Thread Kalle Valo
Lee Jones wrote: > Fixes the following W=1 kernel build warning(s): > > drivers/net/wireless/ath/wil6210/wmi.c:279: warning: Function parameter or > member 'ptr_' not described in 'wmi_buffer_block' > drivers/net/wireless/ath/wil6210/wmi.c:279: warning: Excess function > parameter 'ptr'

[PATCH] bpf: remove duplicate include

2020-11-07 Thread Wang Qing
Remove duplicate header which is included twice. Signed-off-by: Wang Qing --- kernel/bpf/btf.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/bpf/btf.c b/kernel/bpf/btf.c index ed7d02e..6324de8 --- a/kernel/bpf/btf.c +++ b/kernel/bpf/btf.c @@ -22,7 +22,6 @@ #include #include

[PATCH] scsi: core: fix -Wformat

2020-11-07 Thread Nick Desaulniers
Clang is more aggressive about -Wformat warnings when the format flag specifies a type smaller than the parameter. Turns out, struct Scsi_Host's member can_queue is actually an int. Fixes: warning: format specifies type 'short' but the argument has type 'int' [-Wformat] shost_rd_attr(can_queue,

[PATCH] firmware: fix spelling typo of 'wtih'

2020-11-07 Thread Wang Qing
wtih -> with Signed-off-by: Wang Qing --- drivers/firmware/raspberrypi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/firmware/raspberrypi.c b/drivers/firmware/raspberrypi.c index 2371d08..30259dc --- a/drivers/firmware/raspberrypi.c +++

Re: [Linux-kernel-mentees] [PATCH v2 net] rose: Fix Null pointer dereference in rose_send_frame()

2020-11-07 Thread Anmol Karn
Hello Sir, On Fri, Nov 06, 2020 at 01:04:27PM -0800, Saeed Mahameed wrote: > On Thu, 2020-11-05 at 21:26 +0530, Anmol Karn wrote: > > rose_send_frame() dereferences `neigh->dev` when called from > > rose_transmit_clear_request(), and the first occurance of the `neigh` > > is in

Re: WARNING in input_register_device

2020-11-07 Thread Greg KH
On Fri, Nov 06, 2020 at 09:03:14AM -0800, Eric Biggers wrote: > On Fri, Nov 06, 2020 at 03:03:36PM +0100, Greg KH wrote: > > On Fri, Nov 06, 2020 at 04:43:17AM -0800, syzbot wrote: > > > Hello, > > > > > > syzbot found the following issue on: > > > > > > HEAD commit:9e39aef3 usb: misc:

[PATCH] locking: remove duplicate include of percpu-rwsem.h

2020-11-07 Thread Wang Qing
Remove duplicate header include which is unnecessary. Signed-off-by: Wang Qing --- kernel/locking/locktorture.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/locking/locktorture.c b/kernel/locking/locktorture.c index 62d215b..af99e9c 100644 --- a/kernel/locking/locktorture.c +++

[PATCH 1/2] tomoyo: Convert get_user_pages*() to pin_user_pages*()

2020-11-07 Thread Souptick Joarder
In 2019, we introduced pin_user_pages*() and now we are converting get_user_pages*() to the new API as appropriate. [1] & [2] could be referred for more information. This is case 5 as per document [1]. [1] Documentation/core-api/pin_user_pages.rst [2] "Explicit pinning of user-space pages":

[PATCH 2/2] tomoyo: Fixed typo in documentation

2020-11-07 Thread Souptick Joarder
Fixed typo s/Poiner/Pointer Fixes: 5b636857fee6 ("TOMOYO: Allow using argv[]/envp[] of execve() as conditions.") Signed-off-by: Souptick Joarder Cc: John Hubbard --- security/tomoyo/domain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/tomoyo/domain.c

[PATCH v1] i2c: nvidia-gpu: drop empty stub for runtime pm

2020-11-07 Thread Vaibhav Gupta
After the commit c5eb1190074c ("PCI / PM: Allow runtime PM without callback functions") we no more need empty stubs for runtime-pm to work. The driver has no device specific task(s) for .suspend() . The stub was placed just for runtime-pm, which can be dropped now. Reported-by: Bjorn Helgaas

[PATCH] sched: remove duplicate include unnecessary

2020-11-07 Thread Wang Qing
Remove duplicate header include which is unnecessary. Signed-off-by: Wang Qing --- kernel/sched/sched.h | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h index df80bfc..dd91a8b --- a/kernel/sched/sched.h +++ b/kernel/sched/sched.h @@ -351,7 +351,6 @@

Re: [PATCH v2] Make iwmmxt.S support Clang's integrated assembler

2020-11-07 Thread Ard Biesheuvel
On Sat, 7 Nov 2020 at 01:11, Jian Cai wrote: > > This patch replaces 6 IWMMXT instructions Clang's integrated assembler > does not support in iwmmxt.S using macros, while making sure GNU > assembler still emit the same instructions. This should be easier than > providing full IWMMXT support in

[PATCH] ACPI: GED: fix -Wformat

2020-11-07 Thread Nick Desaulniers
Clang is more aggressive about -Wformat warnings when the format flag specifies a type smaller than the parameter. It turns out that gsi is an int. Fixes: drivers/acpi/evged.c:105:48: warning: format specifies type 'unsigned char' but the argument has type 'unsigned int' [-Wformat] trigger ==

Re: [PATCH v2 0/6] platform/chrome: cros_ec_typec: Add cable

2020-11-07 Thread Greg KH
On Fri, Nov 06, 2020 at 10:40:57AM -0800, Prashant Malani wrote: > The following series adds Type C cable registration to the cros-ec-typec > port driver using the Type C connector class framework. The first few > patches perform a few minor re-organizations to prepare for the cable > registration

Re: [PATCH] ACPI: GED: fix -Wformat

2020-11-07 Thread Ard Biesheuvel
On Sat, 7 Nov 2020 at 09:34, Nick Desaulniers wrote: > > Clang is more aggressive about -Wformat warnings when the format flag > specifies a type smaller than the parameter. It turns out that gsi is an > int. Fixes: > > drivers/acpi/evged.c:105:48: warning: format specifies type 'unsigned > char'

Re: [PATCH] Revert "mm/vunmap: add cond_resched() in vunmap_pmd_range"

2020-11-07 Thread Minchan Kim
Hi Andrew, On Fri, Nov 06, 2020 at 05:59:33PM -0800, Andrew Morton wrote: > On Thu, 5 Nov 2020 09:02:49 -0800 Minchan Kim wrote: > > > This reverts commit e47110e90584a22e9980510b00d0dfad3a83354e. > > > > While I was doing zram testing, I found sometimes decompression failed > > since the

[PATCH] wireless: realtek: fix spelling typo of workaround

2020-11-07 Thread Wang Qing
workarould -> workaround Signed-off-by: Wang Qing --- drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c index

[PATCH v2] Revert "kbuild: Do not enable -Wimplicit-fallthrough for clang for now"

2020-11-07 Thread Nick Desaulniers
This reverts commit e2079e93f562c7f7a030eb7642017ee5eabaaa10. This has been fixed up over time thanks to the addition of "fallthrough" pseudo-keyword in commit 294f69e662d1 ("compiler_attributes.h: Add 'fallthrough' pseudo keyword for switch/case use") Link:

Re: [PATCH v4 0/2] iommu/tegra-smmu: Two followup changes

2020-11-07 Thread Nicolin Chen
On Mon, Sep 28, 2020 at 11:13:23PM -0700, Nicolin Chen wrote: > Two followup patches for tegra-smmu: > PATCH-1 is a clean-up patch for the recently applied SWGROUP change. > PATCH-2 fixes a potential race condition These two changes have Acked-by and Reviewed-by for a month already. Who can apply

[PATCH v2] ACPI: GED: fix -Wformat

2020-11-07 Thread Nick Desaulniers
Clang is more aggressive about -Wformat warnings when the format flag specifies a type smaller than the parameter. It turns out that gsi is an int. Fixes: drivers/acpi/evged.c:105:48: warning: format specifies type 'unsigned char' but the argument has type 'unsigned int' [-Wformat] trigger ==

Re: [PATCH v7 0/3] iommu/tegra-smmu: Add PCI support

2020-11-07 Thread Nicolin Chen
On Fri, Oct 09, 2020 at 09:19:33AM -0700, Nicolin Chen wrote: > This series is to add PCI support in tegra-smmu driver. This v7 has fixed all the existing comments and been in the maillist for nearly a month. Can anyone please apply them? Thanks Nic > Changelog (Detail in each patch) > v6->v7 >

[PATCH V3] doc: zh_CN: add translatation for tmpfs

2020-11-07 Thread Wang Qing
Translate Documentation/filesystems/tmpfs.rst into Chinese. Signed-off-by: Wang Qing Changes in v3: - Fix patch format issue. --- .../translations/zh_CN/filesystems/tmpfs.rst | 146 + 1 file changed, 146 insertions(+) create mode 100644

Re: [PATCH 2/2] tomoyo: Fixed typo in documentation

2020-11-07 Thread John Hubbard
On 11/7/20 12:24 AM, Souptick Joarder wrote: Fixed typo s/Poiner/Pointer Fixes: 5b636857fee6 ("TOMOYO: Allow using argv[]/envp[] of execve() as conditions.") Signed-off-by: Souptick Joarder Cc: John Hubbard --- security/tomoyo/domain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Im rechtlichen Kontext

2020-11-07 Thread Wendy
Ich möchte, dass Sie sich mit mir bei der Durchführung eines Geschäftsprojekts zusammenschließen, das einen angemessenen Betrag an Investitionskapital in Höhe von 52 Millionen fünfhunderttausend US-Dollar umfasst. Da die Zeit drängt, ist es unbedingt erforderlich, dass Sie sich zur weiteren

Re: [PATCH 1/2] tomoyo: Convert get_user_pages*() to pin_user_pages*()

2020-11-07 Thread John Hubbard
On 11/7/20 12:24 AM, Souptick Joarder wrote: In 2019, we introduced pin_user_pages*() and now we are converting get_user_pages*() to the new API as appropriate. [1] & [2] could be referred for more information. This is case 5 as per document [1]. It turns out that Case 5 can be implemented via

Re: [PATCH v2] Revert "kbuild: Do not enable -Wimplicit-fallthrough for clang for now"

2020-11-07 Thread Nick Desaulniers
On Sat, Nov 7, 2020 at 12:45 AM Nick Desaulniers wrote: > > This reverts commit e2079e93f562c7f7a030eb7642017ee5eabaaa10. > > This has been fixed up over time thanks to the addition of "fallthrough" > pseudo-keyword in > commit 294f69e662d1 ("compiler_attributes.h: Add 'fallthrough' pseudo >

[PATCH V3] fsl/fman: add missing put_devcie() call in fman_port_probe()

2020-11-07 Thread Yu Kuai
if of_find_device_by_node() succeed, fman_port_probe() doesn't have a corresponding put_device(). Thus add jump target to fix the exception handling for this function implementation. Fixes: 0572054617f3 ("fsl/fman: fix dereference null return value") Signed-off-by: Yu Kuai --- Changes in V3: -

Re: [PATCH v1] i2c: nvidia-gpu: drop empty stub for runtime pm

2020-11-07 Thread Vaibhav Gupta
On Sat, Nov 07, 2020 at 01:51:51PM +0530, Vaibhav Gupta wrote: > After the commit c5eb1190074c ("PCI / PM: Allow runtime PM without callback > functions") we no more need empty stubs for runtime-pm to work. > > The driver has no device specific task(s) for .suspend() . The stub was > placed just

[PATCH] powerpc/32s: Use relocation offset when setting early hash table

2020-11-07 Thread Christophe Leroy
When calling early_hash_table(), the kernel hasn't been yet relocated to its linking address, so data must be addressed with relocation offset. Add relocation offset to write into Hash in early_hash_table(). Reported-by: Erhard Furtner Reported-by: Andreas Schwab Fixes: 69a1593abdbc

[i2c-next,PATCH] i2c: medaitek: Move suspend and resume handling to NOIRQ phase

2020-11-07 Thread qii.wang
From: Qii Wang Some i2c device driver indirectly uses I2C driver when it is now being suspended. The i2c devices driver is suspended during the NOIRQ phase and this cannot be changed due to other dependencies. Therefore, we also need to move the suspend handling for the I2C controller driver to

Re: [PATCH] powerpc/32s: Setup the early hash table at all time.

2020-11-07 Thread Christophe Leroy
Le 29/10/2020 à 22:07, Andreas Schwab a écrit : On Okt 01 2020, Christophe Leroy wrote: At the time being, an early hash table is set up when CONFIG_KASAN is selected. There is nothing wrong with setting such an early hash table all the time, even if it is not used. This is a statically

[PATCH] arm: mm: remove duplicate include

2020-11-07 Thread Wang Qing
Remove duplicate header which is included twice. Signed-off-by: Wang Qing --- arch/arm/mm/mmu.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c index ab69250..4963e1c --- a/arch/arm/mm/mmu.c +++ b/arch/arm/mm/mmu.c @@ -33,7 +33,6 @@ #include #include

Re: [PATCH v22 12/23] LSM: Specify which LSM to display

2020-11-07 Thread Greg KH
On Fri, Nov 06, 2020 at 04:20:43PM -0800, Casey Schaufler wrote: > On 11/5/2020 1:22 AM, Greg KH wrote: > > On Wed, Nov 04, 2020 at 03:41:03PM -0800, Casey Schaufler wrote: > >> Create a new entry "display" in the procfs attr directory for > >> controlling which LSM security information is

[PATCH] locking/lock_events: no need to check return value of debugfs_create functions

2020-11-07 Thread Tiezhu Yang
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Signed-off-by: Tiezhu Yang --- kernel/locking/lock_events.c | 19 --- 1 file changed, 4

[PATCH] tool: selftests: fix spelling typo of 'writting'

2020-11-07 Thread Wang Qing
writting -> writing Signed-off-by: Wang Qing --- tools/testing/selftests/vm/userfaultfd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/testing/selftests/vm/userfaultfd.c b/tools/testing/selftests/vm/userfaultfd.c index 9b0912a..9132fae7 ---

Re: [PATCH] driver core: export device_is_bound() to fix build failure

2020-11-07 Thread Greg Kroah-Hartman
On Fri, Nov 06, 2020 at 03:37:44PM +, Sudip Mukherjee wrote: > When CONFIG_MXC_CLK_SCU is configured as 'm' the build fails as it > is unable to find device_is_bound(). The error being: > ERROR: modpost: "device_is_bound" [drivers/clk/imx/clk-imx-scu.ko] > undefined! > > Export the

[PATCH] arm: mach-sa1100: remove duplicate include

2020-11-07 Thread Wang Qing
Remove duplicate header which is included twice. Signed-off-by: Wang Qing --- 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 3085f1c..3fe34ee --- a/arch/arm/mach-sa1100/hackkit.c +++

Re: [f2fs-dev] [PATCH v4 2/2] f2fs: fix compat F2FS_IOC_{MOVE, GARBAGE_COLLECT}_RANGE

2020-11-07 Thread Chao Yu
On 2020/11/7 2:03, Eric Biggers wrote: On Fri, Nov 06, 2020 at 02:53:31PM +0800, Chao Yu wrote: +#if defined(__KERNEL__) +struct compat_f2fs_gc_range { + u32 sync; + compat_u64 start; + compat_u64 len; +}; There's no need to use '#if defined(__KERNEL__)' in kernel source

[PATCH v8 1/3] USB: PHY: JZ4770: Remove unnecessary function calls.

2020-11-07 Thread Zhou Yanjie
Remove unnecessary "of_match_ptr()", because Ingenic SoCs all depend on Device Tree. Suggested-by: Paul Cercueil Signed-off-by: 周琰杰 (Zhou Yanjie) Reviewed-by: Paul Cercueil --- Notes: v3: New patch. v3->v4: No change. v4->v5: Add Paul Cercueil's Reviewed-by.

[PATCH v8 2/3] dt-bindings: USB: Add bindings for Ingenic JZ4775 and X2000.

2020-11-07 Thread Zhou Yanjie
Move Ingenic USB PHY bindings from Documentation/devicetree/bindings/usb to Documentation/devicetree/bindings/phy, and add bindings for JZ4775 SoC and X2000 SoC. Signed-off-by: 周琰杰 (Zhou Yanjie) --- Notes: v8: New patch. .../{usb/ingenic,jz4770-phy.yaml => phy/ingenic,phy-usb.yaml}

[PATCH v8 3/3] PHY: Ingenic: Add USB PHY driver using generic PHY framework.

2020-11-07 Thread Zhou Yanjie
Used the generic PHY framework API to create the PHY, this driver supoorts USB OTG PHY used in JZ4770 SoC, JZ4775 SoC, JZ4780 SoC, X1000 SoC, X1830 SoC and X2000 SoC. Tested-by: 周正 (Zhou Zheng) Tested-by: H. Nikolaus Schaller Co-developed-by: 漆鹏振 (Qi Pengzhen) Signed-off-by: 漆鹏振 (Qi Pengzhen)

[PATCH v8 0/3] Use the generic PHY framework for Ingenic USB PHY.

2020-11-07 Thread Zhou Yanjie
v3->v4: Only add new generic-PHY driver, without removing the old one. Because the jz4740-musb driver is not ready to use the generic PHY framework. When the jz4740-musb driver is modified to use the generic PHY framework, the old jz4770-phy driver can be "retired". v4->v5: 1.Add an extra blank

Re: [PATCH] powerpc/32s: Use relocation offset when setting early hash table

2020-11-07 Thread Serge Belyshev
Christophe Leroy writes: > When calling early_hash_table(), the kernel hasn't been yet > relocated to its linking address, so data must be addressed > with relocation offset. > > Add relocation offset to write into Hash in early_hash_table(). > > Reported-by: Erhard Furtner > Reported-by:

[PATCH] dma-pool: no need to check return value of debugfs_create functions

2020-11-07 Thread Tiezhu Yang
When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different based on this. Signed-off-by: Tiezhu Yang --- kernel/dma/pool.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH v1] scsi: isci: Don't use PCI helper functions

2020-11-07 Thread Vaibhav Gupta
PCI helper functions such as pci_enable/disable_device(), pci_save/restore_state(), pci_set_power_state(), etc. were used by the legacy framework to perform standard operations related to PCI PM. This driver is using the generic framework and thus calls for those functions should be dropped as

Re: [PATCH v6 1/2] kunit: Support for Parameterized Testing

2020-11-07 Thread Marco Elver
On Sat, 7 Nov 2020 at 05:58, David Gow wrote: > On Sat, Nov 7, 2020 at 3:22 AM Arpitha Raghunandan <98.a...@gmail.com> wrote: > > > > Implementation of support for parameterized testing in KUnit. > > This approach requires the creation of a test case using the > > KUNIT_CASE_PARAM macro that

Re: [PATCH v1] scsi: isci: Don't use PCI helper functions

2020-11-07 Thread Vaibhav Gupta
On Sat, Nov 07, 2020 at 03:34:19PM +0530, Vaibhav Gupta wrote: > PCI helper functions such as pci_enable/disable_device(), > pci_save/restore_state(), pci_set_power_state(), etc. were used by the > legacy framework to perform standard operations related to PCI PM. > > This driver is using the

[tip: x86/urgent] x86/platform/uv: Recognize UV5 hubless system identifier

2020-11-07 Thread tip-bot2 for Mike Travis
The following commit has been merged into the x86/urgent branch of tip: Commit-ID: 801284f9737883a2b2639bd494455a72c82fdedf Gitweb: https://git.kernel.org/tip/801284f9737883a2b2639bd494455a72c82fdedf Author:Mike Travis AuthorDate:Thu, 05 Nov 2020 16:27:41 -06:00

[tip: x86/urgent] x86/platform/uv: Fix missing OEM_TABLE_ID

2020-11-07 Thread tip-bot2 for Mike Travis
The following commit has been merged into the x86/urgent branch of tip: Commit-ID: 1aec69ae56be28b5fd3c9daead5f3840c30153c8 Gitweb: https://git.kernel.org/tip/1aec69ae56be28b5fd3c9daead5f3840c30153c8 Author:Mike Travis AuthorDate:Thu, 05 Nov 2020 16:27:39 -06:00

[tip: x86/urgent] x86/platform/uv: Remove spaces from OEM IDs

2020-11-07 Thread tip-bot2 for Mike Travis
The following commit has been merged into the x86/urgent branch of tip: Commit-ID: 1aee505e0171fc38fd5ed70c7f0dcbb7398c759f Gitweb: https://git.kernel.org/tip/1aee505e0171fc38fd5ed70c7f0dcbb7398c759f Author:Mike Travis AuthorDate:Thu, 05 Nov 2020 16:27:40 -06:00

Re: [PATCH 2/2] arm: lib: xor-neon: disable clang vectorization

2020-11-07 Thread Russell King - ARM Linux admin
On Fri, Nov 06, 2020 at 07:14:36AM +0200, Adrian Ratiu wrote: > diff --git a/arch/arm/lib/xor-neon.c b/arch/arm/lib/xor-neon.c > index e1e76186ec23..84c91c48dfa2 100644 > --- a/arch/arm/lib/xor-neon.c > +++ b/arch/arm/lib/xor-neon.c > @@ -18,6 +18,10 @@ MODULE_LICENSE("GPL"); > * Pull in the

Re: [PATCH] netfilter: conntrack: fix -Wformat

2020-11-07 Thread Joe Perches
On Fri, 2020-11-06 at 23:55 -0800, Nick Desaulniers wrote: > Clang is more aggressive about -Wformat warnings when the format flag > specifies a type smaller than the parameter. Fixes 8 instances of: > > warning: format specifies type 'unsigned short' but the argument has > type 'int' [-Wformat]

Re: [PATCH] arm: mm: remove duplicate include

2020-11-07 Thread Mike Rapoport
On Sat, Nov 07, 2020 at 05:15:50PM +0800, Wang Qing wrote: > Remove duplicate header which is included twice. > > Signed-off-by: Wang Qing Reviewed-by: Mike Rapoport > --- > arch/arm/mm/mmu.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c >

Re: [PATCH] arm: mach-sa1100: remove duplicate include

2020-11-07 Thread Mike Rapoport
On Sat, Nov 07, 2020 at 05:22:26PM +0800, Wang Qing wrote: > Remove duplicate header which is included twice. > > Signed-off-by: Wang Qing Reviewed-by: Mike Rapoport > --- > arch/arm/mach-sa1100/hackkit.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm/mach-sa1100/hackkit.c

[PATCH -next] irq-chip/gic-v3-its: Fixed an issue where the ITS executes the residual commands in the queue again when the ITS wakes up from sleep mode.

2020-11-07 Thread Xu Qiang
On my platform, ITS_FLAGS_SAVE_SUSPEND_STATE is not set,thus do nothing in its suspend and resuse function.On the other hand,firmware stores GITS_CTRL,GITS_CBASER,GITS_CWRITER and GITS_BASER in the suspend, and restores these registers in the resume. As a result, the ITS executes the residual

Re: [PATCH] tool: selftests: fix spelling typo of 'writting'

2020-11-07 Thread Mike Rapoport
On Sat, Nov 07, 2020 at 05:19:35PM +0800, Wang Qing wrote: > writting -> writing > > Signed-off-by: Wang Qing Reviewed-by: Mike Rapoport > --- > tools/testing/selftests/vm/userfaultfd.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git

RE: [PATCH] libbpf: Remove unnecessary conversion to bool

2020-11-07 Thread David Laight
From: Joe Perches > Sent: 06 November 2020 21:50 > > On Fri, 2020-11-06 at 13:32 -0800, Andrii Nakryiko wrote: > > On Thu, Nov 5, 2020 at 11:12 PM wrote: > > > Fix following warning from coccinelle: > > > ./tools/lib/bpf/libbpf.c:1478:43-48: WARNING: conversion to bool not > > > needed here >

Re: [PATCH] firmware: fix spelling typo of 'wtih'

2020-11-07 Thread Nicolas Saenz Julienne
On Sat, 2020-11-07 at 16:19 +0800, Wang Qing wrote: > wtih -> with > > Signed-off-by: Wang Qing > --- Acked-by: Nicolas Saenz Julienne Thanks! > drivers/firmware/raspberrypi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/firmware/raspberrypi.c

Re: [PATCH] MIPS: reserve the memblock right after the kernel

2020-11-07 Thread Thomas Bogendoerfer
On Fri, Nov 06, 2020 at 03:10:01PM +0100, Alexander A Sverdlin wrote: > From: Alexander Sverdlin > > Linux doesn't own the memory immediately after the kernel image. On Octeon > bootloader places a shared structure right close after the kernel _end, > refer to "struct cvmx_bootinfo

Re: [PATCH v3 0/4] drm/bridge: ti-sn65dsi86: Support EDID reading

2020-11-07 Thread Sam Ravnborg
Hi Stephen On Mon, Nov 02, 2020 at 10:11:40AM -0800, Stephen Boyd wrote: > This patch series cleans up the DDC code a little bit so that > it is more efficient time wise and supports grabbing the EDID > of the eDP panel over the aux channel. I timed this on a board > I have on my desk and it

Re: [PATCH] Kbuild: enable -Wfallthrough for clang

2020-11-07 Thread Miguel Ojeda
On Sat, Nov 7, 2020 at 8:08 AM Nick Desaulniers wrote: > > Partial revert of commit e2079e93f562 ("kbuild: Do not enable > -Wimplicit-fallthrough for clang for now") Wait, it says partial revert because it is one, but doing it this way does not enable the option back for GCC (and Clang).

Re: [PATCH net-next 08/11] ath9k: work around false-positive gcc warning

2020-11-07 Thread Kalle Valo
Johannes Berg writes: > On Mon, 2020-11-02 at 18:26 +0200, Kalle Valo wrote: >> Arnd Bergmann writes: >> >> > From: Arnd Bergmann >> > >> > gcc-10 shows a false-positive warning with CONFIG_KASAN: >> > >> > drivers/net/wireless/ath/ath9k/dynack.c: In function >> >

Re: [PATCH 2/2] ath11k: Handle errors if peer creation fails

2020-11-07 Thread Kalle Valo
Alex Dewar writes: > On Tue, Oct 06, 2020 at 10:26:28AM +0300, Kalle Valo wrote: >> Alex Dewar writes: >> >> > ath11k_peer_create() is called without its return value being checked, >> > meaning errors will be unhandled. Add missing check and, as the mutex is >> > unconditionally unlocked on

[PATCH] panic: don't dump stack twice on warn

2020-11-07 Thread Christophe Leroy
Before commit 3f388f28639f ("panic: dump registers on panic_on_warn"), __warn() was calling show_regs() when regs was not NULL, and show_stack() otherwise. After that commit, show_stack() is called regardless of whether show_regs() has been called or not, leading to duplicated Call Trace: [

Re: [PATCH][next] ray_cs: Use fallthrough pseudo-keyword

2020-11-07 Thread Kalle Valo
"Gustavo A. R. Silva" wrote: > In order to enable -Wimplicit-fallthrough for Clang[1], replace the > existing /* fall through */ comments with the new pseudo-keyword > macro fallthrough[2]. > > [1] https://git.kernel.org/linus/e2079e93f562c7f7a030eb7642017ee5eabaaa10 > [2] >

Re: [PATCH][next] wlcore: Use fallthrough pseudo-keyword

2020-11-07 Thread Kalle Valo
"Gustavo A. R. Silva" wrote: > In order to enable -Wimplicit-fallthrough for Clang[1], replace the > existing /* fall-through */ comments with the new pseudo-keyword > macro fallthrough[2]. > > [1] https://git.kernel.org/linus/e2079e93f562c7f7a030eb7642017ee5eabaaa10 > [2] >

Re: [PATCH v2] wireless: remove unneeded break

2020-11-07 Thread Kalle Valo
t...@redhat.com wrote: > From: Tom Rix > > A break is not needed if it is preceded by a return > > Signed-off-by: Tom Rix Patch applied to wireless-drivers-next.git, thanks. 3287953b0399 wireless: remove unneeded break --

Re: [PATCH net-next 08/11] ath9k: work around false-positive gcc warning

2020-11-07 Thread Arnd Bergmann
On Sat, Nov 7, 2020 at 12:21 PM Kalle Valo wrote: > Johannes Berg writes: > > On Mon, 2020-11-02 at 18:26 +0200, Kalle Valo wrote: > >> Arnd Bergmann writes: > >> Isn't there a better way to handle this? I really would not want > >> checking for GCC versions become a common approach in drivers.

Re: [PATCH net-next 02/11] net: hostap: fix function cast warning

2020-11-07 Thread Kalle Valo
Arnd Bergmann wrote: > From: Arnd Bergmann > > gcc -Wextra complains about the function type cast: > > drivers/net/wireless/intersil/hostap/hostap_hw.c:3173:48: warning: cast > between incompatible function types from ‘void (*)(struct tasklet_struct *)’ > to ‘void (*)(long unsigned int)’

Re: [PATCH] rtl8192ce: avoid accessing the data mapped to streaming DMA

2020-11-07 Thread Kalle Valo
Jia-Ju Bai wrote: > In rtl92ce_tx_fill_cmddesc(), skb->data is mapped to streaming DMA on > line 530: > dma_addr_t mapping = dma_map_single(..., skb->data, ...); > > On line 533, skb->data is assigned to hdr after cast: > struct ieee80211_hdr *hdr = (struct ieee80211_hdr *)(skb->data); > >

[PATCH 0/2] Add missing nodes and refresh defconfig for Ingenic SoCs based boards.

2020-11-07 Thread Zhou Yanjie
1.Add missing nodes for Ingenic SoCs and Ingenic SoCs based boards. 2.Refresh defconfig for Ingenic SoCs based boards. 周琰杰 (Zhou Yanjie) (2): MIPS: Ingenic: Add missing nodes for Ingenic SoCs and boards. MIPS: Ingenic: Refresh defconfig for Ingenic SoCs based boards.

[PATCH 2/2] MIPS: Ingenic: Refresh defconfig for Ingenic SoCs based boards.

2020-11-07 Thread Zhou Yanjie
1.Refresh defconfig of CI20 to support OTG and RNG. 2.Refresh defconfig of CU1000-Neo to support OTG/RNG/OST/SC16IS752. 3.Refresh defconfig of CU1830-Neo to support OTG/DTRNG/OST/SC16IS752. Tested-by: 周正 (Zhou Zheng) Signed-off-by: 周琰杰 (Zhou Yanjie) --- arch/mips/configs/ci20_defconfig |

[PATCH 1/2] MIPS: Ingenic: Add missing nodes for Ingenic SoCs and boards.

2020-11-07 Thread Zhou Yanjie
1.Add OTG/OTG PHY/RNG nodes for JZ4780, CGU/OTG nodes for CI20. 2.Add OTG/OTG PHY/RNG/OST nodes for X1000, SSI/CGU/OST/OTG/SC16IS752 nodes for CU1000-Neo. 3.Add OTG/OTG PHY/DTRNG/OST nodes for X1830, SSI/CGU/OST/OTG/SC16IS752 nodes for CU1830-Neo. Tested-by: 周正 (Zhou Zheng) Signed-off-by:

[PATCH V4 3/5] arm64: dts: imx8mn: Add SAI nodes

2020-11-07 Thread Adam Ford
The i.MX8M Nano has several SAI nodes available to it. Enable them. Signed-off-by: Adam Ford --- V4: No Change V3: No Change V2: No Change arch/arm64/boot/dts/freescale/imx8mn.dtsi | 72 +++ 1 file changed, 72 insertions(+) diff --git

[PATCH V4 1/5] dt-bindings: soc: imx: Add binding doc for spba bus

2020-11-07 Thread Adam Ford
Add binding doc for fsl,spba-bus. Signed-off-by: Adam Ford --- V4: Correct errors in YAML V3: New to series .../devicetree/bindings/bus/fsl,spba-bus.yaml | 66 +++ 1 file changed, 66 insertions(+) diff --git a/Documentation/devicetree/bindings/bus/fsl,spba-bus.yaml

[PATCH V4 2/5] arm64: dts: imx8mn: Enable Asynchronous Sample Rate Converter

2020-11-07 Thread Adam Ford
The driver exists for the Enhanced Asynchronous Sample Rate Converter (EASRC) Controller, but there isn't a device tree entry for it. On the vendor kernel, they put this on a spba-bus for SDMA support. Add the node for the spba-bus with the easrc node inside. Signed-off-by: Adam Ford --- V4:

[PATCH V4 5/5] arm64: dts: imx8mn: Add node for SPDIF

2020-11-07 Thread Adam Ford
The i.MX8M Nano can support SPDIF which is compatible to the IP used on the i.MX35. Add the node. Signed-off-by: Adam Ford --- V4: No Change V3: No Change V2: No Change arch/arm64/boot/dts/freescale/imx8mn.dtsi | 24 +++ 1 file changed, 24 insertions(+) diff --git

[PATCH V4 4/5] arm64: dts: imx8mn: Add support for micfil

2020-11-07 Thread Adam Ford
The i.MX8M Nano has supports the MICFIL digital interface. It's a 16-bit audio signal from a PDM microphone bitstream. The driver is already in the kernel, but the node is missing. Add the micfil node. Signed-off-by: Adam Ford --- V4: No Change V3: No Change V2: Change micfil@3008 to

Re: possible deadlock in mnt_want_write

2020-11-07 Thread syzbot
syzbot suspects this issue was fixed by commit: commit 146d62e5a5867fbf84490d82455718bfb10fe824 Author: Amir Goldstein Date: Thu Apr 18 14:42:08 2019 + ovl: detect overlapping layers bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=11e4018450 start commit: 6d906f99

Re: [PATCH 01/19] drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header

2020-11-07 Thread Christian König
Am 06.11.20 um 22:49 schrieb Lee Jones: Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/ttm/ttm_range_manager.c:46: warning: cannot understand function prototype: 'struct ttm_range_manager ' Cc: Christian Koenig Cc: Huang Rui Cc: David Airlie Cc: Daniel Vetter Cc:

[PATCH v2] clk: exynos7: Keep aclk_fsys1_200 enabled

2020-11-07 Thread Paweł Chmiel
This clock must be always enabled to allow access to any registers in fsys1 CMU. Until proper solution based on runtime PM is applied (similar to what was done for Exynos5433), fix this by calling clk_prepare_enable() directly from clock provider driver. It was observed on Samsung Galaxy S6

Re: [PATCH 00/19] [Set 2] Rid W=1 warnings from GPU

2020-11-07 Thread Christian König
Well that's quite a patch set. First of all can you separate this a bit more by driver? I'm assuming we maintainers are supposed to pick that up and apply it. radeon and amdgpu can stick together since that is mostly Alex and me, but I'm not sure if we want to do some of the suggested

[PATCH RESEND 0/2] Add dmaengine bindings for the JZ4775 and the X2000 SoCs.

2020-11-07 Thread Zhou Yanjie
Add the dmaengine bindings for the JZ4775 SoC and the X2000 SoC from Ingenic. 周琰杰 (Zhou Yanjie) (2): dt-bindings: dmaengine: Add JZ4775 bindings. dt-bindings: dmaengine: Add X2000 bindings. include/dt-bindings/dma/jz4775-dma.h | 44 +

[PATCH RESEND 1/2] dt-bindings: dmaengine: Add JZ4775 bindings.

2020-11-07 Thread Zhou Yanjie
Add the dmaengine bindings for the JZ4775 SoC from Ingenic. Signed-off-by: 周琰杰 (Zhou Yanjie) Acked-by: Rob Herring --- include/dt-bindings/dma/jz4775-dma.h | 44 1 file changed, 44 insertions(+) create mode 100644 include/dt-bindings/dma/jz4775-dma.h diff

[PATCH RESEND 2/2] dt-bindings: dmaengine: Add X2000 bindings.

2020-11-07 Thread Zhou Yanjie
Add the dmaengine bindings for the X2000 SoC from Ingenic. Signed-off-by: 周琰杰 (Zhou Yanjie) Acked-by: Rob Herring --- include/dt-bindings/dma/x2000-dma.h | 54 + 1 file changed, 54 insertions(+) create mode 100644 include/dt-bindings/dma/x2000-dma.h diff

About the rockpi4 pcie controller failed to initialize problem in v5.10-rc2

2020-11-07 Thread Qu Wenruo
Hi guys, I see your awesome contribution to support Rock Pi 4B. However in recent rc (v5.10-rc2), I found that even with `vpcie1v8` and `vpcie0v9` added, `regulartor_dev_lookup()` now just returns -EPROBE_DEFER, preventing rockchip pcie controller to be initialized. The full callchain is:

[PATCH V3 1/3] dt-bindings: arm: fsl: Add beacon,imx8mn-beacon-kit

2020-11-07 Thread Adam Ford
Add beacon,imx8mn-beacon-kit to list of compatible options. Signed-off-by: Adam Ford --- V3: Correct Typo and move to Nano section V2: New to series diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml index 85fb24da4a02..5a2608e6bc30

[PATCH V3 2/3] arm64: dts: imx: Add Beacon i.MX8M Nano development kit

2020-11-07 Thread Adam Ford
Beacon Embeddedworks is launching a development kit based on the i.MX8M Nano SoC. The kit consists of a System on Module (SOM) + baseboard. The SOM has the SoC, eMMC, and Ethernet. The baseboard has an wm8962 audio CODEC, a PDM microphone, and a single USB OTG. The baseboard is capable of two

[PATCH V3 3/3] arm64: defconfig: Enable WM8962

2020-11-07 Thread Adam Ford
The Beacon EmbeddedWorks development kits supporting i.MX8M Mini and Nano have an WM8962 audio CODEC installed. Add modules for both CONFIG_SND_SOC_WM8962 and CONFIG_SND_SOC_FSL_ASOC_CARD to enable them. Signed-off-by: Adam Ford --- V3: No Change V2: New to series diff --git

[PATCH 0/3] Switch PineTab DT LCD panel to retail one

2020-11-07 Thread Icenowy Zheng
Retail PineTabs switched to K101-IM2BYL02 panel. This patchset tries to reflect this change, and add a DT for samples that still have K101-IM2BA02. Icenowy Zheng (3): arm64: allwinner: dts: a64: pinetab: switch LCD panel to production one dt-bindings: arm: sunxi: add PineTab developer

[PATCH 1/3] arm64: allwinner: dts: a64: pinetab: switch LCD panel to production one

2020-11-07 Thread Icenowy Zheng
All retail PineTabs use the new panel. Devices with the old panel are only available to several developers as sample. Switch the main PineTab DT to use the new panel, as it should reflect what the retail device uses. Another DT for developers' sample will be added later. Signed-off-by: Icenowy

[PATCH 2/3] dt-bindings: arm: sunxi: add PineTab developer sample DT binding

2020-11-07 Thread Icenowy Zheng
Some developer samples of PineTab are distributed with the old and incompatible LCD panel. Add a device tree binding for this version of PineTab. Signed-off-by: Icenowy Zheng --- Documentation/devicetree/bindings/arm/sunxi.yaml | 5 + 1 file changed, 5 insertions(+) diff --git

Re: [PATCH v2 2/8] firmware: arm_scmi: introduce protocol handles

2020-11-07 Thread Cristian Marussi
Hi Thara thanks for the feedback. On Fri, Nov 06, 2020 at 11:26:44AM -0500, Thara Gopinath wrote: > > > On 11/4/20 12:44 PM, Cristian Marussi wrote: > > Hi > > > > On Wed, Nov 04, 2020 at 11:16:18AM -0500, Thara Gopinath wrote: > > > > > > Hi Cristian, > > > > > > On 10/28/20 4:29 PM,

[PATCH 3/3] arm64: allwinner: dts: a64: add DT for PineTab developer sample

2020-11-07 Thread Icenowy Zheng
Some developers received PineTab samples that used an old LCD panel. Add device tree for these samples. Signed-off-by: Icenowy Zheng --- arch/arm64/boot/dts/allwinner/Makefile| 1 + .../dts/allwinner/sun50i-a64-pinetab-dev.dts | 28 +++ 2 files changed, 29

Re: [PATCH] net/dsa: remove unused macros to tame gcc warning

2020-11-07 Thread Alex Shi
在 2020/11/7 上午12:39, Florian Fainelli 写道: >> Hi Alex >> >> It is good to remember that there are multiple readers of source >> files. There is the compiler which generates code from it, and there >> is the human trying to understand what is going on, what the hardware >> can do, how we could

Re: [PATCH] net/xdp: remove unused macro REG_STATE_NEW

2020-11-07 Thread Alex Shi
在 2020/11/7 上午12:13, Jesper Dangaard Brouer 写道: > Hmm... REG_STATE_NEW is zero, so it is implicitly set via memset zero. > But it is true that it is technically not directly used or referenced. > > It is mentioned in a comment, so please send V2 with this additional change: Hi Jesper, Thanks

  1   2   3   4   >