Re: linux-next: manual merge of the gpio-brgl tree with the arm-soc tree

2021-02-09 Thread Arnd Bergmann
On Tue, Feb 9, 2021 at 11:01 AM Geert Uytterhoeven wrote: > On Thu, Jan 28, 2021 at 7:05 AM Stephen Rothwell > wrote: > > diff --cc arch/arm64/boot/dts/toshiba/tmpv7708-rm-mbrc.dts > > index 37da418393e0,950010a290f0.. > > --- a/arch/arm64/boot/dts/toshiba/tmpv7708-rm-mbrc.dts > > +

Re: [RFC PATCH v3 1/2] mempinfd: Add new syscall to provide memory pin

2021-02-07 Thread Arnd Bergmann
On Sun, Feb 7, 2021 at 9:18 AM Zhou Wang wrote: > diff --git a/arch/arm64/include/asm/unistd32.h > b/arch/arm64/include/asm/unistd32.h > index cccfbbe..3f49529 100644 > --- a/arch/arm64/include/asm/unistd32.h > +++ b/arch/arm64/include/asm/unistd32.h > @@ -891,6 +891,8 @@ __SYSCALL(__NR_faccessa

Re: [PATCH] remoteproc: qcom: fix glink dependencies

2021-02-04 Thread Arnd Bergmann
On Thu, Feb 4, 2021 at 5:16 PM Alex Elder wrote: > > On 2/4/21 9:40 AM, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > Building the remoteproc drivers into the kernel while the qcom_glink > > code is in a loadable module results in a link error: > >

[PATCH] carl9170: fix struct alignment conflict

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Multiple structures in the carl9170 driver have alignment impossible alignment constraints that gcc warns about when building with 'make W=1': drivers/net/wireless/ath/carl9170/fwcmd.h:243:2: warning: alignment 1 of 'union ' is less than 4 [-Wpacked-not-

[PATCH] scsi: pmcraid: fix 'ioarcb' alignment warning

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Building with 'make W=1' enables -Wpacked-not-aligned, and this warns about pmcraid because of incompatible alignment constraints for pmcraid_passthrough_ioctl_buffer: drivers/scsi/pmcraid.h:1044:1: warning: alignment 1 of 'struct pmcraid_passthrough_ioctl_

[PATCH] staging: rtl8723bs: remove unused structures

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Building this with 'make W=1' produces a couple of warnings: rtl8723bs/include/ieee80211.h:730:1: warning: alignment 1 of 'struct ieee80211_assoc_request_frame' is less than 2 [-Wpacked-not-aligned] rtl8723bs/include/ieee80211.h:737:1: warning: al

[PATCH] brcm80211: fix alignment constraints

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann sturct d11txh contains a ieee80211_rts structure, which is required to have at least two byte alignment, and this conflicts with the __packed attribute: drivers/net/wireless/broadcom/brcm80211/brcmsmac/d11.h:786:1: warning: alignment 1 of 'struct d11txh' is l

[PATCH] staging: vt665x: fix alignment constraints

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann multiple structures contains a ieee80211_rts structure, which is required to have at least two byte alignment, but are annotated with a __packed attribute to force single-byte alignment: staging/vt6656/rxtx.h:98:1: warning: alignment 1 of 'struct vnt_rts_g' is le

[PATCH] cwmwl8k: fix alignment constraints

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann sturct mwl8k_dma_data contains a ieee80211_hdr structure, which is required to have at least two byte alignment, and this conflicts with the __packed attribute: vers/net/wireless/marvell/mwl8k.c:811:1: warning: alignment 1 of 'struct mwl8k_dma_data' is less than 2

[PATCH] wl3501: fix alignment constraints

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann struct wl3501_80211_tx_hdr contains a ieee80211_hdr structure, which is required to have at least two byte alignment, and this conflicts with the __packed attribute: wireless/wl3501.h:553:1: warning: alignment 1 of 'struct wl3501_80211_tx_hdr' is less than 2 [-W

[PATCH] can: ucan: fix alignment constraints

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann struct ucan_message_in contains member with 4-byte alignment but is itself marked as unaligned, which triggers a warning: drivers/net/can/usb/ucan.c:249:1: warning: alignment 1 of 'struct ucan_message_in' is less than 4 [-Wpacked-not-aligned] Mark the outer st

[PATCH] xfs: fix unused variable warning

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann When debugging is disabled, the ASSERT() is left out and the 'log' variable becomes unused: fs/xfs/xfs_log.c::16: error: unused variable 'log' [-Werror,-Wunused-variable] Remove the variable declaration and open-code it inside of the assertion.

[PATCH] drivers: soc: atmel: fix type for same7

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann A missing comma caused a build failure: drivers/soc/atmel/soc.c:196:24: error: too few arguments provided to function-like macro invocation Fixes: af3a10513cd6 ("drivers: soc: atmel: add per soc id and version match masks") Signed-off-by: Arnd Bergmann --- It

[PATCH] Bluetooth: btusb: fix excessive stack usage

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Enlarging the size of 'struct btmtk_hci_wmt_cmd' makes it no longer fit on the kernel stack, as seen from this compiler warning: drivers/bluetooth/btusb.c:3365:12: error: stack frame size of 1036 bytes in function 'btusb_mtk_hci_wmt_sync' [-Werror

[PATCH 2/2] crypto: marvell - fix ethernet driver dependency

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann The OcteonTX2 CPT driver force-enables the OCTEONTX2_MBOX symbol, which fails when network drivers are disabled globally WARNING: unmet direct dependencies detected for OCTEONTX2_MBOX Depends on [n]: NETDEVICES [=n] && ETHERNET [=n] && NET_VENDOR_MARVELL [=

[PATCH 1/2] crypto: octeontx2 - fix -Wpointer-bool-conversion warning

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann When CONFIG_CPUMASK_OFFSTACK is disabled, clang reports a warning about a bogus condition: drivers/crypto/marvell/octeontx2/otx2_cptlf.c:334:21: error: address of array 'lfs->lf[slot].affinity_mask' will always evaluate to 'true' [-Werror,-W

[PATCH] remoteproc: qcom: fix glink dependencies

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Building the remoteproc drivers into the kernel while the qcom_glink code is in a loadable module results in a link error: ld.lld: error: undefined symbol: qcom_glink_ssr_notify >>> referenced by vmlinux.o:(glink_subdev_unprepare) Add a Kconfig dependency to a

[PATCH] leds: rt8515: add V4L2_FLASH_LED_CLASS dependency

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann The leds-rt8515 driver can optionall use the v4l2 flash led class, but it causes a link error when that class is in a loadable module and the rt8515 driver itself is built-in: ld.lld: error: undefined symbol: v4l2_flash_init >>> referenced by leds

[PATCH] platform/x86: ideapad-laptop/thinkpad_acpi: mark conflicting symbols static

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Three of the newly added functions are accidently not marked 'static' which causes a warning when building with W=1 drivers/platform/x86/thinkpad_acpi.c:10081:5: warning: no previous prototype for function 'dytc_profile_get' [-Wmissing-prototypes]

[PATCH] net: hns3: avoid -Wpointer-bool-conversion warning

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann clang points out a redundant sanity check: drivers/net/ethernet/hisilicon/hns3/hns3_debugfs.c:497:28: error: address of array 'filp->f_path.dentry->d_iname' will always evaluate to 'true' [-Werror,-Wpointer-bool-conversion] This can never fail

[PATCH] kallsyms: fix nonconverging kallsyms table with lld

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann ARM randconfig builds with lld sometimes show a build failure from kallsyms: Inconsistent kallsyms data Try make KALLSYMS_EXTRA_PASS=1 as a workaround The problem is the veneers/thunks getting added by the linker extend the symbol table, which in turn leads to more

Re: [PATCH v4] misc: bcm-vk: only support ttyVK if CONFIG_TTY is set

2021-02-04 Thread Arnd Bergmann
: Scott Branden > > Acked-by: Randy Dunlap # build-tested Passes my randconfig builds as well and looks reasonable Acked-by: Arnd Bergmann

Re: linux-next: build failure after merge of the akpm-current tree

2021-02-03 Thread Arnd Bergmann
On Wed, Feb 3, 2021 at 6:34 PM Randy Dunlap wrote: > > On 2/3/21 9:09 AM, Arnd Bergmann wrote: > > On Tue, Feb 2, 2021 at 10:12 AM Stephen Rothwell > > wrote: > > > >> > >> 983cb10d3f90 ("mm/gup: do not migrate zero page") > >> >

Re: linux-next: build failure after merge of the akpm-current tree

2021-02-03 Thread Arnd Bergmann
On Tue, Feb 2, 2021 at 10:12 AM Stephen Rothwell wrote: > > 983cb10d3f90 ("mm/gup: do not migrate zero page") > > I have applied the following patch for today: > > From: Stephen Rothwell > Date: Tue, 2 Feb 2021 19:49:00 +1100 > Subject: [PATCH] make is_pinnable_page a macro > > As it is curren

Re: [PATCH v5 13/16] asm-generic/hyperv: introduce hv_device_id and auxiliary structures

2021-02-03 Thread Arnd Bergmann
On Wed, Feb 3, 2021 at 2:26 PM Wei Liu wrote: > On Tue, Feb 02, 2021 at 05:02:48PM +, Wei Liu wrote: > > On Tue, Jan 26, 2021 at 01:26:52AM +, Michael Kelley wrote: > > > From: Wei Liu Sent: Wednesday, January 20, 2021 4:01 > > > AM > > > > +union hv_device_id { > > > > + u64 as_uint64;

Re: [PATCH v2 3/3] arch/arm/configs: Enable VMSPLIT_2G in imx_v6_v7_defconfig

2021-02-02 Thread Arnd Bergmann
On Wed, Feb 3, 2021 at 3:37 AM Alistair Francis wrote: > > On Thu, Jan 28, 2021 at 11:13 PM Shawn Guo wrote: > > > > On Sun, Jan 17, 2021 at 10:03:01AM -0800, Alistair Francis wrote: > > > The reMarkable2 requires VMSPLIT_2G, so lets set this in the > > > imx_v6_v7_defconfig. > > > > Hmm, why is

Re: linux-next: Signed-off-by missing for commit in the arm-soc tree

2021-02-02 Thread Arnd Bergmann
On Tue, Feb 2, 2021 at 9:04 PM Stephen Rothwell wrote: > On Wed, 3 Feb 2021 06:57:54 +1100 Stephen Rothwell > wrote: > > > > Hi all, > > > > Commit > > > > 4e238bfd83f3 ("ARM: dts: sun7i: a20: bananapro: Fix ethernet phy-mode") > > > > is missing a Signed-off-by from its committer. > > This is

Re: [PATCH v2] ASoC: da7218: Drop CONFIG_OF ifdef

2021-02-02 Thread Arnd Bergmann
> of_match_ptr() internally, confusing compilers and causing ifdef a> > pollution. > > > > Reported-by: kernel test robot > > Cc: Geert Uytterhoeven > > Acked-by: Arnd Bergmann > > Cc: Mark Brown > > Signed-off-by: Stephen Boyd > > Thanks for

Re: [PATCH] arm64: Make CPU_BIG_ENDIAN depend on !LD_IS_LLD

2021-02-02 Thread Arnd Bergmann
On Tue, Feb 2, 2021 at 8:51 PM Nathan Chancellor wrote: > On Tue, Feb 02, 2021 at 09:04:34AM +0100, Arnd Bergmann wrote: > > On Tue, Feb 2, 2021 at 3:25 AM Nathan Chancellor wrote: > > > > > > Similar to commit 28187dc8ebd9 ("ARM: 9025/1: Kconfig: CPU_BIG_EN

Re: [PATCH for-next] ASoC: da7218: Drop CONFIG_OF ifdef

2021-02-02 Thread Arnd Bergmann
ollution. > > Reported-by: kernel test robot > Cc: Geert Uytterhoeven > Cc: Arnd Bergmann > Cc: Mark Brown > Signed-off-by: Stephen Boyd > --- > > Please ack so Rob can take through DT tree. Looks reasonable as a compile-time fix, though I'd remove the of_match_p

Re: [PATCH v7 4/4] ARM: Add support for Hisilicon Kunpeng L3 cache controller

2021-02-02 Thread Arnd Bergmann
On Tue, Feb 2, 2021 at 1:18 PM Leizhen (ThunderTown) wrote: > On 2021/2/2 16:44, Arnd Bergmann wrote: > > > > To have a more useful performance number, try mentioning the > > most performance sensitive non-coherent DMA master on one > > of the chips that has this c

Re: [PATCH v7 4/4] ARM: Add support for Hisilicon Kunpeng L3 cache controller

2021-02-02 Thread Arnd Bergmann
On Tue, Feb 2, 2021 at 8:16 AM Zhen Lei wrote: > + > +/* > + * All read and write operations on L3 cache registers are protected by the > + * spinlock, except for l3cache_init(). Each time the L3 cache operation is > + * performed, all related information is filled into its registers. > Therefore

Re: [PATCH] arm64: Make CPU_BIG_ENDIAN depend on !LD_IS_LLD

2021-02-02 Thread Arnd Bergmann
On Tue, Feb 2, 2021 at 3:25 AM Nathan Chancellor wrote: > > Similar to commit 28187dc8ebd9 ("ARM: 9025/1: Kconfig: CPU_BIG_ENDIAN > depends on !LD_IS_LLD"), ld.lld does not support aarch64 big endian, > leading to the following build error when CONFIG_CPU_BIG_ENDIAN is > selected: > > ld.lld: erro

[PATCH] scsi: pmcraid: fix 'ioarcb' alignment warning

2021-02-01 Thread Arnd Bergmann
From: Arnd Bergmann Building with 'make W=1' enables -Wpacked-not-aligned, and this warns about pmcraid because of incompatible alignment constraints for pmcraid_passthrough_ioctl_buffer: drivers/scsi/pmcraid.h:1044:1: warning: alignment 1 of 'struct pmcraid_passthrough_ioctl_

Re: [PATCH v6 2/4] ARM: hisi: add support for Kunpeng50x SoC

2021-02-01 Thread Arnd Bergmann
On Mon, Feb 1, 2021 at 12:49 PM Leizhen (ThunderTown) wrote: > On 2021/2/1 16:35, Arnd Bergmann wrote: > > On Mon, Feb 1, 2021 at 4:35 AM Zhen Lei wrote: > >> > >> Enable support for the Hisilicon Kunpeng506 and Kunpeng509 SoC. > >> > >> Signe

Re: [PATCH 01/20] ata: ahci_dm816: Ignore -Woverride-init

2021-02-01 Thread Arnd Bergmann
On Thu, Jan 28, 2021 at 7:32 PM Lee Jones wrote: > > On Thu, 28 Jan 2021, Christoph Hellwig wrote: > > > On Thu, Jan 28, 2021 at 06:02:20PM +, Lee Jones wrote: > > > Some ATA drivers use the SCSI host template, a series of interwoven > > > macros, to aid with initialisation. Some of these mac

Re: [PATCH v6 2/4] ARM: hisi: add support for Kunpeng50x SoC

2021-02-01 Thread Arnd Bergmann
On Mon, Feb 1, 2021 at 4:35 AM Zhen Lei wrote: > > Enable support for the Hisilicon Kunpeng506 and Kunpeng509 SoC. > > Signed-off-by: Zhen Lei Reviewed-by: Arnd Bergmann Russell, do you have a preference for how to get this series merged after the last comments are resolved? I th

Re: [PATCH v6 4/4] ARM: Add support for Hisilicon Kunpeng L3 cache controller

2021-02-01 Thread Arnd Bergmann
not exceed 36 bits. When the cache > operation is performed based on the address range, the upper 30 bits of > the physical address are recorded in registers L3_MAINT_START and > L3_MAINT_END, and ignore the lower 6 bits cacheline offset. > > Signed-off-by: Zhen Lei Reviewed-by: Ar

Re: [PATCH 2/2] ARM: dts: exynos: use Exynos5420 dedicated USB2 PHY compatible

2021-01-30 Thread Arnd Bergmann
On Wed, Jan 27, 2021 at 8:59 AM Krzysztof Kozlowski wrote: > On Tue, Jan 26, 2021 at 11:44:26PM +0100, Arnd Bergmann wrote: > > On Fri, Nov 20, 2020 at 12:10 PM Marek Szyprowski > > wrote: > > > It won't work easily with both compatibles, because in the 5420 varian

Re: [PATCH v2] ARM: kprobes: rewrite test-[arm|thumb].c in UAL

2021-01-28 Thread Arnd Bergmann
On Thu, Jan 28, 2021 at 10:03 PM Ard Biesheuvel wrote: > On Thu, 28 Jan 2021 at 20:34, Nick Desaulniers > wrote: > > @@ -468,15 +468,15 @@ void kprobe_thumb32_test_cases(void) > > > > TEST_UNSUPPORTED("strexbr0, r1, [r2]") > > TEST_UNSUPPORTED("strexhr0, r1, [r2]"

Re: [PATCH] ARM: kprobes: rewrite in UAL

2021-01-28 Thread Arnd Bergmann
On Thu, Jan 28, 2021 at 7:59 PM 'Nick Desaulniers' via Clang Built Linux wrote: > On Thu, Jan 28, 2021 at 8:15 AM Arnd Bergmann wrote: > > > Plus a lot of extra errors when building with CONFIG_THUMB2_KERNEL, > > as that uses tests in arch/arm/probes/kprobes/test-thum

Re: [PATCH] ARM: kprobes: rewrite in UAL

2021-01-28 Thread Arnd Bergmann
ssembly-language-changes-after-rvctv2-1 > Link: https://github.com/ClangBuiltLinux/linux/issues/1271 > Link: https://reviews.llvm.org/D95586 > Reported-by: Arnd Bergmann > Signed-off-by: Nick Desaulniers > --- > arch/arm/probes/kprobes/test-arm.c | 288 ++--- >

[GIT PULL] asm-generic/ia64 fixes, mark as orphaned

2021-01-28 Thread Arnd Bergmann
architecture as Orphaned and a compile time warning fix I made while working on the regression. Signed-off-by: Arnd Bergmann ---- Arnd Bergmann (3): ia64: fix timer cleanup regression ia64: fix xchg() warning ia64: Mark

Re: [PATCH v5 1/4] ARM: LPAE: Use phys_addr_t instead of unsigned long in outercache hooks

2021-01-28 Thread Arnd Bergmann
c220_inv_range(phys_addr_t pa_start, phys_addr_t pa_end) > { > + unsigned long start = pa_start; > + unsigned long end = pa_end; > > Note that the outercache functions have been doing this cast before this > patch. So now, the cast is just moved into the outercache hook functions. > > No functional change. > > Signed-off-by: Zhen Lei Reviewed-by: Arnd Bergmann

Re: [PATCH v5 2/4] ARM: hisi: add support for Kunpeng50x SoC

2021-01-28 Thread Arnd Bergmann
On Sat, Jan 16, 2021 at 4:32 AM Zhen Lei wrote: > > Enable support for the Hisilicon Kunpeng506 and Kunpeng509 SoC. > > Signed-off-by: Zhen Lei > --- > arch/arm/mach-hisi/Kconfig | 8 > 1 file changed, 8 insertions(+) > > diff --git a/arch/arm/mach-hisi/Kconfig b/arch/arm/mach-hisi/Kcon

Re: [PATCH v5 4/4] ARM: Add support for Hisilicon Kunpeng L3 cache controller

2021-01-28 Thread Arnd Bergmann
On Sat, Jan 16, 2021 at 4:27 AM Zhen Lei wrote: > diff --git a/arch/arm/mm/Makefile b/arch/arm/mm/Makefile > + > +static void l3cache_maint_common(u32 range, u32 op_type) > +{ > + u32 reg; > + > + reg = readl(l3_ctrl_base + L3_MAINT_CTRL); > + reg &= ~(L3_MAINT_RANGE_MASK | L3_MA

Re: [PATCH v5 3/4] dt-bindings: arm: hisilicon: Add binding for Kunpeng L3 cache controller

2021-01-28 Thread Arnd Bergmann
On Sat, Jan 16, 2021 at 4:34 AM Zhen Lei wrote: > > Add devicetree binding for Hisilicon Kunpeng L3 cache controller. > > Signed-off-by: Zhen Lei > --- > .../arm/hisilicon/kunpeng-l3cache.yaml| 40 +++ Reviewed-by: Arnd Bergmann

Re: [PATCH v3 2/4] arm64: dts: correct vendor prefix hisi to hisilicon

2021-01-28 Thread Arnd Bergmann
On Wed, Jan 27, 2021 at 1:42 AM Wei Xu wrote: > On 2021/1/27 6:23, Arnd Bergmann wrote: > > On Tue, Dec 8, 2020 at 1:46 PM Zhen Lei wrote: > >> > >> The vendor prefix of "Hisilicon Limited" is "hisilicon", it is clearly > >> stated in "

Re: [PATCH] kasan: export kasan_poison

2021-01-27 Thread Arnd Bergmann
On Wed, Jan 27, 2021 at 10:25 PM Andrey Konovalov wrote: > On Mon, Jan 25, 2021 at 12:28 PM Arnd Bergmann wrote: > > diff --git a/mm/kasan/shadow.c b/mm/kasan/shadow.c > > index de6b3f074742..32e7a5c148e6 100644 > > --- a/mm/kasan/shadow.c > > +++ b/mm/kasan/shadow.c

Re: [PATCH 2/2] ARM: dts: exynos: use Exynos5420 dedicated USB2 PHY compatible

2021-01-27 Thread Arnd Bergmann
On Fri, Nov 20, 2020 at 12:10 PM Marek Szyprowski wrote: > On 20.11.2020 12:05, Krzysztof Kozlowski wrote: > > On Fri, Nov 20, 2020 at 09:56:37AM +0100, Marek Szyprowski wrote: > >> USB2.0 PHY in Exynos5420 differs from Exynos5250 variant a bit, so use the > >> recently introduced dedicated compat

Re: [PATCH v3 2/4] arm64: dts: correct vendor prefix hisi to hisilicon

2021-01-26 Thread Arnd Bergmann
On Tue, Dec 8, 2020 at 1:46 PM Zhen Lei wrote: > > The vendor prefix of "Hisilicon Limited" is "hisilicon", it is clearly > stated in "vendor-prefixes.yaml". > > Fixes: 35ca8168133c ("arm64: dts: Add dts files for Hisilicon Hi3660 SoC") > Fixes: dd8c7b78c11b ("arm64: dts: Add devicetree for Hisili

Re: linux-next: manual merge of the risc-v tree with the arm-soc tree

2021-01-26 Thread Arnd Bergmann
On Sun, Jan 24, 2021 at 11:14 PM Stephen Rothwell wrote: > > I fixed it up (see below) and can carry the fix as necessary. This > is now fixed as far as linux-next is concerned, but any non trivial > conflicts should be mentioned to your upstream maintainer when your tree > is submitted for mergin

[PATCH] [net-next] ipa: add remoteproc dependency

2021-01-26 Thread Arnd Bergmann
From: Arnd Bergmann Compile-testing without CONFIG_REMOTEPROC results in a build failure: >>> referenced by ipa_main.c >>> net/ipa/ipa_main.o:(ipa_probe) in archive drivers/built-in.a ld.lld: error: undefined symbol: rproc_put >>> referenced by ipa_main.

[PATCH] [net-next] bonding: add TLS dependency

2021-01-26 Thread Arnd Bergmann
From: Arnd Bergmann When TLS is a module, the built-in bonding driver may cause a link error: x86_64-linux-ld: drivers/net/bonding/bond_main.o: in function `bond_start_xmit': bond_main.c:(.text+0xc451): undefined reference to `tls_validate_xmit_skb' Add a dependency to avoid t

Re: [PATCH v2 29/34] Intel tsens i2c slave driver.

2021-01-26 Thread Arnd Bergmann
On Tue, Jan 26, 2021 at 12:39 AM mark gross wrote: > > On Mon, Jan 11, 2021 at 11:15:06PM -0800, Randy Dunlap wrote: > > On 1/8/21 1:25 PM, mgr...@linux.intel.com wrote: > > > diff --git a/drivers/misc/intel_tsens/Kconfig > > > b/drivers/misc/intel_tsens/Kconfig > > > index 8b263fdd80c3..c2138339

[RFC 0/3] kunit vs structleak

2021-01-26 Thread Arnd Bergmann
From: Arnd Bergmann I ran into a couple of problems with kunit tests taking too much stack space, sometimes dangerously so. These the the three instances that cause an increase over the warning limit of some architectures: lib/bitfield_kunit.c:93:1: error: the frame size of 7440 bytes is larger

Re: [PATCH RFC 0/4] Fix arm64 crash for accessing unmapped IO port regions (reboot)

2021-01-26 Thread Arnd Bergmann
On Fri, Jan 15, 2021 at 5:58 PM John Garry wrote: > > This is a reboot of my original series to address the problem of drivers > for legacy ISA devices accessing unmapped IO port regions on arm64 systems > and causing the system to crash. > > There was another recent report of such an issue [0], a

Re: [PATCH] optee: simplify i2c access

2021-01-26 Thread Arnd Bergmann
On Tue, Jan 26, 2021 at 12:50 PM Jorge Ramirez-Ortiz, Foundries wrote: > On 26/01/21, Arnd Bergmann wrote: > > On Tue, Jan 26, 2021 at 9:08 AM Jorge Ramirez-Ortiz, Foundries > > > > The fixes tag only describes which commit introduced the bug, it is > > irrelevant &

Re: [PATCH] optee: simplify i2c access

2021-01-26 Thread Arnd Bergmann
On Tue, Jan 26, 2021 at 9:08 AM Jorge Ramirez-Ortiz, Foundries wrote: > > On 25/01/21, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > Storing a bogus i2c_client structure on the stack adds overhead and > > causes a compile-time warning: > > > > d

Re: [PATCH] [net-next] ipa: add remoteproc dependency

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 4:23 PM Bjorn Andersson wrote: > > On Mon 25 Jan 05:35 CST 2021, Arnd Bergmann wrote: > > > From: Arnd Bergmann > > > > Compile-testing without CONFIG_REMOTEPROC results in a build failure: > > > > >>> referenced by ipa

Re: [PATCH] ath9k: fix build error with LEDS_CLASS=m

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 2:27 PM Krzysztof Kozlowski wrote: > On Mon, 25 Jan 2021 at 14:09, Arnd Bergmann wrote: > > On Mon, Jan 25, 2021 at 12:40 PM Krzysztof Kozlowski > > wrote: > > > On Mon, 25 Jan 2021 at 12:36, Arnd Bergmann wrote: > > > But we do

[PATCH] optee: simplify i2c access

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann Storing a bogus i2c_client structure on the stack adds overhead and causes a compile-time warning: drivers/tee/optee/rpc.c:493:6: error: stack frame size of 1056 bytes in function 'optee_handle_rpc' [-Werror,-Wframe-larger-than=] void optee_handle_rpc(struct t

Re: [PATCH] ath9k: fix build error with LEDS_CLASS=m

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 12:40 PM Krzysztof Kozlowski wrote: > On Mon, 25 Jan 2021 at 12:36, Arnd Bergmann wrote: > > > > From: Arnd Bergmann > > > > When CONFIG_ATH9K is built-in but LED support is in a loadable > > module, both ath9k drivers fails to link: >

[PATCH] mmc: brcmstb: Fix sdhci_pltfm_suspend link error

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann sdhci_pltfm_suspend() is only available when CONFIG_PM_SLEEP support is built into the kernel, which caused a regression in a recent bugfix: ld.lld: error: undefined symbol: sdhci_pltfm_suspend >>> referenced by sdhci-brcmstb.c >>> mmc/ho

[PATCH] arm64: kfence: fix header inclusion

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann Randconfig builds started warning about a missing function declaration after set_memory_valid() is moved to a new file: In file included from mm/kfence/core.c:26: arch/arm64/include/asm/kfence.h:17:2: error: implicit declaration of function 'set_memory_valid'

[PATCH] amdgpu: fix clang build warning

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann clang warns about the -mhard-float command line arguments on architectures that do not support this: clang: error: argument unused during compilation: '-mhard-float' [-Werror,-Wunused-command-line-argument] Move this into the gcc-specific arguments. Fixes: e7

[RFC 3/3] thunderbolt: build kunit tests without structleak plugin

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The structleak plugin causes the stack frame size to grow immensely: drivers/thunderbolt/test.c:1529:1: error: the frame size of 1176 bytes is larger than 1024 bytes [-Werror=frame-larger-than=] Turn it off in this file. Signed-off-by: Arnd Bergmann --- drivers

[RFC 2/3] drivers/base: build kunit tests without structleak plugin

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The structleak plugin causes the stack frame size to grow immensely: drivers/base/test/property-entry-test.c: In function 'pe_test_reference': drivers/base/test/property-entry-test.c:481:1: error: the frame size of 2640 bytes is larger than 2048 bytes [-Werror=fr

[RFC 1/3] bitfield: build kunit tests without structleak plugin

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The structleak plugin causes the stack frame size to grow immensely: lib/bitfield_kunit.c: In function 'test_bitfields_constants': lib/bitfield_kunit.c:93:1: error: the frame size of 7440 bytes is larger than 2048 bytes [-Werror=frame-larger-than=] Turn it off in

[PATCH] PCI: endpoint: Select configfs dependency

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The newly added pci-epf-ntb driver uses configfs, which causes a link failure when that is disabled at compile-time: arm-linux-gnueabi-ld: drivers/pci/endpoint/functions/pci-epf-ntb.o: in function `epf_ntb_add_cfs': pci-epf-ntb.c:(.text+0x954): undefined referen

[PATCH] i915: Fix DRM_I915_WERROR dependencies

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann CONFIG_DRM_I915_DEBUG now selects CONFIG_DRM_I915_WERROR, but fails to honor its dependencies: WARNING: unmet direct dependencies detected for DRM_I915_WERROR Depends on [n]: HAS_IOMEM [=y] && DRM_I915 [=m] && EXPERT [=y] && !COMPILE_TE

[PATCH] ath9k: fix build error with LEDS_CLASS=m

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann When CONFIG_ATH9K is built-in but LED support is in a loadable module, both ath9k drivers fails to link: x86_64-linux-ld: drivers/net/wireless/ath/ath9k/gpio.o: in function `ath_deinit_leds': gpio.c:(.text+0x36): undefined reference to `led_classdev_unregister' x8

Re: [PATCH v2 6/6] of/device: Don't NULLify match table in of_match_device() with CONFIG_OF=n

2021-01-25 Thread Arnd Bergmann
; Signed-off-by: Stephen Boyd > Acked-by: Frank Rowand > Cc: Arnd Bergmann > Cc: Geert Uytterhoeven > Cc: Rob Herring > Cc: Frank Rowand Reviewed-by: Arnd Bergmann

[PATCH] f2fs: fix unused function warning

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The f2fs_compress_set_level() function is only used for zstd and lz4 compression modes, and otherwise causes a compile-time warning for having no users. Change the #ifdef check to match what calls it. Fixes: c08e95fc6466 ("f2fs: compress: support compress level")

[PATCH] drm/i915/gem: fix non-SMP build failure

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The x86-specific wbinvd_on_all_cpus() function is exported through asm/smp.h, causing a build failure in the i915 driver when SMP is disabled: drivers/gpu/drm/i915/i915_gem.c:1182:2: error: implicit declaration of function 'wbinvd_on_all_cpus' [-Werror,-Wimplici

Re: [PATCH] usb: host: ehci-tegra: fix Kconfig depencies

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 5:22 PM Alan Stern wrote: > > On Mon, Jan 25, 2021 at 12:32:30PM +0100, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > Selecting the chipidea driver from the old Kconfig symbol > > can lead to a missing dependency: > > Arnd: &

Re: [PATCH] ath9k: fix build error with LEDS_CLASS=m

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 4:04 PM Krzysztof Kozlowski wrote: > On Mon, 25 Jan 2021 at 15:38, Arnd Bergmann wrote: > > On Mon, Jan 25, 2021 at 2:27 PM Krzysztof Kozlowski wrote: > > I meant that having MAC80211_LEDS selected causes the ath9k driver to > toggle on/off the WiFi

Re: LTP: madvise08.c:203: TFAIL: No sequence in dump after MADV_DODUMP.

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 3:48 PM Naresh Kamboju wrote: > > LTP syscalls madvise08 test case failed on all devices from > Linux next 20210118 to till day. > strace log attached to this email and link provided below. > > BAD: next-20210118 > GOOD: next-20210115 > > This failure is easily reproducible

[PATCH] drm/amd/display: use div_s64() for 64-bit division

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The open-coded 64-bit division causes a link error on 32-bit machines: ERROR: modpost: "__udivdi3" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! ERROR: modpost: "__divdi3" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! Use the div_s64() to perfo

Re: [PATCH] drm/amd/display: use div_s64() for 64-bit division

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 1:51 PM Chen, Guchun wrote: > > [AMD Public Use] > > Hi Arnd Bergmann, > > Thanks for your patch. This link error during compile has been fixed by below > commit and been submitted to drm-next branch already. > > 5da047444e82 drm/amd/display: f

Re: [PATCH] i915: Fix DRM_I915_WERROR dependencies

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 1:33 PM Chris Wilson wrote: > > Quoting Arnd Bergmann (2021-01-25 12:26:44) > > From: Arnd Bergmann > > > > CONFIG_DRM_I915_DEBUG now selects CONFIG_DRM_I915_WERROR, but fails > > to honor its dependencies: > > > > WARNI

[PATCH] drm/amd/display: fix unused variable warning

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann After all users of the 'dm' warnings got hidden in an #ifdef, the compiler started warning about it being unused: drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5380:33: error: unused variable 'dm' [-Werror,-Wunused-variable] Add another

[PATCH] usb: host: ehci-tegra: fix Kconfig depencies

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann Selecting the chipidea driver from the old Kconfig symbol can lead to a missing dependency: WARNING: unmet direct dependencies detected for USB_CHIPIDEA Depends on [m]: USB_SUPPORT [=y] && (USB_EHCI_HCD [=y] && USB_GADGET [=m] || USB_EHCI_HCD [=y] &a

[PATCH] mmc: aspeed: move kunit test into separate module

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann Having both the driver and the unit test in the same module leads to a link failure because of the extra init/exit functions: drivers/mmc/host/sdhci-of-aspeed-test.c:98:1: error: redefinition of '__inittest' kunit_test_suite(aspeed_sdhci_test_suite); Make it

[PATCH] kasan: export kasan_poison

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The unit test module fails to build after adding a reference to kasan_poison: ERROR: modpost: "kasan_poison" [lib/test_kasan.ko] undefined! Export this symbol to make it available to loadable modules. Fixes: b9b322c2bba9 ("kasan: add match-all tag tests&

[PATCH] [5.8 regression] net: ks8851: fix link error

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann An object file cannot be built for both loadable module and built-in use at the same time: arm-linux-gnueabi-ld: drivers/net/ethernet/micrel/ks8851_common.o: in function `ks8851_probe_common': ks8851_common.c:(.text+0xf80): undefined reference to `__this_module'

Re: [PATCH] percpu: fix clang modpost warning in pcpu_build_alloc_info()

2021-01-25 Thread Arnd Bergmann
here. If another issue comes up I'll drop this and tag > it as __refdata. I've come across this one again in linux-next today, and found that I had an old patch for it already, that I had never submitted: >From 7d6f40414490092b86f1a64d8c42426ee350da1a Mon Sep 17 00:00:00 2001

Re: [PATCH 4/5] watchdog: remove tango driver

2021-01-23 Thread Arnd Bergmann
On Sat, Jan 23, 2021 at 7:36 PM Guenter Roeck wrote: > > On Wed, Jan 20, 2021 at 05:27:44PM +0100, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > The tango platform is getting removed, so the driver is no > > longer needed. > > > > Cc: Marc Gonza

Re: [PATCH v1] misc: pti: Remove driver for deprecated platform

2021-01-22 Thread Arnd Bergmann
to above and to reduce a burden of supporting outdated drivers > we remove the support of outdated platforms completely. > > Cc: Alexander Shishkin > Signed-off-by: Andy Shevchenko > Acked-by: Linus Walleij Acked-by: Arnd Bergmann Does this mean drivers/sfi/ can get killed off alon

[PATCH 2/2] MIPS: make kgdb depend on FPU support

2021-01-22 Thread Arnd Bergmann
From: Arnd Bergmann kgdb fails to build when the FPU support is disabled: arch/mips/kernel/kgdb.c: In function 'dbg_set_reg': arch/mips/kernel/kgdb.c:147:35: error: 'struct thread_struct' has no member named 'fpu' 147 |memcpy((vo

[PATCH 1/2] MIPS: jazz: always allow little-endian builds

2021-01-22 Thread Arnd Bergmann
From: Arnd Bergmann The kernel test robot keeps reporting the same bug when it shows up in new files after random unrelated patches: In file included from arch/mips/include/uapi/asm/byteorder.h:13, from arch/mips/include/asm/bitops.h:20, from include/linux

Re: [PATCH 4/5] watchdog: remove tango driver

2021-01-21 Thread Arnd Bergmann
On Thu, Jan 21, 2021 at 3:01 PM Måns Rullgård wrote: > Arnd Bergmann writes: > > From: Arnd Bergmann > > > > The tango platform is getting removed, so the driver is no > > longer needed. > > > > Cc: Marc Gonzalez > > Cc: Mans Rullgard > >

Re: [RFC PATCH 7/7] irqchip/apple-aic: add SMP support to the Apple AIC driver.

2021-01-21 Thread Arnd Bergmann
On Thu, Jan 21, 2021 at 3:10 PM Marc Zyngier wrote: > On 2021-01-21 13:34, Mohamed Mediouni wrote: > >> On 21 Jan 2021, at 14:22, Marc Zyngier wrote: > > Because UART access adapters for the new M1 Macs aren’t plentiful > > at all, I actually use this for development, with iPhones which have > >

Re: [RFC PATCH 7/7] irqchip/apple-aic: add SMP support to the Apple AIC driver.

2021-01-21 Thread Arnd Bergmann
On Thu, Jan 21, 2021 at 1:50 PM Mohamed Mediouni wrote: > > On 21 Jan 2021, at 13:44, Arnd Bergmann wrote: > >> @@ -186,8 +325,11 @@ static int __init apple_aic_init(struct device_node > >> *node, > >> if (WARN(!aic.base, "unable to map aic registers\

Re: [RFC PATCH 7/7] irqchip/apple-aic: add SMP support to the Apple AIC driver.

2021-01-21 Thread Arnd Bergmann
On Wed, Jan 20, 2021 at 2:27 PM Mohamed Mediouni wrote: > +#ifdef CONFIG_SMP > +static void apple_aic_ipi_send_mask(struct irq_data *d, > + const struct cpumask *mask) Not sure we care about the #ifdef here, given that arch/arm64 does not allow building a kernel

Re: [RFC PATCH 3/7] arm64: mm: use nGnRnE instead of nGnRE on Apple processors

2021-01-21 Thread Arnd Bergmann
On Thu, Jan 21, 2021 at 12:32 PM Will Deacon wrote: > On Wed, Jan 20, 2021 at 02:27:13PM +0100, Mohamed Mediouni wrote: > > Use nGnRnE instead of nGnRE on Apple SoCs to workaround a serious hardware > > quirk. > > /* > > * Default MAIR_EL1. MT_NORMAL_TAGGED is initially mapped as Normal memory

Re: [RFC PATCH 2/7] arm64: kernel: Add a WFI hook.

2021-01-21 Thread Arnd Bergmann
On Thu, Jan 21, 2021 at 12:01 PM Mohamed Mediouni wrote: > > On 21 Jan 2021, at 11:52, Arnd Bergmann wrote: > > > > On Wed, Jan 20, 2021 at 2:27 PM Mohamed Mediouni > > wrote: > >> --- a/arch/arm64/kernel/cpu_ops.c > >> +++ b/arch/a

Re: [RFC PATCH 6/7] arm64: kernel: Apple CPU start driver

2021-01-21 Thread Arnd Bergmann
On Wed, Jan 20, 2021 at 2:27 PM Mohamed Mediouni wrote: > diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml > b/Documentation/devicetree/bindings/arm/cpus.yaml > index 14cd727d3c4b..a6ff8cb3db1e 100644 > --- a/Documentation/devicetree/bindings/arm/cpus.yaml > +++ b/Documentation/device

Re: [RFC PATCH 2/7] arm64: kernel: Add a WFI hook.

2021-01-21 Thread Arnd Bergmann
On Wed, Jan 20, 2021 at 2:27 PM Mohamed Mediouni wrote: > --- a/arch/arm64/kernel/cpu_ops.c > +++ b/arch/arm64/kernel/cpu_ops.c > #if defined(CONFIG_STACKPROTECTOR) && > !defined(CONFIG_STACKPROTECTOR_PER_TASK) > #include > @@ -74,8 +75,14 @@ void (*arm_pm_restart)(enum reboot_mode reboot_mod

Re: [RFC PATCH 4/7] irqchip/apple-aic: Add support for Apple AIC

2021-01-21 Thread Arnd Bergmann
On Thu, Jan 21, 2021 at 10:48 AM Linus Walleij wrote: > > Hi Mohamed, > > thanks for your patch! > > On Wed, Jan 20, 2021 at 2:31 PM Mohamed Mediouni > wrote: > > > +properties: > > + compatible: > > +items: > > + - const: apple,aic > > However weird it may seem, Apple is not in the fil

<    1   2   3   4   5   6   7   8   9   10   >