Re: [PATCH] openrisc: Add support for more module relocations

2024-04-11 Thread Geert Uytterhoeven
ine R_OR1K_NONE0 and add definitions for backwards compatibility? #define R_OR32_NONER_OR1K_NONE Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technic

[PATCH 18/20] sh: kprobes: Remove unneeded kprobe_opcode_t casts

2024-03-01 Thread Geert Uytterhoeven
There is no need to cast a kprobe_opcode_t pointer to a kprobe_opcode_t pointer. Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/kprobes.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/sh/kernel/kprobes.c b/arch/sh/kernel/kprobes.c index d8c2e399d6e50794

[PATCH 20/20] [RFC] sh: dma: Remove unused functionality

2024-03-01 Thread Geert Uytterhoeven
dma_extend(), get_dma_info_by_name(), register_chan_caps(), and request_dma_bycap() are unused. Remove them, and all related code. Signed-off-by: Geert Uytterhoeven --- arch/sh/drivers/dma/dma-api.c | 116 -- arch/sh/include/asm/dma.h | 7 -- 2 files

[PATCH 16/20] sh: kprobes: Merge arch_copy_kprobe() into arch_prepare_kprobe()

2024-03-01 Thread Geert Uytterhoeven
("[PATCH] kprobes: changed from using spinlock to mutex"). Fixes: d39f5450146ff39f ("sh: Add kprobes support.") Signed-off-by: Geert Uytterhoeven --- Compile-tested only. --- arch/sh/kernel/kprobes.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --

[PATCH 05/20] sh: return_address: Add missing #include

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/return_address.c:49:7: warning: no previous prototype for ‘return_address’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/return_address.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/sh/kernel/return_address.c b/arch/sh/kernel

[PATCH 12/20] sh: dma: Remove unused dmac_search_free_channel()

2024-03-01 Thread Geert Uytterhoeven
arch/sh/drivers/dma/dma-api.c:164:5: warning: no previous prototype for 'dmac_search_free_channel' [-Wmissing-prototypes] dmac_search_free_channel() never had a user in upstream, remove it. Signed-off-by: Geert Uytterhoeven --- dma_extend(), get_dma_info_by_name(), register_chan_caps

[PATCH 00/20] sh: Fix missing prototypes

2024-03-01 Thread Geert Uytterhoeven
.kernel.org/linux-sh/c395b02613572131568bc1fd1bc456d20d1a5426.1709325647.git.geert+rene...@glider.be Geert Uytterhoeven (20): sh: pgtable: Fix missing prototypes sh: fpu: Add missing forward declarations sh: syscall: Add missing forward declaration for sys_cacheflush() sh: tlb: Add missing forward declaration for h

[PATCH 15/20] sh: smp: Fix missing prototypes

2024-03-01 Thread Geert Uytterhoeven
the other warning. There are no users outside this file, so make it static. Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/smp.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/sh/kernel/smp.c b/arch/sh/kernel/smp.c index 5cf35a774dc70082..b3ea50aabba3d7f2

[PATCH 08/20] sh: boot: Add proper forward declarations

2024-03-01 Thread Geert Uytterhoeven
for ‘arch_ftrace_ops_list_func’ [-Wmissing-prototypes] arch/sh/boot/compressed/misc.c:128:6: warning: no previous prototype for ‘decompress_kernel’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/boot/compressed/cache.c | 3 +++ arch/sh/boot/compressed/cache.h | 10

[PATCH 06/20] sh: traps: Add missing #include

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/traps_32.c:735:6: warning: no previous prototype for ‘per_cpu_trap_init’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/traps_32.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/sh/kernel/traps_32.c b/arch/sh/kernel/traps_32.c index

[PATCH 07/20] sh: hw_breakpoint: Add missing forward declaration for arch_bp_generic_fields()

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/hw_breakpoint.c:135:5: warning: no previous prototype for ‘arch_bp_generic_fields’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/include/asm/hw_breakpoint.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/sh/include/asm/hw_breakpoint.h b/arch

[PATCH 01/20] sh: pgtable: Fix missing prototypes

2024-03-01 Thread Geert Uytterhoeven
pgd_ctor() static, as it is only used in this file. Include to fix the other warnings. Signed-off-by: Geert Uytterhoeven --- arch/sh/mm/pgtable.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/sh/mm/pgtable.c b/arch/sh/mm/pgtable.c index cf7ce4b5735954bf

[PATCH 13/20] sh: sh2a: Add missing #include

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/cpu/sh2a/opcode_helper.c:34:14: warning: no previous prototype for 'instruction_size' [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/cpu/sh2a/opcode_helper.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/sh/kernel/cpu/sh2a

[PATCH 09/20] sh: ftrace: Fix missing prototypes

2024-03-01 Thread Geert Uytterhoeven
for ‘prepare_ftrace_return’ [-Wmissing-prototypes] Fix this by moving existing forward declarations to , and adding the missing forward declaration for prepare_ftrace_return(). Signed-off-by: Geert Uytterhoeven --- arch/sh/include/asm/ftrace.h | 10 ++ arch/sh/kernel/traps.c | 10

[PATCH 14/20] sh: sh7786: Remove unused sh7786_usb_use_exclock()

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/cpu/sh4a/setup-sh7786.c:411:13: warning: no previous prototype for 'sh7786_usb_use_exclock' [-Wmissing-prototypes] Upstream never had a user of sh7786_usb_use_exclock(), remove it. Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/cpu/sh4a/setup-sh7786.c | 14

[PATCH 11/20] sh: math-emu: Add missing #include

2024-03-01 Thread Geert Uytterhoeven
arch/sh/math-emu/math.c:492:5: warning: no previous prototype for 'do_fpu_inst' [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/math-emu/math.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/sh/math-emu/math.c b/arch/sh/math-emu/math.c index cdaef6501d764a0c

[PATCH 10/20] sh: nommu: Add missing #include

2024-03-01 Thread Geert Uytterhoeven
-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/mm/nommu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/sh/mm/nommu.c b/arch/sh/mm/nommu.c index 78c4b6e6d33ba3af..fa3dc9428a737ffe 100644 --- a/arch/sh/mm/nommu.c +++ b/arch/sh/mm/nommu.c @@ -10,6 +10,8 @@ #include #include

[PATCH 02/20] sh: fpu: Add missing forward declarations

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/cpu/sh4/fpu.c:389:6: warning: no previous prototype for ‘float_raise’ [-Wmissing-prototypes] arch/sh/kernel/cpu/sh4/fpu.c:394:5: warning: no previous prototype for ‘float_rounding_mode’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/include/asm/fpu.h | 3

[PATCH 17/20] sh: kprobes: Make trampoline_probe_handler() static

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/kprobes.c:299:15: warning: no previous prototype for 'trampoline_probe_handler' [-Wmissing-prototypes] There are no users outside this file, so make it static. Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/kprobes.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 04/20] sh: tlb: Add missing forward declaration for handle_tlbmiss()

2024-03-01 Thread Geert Uytterhoeven
arch/sh/mm/tlbex_32.c:22:1: warning: no previous prototype for ‘handle_tlbmiss’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/include/asm/tlb.h | 4 arch/sh/mm/tlbex_32.c | 1 + 2 files changed, 5 insertions(+) diff --git a/arch/sh/include/asm/tlb.h b/arch/sh

[PATCH 03/20] sh: syscall: Add missing forward declaration for sys_cacheflush()

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/sys_sh.c:58:16: warning: no previous prototype for ‘sys_cacheflush’ [-Wmissing-prototypes] Signed-off-by: Geert Uytterhoeven --- arch/sh/include/asm/syscalls.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/sh/include/asm/syscalls.h b/arch/sh/include/asm/syscalls.h

[PATCH 19/20] sh: dwarf: Make dwarf_lookup_fde() static

2024-03-01 Thread Geert Uytterhoeven
arch/sh/kernel/dwarf.c:347:19: warning: no previous prototype for 'dwarf_lookup_fde' [-Wmissing-prototypes] There are no users outside this file, so make it static. Signed-off-by: Geert Uytterhoeven --- arch/sh/kernel/dwarf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 0/4] apply page shift to PFN instead of VA in pfn_to_virt

2024-02-01 Thread Geert Uytterhoeven
ting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that.

Re: [PATCH] dt-bindings: correct white-spaces in examples

2023-11-27 Thread Geert Uytterhoeven
On Fri, Nov 24, 2023 at 10:21 AM Krzysztof Kozlowski wrote: > Use only one and exactly one space around '=' in DTS example. > > Signed-off-by: Krzysztof Kozlowski > .../bindings/pinctrl/renesas,rzg2l-pinctrl.yaml | 6 +++--- Reviewed-by: Geert Uytterhoeven Ack

Re: [PATCH v2] locking/atomic: sh: Use generic_cmpxchg_local for arch_cmpxchg_local()

2023-10-26 Thread Geert Uytterhoeven
st robot > Closes: > https://lore.kernel.org/oe-kbuild-all/202310241310.ir5uukog-...@intel.com/ > Signed-off-by: Masami Hiramatsu (Google) Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@

Re: [PATCH] locking/atomic: sh: Use generic_cmpxchg_local for arch_cmpxchg_local()

2023-10-25 Thread Geert Uytterhoeven
___r = raw_cmpxchg_local((_ptr), ___o, (_new)); \ |^ ./include/linux/atomic/atomic-instrumented.h:4980:9: note: in expansion of macro ‘raw_try_cmpxchg_local’ 4980 | raw_try_cmpxchg_local(__ai_ptr, __ai_oldp, __VA_ARGS__); \ | ^~~

[PATCH] neighbor: tracing: Move pin6 inside CONFIG_IPV6=y section

2023-10-16 Thread Geert Uytterhoeven
for CONFIG_IPV6. Fixes: fc651001d2c5ca4f ("neighbor: Add tracepoint to __neigh_create") Signed-off-by: Geert Uytterhoeven --- No changes in generated code. include/trace/events/neigh.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/trace/events/neigh.h b/inc

Re: [PATCH v2 7/9] sched: define TIF_ALLOW_RESCHED

2023-10-06 Thread Geert Uytterhoeven
hidden: although openrisc does not select ARCH_NO_PREEMPT, it does not call preempt_schedule_irq() or select GENERIC_ENTRY? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical

Re: [PATCH v2 2/2] powerpc: Replace GPL 2.0+ README.legal boilerplate with SPDX

2023-09-19 Thread Geert Uytterhoeven
Hi Michael, On Tue, Sep 19, 2023 at 4:13 AM Michael Ellerman wrote: > Geert Uytterhoeven writes: > > Upstream Linux never had a "README.legal" file, but it was present > > in early source releases of Linux/m68k. It contained a simple copyright > > notice and a li

[PATCH v2 2/2] powerpc: Replace GPL 2.0+ README.legal boilerplate with SPDX

2023-09-18 Thread Geert Uytterhoeven
se. Get rid of the references to non-existent files by replacing the boilerplate with SPDX license identifiers. Signed-off-by: Geert Uytterhoeven --- v2: - s/existant/existent/. --- arch/powerpc/kernel/ptrace/ptrace.c | 5 + arch/powerpc/kernel/signal.c| 5 + arch/powerpc/kernel/

[PATCH v2 0/2] m68k/powerpc: Kill references to non-existent README.legal

2023-09-18 Thread Geert Uytterhoeven
sx-11.mit.edu/Oct-07-1996/680x0/v0.9/linux-0.9.tar.gz [2] "[PATCH 0/2] m68k/powerpc: Kill references to non-existent README.legal https://lore.kernel.org/r/cover.1694095086.git.ge...@linux-m68k.org Geert Uytterhoeven (2): m68k: Replace GPL 2.0+ README.legal boilerplate with SPDX powerp

[PATCH v2 1/2] m68k: Replace GPL 2.0+ README.legal boilerplate with SPDX

2023-09-18 Thread Geert Uytterhoeven
se. Get rid of the references to non-existent files by replacing the boilerplate with SPDX license identifiers. Signed-off-by: Geert Uytterhoeven Acked-by: Greg Ungerer --- v2: - Add Acked-by, - s/existant/existent/. --- arch/m68k/68000/entry.S | 7 ++- arch/m68k/bvme6000/config.c | 5 +--

Re: [PATCH] um,ethertap: refactor deprecated strncpy

2023-09-12 Thread Geert Uytterhoeven
tup_args; > > --- > base-commit: 2dde18cd1d8fac735875f2e4987f11817cc0bc2c > change-id: > 20230911-strncpy-arch-um-os-linux-drivers-ethertap_user-c-859160d13f59 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linu

Re: Build regressions/improvements in v5.19-rc1

2022-06-06 Thread Geert Uytterhoeven
On Mon, 6 Jun 2022, Geert Uytterhoeven wrote: Below is the list of build error/warning regressions/improvements in v5.19-rc1[1] compared to v5.18[2]. Summarized: - build errors: +9/-10 [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/f2906aa863381afb0015a9eb7fefad885d4e5a56/ (all

Re: [PATCH 1/2] mm: Fix struct page layout on 32-bit systems

2021-04-20 Thread Geert Uytterhoeven
, dma_addr_t addr) > +{ > + page->dma_addr[0] = addr; > + if (sizeof(dma_addr_t) > sizeof(unsigned long)) > + page->dma_addr[1] = addr >> 16 >> 16; ... but we do have upper_32_bits() for a 32-bit right shift.

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-19 Thread Geert Uytterhoeven
Hi Andy, On Mon, Apr 19, 2021 at 4:14 PM Andy Shevchenko wrote: > On Mon, Apr 19, 2021 at 4:58 PM Andy Shevchenko > wrote: > > On Mon, Apr 19, 2021 at 4:54 PM Geert Uytterhoeven > > wrote: > > > > > In any case it's not true. We have the platform

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-19 Thread Geert Uytterhoeven
Hi Andy, On Mon, Apr 19, 2021 at 3:58 PM Andy Shevchenko wrote: > On Mon, Apr 19, 2021 at 4:54 PM Geert Uytterhoeven > wrote: > > On Mon, Apr 19, 2021 at 3:35 PM Andy Shevchenko > > wrote: > > > On Mon, Apr 19, 2021 at 4:02 PM Geert Uytterhoeven > > >

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-19 Thread Geert Uytterhoeven
Hi Andy, On Mon, Apr 19, 2021 at 3:35 PM Andy Shevchenko wrote: > On Mon, Apr 19, 2021 at 4:02 PM Geert Uytterhoeven > wrote: > > On Thu, Apr 15, 2021 at 10:50 AM Andy Shevchenko > > wrote: > > > On Thu, Apr 15, 2021 at 3:43 AM Geert Uytterhoeven > > >

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-19 Thread Geert Uytterhoeven
Hi Andy, On Thu, Apr 15, 2021 at 10:50 AM Andy Shevchenko wrote: > On Thu, Apr 15, 2021 at 3:43 AM Geert Uytterhoeven > wrote: > > On Wed, Apr 14, 2021 at 9:14 PM Andy Shevchenko > > wrote: > > > On Wed, Apr 14, 2021 at 08:55:21PM +0200, Geert Uytterhoeven wrote: &

Re: [PATCH V3 5/5] arm64: dts: renesas: beacon kits: Setup AVB refclk

2021-04-19 Thread Geert Uytterhoeven
Hi Adam, On Sat, Apr 17, 2021 at 3:54 PM Adam Ford wrote: > On Thu, Mar 4, 2021 at 2:04 AM Geert Uytterhoeven > wrote: > > On Wed, Feb 24, 2021 at 12:52 PM Adam Ford wrote: > > > The AVB refererence clock assumes an external clock that runs > > > > reference

Re: [PATCH V3 3/5] arm64: dts: renesas: Add fck to etheravb-rcar-gen3 clock-names list

2021-04-19 Thread Geert Uytterhoeven
On Wed, Feb 24, 2021 at 12:52 PM Adam Ford wrote: > The bindings have been updated to support two clocks, but the > original clock now requires the name fck. Add a clock-names > list in the device tree with fck in it. > > Signed-off-by: Adam Ford > Reviewed-by: Geert Uytte

Build regressions/improvements in v5.12-rc8

2021-04-19 Thread Geert Uytterhoeven
kb/src/drivers/net/ethernet/freescale/fec_main.c: warning: unused variable 'pdata' [-Wunused-variable]: 1667:28 => - /kisskb/src/drivers/net/ethernet/freescale/fec_main.c: warning: unused variable 'val' [-Wunused-variable]: 948:6 => - /kisskb/src/drivers/rtc/rtc-rx6110.c: warning: 'rx6

Re: [RFC v1 PATCH 3/3] driver: update all the code that use soc_device_match

2021-04-19 Thread Geert Uytterhoeven
ed on multiple platforms, some of them registering an SoC device, some of them not registering an SoC device. So there is no way to know the difference between "SoC device not registered, intentionally", and "SoC device not yet registered". soc_device_match() should only be

Re: [RFC v1 PATCH 1/3] drivers: soc: add support for soc_device_match returning -EPROBE_DEFER

2021-04-19 Thread Geert Uytterhoeven
_type.p && !early_soc_dev_attr) > + return ERR_PTR(-EPROBE_DEFER); > + > while (!ret) { > if (!(matches->machine || matches->family || > matches->revision || matches->soc_id)) Gr{oetje,eeting}s,

Re: [PATCH V2] mm: Define default value for FIRST_USER_ADDRESS

2021-04-19 Thread Geert Uytterhoeven
s > makes it much cleaner with reduced code. > > The default FIRST_USER_ADDRESS here would be skipped in > when the given platform overrides its value via . > Signed-off-by: Anshuman Khandual > arch/m68k/include/asm/pgtable_mm.h | 1 - Acked-by: Geert Uytte

Re: [PATCH] net: ethernet: ravb: Fix release of refclk

2021-04-19 Thread Geert Uytterhoeven
efclk") > Signed-off-by: Adam Ford Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talki

Re: [PATCH v2 5/6] kunit: mptcp: adhear to KUNIT formatting standard

2021-04-19 Thread Geert Uytterhoeven
est config patch when all other parts have entered mainline. Note that I would have made the same changes myself anyway, on -rc1 defconfig refresh. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal c

Re: [PATCH v4 4/7] arm64: dts: renesas: r8a77970: Add csi40 port@0

2021-04-15 Thread Geert Uytterhoeven
t; > + > > port@1 { > > #address-cells = <1>; > > #size-cells = <0>; \ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org I

Re: [PATCH bpf-next v3 3/6] bpf: Add a bpf_snprintf helper

2021-04-15 Thread Geert Uytterhoeven
Hi Andrii, On Thu, Apr 15, 2021 at 12:58 AM Andrii Nakryiko wrote: > On Wed, Apr 14, 2021 at 11:30 AM Florent Revest wrote: > > On Wed, Apr 14, 2021 at 8:02 PM Geert Uytterhoeven > > wrote: > > > On Wed, Apr 14, 2021 at 9:41 AM Andrii Nakryiko > > > wrote:

Re: [PATCH] m68k: Remove unneeded semicolon

2021-04-15 Thread Geert Uytterhoeven
On Thu, Apr 15, 2021 at 5:15 AM Wan Jiabing wrote: > Fix the following coccicheck warning: > > ./arch/m68k/include/asm/sun3xflop.h:109:2-3: Unneeded semicolon > > Signed-off-by: Wan Jiabing Reviewed-by: Geert Uytterhoeven i.e. will queue in the m68k for-v5.14 branch. Gr

Re: [PATCH v3 1/5] dt-bindings: media: max9286: Define 'maxim,gpio-poc'

2021-04-15 Thread Geert Uytterhoeven
; > I would have written poc-gpio to match the order of the GPIO bindings > > syntax. > > > > That's what I had :) but then the property gets matched against the > gpio schema and I get complains because it expects a phandle as first > argument... Maybe there's a

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-14 Thread Geert Uytterhoeven
Hi Andy, On Wed, Apr 14, 2021 at 9:14 PM Andy Shevchenko wrote: > On Wed, Apr 14, 2021 at 08:55:21PM +0200, Geert Uytterhoeven wrote: > > On Wed, Apr 14, 2021 at 8:18 PM Andy Shevchenko > > wrote: > > > On Wed, Apr 14, 2021 at 08:06:18PM +0200, Geert Uytterhoeven wrote

Re: [PATCH 4/5] sh: Get rid of oprofile leftovers

2021-04-14 Thread Geert Uytterhoeven
On Wed, Apr 14, 2021 at 3:53 PM Marc Zyngier wrote: > perf_pmu_name() and perf_num_counters() are unused. Drop them. > > Signed-off-by: Marc Zyngier Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux be

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-14 Thread Geert Uytterhoeven
Hi Andy, On Wed, Apr 14, 2021 at 8:18 PM Andy Shevchenko wrote: > On Wed, Apr 14, 2021 at 08:06:18PM +0200, Geert Uytterhoeven wrote: > > On Wed, Apr 14, 2021 at 11:24 AM Yicong Yang > > wrote: > > > On 2021/4/13 20:26, Geert Uytterhoeven wrote: > > > >

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-14 Thread Geert Uytterhoeven
Hi Yicong, On Wed, Apr 14, 2021 at 11:24 AM Yicong Yang wrote: > On 2021/4/13 20:26, Geert Uytterhoeven wrote: > > The HiSilicon Kunpeng I2C controller is only present on HiSilicon > > Kunpeng SoCs, and its driver relies on ACPI to probe for its presence. > > Hence add depe

Re: [PATCH bpf-next v3 3/6] bpf: Add a bpf_snprintf helper

2021-04-14 Thread Geert Uytterhoeven
map_direct_value_addr is set. > > +*/ > > + fmt_map_off = fmt_reg->off + fmt_reg->var_off.value; > > + err = fmt_map->ops->map_direct_value_addr(fmt_map, _addr, > > + fmt_map_off); > > + if (

Re: [PATCH V4 2/2] net: ethernet: ravb: Enable optional refclk

2021-04-14 Thread Geert Uytterhoeven
Hi Adam, On Wed, Apr 14, 2021 at 3:08 PM Adam Ford wrote: > On Tue, Apr 13, 2021 at 2:33 AM Geert Uytterhoeven > wrote: > > On Mon, Apr 12, 2021 at 3:27 PM Adam Ford wrote: > > > For devices that use a programmable clock for the AVB reference clock, > > > the

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-13 Thread Geert Uytterhoeven
Hi Andy, On Tue, Apr 13, 2021 at 4:41 PM Andy Shevchenko wrote: > On Tue, Apr 13, 2021 at 02:48:15PM +0200, Geert Uytterhoeven wrote: > > On Tue, Apr 13, 2021 at 2:37 PM Andy Shevchenko > > wrote: > > > On Tue, Apr 13, 2021 at 02:26:15PM +0200, Geert Uytterhoeven wrot

Re: [PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-13 Thread Geert Uytterhoeven
Hi Andy, On Tue, Apr 13, 2021 at 2:37 PM Andy Shevchenko wrote: > On Tue, Apr 13, 2021 at 02:26:15PM +0200, Geert Uytterhoeven wrote: > > The HiSilicon Kunpeng I2C controller is only present on HiSilicon > > Kunpeng SoCs, and its driver relies on ACPI to probe for its presence

[PATCH] spi: SPI_HISI_KUNPENG should depend on ARCH_HISI

2021-04-13 Thread Geert Uytterhoeven
river for Kunpeng SoCs") Signed-off-by: Geert Uytterhoeven --- drivers/spi/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index 1ffcad19c0562a39..7d438ad343598bb8 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig

[PATCH] i2c: I2C_HISI should depend on ARCH_HISI && ACPI

2021-04-13 Thread Geert Uytterhoeven
support. Fixes: d62fbdb99a85730a ("i2c: add support for HiSilicon I2C controller") Signed-off-by: Geert Uytterhoeven --- drivers/i2c/busses/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig index b5b4e0

[PATCH] irqchip: APPLE_AIC should depend on ARCH_APPLE

2021-04-13 Thread Geert Uytterhoeven
("irqchip/apple-aic: Add support for the Apple Interrupt Controller") Signed-off-by: Geert Uytterhoeven --- drivers/irqchip/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig index d3a14f304ec80caa..b165b3

Re: [PATCH V4 2/2] net: ethernet: ravb: Enable optional refclk

2021-04-13 Thread Geert Uytterhoeven
t; out_release: > + clk_disable_unprepare(priv->refclk); > free_netdev(ndev); > > pm_runtime_put(>dev); Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux

Re: New 'make dtbs_check W=1' warnings

2021-04-12 Thread Geert Uytterhoeven
oc. > renesas/r8a774a1-beacon-rzg2m-kit.dt.yaml: csi2@feaa: ports: > 'port@0' is a required property [...] I've replied to these as a response to your PR reply, see https://lore.kernel.org/linux-renesas-soc/camuhmdwhlnxgbsjp7vkudx-ynr9rskfke5ge5q_taru6hp9...@mail.gmail.com/ Gr{oetje,eeting}s,

Build regressions/improvements in v5.12-rc7

2021-04-12 Thread Geert Uytterhoeven
- /kisskb/src/drivers/net/ethernet/freescale/fec_main.c: warning: unused variable 'pdata' [-Wunused-variable]: 1667:28 => - /kisskb/src/drivers/net/ethernet/freescale/fec_main.c: warning: unused variable 'val' [-Wunused-variable]: 948:6 => - /kisskb/src/drivers/rtc/rtc-rx6110.c: warning: 'rx6

Re: [PATCH v2 1/1] serial: sh-sci: Respect deferred probe when getting IRQ

2021-04-12 Thread Geert Uytterhoeven
ARRAY_SIZE(sci_port->irqs); ++i) { > > + sci_port->irqs[i] = platform_get_irq_optional(dev, i); > > + if (sci_port->irqs[i] < 0) > > + return sci_port->irqs[i]; > > + if (sci_port->irqs[i] == 0)

Re: [PATCH] clk: renesas: rcar-usb2-clock-sel: Fix error handling in rcar_usb2_clock_sel_probe

2021-04-12 Thread Geert Uytterhoeven
The error path can be simplified by replacing the call to clk_register() by a call to devm_clk_register(), to match the style of the other initialization steps. > +pm_put: > + pm_runtime_put(dev); > + pm_runtime_disable(dev); > + return ret; This part has to

Re: [PATCH] kbuild: use ?= to assign CROSS_COMPILE by arch-Makefile

2021-04-12 Thread Geert Uytterhoeven
nger pick up the cross-compiler, but fall back to the native compiler, thus breaking the build. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technic

Re: [PATCH] arch/m68k/kernel/sys_m68k: Add missing mmap_read_lock() to sys_cacheflush()

2021-04-12 Thread Geert Uytterhoeven
On Wed, Apr 7, 2021 at 10:00 PM Liam Howlett wrote: > When the superuser flushes the entire cache, the mmap_read_lock() is not > taken, but mmap_read_unlock() is called. Add the missing > mmap_read_lock() call. > > Signed-off-by: Liam R. Howlett Reviewed-by: Geert Uytterhoeven

Re: [PATCH 12/20] kbuild: m68k: use common install script

2021-04-07 Thread Geert Uytterhoeven
e the m68k-only version of the install script. > > Cc: Geert Uytterhoeven > Cc: linux-m...@lists.linux-m68k.org > Signed-off-by: Greg Kroah-Hartman Thanks for your patch! Acked-by: Geert Uytterhoeven I guess it would "work" with the version number, too. Gr{oetje,eet

Re: [PATCH 00/20] kbuild: unify the install.sh script usage

2021-04-07 Thread Geert Uytterhoeven
or appended DTB? Even on x86, the script will bail out with "Cannot find LILO." if you're using Grub. Anyway, having less of them is good. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In pe

Re: [PATCH 4/9] sh: intc: Drop the use of irq_create_identity_mapping()

2021-04-06 Thread Geert Uytterhoeven
n, irq)) > + continue; > > intc_irq_xlate_set(irq, vect->enum_id, d); > intc_register_irq(desc, d, vect->enum_id, irq); Otherwise this seems to work fine on real hardware (landisk) and qemu (rts7751r2d). I did verify that the new fun

Re: [PATCH] media: VIDEO_IMX8_JPEG should depend on ARCH_MXC and not default to m

2021-04-06 Thread Geert Uytterhoeven
Hoi Hans, On Tue, Apr 6, 2021 at 11:24 AM Hans Verkuil wrote: > On 31/03/2021 10:17, Geert Uytterhoeven wrote: > > The i.MX8 QXP/QM integrated JPEG encoder/decoder is only present on > > Freescale/NXP i.MX8 QXP and QM SoCs. Hence add a dependency on > > ARCH_MXC, to pr

[PATCH v2] media: VIDEO_IMX8_JPEG should depend on ARCH_MXC and not default to m

2021-04-06 Thread Geert Uytterhoeven
"default y" if CONFIG_MODULES is not enabled), as merely enabling CONFIG_COMPILE_TEST should not enable additional code. Fixes: 2db16c6ed72ce644 ("media: imx-jpeg: Add V4L2 driver for i.MX8 JPEG Encoder/Decoder") Signed-off-by: Geert Uytterhoeven --- v2: - Really drop the "default m&quo

Re: [PATCH v3 3/5] RISC-V: Improve init_resources

2021-04-06 Thread Geert Uytterhoeven
Hi Nick, On Tue, Apr 6, 2021 at 10:11 AM Nick Kossifidis wrote: > Hello Geert, > Στις 2021-04-06 10:19, Geert Uytterhoeven έγραψε: > > On Mon, Apr 5, 2021 at 10:57 AM Nick Kossifidis > > wrote: > >> * Kernel region is always present and we know where it is, no >

Re: [PATCH] m68k: mvme147, mvme16x: Don't wipe PCC timer config bits

2021-04-06 Thread Geert Uytterhoeven
;m68k: mvme16x: Handle timer counter overflow") > Reported-and-tested-by: Michael Pavone > Signed-off-by: Finn Thain Thanks, applying, and queueing in the m68k for-v5.13 branch. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond i

Re: [PATCH v3 3/5] RISC-V: Improve init_resources

2021-04-06 Thread Geert Uytterhoeven
lled, so increase the > number of pre-allocated regions by one to be on the safe side > (reported and patched by Geert Uytterhoeven) > > Signed-off-by: Geert Uytterhoeven Where does this SoB come from? > Signed-off-by: Nick Kossifidis > --- a/arch/riscv/kernel/setup.c > ++

Build regressions/improvements in v5.12-rc6

2021-04-05 Thread Geert Uytterhoeven
ers/net/ethernet/freescale/fec_main.c: warning: unused variable 'pdata' [-Wunused-variable]: 1667:28 => - /kisskb/src/drivers/net/ethernet/freescale/fec_main.c: warning: unused variable 'val' [-Wunused-variable]: 948:6 => - /kisskb/src/drivers/rtc/rtc-rx6110.c: warning: 'rx6110_probe' d

[PATCH] dt-bindings: fpga: fpga-region: Convert to sugar syntax

2021-04-02 Thread Geert Uytterhoeven
Using overlay sugar syntax makes the DTS files easier to read (and write). While at it, fix two build issues: - "/dts-v1/" and "/plugin/" must be separate statements. - Add a missing closing curly brace. Signed-off-by: Geert Uytterhoeven --- .../devicetree/bindings/

Re: [PATCH] zero-fill colormap in drivers/video/fbdev/core/fbcmap.c

2021-04-01 Thread Geert Uytterhoeven
ak bug reported by syzbot at: > https://syzkaller.appspot.com/bug?id=741578659feabd108ad9e06696f0c1f2e69c4b6e > > Reported-by: syzbot+47fa9c9c648b76530...@syzkaller.appspotmail.com > Signed-off-by: Phillip Potter Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s,

Re: [PATCH 2/3] tracing: Use pr_crit() instead of long fancy messages

2021-04-01 Thread Geert Uytterhoeven
Hi Steven, On Wed, Mar 31, 2021 at 3:40 PM Steven Rostedt wrote: > On Wed, 31 Mar 2021 11:31:03 +0200 > Geert Uytterhoeven wrote: > > > This reduces kernel size by ca. 0.5 KiB. > > If you are worried about size, disable tracing and it will go away > entirely. 0.5KiB

Re: [PATCH 2/2] m68k: remove meaningless EXTRA_LDFLAGS

2021-03-31 Thread Geert Uytterhoeven
Reviewed-by: Geert Uytterhoeven i.e. will queue in the m68k for-v5.13 branch. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. Bu

Re: [PATCH 1/2] m68k: remove meaningless $(OS_OBJS)

2021-03-31 Thread Geert Uytterhoeven
On Wed, Mar 31, 2021 at 4:44 PM Masahiro Yamada wrote: > 'git grep OS_OBJS' hits only this line. $(OS_OBJS) is just empty. Indeed. All other users were removed in 2002. > Signed-off-by: Masahiro Yamada Reviewed-by: Geert Uytterhoeven i.e. will queue in the m68k for-v5.13 branch. Gr

[PATCH 2/3] tracing: Use pr_crit() instead of long fancy messages

2021-03-31 Thread Geert Uytterhoeven
KERN_WARNING to KERN_CRIT, and removing irrelevant text and graphics. This reduces kernel size by ca. 0.5 KiB. Fixes: 2184db46e425c2b8 ("tracing: Print nasty banner when trace_printk() is in use") Signed-off-by: Geert Uytterhoeven --- kernel/trace/trace.c | 17 +++-- 1 file

[PATCH] clk: Align provider-specific CLK_* bit definitions

2021-03-31 Thread Geert Uytterhoeven
at 40 columns, to increase uniformity. Signed-off-by: Geert Uytterhoeven --- include/linux/clk-provider.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h index 58f6fe866ae9b797..8f37829565f9640e 100644

[PATCH 0/3] Use pr_crit() instead of long fancy messages

2021-03-31 Thread Geert Uytterhoeven
level of such messages from KERN_WARNING to KERN_CRIT, and removes irrelevant text and graphics. It was made against linux-next, but applies to v5.12-rc5 with an offset for the last patch. Each of these patches reduces kernel size by ca. 0.5 KiB. Thanks for your comments! Geert Uytterhoeven (3

[PATCH 3/3] lib/vsprintf: Use pr_crit() instead of long fancy messages

2021-03-31 Thread Geert Uytterhoeven
KERN_WARNING to KERN_CRIT, and removing irrelevant text and graphics. This reduces kernel size by ca. 0.5 KiB. Fixes: 5ead723a20e0447b ("lib/vsprintf: no_hash_pointers prints all addresses as unhashed") Signed-off-by: Geert Uytterhoeven --- lib/vsprintf.c | 17 +++-- 1 file

[PATCH 1/3] iommu: Use pr_crit() instead of long fancy messages

2021-03-31 Thread Geert Uytterhoeven
KERN_WARNING to KERN_CRIT, and removing irrelevant text and graphics. This reduces kernel size by ca. 0.5 KiB. Fixes: bad614b24293ae46 ("iommu: Enable debugfs exposure of IOMMU driver internals") Signed-off-by: Geert Uytterhoeven --- drivers/iommu/iommu-debugfs.c | 19 -

[PATCH] phy: microchip: PHY_SPARX5_SERDES should depend on ARCH_SPARX5

2021-03-31 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven --- drivers/phy/microchip/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/phy/microchip/Kconfig b/drivers/phy/microchip/Kconfig index 0b1a818e01b88cc5..3728a284bf643d48 100644 --- a/drivers/phy/microchip/Kconfig +++ b/drivers/phy/microchip/Kconf

[PATCH] reset: RESET_INTEL_GW should depend on X86

2021-03-31 Thread Geert Uytterhoeven
t;) Signed-off-by: Geert Uytterhoeven --- drivers/reset/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig index 6dba675bcec4..c5dc1503de7909fa 100644 --- a/drivers/reset/Kconfig +++ b/drivers/reset/Kconfig @@ -83,6 +83,7 @@ config

[PATCH] media: VIDEO_IMX8_JPEG should depend on ARCH_MXC and not default to m

2021-03-31 Thread Geert Uytterhoeven
"default y" if CONFIG_MODULES is not enabled), as merely enabling CONFIG_COMPILE_TEST should not enable additional code. Fixes: 2db16c6ed72ce644 ("media: imx-jpeg: Add V4L2 driver for i.MX8 JPEG Encoder/Decoder") Signed-off-by: Geert Uytterhoeven --- drivers/media/platform/imx-jpeg/Kc

Re: [PATCH] clk: Mark fwnodes when their clock provider is added

2021-03-31 Thread Geert Uytterhoeven
Hi Stephen, On Wed, Mar 31, 2021 at 4:22 AM Stephen Boyd wrote: > Quoting Geert Uytterhoeven (2021-03-29 23:58:23) > > On Tue, Mar 30, 2021 at 3:53 AM Stephen Boyd wrote: > > > Quoting Saravana Kannan (2021-03-29 16:28:20) > > > > On Mon, Mar 29, 2021

Re: [PATCH v3 2/2] riscv: Cleanup KASAN_VMALLOC support

2021-03-30 Thread Geert Uytterhoeven
p = memblock_alloc(PAGE_SIZE, PAGE_SIZE); > > + set_pgd(pgd_k, pfn_pgd(PFN_DOWN(__pa(p)), > > PAGE_TABLE)); > > } > > - vaddr += PAGE_SIZE; > > - } > > + } while (pgd_k++, vaddr = next, vaddr != end); > > +} > &

Re: linux-next: manual merge of the drm tree with Linus' tree

2021-03-30 Thread Geert Uytterhoeven
29? Your tree has a second copy of the first conflict block after the second: /* skip suspend of gfx and psp for S0ix [...] /* skip unnecessary suspend if we do not initialize them yet */ [...] /* skip suspend of gfx

Re: [PATCH] clk: Mark fwnodes when their clock provider is added

2021-03-30 Thread Geert Uytterhoeven
Hi Stephen, On Tue, Mar 30, 2021 at 3:53 AM Stephen Boyd wrote: > Quoting Saravana Kannan (2021-03-29 16:28:20) > > On Mon, Mar 29, 2021 at 2:25 PM Stephen Boyd wrote: > > > Quoting Geert Uytterhoeven (2021-03-26 11:29:55) > > > > On Fri, Mar 26, 2021

Re: [PATCH V3 4/5] net: ethernet: ravb: Enable optional refclk

2021-03-29 Thread Geert Uytterhoeven
Hi Adam, On Mon, Mar 29, 2021 at 2:45 PM Adam Ford wrote: > On Thu, Mar 4, 2021 at 2:08 AM Geert Uytterhoeven > wrote: > > On Wed, Feb 24, 2021 at 12:52 PM Adam Ford wrote: > > > For devices that use a programmable clock for the AVB reference clock, > > > the

Build regressions/improvements in v5.12-rc5

2021-03-29 Thread Geert Uytterhoeven
g: missing braces around initializer [-Wmissing-braces]: 36:8 => - /kisskb/src/drivers/net/ethernet/aurora/nb8800.h: warning: "TCR_DIE" redefined: 92 => - /kisskb/src/drivers/net/ethernet/freescale/fec_main.c: warning: unused variable 'pdata' [-Wunused-variable]: 1667:28 => - /kiss

Re: [PATCH v2] arm64: dts: renesas: falcon-cpu: Add GP LEDs

2021-03-29 Thread Geert Uytterhoeven
function-enumerator = <3>; > + }; > + }; > + > memory@4800 { > device_type = "memory"; > /* first 128MB is reserved for secure area. */ Reviewed-by: Geert Uytterhoeven i.e. will queue in

Re: [PATCH 16/17] auxdisplay: ht16k33: Add support for segment displays

2021-03-29 Thread Geert Uytterhoeven
Hoi Robin, On Mon, Mar 29, 2021 at 9:09 AM Robin van der Gracht wrote: > On 2021-03-22 15:48, Geert Uytterhoeven wrote: > > The Holtek HT16K33 LED controller is not only used for driving > > dot-matrix displays, but also for driving segment displays. > > > > Add su

Re: [PATCH v2] lib: fix kconfig dependency on ARCH_WANT_FRAME_POINTERS

2021-03-28 Thread Geert Uytterhoeven
p;& !PPC && !S390 && !MICROBLAZE && !ARM > && !ARC && !X86 > + select FRAME_POINTER if MIPS || PPC || S390 || MICROBLAZE || ARM || > ARC || X86 s/select/depends on/? > select KALLSYMS > select KALLSYMS_ALL >

  1   2   3   4   5   6   7   8   9   10   >