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 pinctrl.

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/K

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. > > > One

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, the au

[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
is not defined (so there are checks for it). > + SECCOMP_ARCH_NATIVE_NR); > + > +#ifdef SECCOMP_ARCH_COMPAT > + proc_pid_seccomp_cache_arch(m, SECCOMP_ARCH_COMPAT_NAME, > + f->cache.allow_compat, > +

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

2020-12-17 Thread Geert Uytterhoeven
t;&cpg CPG_MOD 723>, > + <&cpg CPG_MOD 721>, > + <&versaclock5 1>, > + <&x302_clk>, > + <&versaclock5 2>; > + clock-names = "du.0", "du.1", "du.3&

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

2020-12-17 Thread Geert Uytterhoeven
+&du { > + status = "okay"; Missing pinctrl properties? > + > + clocks = <&cpg CPG_MOD 724>, > + <&cpg CPG_MOD 723>, > + <&cpg CPG_MOD 721>, > + <&versaclock5 1>, >

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
gt; Signed-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,

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
n 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
o5 22 GPIO_ACTIVE_LOW>; > linux,code = ; > - label = "Switch-4"; > + label = "Center"; > wakeup-source; > debounce-interval = <20>; >

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
use the display 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 -- Gee

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

2020-12-17 Thread Geert Uytterhoeven
aliases may lead to failures, depending 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 Uytterho

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

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

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

2020-12-17 Thread Geert Uytterhoeven
date the bindings. Note that arch/arm/boot/dts/imx6-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 = <

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-m6

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

2020-12-17 Thread Geert Uytterhoeven
gt; @@ -591,7 +594,7 @@ sound_pins: sound { > }; > > sound_clk_pins: sound_clk { > - groups = "audio_clk_a_a"; > + groups = "audio_clk_a_a", "audio_clk_b_a"; > function = "audio_clk"; >

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
ing the 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
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", and the driver parses "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
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 m

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

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

2020-12-15 Thread Geert Uytterhoeven
unsigned 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) { > > -

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

2020-12-15 Thread Geert Uytterhoeven
r a division-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,eetin

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

2020-12-15 Thread Geert Uytterhoeven
al-reverse-channel-mV: > > > > 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'. > &g

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

2020-12-15 Thread Geert Uytterhoeven
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
pto/keembay/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

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

2020-12-15 Thread Geert Uytterhoeven
w the 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
(void)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,

[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
- /kisskb/src/net/smc/smc_llc.c: warning: (near initialization for 'del_llc.hd') [-Wmissing-braces]: 1245:9 => - /kisskb/src/net/smc/smc_llc.c: warning: (near initialization for 'delllc.hd') [-Wmissing-braces]: 1317:9 => - /kisskb/src/net/smc/smc_llc.c: wa

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

2020-12-13 Thread Geert Uytterhoeven
stophe JAILLET 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 jou

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 jou

[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
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(-)

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

2020-12-10 Thread Geert Uytterhoeven
Shevchenko > 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

[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
tatic int pseries_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,

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
> +#define BD9574MWF_VDCORE_PGD_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 "

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

2020-12-09 Thread Geert Uytterhoeven
; unsigned int? > + char *part_number; part_name? > + 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()

[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

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

2020-12-09 Thread Geert Uytterhoeven
for the reported issue in the > patch is 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-com

[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(&soc_dev->dev); Gr{oetje,eeting}s, Geert --

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

2020-12-08 Thread Geert Uytterhoeven
upport for R-Car > V3U") > > has these problem(s): > > - Target SHA1 does not exist Oops, my bad. > Maybe you meant > > Fixes: 17bcc8035d2d ("clk: renesas: cpg-mssr: Add support for R-Car V3U") Yes I did. Mike/Stephen: do you want me to respin my pull reques

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

2020-12-07 Thread Geert Uytterhoeven
Hi Doug, 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 at 9:23 PM Geert Uytterhoeven > > wrote: > > > On Tue, Dec 1, 2020 at 3:06 PM Arnd Bergmann wrote: > > > > On Tue,

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

2020-12-07 Thread Geert Uytterhoeven
ithout aliases, as there is no support for dynamically updating the aliases list). 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

Re: [PATCH 1/2] clocksource/drivers/sh_cmt: Fix potential deadlock when calling runtime PM

2020-12-07 Thread Geert Uytterhoeven
rying to lock: c254cd2c (&dev->power.lock){}-{3:3}, at: __pm_runtime_resume+0x54/0x80 other info that might help us debug this: context-{5:5} 3 locks held by swapper/1: #0: c254ccd0 (&dev->mutex){}-{4:4}, at: device_driver_attach+0x18/0x5c #1: c0bed23

Re: Build regressions/improvements in v5.10-rc7

2020-12-07 Thread Geert Uytterhoeven
On Mon, Dec 7, 2020 at 1:08 PM Geert Uytterhoeven wrote: > JFYI, when comparing v5.10-rc7[1] to v5.10-rc6[3], the summaries are: > - build errors: +1/-0 + /kisskb/src/arch/powerpc/platforms/powermac/smp.c: error: implicit declaration of function 'cleanup_cpu_mmu_context' [

Build regressions/improvements in v5.10-rc7

2020-12-07 Thread Geert Uytterhoeven
/src/net/smc/smc_llc.c: warning: (near initialization for 'add_llc.hd') [-Wmissing-braces]: 1212:9 => - /kisskb/src/net/smc/smc_llc.c: warning: (near initialization for 'del_llc.hd') [-Wmissing-braces]: 1245:9 => - /kisskb/src/net/smc/smc_llc.c: warning: (near in

Re: [PATCH] MAINTAINERS: Update 68k Mac entry

2020-12-06 Thread Geert Uytterhoeven
ners to deal with. Update the "M68K ON APPLE MACINTOSH" section > accordingly. > > Cc: Michael Ellerman > Cc: Benjamin Herrenschmidt > Cc: Joshua Thompson > Cc: linuxppc-...@lists.ozlabs.org > Cc: linux-m...@lists.linux-m68k.org > Signed-off-by: Finn Thain Re

Re: [PATCH v2] clk: renesas: r9a06g032: Drop __packed for portability

2020-12-05 Thread Geert Uytterhoeven
Hi Stephen, On Sat, Dec 5, 2020 at 7:24 AM Stephen Boyd wrote: > Quoting Geert Uytterhoeven (2020-11-30 00:57:43) > > The R9A06G032 clock driver uses an array of packed structures to reduce > > kernel size. However, this array contains pointers, which are no longer > > a

Re: [PATCH] macintosh/adb-iop: Always wait for reply message from IOP

2020-12-04 Thread Geert Uytterhoeven
} So if we're not expecting a reply. It's ignored. Just wondering: what kind of messages are being dropped? If reply packets from different ADB commands become mixed up, they are still (expected?) replies to messages we sent before. Why shouldn't we depend on receiving

Re: [PATCH] macintosh/adb-iop: Send correct poll command

2020-12-04 Thread Geert Uytterhoeven
Implement idle -> sending state > transition") WARNING: Unknown commit id 'fa3b5a9929fc', maybe rebased or not pulled? 32226e817043? > Tested-by: Stan Johnson > Signed-off-by: Finn Thain Thanks, will queue in the m68k for-v5.11 branch. Gr{oetje,eeting}s,

Re: [PATCH v10 1/3] ARM: uncompress: Add be32tocpu macro

2020-12-04 Thread Geert Uytterhoeven
Hi Ard, On Thu, Dec 3, 2020 at 1:50 PM Ard Biesheuvel wrote: > On Thu, 3 Dec 2020 at 13:19, Geert Uytterhoeven > wrote: > > DTB stores all values as 32-bit big-endian integers. > > Add a macro to convert such values to native CPU endianness, to reduce > > duplicatio

Re: [PATCH v10 2/3] ARM: uncompress: Add OF_DT_MAGIC macro

2020-12-04 Thread Geert Uytterhoeven
On Thu, Dec 3, 2020 at 5:27 PM Nicolas Pitre wrote: > On Thu, 3 Dec 2020, Geert Uytterhoeven wrote: > > The DTB magic marker is stored as a 32-bit big-endian value, and thus > > depends on the CPU's endianness. Add a macro to define this value in > > native endianness

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

2020-12-04 Thread Geert Uytterhoeven
Hi Linus, On Fri, Dec 4, 2020 at 9:08 AM Linus Walleij wrote: > On Wed, Nov 18, 2020 at 8:13 PM Geert Uytterhoeven > wrote: > > The dbgadtb macro is passed the size of the appended DTB, not the end > > address. > > > > Fixes: c03e41470e901123 ("ARM: 9010/1

Re: [PATCH] scsi/NCR5380: Remove in_interrupt() test

2020-12-04 Thread Geert Uytterhoeven
ecause it is ignored in atomic context. So you may as well > specify 0 jiffies at these call sites. (There will be a 1 jiffy timeout > applied regardless.) Notwithstanding HZ / 64 breaks if HZ < 64. Possible values of HZ in the kernel: 24 32 48 64 100 128 200 250 256 300 500 1000 1024 1200

[tip: timers/core] dt-bindings: timer: renesas: tmu: Convert to json-schema

2020-12-03 Thread tip-bot2 for Geert Uytterhoeven
The following commit has been merged into the timers/core branch of tip: Commit-ID: b7c0fed5ccf2c5cb4bb43ddc6b1625f042a83d0a Gitweb: https://git.kernel.org/tip/b7c0fed5ccf2c5cb4bb43ddc6b1625f042a83d0a Author:Geert Uytterhoeven AuthorDate:Tue, 10 Nov 2020 17:20:14 +01:00

Re: [PATCH v10 3/3] ARM: uncompress: Validate start of physical memory against passed DTB

2020-12-03 Thread Geert Uytterhoeven
Hi Russell, On Thu, Dec 3, 2020 at 3:05 PM Russell King - ARM Linux admin wrote: > On Thu, Dec 03, 2020 at 01:19:16PM +0100, Geert Uytterhoeven wrote: > > diff --git a/arch/arm/boot/compressed/head.S > > b/arch/arm/boot/compressed/head.S > > index d9cce7238a365081..1b

[PATCH v2] ARM: Parse kdump DT properties

2020-12-03 Thread Geert Uytterhoeven
derive the start of memory is still used if "linux,usable-memory-range" is absent, and the "elfcorehdr=" and "mem=" kernel parameters are still parsed. Loosely based on the ARM64 version by Akashi Takahiro. Signed-off-by: Geert Uytterhoeven --- This depends on &

[PATCH v10 2/3] ARM: uncompress: Add OF_DT_MAGIC macro

2020-12-03 Thread Geert Uytterhoeven
The DTB magic marker is stored as a 32-bit big-endian value, and thus depends on the CPU's endianness. Add a macro to define this value in native endianness, to reduce #ifdef clutter and (future) duplication. Signed-off-by: Geert Uytterhoeven --- v10: - New. --- arch/arm/boot/compr

[PATCH v10 3/3] ARM: uncompress: Validate start of physical memory against passed DTB

2020-12-03 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 --- v10

[PATCH v10 0/3] ARM: uncompress: Validate start of physical memory against passed DTB

2020-12-03 Thread Geert Uytterhoeven
variety of boards using a passed or appended DTB. Thanks for your comments! [*] "[PATCH v9] ARM: boot: Validate start of physical memory against DTB" (https://lore.kernel.org/r/20200902153606.13652-1-geert+rene...@glider.be) Geert Uytterhoeven (3): ARM: uncompress: Add be32tocpu m

[PATCH v10 1/3] ARM: uncompress: Add be32tocpu macro

2020-12-03 Thread Geert Uytterhoeven
DTB stores all values as 32-bit big-endian integers. Add a macro to convert such values to native CPU endianness, to reduce duplication. Signed-off-by: Geert Uytterhoeven --- v10: - New. --- arch/arm/boot/compressed/head.S | 26 -- 1 file changed, 12 insertions(+), 14

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