Re: [PATCH] arm64: smp: remove unused variable

2020-12-28 Thread Geert Uytterhoeven
03026.2872532-1-geert+rene...@glider.be/ Note that the Fixes tag in my patch is different, as apparently the offending commit was rebased a few hours later. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In person

Re: [PATCH 01/18] arm64: dts: renesas: beacon kit: Configure programmable clocks

2020-12-28 Thread Geert Uytterhoeven
Hi Adam, On Thu, Dec 24, 2020 at 2:53 PM Adam Ford wrote: > On Tue, Dec 22, 2020 at 2:03 AM Geert Uytterhoeven > wrote: > > On Tue, Dec 22, 2020 at 2:39 AM Adam Ford wrote: > > > On Fri, Dec 18, 2020 at 7:16 AM Geert Uytterhoeven > > > wrote: > > >

[PATCH] m68k: defconfig: Update defconfigs for v5.11-rc1

2020-12-28 Thread Geert Uytterhoeven
- Enable modular build of netfilter nf_tables netdev REJECT support, - Enable modular build of the resource and cmdline API unit tests. Signed-off-by: Geert Uytterhoeven --- To be queued for v5.12. arch/m68k/configs/amiga_defconfig| 3 +++ arch/m68k/configs/apollo_defconfig | 3

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

2020-12-28 Thread Geert Uytterhoeven
On Mon, Dec 28, 2020 at 10:38 AM Geert Uytterhoeven wrote: > Below is the list of build error/warning regressions/improvements in > v5.11-rc1[1] compared to v5.10[2]. > > Summarized: > - build errors: +1/-3 > - build warnings: +12/-132 > > Happy fixing! ;-) > >

Build regressions/improvements in v5.11-rc1

2020-12-28 Thread Geert Uytterhoeven
; - /kisskb/src/net/bridge/netfilter/ebtables.c: warning: 'compat_copy_everything_to_user' uses dynamic stack allocation: 1767:1 => - /kisskb/src/net/sched/sch_cake.c: warning: the frame size of 1480 bytes is larger than 1280 bytes [-Wframe-larger-than=]: 2942:1 => - /kisskb/src/samples/se

[PATCH] openrisc: io: Add missing __iomem annotation to iounmap()

2020-12-28 Thread Geert Uytterhoeven
drivers/soc/renesas/rmobile-sysc.c:330:33: sparse: got void [noderef] __iomem *[assigned] base Fix this by adding the missing __iomem annotation to iounmap(). Reported-by: kernel test robot Signed-off-by: Geert Uytterhoeven --- arch/openrisc/include/asm/io.h | 2 +- arch/openrisc/mm

Re: [PATCH] m68k: let clk_enable() return immediately if clk is NULL

2020-12-27 Thread Geert Uytterhoeven
nts (no complaint from scripts/checkpatch.pl? See Documentation/process/submitting-patches.rst ). > + if (!clk) > + return -EINVAL; > + > spin_lock_irqsave(_lock, flags); > if ((clk->enabled++ == 0) && clk->clk_ops) > clk->

Re: [PATCH v2] local64.h: make mandatory

2020-12-27 Thread Geert Uytterhoeven
CC Arnd On Sun, Dec 27, 2020 at 3:48 AM Randy Dunlap wrote: > > Make mandatory in include/asm-generic/Kbuild > and remove all arch/*/include/asm/local64.h arch-specific files since > they only #include . > > This fixes build errors on arch/c6x/ and arch/nios2/ for > block/blk-iocost.c. > >

Re: [PATCH] arch: consolidate pm_power_off callback

2020-12-23 Thread Geert Uytterhoeven
Hi Enrico, On Tue, Dec 22, 2020 at 9:15 PM Enrico Weigelt, metux IT consult wrote: > On 22.12.20 19:54, Geert Uytterhoeven wrote: > > On Tue, Dec 22, 2020 at 7:46 PM Enrico Weigelt, metux IT consult > > wrote: > >> Move the pm_power_off callback into one g

Re: [PATCH] arch: consolidate pm_power_off callback

2020-12-22 Thread Geert Uytterhoeven
ny > - */ > -void (*pm_power_off)(void) = machine_power_off; Assignments like these are lost in the conversion. > -EXPORT_SYMBOL(pm_power_off); Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal c

Re: [PATCH v4 12/12] mfd: bd9571mwv: Add support for BD9574MWF

2020-12-22 Thread Geert Uytterhoeven
Hi Shimoda-san, On Tue, Dec 22, 2020 at 10:23 AM Yoshihiro Shimoda wrote: > > From: Geert Uytterhoeven, Sent: Tuesday, December 22, 2020 5:53 PM > > On Mon, Dec 21, 2020 at 3:56 AM Yoshihiro Shimoda > > wrote: > > > > --- a/drivers/mfd/bd9571mwv.c >

Re: [PATCH 5/5] WIP soc: v3u: allow WDT reset

2020-12-22 Thread Geert Uytterhoeven
iewed-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 talking to journalists I just say "programme

Re: [PATCH 4/5] arm64: dts: renesas: falcon: Enable watchdog timer

2020-12-22 Thread Geert Uytterhoeven
On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang wrote: > From: Hoang Vo > > Enable the watchdog on the Falcon board. > > Signed-off-by: Hoang Vo > [wsa: rebased to mainline] > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-devel

Re: [PATCH 3/5] arm64: dts: renesas: r8a779a0: Add RWDT node

2020-12-22 Thread Geert Uytterhoeven
On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang wrote: > From: Hoang Vo > > Add a device node for the Watchdog Timer (WDT) controller on the > R8A779A0 SoC. > > Signed-off-by: Hoang Vo > [wsa: rebased to mainline] > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytte

Re: [PATCH 2/5] clk: renesas: r8a779a0: Add RWDT clocks

2020-12-22 Thread Geert Uytterhoeven
On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang wrote: > And introduce critical clocks, too, because RWDT is one. > > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-clk-for-v5.12. Gr{oetje,eeting}s, Geert -- Geert Uyt

Re: [PATCH 1/5] dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support

2020-12-22 Thread Geert Uytterhoeven
On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang wrote: > Signed-off-by: Wolfram Sang 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 techni

Re: [PATCH 2/2] arm64: proper comment formatting in reboot handler

2020-12-22 Thread Geert Uytterhoeven
On Sat, Dec 19, 2020 at 3:38 PM Wolfram Sang wrote: > This comment was probably copied from arm32 and then shortened. It fits > to single line now. > > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert

Re: [PATCH v4 12/12] mfd: bd9571mwv: Add support for BD9574MWF

2020-12-22 Thread Geert Uytterhoeven
= { > - { "bd9571mwv", 0 }, > + { "bd9571mwv", ROHM_CHIP_TYPE_BD9571 }, > + { "bd9574mwf", ROHM_CHIP_TYPE_BD9574 }, Why add the chip types? These are unused, and the driver uses autodetection of the chip variant anyway. Gr{oetje,eeting}s,

Re: [PATCH v4 10/12] mfd: bd9571mwv: Use devm_regmap_add_irq_chip()

2020-12-22 Thread Geert Uytterhoeven
On Mon, Dec 21, 2020 at 3:57 AM Yoshihiro Shimoda wrote: > Use dev_regmap_add_irq_chip() to simplify the code. > > Signed-off-by: Yoshihiro Shimoda > Acked-for-MFD-by: Lee Jones Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert

Re: [PATCH v4 01/12] mfd: bd9571mwv: Use devm_mfd_add_devices()

2020-12-22 Thread Geert Uytterhoeven
MFD-by: Lee Jones 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 talking to journa

Re: [PATCH 01/18] arm64: dts: renesas: beacon kit: Configure programmable clocks

2020-12-22 Thread Geert Uytterhoeven
Hi Adam, On Tue, Dec 22, 2020 at 2:39 AM Adam Ford wrote: > On Fri, Dec 18, 2020 at 7:16 AM Geert Uytterhoeven > wrote: > > On Thu, Dec 17, 2020 at 12:52 PM Adam Ford wrote: > > > On Thu, Dec 17, 2020 at 2:16 AM Geert Uytterhoeven > > > wrote: > > >

Re: linux-next: Fixes tag needs some work in the clk tree

2020-12-21 Thread Geert Uytterhoeven
Hi Stephen, On Sun, Dec 20, 2020 at 10:35 PM Stephen Rothwell wrote: > On Thu, 10 Dec 2020 08:52:41 +0100 Geert Uytterhoeven > wrote: > > > trees can be pulled into linux-next? That would find this earlier. > > > > That sounds like a great idea, also for pinc

Re: [PATCH] m68k: Enable seccomp architecture tracking

2020-12-20 Thread Geert Uytterhoeven
Hi Adrian, On Sun, Dec 20, 2020 at 12:24 PM John Paul Adrian Glaubitz wrote: > On 12/20/20 9:51 AM, Geert Uytterhoeven wrote: > > To enable seccomp constant action bitmaps, we need to have a static > > mapping to the audit architecture and system call table size. > > >

[PATCH] m68k: Enable seccomp architecture tracking

2020-12-20 Thread Geert Uytterhoeven
To enable seccomp constant action bitmaps, we need to have a static mapping to the audit architecture and system call table size. Signed-off-by: Geert Uytterhoeven --- Needed for CONFIG_SECCOMP_CACHE_DEBUG. Note that upstream doesn't have m68k seccomp support yet. arch/m68k/include/asm/Kbuild

Re: [PATCH] clk: vc5: Use "idt,voltage-microvolt" instead of "idt,voltage-microvolts"

2020-12-19 Thread Geert Uytterhoeven
Hi Luca, On Fri, Dec 18, 2020 at 11:18 PM Luca Ceresoli wrote: > On 18/12/20 13:52, Geert Uytterhoeven wrote: > > Commit 45c940184b501fc6 ("dt-bindings: clk: versaclock5: convert to > > yaml") accidentally changed "idt,voltage-microvolts" to > > "id

Re: [PATCH 01/18] arm64: dts: renesas: beacon kit: Configure programmable clocks

2020-12-18 Thread Geert Uytterhoeven
Hi Adam, CC Shimoda-san On Thu, Dec 17, 2020 at 12:52 PM Adam Ford wrote: > On Thu, Dec 17, 2020 at 2:16 AM Geert Uytterhoeven > wrote: > > On Wed, Dec 16, 2020 at 6:03 PM Adam Ford wrote: > > > On Wed, Dec 16, 2020 at 8:55 AM Geert Uytterhoeven > > > wrote

Re: [PATCH 04/18] arm64: dts: renesas: beacon kit: Fix Audio Clock sources

2020-12-18 Thread Geert Uytterhoeven
Hi Adam, On Thu, Dec 17, 2020 at 1:01 PM Adam Ford wrote: > On Thu, Dec 17, 2020 at 4:54 AM Geert Uytterhoeven > wrote: > > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > > > The SoC was expecting two clock sources with different frequencies. > > >

Re: [PATCH 02/18] arm64: dts: renesas: beacon kit: Fix choppy Bluetooth Audio

2020-12-18 Thread Geert Uytterhoeven
Hi Adam, On Thu, Dec 17, 2020 at 1:16 PM Adam Ford wrote: > On Thu, Dec 17, 2020 at 4:41 AM Geert Uytterhoeven > wrote: > > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > > > The Bluetooth chip is capable of operating at 4Mbps, but the > > > max-speed settin

Re: [PATCH] ASoC: wm8962: Add optional mclk device tree binding

2020-12-18 Thread Geert Uytterhoeven
On Thu, Dec 17, 2020 at 5:27 PM Adam Ford wrote: > The driver can request an optional clock for mclk. > Update the txt file to reflect this. > > Suggested-by: Geert Uytterhoeven > Signed-off-by: Adam Ford Reviewed-by: Geert Uytterhoeven > --- a/Documentation/devicetree/bin

Re: [PATCH 06/18] arm64: dts: renesas: beacon: Configure Audio CODEC clocks

2020-12-18 Thread Geert Uytterhoeven
Hi Adam, On Thu, Dec 17, 2020 at 2:33 PM Adam Ford wrote: > On Thu, Dec 17, 2020 at 5:12 AM Geert Uytterhoeven > wrote: > > CC alsa-devel > > > > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > > > With the newly added configurable clock options, th

[PATCH] clk: vc5: Use "idt,voltage-microvolt" instead of "idt,voltage-microvolts"

2020-12-18 Thread Geert Uytterhoeven
n/devicetree/bindings/property-units.txt actually recommends using "microvolt". Fixes: 260249f929e81d3d ("clk: vc5: Enable addition output configurations of the Versaclock") Signed-off-by: Geert Uytterhoeven --- There are no upstream users yet, but they are planned for v5.1

Re: [PATCH v5 seccomp 5/5] seccomp/cache: Report cache data through /proc/pid/seccomp_cache

2020-12-18 Thread Geert Uytterhoeven
Hi YiFei, On Thu, Dec 17, 2020 at 7:34 PM YiFei Zhu wrote: > On Thu, Dec 17, 2020 at 6:14 AM Geert Uytterhoeven > wrote: > > Should there be a dependency on SECCOMP_ARCH_NATIVE? > > Should all architectures that implement seccomp have this? > > &

Re: [PATCH v5 seccomp 5/5] seccomp/cache: Report cache data through /proc/pid/seccomp_cache

2020-12-17 Thread Geert Uytterhoeven
SECCOMP_ARCH_NATIVE_NR); > + > +#ifdef SECCOMP_ARCH_COMPAT > + proc_pid_seccomp_cache_arch(m, SECCOMP_ARCH_COMPAT_NAME, > + f->cache.allow_compat, > + SECCOMP_ARCH_COMPA

Re: [PATCH 18/18] arm64: dts: renesas: Introduce r8a774e1-beacon-rzg2h-kit

2020-12-17 Thread Geert Uytterhoeven
> + < CPG_MOD 721>, > + < 1>, > + <_clk>, > + < 2>; > + clock-names = "du.0", "du.1", "du.3", > + "dclkin.0", "dclkin.1", "

Re: [PATCH 17/18] arm64: dts: renesas: Introduce r8a774b1-beacon-rzg2n-kit

2020-12-17 Thread Geert Uytterhoeven
t; + > + clocks = < CPG_MOD 724>, > + < CPG_MOD 723>, > + < CPG_MOD 721>, > + < 1>, > + <_clk>, > + < 2>; > + clock-names = "du.0", "du.1", "du.3", &g

Re: [PATCH 15/18] arm64: dts: renesas: beacon-rzg2m-kit: Rearange SoC unique functions

2020-12-17 Thread Geert Uytterhoeven
Hi Adam, On Thu, Dec 17, 2020 at 12:41 PM Geert Uytterhoeven wrote: > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > > In preparation for adding new dev kits, move anything specific to the > > RZ/G2M from the SOM-level and baseboard-levels and move them to the > > ki

Re: [PATCH 16/18] dt-bindings: arm: renesas: Add Beacon RZ/G2N and RZ/G2H boards

2020-12-17 Thread Geert Uytterhoeven
On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > Add beacon,beacon-rzg2n and beacon,beacon-rzg2h to the bindings > list. > > Signed-off-by: Adam Ford Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-devel for v5.12. Gr{oetje,eeting}s, Geer

Re: [PATCH 15/18] arm64: dts: renesas: beacon-rzg2m-kit: Rearange SoC unique functions

2020-12-17 Thread Geert Uytterhoeven
igned-off-by: Adam Ford Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-devel for v5.12. 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 ca

Re: [PATCH 14/18] arm64: dts: renesas: beacon: Correct I2C bus speeds

2020-12-17 Thread Geert Uytterhoeven
the bus support 400 kHz. Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-devel for v5.12. 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

Re: [PATCH 13/18] arm64: dts: renesas: beacon: Enable SPI

2020-12-17 Thread Geert Uytterhoeven
than a bunch of duplicates later. > > Signed-off-by: Adam Ford Thanks, I have to trust you on this one, i.e. will queue in renesas-devel for v5.12. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In pers

Re: [PATCH 12/18] arm64: dts: renesas: beacon: Better describe keys

2020-12-17 Thread Geert Uytterhoeven
inux,code = ; > - label = "Switch-4"; > + label = "Center"; > wakeup-source; > debounce-interval = <20>; > }; Wouldn't it make sense for the linux,

Re: [PATCH 10/18] arm64: dts: renesas: Don't make vccq_sdhi0 always on

2020-12-17 Thread Geert Uytterhoeven
On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > vccq_sdhi0 is referenced from sdhi0, so there is no need to force > this regualtor to be always-on. In theory it could help with regulator > low power modes in the future. > > Signed-off-by: Adam Ford Reviewed-by: Geert Uy

Re: [PATCH 09/18] arm64: dts: renesas: beacon: Fix RGB Display PWM Backlight

2020-12-17 Thread Geert Uytterhoeven
splay driver refers to the display as DPI, this also > renames the backlight to use DPI for consistency. > > Signed-off-by: Adam Ford Thanks, I have to trust you on this one, i.e. will queue in renesas-devel for v5.12. Gr{oetje,eeting}s, Geert -- Geert Uytterh

Re: [PATCH 08/18] arm64: dts: renesas: beacon: Enable SCIF4

2020-12-17 Thread Geert Uytterhoeven
on probe order, you want to add an aliases for scif4 to arch/arm64/boot/dts/renesas/r8a774a1-beacon-rzg2m-kit.dts. I see you did that for the rzg2h and rzg2n kits, but rzh2m lacks it. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's

Re: [PATCH 07/18] arm64: dts: renesas: beacon: Fix LVDS PWM Backlight

2020-12-17 Thread Geert Uytterhoeven
; > pwm2_pins: pwm2 { > groups = "pwm2_a"; > - function = "pwm2_a"; > + function = "pwm2"; > }; This part is definitely correct. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -

Re: [PATCH 06/18] arm64: dts: renesas: beacon: Configure Audio CODEC clocks

2020-12-17 Thread Geert Uytterhoeven
x6-logicpd-baseboard.dtsi and arch/arm64/boot/dts/freescale/imx8mm-beacon-baseboard.dtsi (both by your hand) use "xclk" instead of "mclk"? > DCVDD-supply = <_audio>; > DBVDD-supply = <_audio>; > AVDD-supp

Re: [PATCH 05/18] arm64: dts: renesas: beacon: Fix audio-1.8V pin enable

2020-12-17 Thread Geert Uytterhoeven
4a1-beacon-rzg2m-kit") > Signed-off-by: Adam Ford Thanks, I have to trust you on this one, i.e. will queue in renesas-devel for v5.12 (with the above fixed). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.or

Re: [PATCH 04/18] arm64: dts: renesas: beacon kit: Fix Audio Clock sources

2020-12-17 Thread Geert Uytterhoeven
> > sound_clk_pins: sound_clk { > - groups = "audio_clk_a_a"; > + groups = "audio_clk_a_a", "audio_clk_b_a"; > function = "audio_clk"; > }; Yes, this part was definitely missing. Gr{oetje,eeting}s

Re: [PATCH 03/18] arm64: dts: renesas: beacon kit: Remove unnecessary nodes

2020-12-17 Thread Geert Uytterhoeven
On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > VSPI0 and VSPB are already enabled by default. There is no need > to add extra nodes to enable them. Remove the redundant nodes. > > Signed-off-by: Adam Ford Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-devel fo

Re: [PATCH 02/18] arm64: dts: renesas: beacon kit: Fix choppy Bluetooth Audio

2020-12-17 Thread Geert Uytterhoeven
he max-speed in the proper > node. > > Fixes: a1d8a344f1ca ("arm64: dts: renesas: Introduce > r8a774a1-beacon-rzg2m-kit") > Signed-off-by: Adam Ford Reviewed-by: Geert Uytterhoeven i.e. will queue in renesas-devel for v5.12. Gr{oetje,eeting}s,

Re: [PATCH 01/18] arm64: dts: renesas: beacon kit: Configure programmable clocks

2020-12-17 Thread Geert Uytterhoeven
Hi Adam, On Wed, Dec 16, 2020 at 6:03 PM Adam Ford wrote: > On Wed, Dec 16, 2020 at 8:55 AM Geert Uytterhoeven > wrote: > > On Sun, Dec 13, 2020 at 7:38 PM Adam Ford wrote: > > > When the board was added, clock drivers were being updated done at > > > the sam

Re: [PATCH] hwmon: SENSORS_SBTSI should depend on X86

2020-12-17 Thread Geert Uytterhoeven
Hi Kun, Günter, On Wed, Dec 16, 2020 at 9:17 PM Guenter Roeck wrote: > On Wed, Dec 16, 2020 at 09:27:28AM -0800, Kun Yi wrote: > > On Wed, Dec 16, 2020 at 8:31 AM Guenter Roeck wrote: > > > > > > On Wed, Dec 16, 2020 at 02:46:41PM +0100, Geert Uytterhoeven wrote: >

Re: [PATCH 01/18] arm64: dts: renesas: beacon kit: Configure programmable clocks

2020-12-16 Thread Geert Uytterhoeven
t; <24576000>; > + > + OUT1 { > + idt,mode = ; > + idt,voltage-microvolts = <180>; Oops. The DT bindings say "idt,voltage-microvolt", the example in the DT bindings says "idt,voltage-microvolts",

Re: [PATCH 1/3] ARM: uncompress: Fix dbgadtb size parameter name

2020-12-16 Thread Geert Uytterhoeven
On Wed, Dec 16, 2020 at 2:46 PM Geert Uytterhoeven wrote: > From: Geert Uytterhoeven > > The dbgadtb macro is passed the size of the appended DTB, not the end > address. > > Fixes: c03e41470e901123 ("ARM: 9010/1: uncompress: Print the location of > appended DT

[PATCH 1/3] ARM: uncompress: Fix dbgadtb size parameter name

2020-12-16 Thread Geert Uytterhoeven
From: Geert Uytterhoeven The dbgadtb macro is passed the size of the appended DTB, not the end address. Fixes: c03e41470e901123 ("ARM: 9010/1: uncompress: Print the location of appended DTB") Signed-off-by: Geert Uytterhoeven Reviewed-by: Linus Walleij --- arch/arm/boot/compres

[PATCH] hwmon: SENSORS_SBTSI should depend on X86

2020-12-16 Thread Geert Uytterhoeven
nsors") Signed-off-by: Geert Uytterhoeven --- drivers/hwmon/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig index 1ecf697d8d99b70c..63d28f98108d4bb5 100644 --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -1536,6 +1536,7

[PATCH] platform/surface: SURFACE_PLATFORMS should depend on ACPI

2020-12-16 Thread Geert Uytterhoeven
this by moving the dependency on ACPI from the individual driver symbols to SURFACE_PLATFORMS. Signed-off-by: Geert Uytterhoeven --- drivers/platform/surface/Kconfig | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/platform/surface/Kconfig b/drivers/platform/surface

[PATCH trivial] dm crypt: Spelling s/cihper/cipher/

2020-12-16 Thread Geert Uytterhoeven
Fix a misspelling of "cipher". Signed-off-by: Geert Uytterhoeven --- drivers/md/dm-crypt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/md/dm-crypt.c b/drivers/md/dm-crypt.c index 392337f16ecfd87f..3e6a06c93865e311 100644 --- a/drivers/md/dm-crypt.c +++

[PATCH] crypto: CRYPTO_DEV_KEEMBAY_OCS_AES_SM4 should depend on ARCH_KEEMBAY

2020-12-16 Thread Geert Uytterhoeven
t;. Fixes: 88574332451380f4 ("crypto: keembay - Add support for Keem Bay OCS AES/SM4") Signed-off-by: Geert Uytterhoeven --- drivers/crypto/keembay/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/keembay/Kconfig b/drivers/crypto/k

Re: [PATCH] spi: Fix the clamping of spi->max_speed_hz

2020-12-16 Thread Geert Uytterhoeven
gt;controller->max_speed_hz non-zero check higher in the if, > so that we avoid a superfluous init to zero when both spi->max_speed_hz > and spi->controller->max_speed_hz are zero. > > Fixes: 9326e4f1e5dd ("spi: Limit the spi device max speed to controller's max &

Re: [PATCH tip/core/rcu 3/4] rcutorture: Make grace-period kthread report match RCU flavor being tested

2020-12-16 Thread Geert Uytterhoeven
Hi Paul, On Tue, Dec 15, 2020 at 7:24 PM Paul E. McKenney wrote: > On Tue, Dec 15, 2020 at 09:40:26AM +0100, Geert Uytterhoeven wrote: > > On Fri, Nov 6, 2020 at 12:40 AM wrote: > > > From: "Paul E. McKenney" > > > > > > At the end of the test and

[PATCH] dt-bindings: phy: Rename Intel Keem Bay USB PHY bindings

2020-12-16 Thread Geert Uytterhoeven
This is the only file not using the "intel,keembay-*" pattern. Fortunately the actual compatible value is already following the standard scheme. Fixes: 4086afa2a1627939 ("dt-bindings: phy: Add Intel Keem Bay USB PHY bindings") Signed-off-by: Geert Uytterhoeven --- ..

Re: [PATCH v2 03/10] regulator: bd9571mwv: rid of using struct bd9571mwv

2020-12-15 Thread Geert Uytterhoeven
On Tue, Dec 15, 2020 at 5:02 PM Geert Uytterhoeven wrote: > On Fri, Dec 11, 2020 at 3:03 PM Vaittinen, Matti > wrote: > > On Fri, 2020-12-11 at 20:27 +0900, Yoshihiro Shimoda wrote: > > > To simplify this driver, use dev_get_regmap() and > > > rid of using struct

Re: [PATCH v2 05/10] gpio: bd9571mwv: Use the SPDX license identifier

2020-12-15 Thread Geert Uytterhoeven
On Fri, Dec 11, 2020 at 2:47 PM Yoshihiro Shimoda wrote: > Use the SPDX license identifier instead of a local description. > > Signed-off-by: Yoshihiro Shimoda Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots

Re: [PATCH v2 03/10] regulator: bd9571mwv: rid of using struct bd9571mwv

2020-12-15 Thread Geert Uytterhoeven
signed int *mode) > > { > > int ret; > > > > - ret = regmap_read(bd->regmap, BD9571MWV_BKUP_MODE_CNT, mode); > > + ret = regmap_read(bdreg->regmap, BD9571MWV_BKUP_MODE_CNT, > > mode); > > if (ret) { > > - dev_err

Re: [PATCH] spi: Limit the spi device max speed to controller's max speed

2020-12-15 Thread Geert Uytterhoeven
ivision-by-zero later... Arguably all SPI controller drivers should fill in spi_controller.{min,max}_speed_hz, but as long as that is not the case, I think this patch should be reverted, or the check should be enhanced to make sure spi->controller->max_speed_hz is non-zero. Gr{oetje,eeting}s,

Re: [PATCH v5 2/8] dt-bindings: media: max9286: Document 'maxim,initial-reverse-channel-mV'

2020-12-15 Thread Geert Uytterhoeven
V: > > > > Use standard unit suffix. > > > > Which one ? :) Documentation/devicetree/bindings/property-units.txt > I see in v5.10 one 'mV', three 'mv', one 'millivolts', several > 'microvolts'. > > I'll go with the majority and make this > 'maxim,in

[PATCH] arm64/smp: Remove unused irq variable in arch_show_interrupts()

2020-12-15 Thread Geert Uytterhoeven
the variable. Fixes: 2f516e34383d0e97 ("arm64/smp: Use irq_desc_kstat_cpu() in arch_show_interrupts()") Signed-off-by: Geert Uytterhoeven --- One more issue in irq-core-2020-12-14. arch/arm64/kernel/smp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm64/kernel/smp.c b/

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

2020-12-15 Thread Geert Uytterhoeven
eembay/ocs-aes.c > +F:drivers/crypto/keembay/ocs-aes.h > + > INTEL MANAGEMENT ENGINE (mei) > M:Tomas Winkler > L:linux-kernel@vger.kernel.org Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots

Re: [PATCH tip/core/rcu 3/4] rcutorture: Make grace-period kthread report match RCU flavor being tested

2020-12-15 Thread Geert Uytterhoeven
e file is included only once. > +#endif // !defined(CONFIG_TINY_RCU) > +#endif /* #ifdef CONFIG_TASKS_RUDE_RCU */ > > > // 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 talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

Re: [GIT PULL] ARM: SoC fixes for v5.10, part 3

2020-12-15 Thread Geert Uytterhoeven
Hi Ulf, On Mon, Dec 14, 2020 at 9:23 PM Ulf Hansson wrote: > On Tue, 8 Dec 2020 at 08:32, Geert Uytterhoeven wrote: > > On Mon, Dec 7, 2020 at 11:15 PM Doug Anderson wrote: > > > On Mon, Dec 7, 2020 at 1:55 PM Arnd Bergmann wrote: > > > > On Mon, Dec 7, 2020

Re: [RFC] ravb: Add support for optional txc_refclk

2020-12-14 Thread Geert Uytterhoeven
clk_prepare_enable(priv->ref_clk); This can fail. Does this clock need to be enabled all the time? At least it should be disabled in the probe failure path, and in ravb_remove(). [1] Documentation/devicetree/bindings/net/renesas,etheravb.yaml Gr{oetje,eeting}s, Geert -- Gee

[GIT PULL] m68k updates for 5.11

2020-12-14 Thread Geert Uytterhoeven
command MAINTAINERS: Update m68k Mac entry Geert Uytterhoeven (2): m68k: defconfig: Update defconfigs for v5.10-rc1 m68k: defconfig: Enable KUnit tests Laurent Vivier (1): m68k: Remove unused mach_max_dma_address Youling Tang (2): m68k: Drop redundant NOTES in link

Build regressions/improvements in v5.10

2020-12-14 Thread Geert Uytterhoeven
missing braces around initializer [-Wmissing-braces]: 1212:9, 1245:9, 1317:9 => - warning: 136 bad relocations: N/A => - warning: 140 bad relocations: N/A => - warning: 148 bad relocations: N/A => - warning: You need at least binutils >= 2.19 to build a CONFIG_RELO

Re: [PATCH] media: vsp1: Fix an error handling path in the probe function

2020-12-13 Thread Geert Uytterhoeven
LLET 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 talking to journalists I just say

Re: [PATCH] powerpc/ps3: use dma_mapping_error()

2020-12-13 Thread Geert Uytterhoeven
ed-off-by: Vincent Stehlé 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 talking to journalis

[irqchip: irq/irqchip-next] irqchip/gic: Spelling s/REturn/Return/

2020-12-11 Thread irqchip-bot for Geert Uytterhoeven
The following commit has been merged into the irq/irqchip-next branch of irqchip: Commit-ID: 42a590b0fdf72498ebf47b01ddf006ee92cbfc70 Gitweb: https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms/42a590b0fdf72498ebf47b01ddf006ee92cbfc70 Author:Geert Uytterhoeven

[PATCH] serial: 8250_pci: Drop bogus __refdata annotation

2020-12-11 Thread Geert Uytterhoeven
uppressing future section warnings. Signed-off-by: Geert Uytterhoeven --- drivers/tty/serial/8250/8250_pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/8250/8250_pci.c b/drivers/tty/serial/8250/8250_pci.c index d5a513efb2613dff..689d8227f95f7dfb 100644 ---

[PATCH] media: sh_vou: Drop bogus __refdata annotation

2020-12-11 Thread Geert Uytterhoeven
ing future section warnings. Signed-off-by: Geert Uytterhoeven --- drivers/media/platform/sh_vou.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/sh_vou.c b/drivers/media/platform/sh_vou.c index b22dc1d725276f61..4ac48441f22c44a3 100644 --- a/drivers/medi

[PATCH] mwifiex: pcie: Drop bogus __refdata annotation

2020-12-11 Thread Geert Uytterhoeven
As the Marvell PCIE WiFi-Ex driver does not have any code or data located in initmem, there is no need to annotate the mwifiex_pcie structure with __refdata. Drop the annotation, to avoid suppressing future section warnings. Signed-off-by: Geert Uytterhoeven --- drivers/net/wireless/marvell

[PATCH] Input: adc-keys - drop bogus __refdata annotation

2020-12-11 Thread Geert Uytterhoeven
As the ADC ladder input driver does not have any code or data located in initmem, there is no need to annotate the adc_keys_driver structure with __refdata. Drop the annotation, to avoid suppressing future section warnings. Signed-off-by: Geert Uytterhoeven --- drivers/input/keyboard/adc

[PATCH] hwmon: (iio_hwmon) Drop bogus __refdata annotation

2020-12-11 Thread Geert Uytterhoeven
As the IIO hardware monitoring driver does not have any code or data located in initmem, there is no need to annotate the iio_hwmon_driver structure with __refdata. Drop the annotation, to avoid suppressing future section warnings. Signed-off-by: Geert Uytterhoeven --- drivers/hwmon

[PATCH] hwmon: (xgene) Drop bogus __refdata annotation

2020-12-11 Thread Geert Uytterhoeven
As the X-Gene hardware monitoring driver does not have any code or data located in initmem, there is no need to annotate the xgene_hwmon_driver structure with __refdata. Drop the annotation, to avoid suppressing future section warnings. Signed-off-by: Geert Uytterhoeven --- drivers/hwmon/xgene

[PATCH] clocksource/drivers/sh_cmt: Make sure channel clock supply is enabled

2020-12-10 Thread Geert Uytterhoeven
the clk_disable() call, else the timer's registers cannot be accessed in sh_cmt_setup_channel(). Signed-off-by: Geert Uytterhoeven --- Tested on R-Mobile APE6, R-Car M2-W, and R-Car H3 ES2.0. --- drivers/clocksource/sh_cmt.c | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff

Re: [PATCH v2] pinctrl: remove empty lines in pinctrl subsystem

2020-12-10 Thread Geert Uytterhoeven
enko > Signed-off-by: Zhaoyu Liu Thanks for your patch! > drivers/pinctrl/renesas/pfc-r8a77950.c| 1 - > drivers/pinctrl/renesas/pfc-r8a77951.c| 3 --- > drivers/pinctrl/renesas/pfc-r8a7796.c | 1 - > drivers/pinctrl/renesas/pfc-r8a77965.c| 1 -

[PATCH v11] ARM: uncompress: Validate start of physical memory against passed DTB

2020-12-10 Thread Geert Uytterhoeven
memory itself. This allows to boot Linux on r7s9210/rza2mevb using the 64 MiB of SDRAM on the RZA2MEVB sub board, which is located at 0x0C00 (CS3 space), i.e. not at a multiple of 128 MiB. Suggested-by: Nicolas Pitre Suggested-by: Ard Biesheuvel Signed-off-by: Geert Uytterhoeven Reviewed

Re: [PATCH 2/2] powerpc/64s: Trim offlined CPUs from mm_cpumasks

2020-12-10 Thread Geert Uytterhoeven
es_cpu_disable(void) > xive_smp_disable_cpu(); > else > xics_migrate_irqs_away(); > + > + cleanup_cpu_mmu_context(); > + > return 0; > } > > -- > 2.23.0 > -- Gr{oetje,eeting}s, Geert

Re: [PATCH 2/3] mfd: bd9571mwv: Make the driver more generic

2020-12-10 Thread Geert Uytterhoeven
Hi Shimoda-san, On Thu, Dec 10, 2020 at 5:10 AM Yoshihiro Shimoda wrote: > > From: Geert Uytterhoeven, Sent: Wednesday, December 9, 2020 10:26 PM > > On Tue, Dec 8, 2020 at 9:06 AM Yoshihiro Shimoda > > wrote: > > > > index 80c6ef0..57bdb6a 100644 >

Re: [PATCH 3/3] mfd: bd9571mwv: Add support for BD9574MWF

2020-12-10 Thread Geert Uytterhoeven
Hi Matti, Shimoda-san, On Thu, Dec 10, 2020 at 8:33 AM Vaittinen, Matti wrote: > On Thu, 2020-12-10 at 04:44 +, Yoshihiro Shimoda wrote: > > > From: Geert Uytterhoeven, Sent: Wednesday, December 9, 2020 10:30 > > > PM > > > > > > --- a/drivers/mf

Re: linux-next: Fixes tag needs some work in the clk tree

2020-12-09 Thread Geert Uytterhoeven
Hi Stephen², On Wed, Dec 9, 2020 at 6:29 PM Stephen Boyd wrote: > Quoting Geert Uytterhoeven (2020-12-08 00:37:00) > > On Mon, Dec 7, 2020 at 11:06 PM Stephen Rothwell > > wrote: > > > In commit > > > > > > c3f207f6d23d ("clk: renesas: r8a77

Re: [PATCH 3/3] mfd: bd9571mwv: Add support for BD9574MWF

2020-12-09 Thread Geert Uytterhoeven
CNT 0x56 Some of the above are the same as the corresponding BD9571MWV registers, so using the same define may simplify regulator support (cfr. BD9571MWV_DVFS_SETVID and BD9571MWV_DVFS_MONIVDAC). > +#define BD9574MWF_PART_NUMBER "BD9574MWF" BD9574MWF_PART_NAME

Re: [PATCH 2/3] mfd: bd9571mwv: Make the driver more generic

2020-12-09 Thread Geert Uytterhoeven
+ const struct regmap_config *regmap_config; > + const struct regmap_irq_chip *irq_chip; > + const struct mfd_cell *cells; > + int num_cells; I'd say "unsigned int", but mfd_add_devices() takes plain "int". Please put the "product_cod

[PATCH trivial] irqchip/gic: Spelling s/REturn/Return/

2020-12-09 Thread Geert Uytterhoeven
Fix a capitalization typo. Signed-off-by: Geert Uytterhoeven --- drivers/irqchip/irq-gic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/irqchip/irq-gic.c b/drivers/irqchip/irq-gic.c index 6053245a4754c092..a3c2f18131b2b1da 100644 --- a/drivers/irqchip/irq-gic.c

[PATCH trivial] ARM: uncompress: atags_to_fdt: Spelling s/REturn/Return/

2020-12-09 Thread Geert Uytterhoeven
Fix a capitalization typo. Signed-off-by: Geert Uytterhoeven --- arch/arm/boot/compressed/atags_to_fdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/compressed/atags_to_fdt.c b/arch/arm/boot/compressed/atags_to_fdt.c index 8452753efebe5621

Re: [PATCH 1/3] mfd: bd9571mwv: Use the SPDX license identifier

2020-12-09 Thread Geert Uytterhoeven
On Tue, Dec 8, 2020 at 9:06 AM Yoshihiro Shimoda wrote: > Use the SPDX license identifier instead of a local description. > > Signed-off-by: Yoshihiro Shimoda Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots

Re: [Ksummit-discuss] crediting bug reports and fixes folded into original patch

2020-12-09 Thread Geert Uytterhoeven
folded, as it's no longer clear whether the bot reported the original > issue the patch is fixing, or a bug in the fix. So we should have a different > variant. "Fixes-reported-by:" so it has the same prefix? Taken-into-account-comments-from: Any terse English word for that? Gr{

[PATCH 0/2] Documentation/kbuild: Document COMPILE_TEST and platform dependencies

2020-12-08 Thread Geert Uytterhoeven
Hi all, This patch series documents best practices for COMPILE_TEST and architecture/platform dependencies, like already in use in most subsystems, to serve as a point of reference. Thanks for your comments! Geert Uytterhoeven (2): Documentation/kbuild: Document COMPILE_TEST

[PATCH 1/2] Documentation/kbuild: Document COMPILE_TEST dependencies

2020-12-08 Thread Geert Uytterhoeven
Document best practises for using COMPILE_TEST dependencies. Signed-off-by: Geert Uytterhoeven --- Documentation/kbuild/kconfig-language.rst | 11 +++ 1 file changed, 11 insertions(+) diff --git a/Documentation/kbuild/kconfig-language.rst b/Documentation/kbuild/kconfig-language.rst

[PATCH 2/2] Documentation/kbuild: Document platform dependency practises

2020-12-08 Thread Geert Uytterhoeven
Document best practises for using architecture and platform dependencies. Signed-off-by: Geert Uytterhoeven --- Documentation/kbuild/kconfig-language.rst | 24 +++ 1 file changed, 24 insertions(+) diff --git a/Documentation/kbuild/kconfig-language.rst b/Documentation

Re: [PATCH] soc: fix comment for freeing soc_dev_attr

2020-12-08 Thread Geert Uytterhoeven
by the caller. What about "soc_dev->attr should be freed after calling soc_device_unregister."? > void soc_device_unregister(struct soc_device *soc_dev) > { > device_unregister(_dev->dev); Gr{oetje,eeting}s, Geert -- Geert Uytterhoeve

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