Re: [PATCH] remoteproc: st: Use device_get_match_data()

2023-10-09 Thread Patrice CHOTARD
; - ddata->config = (struct st_rproc_config *)match->data; > + ddata->config = (struct st_rproc_config *)device_get_match_data(dev); > + if (!ddata->config) > + goto free_rproc; > > platform_set_drvdata(pdev, rproc); > Reviewed-by: Patrice Chotard Thanks Patrice

Re: linux-next: build warning after merge of the spi tree

2021-04-20 Thread Patrice CHOTARD
Hi Mark I will send a patch to fix this warning, sorry for that. Patrice. On 4/20/21 5:48 AM, Stephen Rothwell wrote: > Hi all, > > After merging the spi tree, today's linux-next build (x86_64 allmodconfig) > produced this warning: > > In file included from include/linux/printk.h:409, >

Re: [Linux-stm32] [RESEND PATCH 0/3] MAINTAINERS: update STMicroelectronics email

2021-04-12 Thread Patrice CHOTARD
On 4/12/21 3:48 PM, Arnd Bergmann wrote: > On Mon, Apr 12, 2021 at 12:19 PM Patrice CHOTARD > wrote: >> >> Hi >> >> I think this series has been forgotten, any chance to see it merged into >> v5.13 ? > > It's in -rc7, but it appears that

Re: [Linux-stm32] [RESEND PATCH 0/3] MAINTAINERS: update STMicroelectronics email

2021-04-12 Thread Patrice CHOTARD
Hi I think this series has been forgotten, any chance to see it merged into v5.13 ? Thanks Patrice On 3/11/21 4:24 PM, Patrice CHOTARD wrote: > Hi > > It's just a gentle reminder to not forgot this series. > > Thanks > > Patrice > > On 2/16/21 8:49 AM, p

Re: [PATCH -next] mmc: sdhci-st: Remove unnecessary error log

2021-04-09 Thread Patrice CHOTARD
dev_warn(&pdev->dev, "FlashSS Top Dly registers not available"); > + if (IS_ERR(pdata->top_ioaddr)) > pdata->top_ioaddr = NULL; > - } > > pltfm_host->clk = clk; > pdata->icnclk = icnclk; > Reviewed-by: Patrice Chotard Thanks Patrice

Re: [PATCH v3 4/7] clk: st: clkgen-pll: embed soc clock outputs within compatible data

2021-03-31 Thread Patrice CHOTARD
t; + clkgen_c32_pll_setup(np, > + (struct clkgen_pll_data_clks *) &st_pll3200c32_c1_data); > +} > +CLK_OF_DECLARE(c32_pll1_c0, "st,clkgen-pll1-c0", clkgen_c32_pll1_c0_setup); > + > static void __init clkgen_c32_plla9_setup(struct device_node *np) > { > clkgen_c32_pll_setup(np, > - (struct clkgen_pll_data *) &st_pll3200c32_407_a9); > + (struct clkgen_pll_data_clks *) &st_pll3200c32_407_a9_data); > } > CLK_OF_DECLARE(c32_plla9, "st,stih407-clkgen-plla9", clkgen_c32_plla9_setup); > > static void __init clkgen_c28_plla9_setup(struct device_node *np) > { > clkgen_c32_pll_setup(np, > - (struct clkgen_pll_data *) &st_pll4600c28_418_a9); > + (struct clkgen_pll_data_clks *) &st_pll4600c28_418_a9_data); > } > CLK_OF_DECLARE(c28_plla9, "st,stih418-clkgen-plla9", clkgen_c28_plla9_setup); > Reviewed-by: Patrice Chotard Thanks

Re: [PATCH v3 6/7] clk: st: clkgen-fsyn: embed soc clock outputs within compatible data

2021-03-31 Thread Patrice CHOTARD
void __init st_of_quadfs660D_setup(struct device_node *np) > { > - st_of_quadfs_setup(np, (struct clkgen_quadfs_data *) &st_fs660c32_D); > + st_of_quadfs_setup(np, > + (struct clkgen_quadfs_data_clks *) &st_fs660c32_D_data); > } > CLK_OF_DECLARE(quadfs660D, "st,quadfs&

Re: [PATCH v3 1/7] clk: st: clkgen-pll: remove unused variable of struct clkgen_pll

2021-03-31 Thread Patrice CHOTARD
ndiv; > u32 idf; > - u32 odf; > u32 cp; > }; > > Reviewed-by: Patrice Chotard Thanks

Re: [PATCH v3 2/7] clk: st: flexgen: embed soc clock outputs within compatible data

2021-03-31 Thread Patrice CHOTARD
0", }, > + { .name = "clk-st231-gp-1", }, > + { .name = "clk-st231-dmu", }, > + /* This clk needs to be on to keep bus interconnect alive */ > + { .name = "clk-icn-lmi", .flags = CLK_IS_CRITICAL }, > + { .

Re: [RESEND PATCH 0/3] MAINTAINERS: update STMicroelectronics email

2021-03-11 Thread Patrice CHOTARD
Hi It's just a gentle reminder to not forgot this series. Thanks Patrice On 2/16/21 8:49 AM, patrice.chot...@foss.st.com wrote: > From: Patrice Chotard > > This series: > _ Update st.com to foss.st.com email for some maintainers. > _ Remove Vincent Abriou as STI/STM

Re: [RESEND PATCH 0/3] MAINTAINERS: update STMicroelectronics email

2021-03-11 Thread Patrice CHOTARD
Hi It's just a gentle reminder to not forgot this series. Thanks Patrice On 2/16/21 8:49 AM, patrice.chot...@foss.st.com wrote: > From: Patrice Chotard > > This series: > _ Update st.com to foss.st.com email for some maintainers. > _ Remove Vincent Abriou as STI/STM

Re: [PATCH 0/3] MAINTAINERS: update STMicroelectronics email

2021-02-15 Thread Patrice CHOTARD
Hi Arnd On 2/15/21 3:19 PM, Arnd Bergmann wrote: > On Mon, Feb 15, 2021 at 2:17 PM Patrice CHOTARD > wrote: >> >> Hi Arnd, Olof, Kevin >> >> What is the best way to get this series merged ? >> Do you pick it and apply it directly, or do we integrate it in the

Re: [PATCH 0/3] MAINTAINERS: update STMicroelectronics email

2021-02-15 Thread Patrice CHOTARD
Hi Arnd, Olof, Kevin What is the best way to get this series merged ? Do you pick it and apply it directly, or do we integrate it in the next STM32 pull request ? Thanks Patrice On 2/1/21 11:00 AM, patrice.chot...@foss.st.com wrote: > From: Patrice Chotard > > This series: >

Re: [PATCH] ARM: sti: Fix spelling mistake in Kconfig "targetted" -> "targeted"

2020-12-17 Thread Patrice CHOTARD
les support for STMicroelectronics Digital Consumer > - Electronics family StiH407 parts, targetted at set-top-box > + Electronics family StiH407 parts, targeted at set-top-box > and other digital audio/video applications using Flattened Device > Trees. > Reviewed-by: Patrice Chotard Thanks Patrice

Re: [PATCH v3 07/10] arm: dts: stih407-family: Harmonize DWC USB3 DT nodes name

2020-12-09 Thread Patrice CHOTARD
status = "disabled"; > > - dwc3: dwc3@990 { > + dwc3: usb@990 { > compatible = "snps,dwc3"; > reg = <0x0990 0x10>; > interrupts = IRQ_TYPE_LEVEL_HIGH>; Reviewed-by: Patrice Chotard Thanks Patrice

Re: [PATCH v2 0/2] arm: sti: LL_UART updates & STiH418 addition

2020-11-19 Thread Patrice CHOTARD
After discussion with Alain, he will take care to submit them to Russell's patch tracker. Thanks On 11/19/20 10:57 AM, Patrice CHOTARD wrote: > Hi Linus > > On 11/19/20 9:26 AM, Linus Walleij wrote: >> On Thu, Nov 19, 2020 at 9:08 AM Patrice CHOTARD >> wrote: >

Re: [PATCH v2 0/2] arm: sti: LL_UART updates & STiH418 addition

2020-11-19 Thread Patrice CHOTARD
Hi Linus On 11/19/20 9:26 AM, Linus Walleij wrote: > On Thu, Nov 19, 2020 at 9:08 AM Patrice CHOTARD > wrote: > >> Is there any reason for this series not being merged ? > Did you put them into Russell's patch tracker as I asked? > I haven't seen them there

Re: [PATCH v2 0/2] arm: sti: LL_UART updates & STiH418 addition

2020-11-19 Thread Patrice CHOTARD
Hi Russell Is there any reason for this series not being merged ? Thanks Patrice On 8/30/20 9:57 PM, Alain Volmat wrote: > This serie update the STi Platform LL_UART code to rely on > DEBUG_UART_PHYS & DEBUG_UART_VIRT and add the STiH418 SoC support. > > Alain Volmat (2): > arm: use DEBUG_UA

Re: [PATCH] reset: sti: reset-syscfg: fix struct description warnings

2020-11-18 Thread Patrice CHOTARD
tem configuration registers. > + * struct syscfg_reset_controller - A reset controller which groups together > + * a set of related reset bits, which may be located in different system > + * configuration registers. > * > * @rst: base reset controller structure. > * @active_low: are the resets in this controller active low, i.e. clearing Reviewed-by: Patrice Chotard Thanks Patrice

Re: [PATCH v1 0/4] PM: Add dev_wakeup_path() helper

2020-11-18 Thread Patrice CHOTARD
Hi Rafael On 11/18/20 2:08 PM, Rafael J. Wysocki wrote: > On Fri, Nov 6, 2020 at 8:34 AM wrote: >> From: Patrice Chotard >> >> Add dev_wakeup_path() helper to avoid to spread >> dev->power.wakeup_path test in drivers. > OK > >> Cc: amelie.

Re: [PATCH 2/3] watchdog: st_lpc_wdt: remove unnecessary CONFIG_PM_SLEEP

2020-10-29 Thread Patrice CHOTARD
evice *dev)   |    ^~~ Can you add missing "__maybe unused"  attribute ? Thanks Patrice On 10/29/20 8:58 AM, Patrice CHOTARD wrote: > Hi Coiby > > On 10/29/20 8:53 AM, Coiby Xu wrote: >> SIMPLE_DEV_PM_OPS has already took good care of CONFIG_PM_CONFIG. >> >&

Re: [PATCH 2/3] watchdog: st_lpc_wdt: remove unnecessary CONFIG_PM_SLEEP

2020-10-29 Thread Patrice CHOTARD
LEEP > static int st_wdog_suspend(struct device *dev) > { > struct st_wdog *st_wdog = watchdog_get_drvdata(&st_wdog_dev); > @@ -285,7 +284,6 @@ static int st_wdog_resume(struct device *dev) > > return 0; > } > -#endif > > static SIMPLE_DEV_PM_O

Re: [PATCH v2 2/2] arm: sti LL_UART: add STiH418 SBC UART0 support

2020-10-06 Thread Patrice CHOTARD
TI81XXUART1 > default 0xfa022000 if DEBUG_TI81XXUART2 > @@ -1864,7 +1878,8 @@ config DEBUG_UART_VIRT > DEBUG_BCM63XX_UART || DEBUG_ASM9260_UART || \ > DEBUG_SIRFSOC_UART || DEBUG_DIGICOLOR_UA0 || \ > DEBUG_AT91_UART || DEBUG_STM32_UART || \ &g

Re: [PATCH v2 1/2] arm: use DEBUG_UART_PHYS and DEBUG_UART_VIRT for sti LL_UART

2020-10-06 Thread Patrice CHOTARD
> -#endif > - > -#ifndef DEBUG_LL_UART_BASE > -#error "DEBUG UART is not Configured" > -#endif > - > #define ASC_TX_BUF_OFF 0x04 > #define ASC_CTRL_OFF0x0c > #define ASC_STA_OFF 0x14 > @@ -37,8 +15,8 @@ > > > .macro addruart, rp, rv, tmp > - ldr \rp, =DEBUG_LL_UART_BASE @ physical base > - ldr \rv, =VIRT_ADDRESS(DEBUG_LL_UART_BASE) @ virt base > + ldr \rp, =CONFIG_DEBUG_UART_PHYS @ physical base > + ldr \rv, =CONFIG_DEBUG_UART_VIRT @ virt base > .endm > > .macro senduart,rd,rx Reviewed-by: Patrice Chotard Thanks Patrice

Re: [PATCH 4/7] usb: dwc3: convert to devm_platform_ioremap_resource_byname

2020-07-27 Thread Patrice CHOTARD
Hi Chunfeng On 7/22/20 9:51 AM, Chunfeng Yun wrote: > Use devm_platform_ioremap_resource_byname() to simplify code > > Cc: Patrice Chotard > Signed-off-by: Chunfeng Yun > --- > drivers/usb/dwc3/dwc3-st.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > &g

Re: [PATCH -next 11/13] hwrng: st - use devm_platform_ioremap_resource() to simplify code

2019-10-16 Thread Patrice CHOTARD
m_device *pdev) > if (!ddata) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - base = devm_ioremap_resource(&pdev->dev, res); > + base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(base)) > return PTR_ERR(base); > Reviewed-by: Patrice Chotard Thanks

Re: ARM: multi_v7_defconfig: Enable SPI_STM32_QSPI support

2019-07-31 Thread Patrice CHOTARD
, Jul 29, 2019 at 3:55 PM wrote: >> From: Patrice Chotard >> >> Enable support for QSPI block on STM32 SoCs. >> >> Signed-off-by: Patrice Chotard >> --- >> arch/arm/configs/multi_v7_defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >>

Re: [PATCH 1/6] ARM: dts: STi: Switch to SPDX header

2019-06-14 Thread Patrice CHOTARD
dts/stih418.dtsi b/arch/arm/boot/dts/stih418.dtsi > index 0efb3cd6a86e..f3f0a0e0f23c 100644 > --- a/arch/arm/boot/dts/stih418.dtsi > +++ b/arch/arm/boot/dts/stih418.dtsi > @@ -1,10 +1,7 @@ > +// SPDX-License-Identifier: GPL-2.0 > /* > * Copyright (C) 2014 STMicroelectronics Limited. > * Author: Peter Griffin > - * > - * This program is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License version 2 as > - * publishhed by the Free Software Foundation. > */ > #include "stih418-clock.dtsi" > #include "stih407-family.dtsi" Acked-by: Patrice Chotard Thanks

Re: [PATCH -next] usb: host: ehci-st: Remove set but not used variable 'ehci'

2019-06-03 Thread Patrice CHOTARD
_data = pdata; > priv = hcd_to_ehci_priv(hcd); > - ehci = hcd_to_ehci(hcd); > > priv->phy = devm_phy_get(&dev->dev, "usb"); > if (IS_ERR(priv->phy)) { > Acked-by: Patrice Chotard Thanks Patrice

Re: [PATCH -next] usb: host: ohci-st: Remove set but not used variable 'ohci'

2019-06-03 Thread Patrice CHOTARD
tform_data = pdata; > priv = hcd_to_ohci_priv(hcd); > - ohci = hcd_to_ohci(hcd); > > priv->phy = devm_phy_get(&dev->dev, "usb"); > if (IS_ERR(priv->phy)) { > Acked-by: Patrice Chotard Thanks

Re: [PATCH 3/5] pinctrl: st: fix leaked of_node references

2019-04-12 Thread Patrice CHOTARD
this function. > ./drivers/pinctrl/pinctrl-st.c:1199:2-8: ERROR: missing of_node_put; acquired > a node pointer with refcount incremented on line 1175, but without a > corresponding object release within this function. > > Signed-off-by: Wen Yang > Cc: Patrice Chotard >

Re: [PATCH 19/23] watchdog: st_lpc_wdt: Convert to use device managed functions

2019-04-10 Thread Patrice CHOTARD
d use it instead of > dereferencing it repeatedly > - Use devm_watchdog_register_driver() to register watchdog device > > Cc: Patrice Chotard > Signed-off-by: Guenter Roeck > --- > drivers/watchdog/st_lpc_wdt.c | 47 > --- >

Re: [PATCH] watchdog: Convert to use devm_platform_ioremap_resource

2019-04-04 Thread Patrice CHOTARD
; Cc: Kevin Hilman > Cc: Matthias Brugger > Cc: Avi Fishman > Cc: Nancy Yuen > Cc: Brendan Higgins > Cc: Wan ZongShun > Cc: Michal Simek > Cc: Sylvain Lemieux > Cc: Kukjin Kim > Cc: Barry Song > Cc: Orson Zhai > Cc: Patrice Chotard > Cc: Maxime Coqu

Re: [PATCH] USB: dwc3: add missing of_node_put()

2019-02-06 Thread Patrice CHOTARD
we also should release that reference. > This patch fixes those 2 issues. > > Signed-off-by: Wen Yang > Cc: Patrice Chotard > Cc: Felipe Balbi > Cc: Greg Kroah-Hartman > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-...@vger.kernel.org > Cc: linux-kernel@vger

Re: [PATCH] phy: Variable "val" in function miphy_osc_is_ready() could be uninitialized

2019-02-06 Thread Patrice CHOTARD
gmap, > + ret = regmap_read(miphy_dev->regmap, > miphy_phy->syscfg_reg[SYSCFG_STATUS], &val); > + if (ret) > + return ret; > > if ((val & MIPHY_OSC_RDY) != MIPHY_OSC_RDY) > cpu_relax(); > Reviewed-by: Patrice Chotard Thanks

Re: [PATCH] dmaengine: st_fdma: use struct_size() in kzalloc()

2019-01-07 Thread Patrice CHOTARD
int i; > > - fdesc = kzalloc(sizeof(*fdesc) + > - sizeof(struct st_fdma_sw_node) * sg_len, GFP_NOWAIT); > + fdesc = kzalloc(struct_size(fdesc, node, sg_len), GFP_NOWAIT); > if (!fdesc) > return NULL; > > Acked-by: Patrice Chotard Thanks

[GIT PULL] STi SoC update for v4.21 round 1

2018-12-20 Thread Patrice CHOTARD
boot_lock cleanup initiated by Russell King, .smp_prepare_cpus and .smp_boot_secondary STi callbacks must be reworked to keep secondary CPU's bringup. Patrice Chotard (1): ARM: sti: remove pen_release and boot_lock arc

Re: [PATCH 1/1] ARM: STi: Restore secondary CPU's bringup

2018-12-19 Thread Patrice CHOTARD
On 12/19/18 12:28 PM, Russell King - ARM Linux wrote: > On Wed, Dec 19, 2018 at 10:31:35AM +0000, Patrice CHOTARD wrote: >> Hi Russell >> >> On 12/18/18 6:27 PM, Russell King - ARM Linux wrote: >>> On Tue, Dec 18, 2018 at 05:05:18PM +, Patrice CHOTARD wrote: >

Re: [PATCH 1/1] ARM: STi: Restore secondary CPU's bringup

2018-12-19 Thread Patrice CHOTARD
Hi Russell On 12/18/18 6:27 PM, Russell King - ARM Linux wrote: > On Tue, Dec 18, 2018 at 05:05:18PM +0000, Patrice CHOTARD wrote: >> Hi Russell >> >> On 12/18/18 4:52 PM, Russell King - ARM Linux wrote: >>> On Tue, Dec 18, 2018 at 03:48:13PM +0100, patrice.c

Re: [PATCH 1/1] ARM: STi: Restore secondary CPU's bringup

2018-12-18 Thread Patrice CHOTARD
Hi Russell On 12/18/18 4:52 PM, Russell King - ARM Linux wrote: > On Tue, Dec 18, 2018 at 03:48:13PM +0100, patrice.chot...@st.com wrote: >> From: Patrice Chotard >> >> Due to pen_release and boot_lock removal, secondary CPU's bringup >> was broken. Restore CPU

[GIT PULL] STi DT update for v4.20 round 1

2018-09-25 Thread Patrice CHOTARD
Hi Arnd, Kevin, Olof PLease consider this first round of STi dts update for v4.20 The following changes since commit 5b394b2ddf0347bef56e50c69a58773c94343ff3: Linux 4.19-rc1 (2018-08-26 14:11:59 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/pc

Re: [PATCH] pinctrl: Convert to using %pOFn instead of device_node.name

2018-09-03 Thread Patrice CHOTARD
olas Ferre > Cc: Alexandre Belloni > Cc: Heiko Stuebner > Cc: Tony Lindgren > Cc: Haojian Zhuang > Cc: Patrice Chotard > Cc: Barry Song > Cc: Maxime Coquelin > Cc: Alexandre Torgue > Cc: Maxime Ripard > Cc: Chen-Yu Tsai > Cc: linux-g...@vger.kernel.org >

Re: [PATCH v2 3/3] ARM: dts: stm32: Fix DT dtc warnings for stm32f4

2018-08-07 Thread Patrice CHOTARD
Hi Rob On 08/07/2018 04:08 PM, Rob Herring wrote: > On Tue, Aug 7, 2018 at 6:55 AM wrote: >> >> From: Patrice Chotard >> >> Fix the following DT dtc warnings for stm32f429 and >> stm32f469 boards: >> >> Warning (node_name_chars_strict): /soc/pin-

Re: [PATCH 1/3] ARM: dts: stm32: Fix DT dtc warnings for stm32h7

2018-08-07 Thread Patrice CHOTARD
I will send a v2 Thanks for reviewing ;-) Patrice On 08/07/2018 01:56 PM, Alexandre Torgue wrote: > Hi Patrice > > On 08/07/2018 01:38 PM, Alexandre Torgue wrote: >>> +    #address-cells = <1>; >>> +    #size-cells = <1>; >>> + >>>   clocks { >>>   clk_hse: clk-hse { >>>    

Re: [U-BOOT internal] ARM: dts: stm32: Add clk-lse node's label on stm32f429

2018-07-19 Thread Patrice CHOTARD
Sorry, don't take care about this patch, it was send using the wrong script Patrice On 07/19/2018 03:10 PM, patrice.chot...@st.com wrote: > From: Patrice Chotard > > Add missing clk_lse label for node clk-lse. > > Signed-off-by: Patrice Chotard > --- > arch/arm/b

Re: [PATCH V2] ARM: STi: warn if scu mapping fails

2018-07-18 Thread Patrice CHOTARD
cholas Mc Guire > Link: http://lkml.org/lkml/2018/7/16/219 > Fixes: commit 65ebcc115889 ("ARM: sti: Add STiH415 SOC support") > --- > > V2: replacement of WARN_ON() by a meaningful pr_err as suggested by > Patrice CHOTARD . > > Problem was found by an expe

Re: [PATCH] ARM: STi: at least warn if of_iomap fails

2018-07-16 Thread Patrice CHOTARD
Hi Nicholas On 07/12/2018 05:48 PM, Nicholas Mc Guire wrote: > The call to of_iomap() is unchecked but scu_enable(), which the returned > address is passed on to, assumes a valid mapping. If the mapping is > invalid this could probably lead to undefined system state so at least > a warning should

Re: [PATCH] mmc: sdhci-*: Don't emit error msg if sdhci_add_host() fails

2018-05-25 Thread Patrice CHOTARD
st); > > diff --git a/drivers/mmc/host/sdhci-st.c b/drivers/mmc/host/sdhci-st.c > index c32daed0d418..8f95647195d9 100644 > --- a/drivers/mmc/host/sdhci-st.c > +++ b/drivers/mmc/host/sdhci-st.c > @@ -422,10 +422,8 @@ static int sdhci_st_probe(struct platform_device *pdev) > st_mmcss_cconfig(np, host); > > ret = sdhci_add_host(host); > - if (ret) { > - dev_err(&pdev->dev, "Failed sdhci_add_host\n"); > + if (ret) > goto err_out; > - } > > host_version = readw_relaxed((host->ioaddr + SDHCI_HOST_VERSION)); > > For drivers/mmc/host/sdhci-st.c: Acked-by: Patrice Chotard Thanks

Re: [PATCH v2] rtc: st-lpc: add range

2018-05-23 Thread Patrice CHOTARD
p;st_rtc_ops; > + rtc->rtc_dev->range_max = U64_MAX; > + do_div(rtc->rtc_dev->range_max, rtc->clkrate); > > ret = rtc_register_device(rtc->rtc_dev); > if (ret) { > Acked-by: Patrice Chotard Patrice

Re: [PATCH 1/2] rtc: st-lpc: fix possible race condition

2018-05-23 Thread Patrice CHOTARD
gt; - return 0; > -} > - > #ifdef CONFIG_PM_SLEEP > static int st_rtc_suspend(struct device *dev) > { > @@ -325,7 +320,6 @@ static struct platform_driver st_rtc_platform_driver = { > .of_match_table = st_rtc_match, > }, > .probe = st_rtc_probe, > - .remove = st_rtc_remove, > }; > > module_platform_driver(st_rtc_platform_driver); > Acked-by: Patrice Chotard Thanks Patrice

[GIT PULL] STi DT update for v4.18 round 1

2018-05-14 Thread Patrice CHOTARD
for 4.18: - Fix complain about IRQ_TYPE_NONE_usage Patrice Chotard (5): ARM: dts: stih407-family: Fix complain about IRQ_TYPE_NONE usage ARM: dts: stih407-pinctrl: Fix complain about IRQ_TYPE_NONE usage ARM: dts

Re: [PATCH 53/61] tty: serial: simplify getting .drvdata

2018-04-20 Thread Patrice CHOTARD
platform_get_drvdata(pdev); > + struct uart_port *port = dev_get_drvdata(dev); > struct cdns_uart *cdns_uart = port->private_data; > > clk_disable(cdns_uart->uartclk); > @@ -1441,8 +1440,7 @@ static int __maybe_unused cdns_runtime_suspend(struct > device *dev) > > static int __maybe_unused cdns_runtime_resume(struct device *dev) > { > - struct platform_device *pdev = to_platform_device(dev); > - struct uart_port *port = platform_get_drvdata(pdev); > + struct uart_port *port = dev_get_drvdata(dev); > struct cdns_uart *cdns_uart = port->private_data; > > clk_enable(cdns_uart->pclk); > For st-asc: Acked-by: Patrice Chotard Thanks

Re: [PATCH] ata: ahci-platform: add reset control support

2018-04-05 Thread Patrice CHOTARD
Hi Hans On 04/05/2018 03:27 PM, Hans de Goede wrote: > Hi, > > On 05-04-18 15:17, Patrice CHOTARD wrote: >> Hi Thierry >> >> On 04/05/2018 11:54 AM, Thierry Reding wrote: >>> On Fri, Mar 23, 2018 at 10:30:53AM +0900, Kunihiko Hayashi wrote: >>>>

Re: [PATCH] ata: ahci-platform: add reset control support

2018-04-05 Thread Patrice CHOTARD
Hi Thierry On 04/05/2018 11:54 AM, Thierry Reding wrote: > On Fri, Mar 23, 2018 at 10:30:53AM +0900, Kunihiko Hayashi wrote: >> Add support to get and control a list of resets for the device >> as optional and shared. These resets must be kept de-asserted until >> the device is enabled. >> >> This

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

2018-03-23 Thread Patrice CHOTARD
On 03/22/2018 10:01 PM, Alexandre Belloni wrote: > On 16/03/2018 at 07:34:06 +0000, Patrice CHOTARD wrote: >> Hi Arnd, Stephen >> >> On 03/15/2018 10:06 PM, Arnd Bergmann wrote: >>> On Thu, Mar 15, 2018 at 9:56 PM, Stephen Rothwell >>&g

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

2018-03-16 Thread Patrice CHOTARD
Hi Arnd, Stephen On 03/15/2018 10:06 PM, Arnd Bergmann wrote: > On Thu, Mar 15, 2018 at 9:56 PM, Stephen Rothwell > wrote: >> Hi all, >> >> Commit >> >>9a5f4879cd8d ("ARM: dts: stih4xx: Add missing #sound-dai-cells") >> >> is missing a Signed-off-by from its committer. > > Thanks for the re

Re: [GIT PULL] STi DT update for v4.17 round 2

2018-03-15 Thread Patrice CHOTARD
Hi Arnd On 03/15/2018 03:59 PM, Arnd Bergmann wrote: > On Thu, Mar 15, 2018 at 10:06 AM, Patrice CHOTARD > wrote: >> Hi Arnd, Kevin, Olof >> >> PLease consider this second round of STi dts update for v4.17 >> >> The following changes since commit 0e04

[GIT PULL] STi DT update for v4.17 round 2

2018-03-15 Thread Patrice CHOTARD
Hi Arnd, Kevin, Olof PLease consider this second round of STi dts update for v4.17 The following changes since commit 0e04ce02e6a2ac30d21294d8bb2aeceb8489e52b: ARM: dts: STi: Remove unused clk_ext2f_a9 clock (2018-02-13 13:31:38 +0100) are available in the git repository at: git://git.k

Re: [RESEND PATCH v4 3/3] ARM: dts: STi: Remove console=ttyASN from bootargs for STi boards] ARM: dts: STi: Remove "console=ttyASN" from bootargs for STi boards

2018-03-14 Thread Patrice CHOTARD
Sorry I just noticed that the commit header is corrupted I will resend this patch Patrice On 03/14/2018 03:52 PM, patrice.chot...@st.com wrote: > From: Patrice Chotard > > As serial interface is already specified into stdout-path property, > "console=ttyASN,115200"

Re: [PATCH v4 3/3] ARM: dts: STi: Remove "console=ttyASN" from bootargs for STi boards

2018-03-14 Thread Patrice CHOTARD
Hi Greg On 03/14/2018 02:57 PM, Greg KH wrote: > On Wed, Mar 07, 2018 at 06:35:34PM +0100, patrice.chot...@st.com wrote: >> From: Patrice Chotard >> >> As serial interface is already specified into stdout-path property, >> "console=ttyASN,115200" from bootarg

Re: [PATCH v3 04/10] otm

2018-03-13 Thread Patrice CHOTARD
Hi Yannick There is a problem with patch 4. I received two patches 04/10, this one and [PATCH v3 04/10] video: add support of panel OTM8009A. Patrice On 03/13/2018 02:50 PM, yannick fertre wrote: > --- > drivers/video/orisetech_otm8009a.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deleti

Re: [PATCH v3 10/10] board: Add STM32F769 SoC, discovery board support

2018-03-13 Thread Patrice CHOTARD
Hi Yannick You have forgotten the commit message Patrice On 03/13/2018 02:50 PM, yannick fertre wrote: > Signed-off-by: yannick fertre > --- > configs/stm32f769-disco_defconfig | 65 > +++ > 1 file changed, 65 insertions(+) > create mode 100644 configs/

Re: [PATCH v2 07/10] video: add support of panel rm68200

2018-03-13 Thread Patrice CHOTARD
Hi yannick On 03/02/2018 04:44 PM, yannick fertre wrote: > Support for Raydium rm68200 720p dsi 2dl video mode panel. > > Signed-off-by: yannick fertre > --- > drivers/video/Kconfig | 8 + > drivers/video/Makefile | 1 + > drivers/video/raydium-rm68200.c | 329 > +

Re: [PATCH v2 06/10] video: add support of STM32 MIPI DSI controller driver

2018-03-13 Thread Patrice CHOTARD
Hi yannick On 03/02/2018 04:44 PM, yannick fertre wrote: > Add the STM32 DSI controller driver that uses the Synopsys DesignWare > MIPI DSI host controller bridge. > > Signed-off-by: yannick fertre > --- > drivers/video/stm32/Kconfig | 10 + > drivers/video/stm32/Makefile| 1 + >

Re: [PATCH v2 04/10] video: add support of panel OTM8009A

2018-03-13 Thread Patrice CHOTARD
Hi Yannick On 03/02/2018 04:44 PM, yannick fertre wrote: > Support for Orise Tech otm8009a 480p dsi 2dl video mode panel. > > Signed-off-by: yannick fertre > --- > drivers/video/Kconfig | 8 + > drivers/video/Makefile | 1 + > drivers/video/orisetech_otm8009a.c |

Re: [PATCH v2 02/10] video: stm32: stm32_ltdc: update debug log

2018-03-13 Thread Patrice CHOTARD
Hi Yannick On 03/02/2018 04:44 PM, yannick fertre wrote: > Replace macro debug by pr_error, pr_warn or pr_info. > > Signed-off-by: yannick fertre > --- > drivers/video/stm32/stm32_ltdc.c | 62 > > 1 file changed, 31 insertions(+), 31 deletions(-) >

Re: [PATCH v2 01/10] video: stm32: stm32_ltdc: add bridge to display controller

2018-03-13 Thread Patrice CHOTARD
Hi Yannick On 03/02/2018 04:44 PM, yannick fertre wrote: > Manage a bridge insert between the display controller & a panel. > > Signed-off-by: yannick fertre > --- > drivers/video/stm32/stm32_ltdc.c | 107 > ++- > 1 file changed, 71 insertions(+), 36 delet

Re: [PATCH v2 10/10] board: Add STM32F769 SoC, discovery board support

2018-03-13 Thread Patrice CHOTARD
Hi Yannick On 03/02/2018 04:44 PM, yannick fertre wrote: > Signed-off-by: yannick fertre Can you add a commit message explaining why you add a specific defconfig for this board. FYI, previously, the same defconfig was used for all STM32F7 boards (ie /stm32f746-disco_defconfig). You will also

Re: [PATCH v3 1/3] ARM: dts: STi: Fix aliases property name for STi boards

2018-03-07 Thread Patrice CHOTARD
Hi Rob On 03/07/2018 05:15 PM, Rob Herring wrote: > On Wed, Mar 7, 2018 at 2:49 AM, wrote: >> From: Patrice Chotard >> >> Update serial aliases from "ttyASN" to more common "serialN". >> >> Since dtc v1.4.6-9-gaadd0b65c987, aliases proper

Re: [PATCH v2 0/6] Add mmc support for STM32F7 boards

2018-03-07 Thread Patrice CHOTARD
Hi Lee On 03/07/2018 09:35 AM, Lee Jones wrote: > On Mon, 05 Mar 2018, Alexandre Torgue wrote: > >> Hi Patrice, >> >> On 03/01/2018 11:53 AM, patrice.chot...@st.com wrote: >>> From: Patrice Chotard >>> >>> This series adds : >>>

Re: [PATCH v2 3/3] tty: st-asc: Update tty alias

2018-03-07 Thread Patrice CHOTARD
Hi Rob On 03/06/2018 02:29 PM, Rob Herring wrote: > On Tue, Mar 6, 2018 at 2:07 AM, wrote: >> From: Patrice Chotard >> >> Since dtc v1.4.6-9-gaadd0b65c987, aliases property name >> must include only lowercase and '-'. >> >> After having updated

Re: [PATCH v2 2/3] ARM: dts: STi: Remove useless stdout-path for STi boards

2018-03-07 Thread Patrice CHOTARD
Hi Rob On 03/06/2018 02:26 PM, Rob Herring wrote: > On Tue, Mar 6, 2018 at 2:07 AM, wrote: >> From: Patrice Chotard >> >> As serial interface is already specified into bootargs, >> stdout-path can be removed. > > Wrong way around. stdout-path is preferred ov

Re: [PATCH 1/2] ARM: dts: STi: Fix aliases property name for STi boards

2018-03-05 Thread Patrice CHOTARD
Hi Rob On 03/05/2018 04:28 PM, Rob Herring wrote: > On Mon, Mar 5, 2018 at 9:00 AM, wrote: >> From: Patrice Chotard >> >> Since dtc v1.4.6-9-gaadd0b65c987, aliases property name must >> be lowercase only. >> This allows to fix following warnings when compiling

Re: [PATCH 1/6] ARM: dts: stm32: Add sdio pins definition for stm32f7

2018-03-01 Thread Patrice CHOTARD
Hi Alex On 03/01/2018 10:51 AM, Alexandre Torgue wrote: > Hi Patrice > > On 03/01/2018 10:43 AM, patrice.chot...@st.com wrote: >> From: Patrice Chotard >> >> Add sdio pins definition for the 2 sdio instances embeds in stm32f746. >> >> Signed-off-by: Pat

Re: [PATCH v2 14/15] clk: stm32: Add clk entry for SDMMC2 on stm32F769

2018-03-01 Thread Patrice CHOTARD
Hi Linus On 03/01/2018 10:12 AM, Linus Walleij wrote: > On Mon, Feb 26, 2018 at 9:41 AM, Patrice CHOTARD > wrote: > >> Hi Stephen, Michael >> >> It's a gentle reminder as this patch is present on mailing list and >> acked-by since 01/12/2018 > > Yo

Re: [PATCH v2 14/15] clk: stm32: Add clk entry for SDMMC2 on stm32F769

2018-02-26 Thread Patrice CHOTARD
Hi Stephen, Michael It's a gentle reminder as this patch is present on mailing list and acked-by since 01/12/2018 Thanks Patrice On 01/16/2018 01:30 PM, patrice.chot...@st.com wrote: > From: Patrice Chotard > > STM32F769 has 2 SDMMC port, add clock entry for the second one. &

[GIT PULL] STi DT update for v4.17 round 1

2018-02-26 Thread Patrice CHOTARD
node, the addition of a fake reg property fixes these warnings. For others nodes, their location in device tree have been updated. -------- Patrice Chotard (11): ARM: dts: STi: Fix bindings notation ARM: dts: STi: Move leds node o

Re: [PATCH v2 10/12] ARM: dts: STi: Add fake reg property for clocks

2018-02-13 Thread Patrice CHOTARD
Hi Rob On 02/12/2018 04:22 PM, Rob Herring wrote: > On Mon, Feb 12, 2018 at 8:43 AM, wrote: >> From: Patrice Chotard >> >> For all clock's element inside SoC, add a fake reg property, >> this allows to fix the following warnings when compiling >> dtb wit

Re: [PATCH v2 09/12] ARM: dts: STi: Add fake reg property for sound related nodes

2018-02-13 Thread Patrice CHOTARD
Hi Rob On 02/12/2018 04:40 PM, Rob Herring wrote: > On Mon, Feb 12, 2018 at 8:43 AM, wrote: >> From: Patrice Chotard >> >> Add fake reg property to sound, simple-audio-card, dai-link and >> sti_sasg_codec nodes, this allows to fix the following warning >> w

Re: [PATCH 2/5] pinctrl: stm32: add STM32F769 MCU support

2018-01-18 Thread Patrice CHOTARD
Hi Linus It's a gentle reminder because this patch seems not yet merged in any of your pinctrl branch. Thanks Patrice On 12/11/2017 09:54 AM, Alexandre Torgue wrote: > This patch which adds STM32F769 pinctrl and GPIO support, relies on the > generic STM32 pinctrl driver. > > Signed-off-by: Al

Re: [PATCH v2 04/15] mmc: mmci: Add support for setting pad type via pinctrl

2018-01-18 Thread Patrice CHOTARD
Hi Ulf On 01/17/2018 10:34 AM, Ulf Hansson wrote: > [...] > >> /* >> @@ -1616,6 +1625,32 @@ static int mmci_probe(struct amba_device *dev, >> host = mmc_priv(mmc); >> host->mmc = mmc; >> >> + /* >> +* Some variant (STM32) doesn't have opendrain bit, nevert

Re: [PATCH v2 03/15] mmc: mmci: Don't pretend all variants to have OPENDRAIN bit

2018-01-18 Thread Patrice CHOTARD
Hi Ulf On 01/17/2018 10:33 AM, Ulf Hansson wrote: > [...] > >> /* Busy detection for the ST Micro variant */ >> @@ -1455,16 +1465,13 @@ static void mmci_set_ios(struct mmc_host *mmc, >> struct mmc_ios *ios) >> ~MCI_ST_DATA2DIREN); >> } >> >> - if

Re: [PATCH 2/7] ARM: dts: STi: Add gpio polarity for "hdmi,hpd-gpio" property

2018-01-15 Thread Patrice CHOTARD
Hi Arnd On 01/15/2018 03:59 PM, Arnd Bergmann wrote: > On Wed, Jan 10, 2018 at 9:21 AM, wrote: >> From: Patrice Chotard >> >> The GPIO polarity is missing in the hdmi,hpd-gpio property, this >> fixes the following DT warnings: >> >> arch/arm/boot/dts/sti

Re: [PATCH 03/14] mmc: mmci: Add support for setting pad type via pinctrl

2018-01-15 Thread Patrice CHOTARD
Hi Ulf On 01/15/2018 01:43 PM, Ulf Hansson wrote: > On 12 January 2018 at 13:15, wrote: >> From: Patrice Chotard >> >> The STM32 variant hasn't the control bit to switch pads in opendrain mode. >> In this case we can achieve the same result by asking to the pi

Re: [PATCH 02/14] mmc: mmci: Don't pretend all variants to have MCI_STARBITERR flag

2018-01-15 Thread Patrice CHOTARD
Hi Ulf On 01/15/2018 01:32 PM, Ulf Hansson wrote: > On 12 January 2018 at 13:15, wrote: >> From: Patrice Chotard >> >> This patch prepares for supporting the STM32 variant that >> has no such bit in the status register. >> >> Signed-off-by: Andrea Mer

Re: [PATCH 04/14] mmc: mmci: Add STM32 variant

2018-01-15 Thread Patrice CHOTARD
Hi Linus On 01/15/2018 02:13 AM, Linus Walleij wrote: > On Fri, Jan 12, 2018 at 1:15 PM, wrote: > >> From: Patrice Chotard >> >> STM32F4 and STM32F7 MCUs has a SDIO controller that looks like >> an ARM pl810. > > PL180 you mean. Ulf can fix while applying.

Re: [PATCH 0/7] ARM: dts: STi: Fix DT dtc warnings

2018-01-10 Thread Patrice CHOTARD
HI Arnd On 01/10/2018 09:52 PM, Arnd Bergmann wrote: > On Wed, Jan 10, 2018 at 6:17 PM, Patrice CHOTARD > wrote: >> Hi Arnd >> >> On 01/10/2018 04:52 PM, Arnd Bergmann wrote: >>> On Wed, Jan 10, 2018 at 9:21 AM, wrote: >>>> From: Patrice Chotard

Re: [PATCH 0/7] ARM: dts: STi: Fix DT dtc warnings

2018-01-10 Thread Patrice CHOTARD
Hi Arnd On 01/10/2018 04:52 PM, Arnd Bergmann wrote: > On Wed, Jan 10, 2018 at 9:21 AM, wrote: >> From: Patrice Chotard >> >> This series fixes all dtc warnings related to STi platforms dt files. >> It has been triggered by Rob Herring [1] >> >> [1] h

Re: [PATCH] [media] c8sectpfe: Use resource_size function on memory resource

2017-11-21 Thread Patrice CHOTARD
(IS_ERR(fei->sram)) > return PTR_ERR(fei->sram); > > - fei->sram_size = res->end - res->start; > + fei->sram_size = resource_size(res); > > fei->idle_irq = platform_get_irq_byname(pdev, "c8sectpfe-idle-irq"); > if (fei->idle_irq < 0) { > Acked-by: Patrice Chotard Thanks

Re: [PATCH] c8sectpfe: fix potential NULL pointer dereference in c8sectpfe_timer_interrupt

2017-11-21 Thread Patrice CHOTARD
> > + fei = channel->fei; > + > wp = readl(channel->irec + DMA_PRDS_BUSWP_TP(0)); > rp = readl(channel->irec + DMA_PRDS_BUSRP_TP(0)); > > Acked-by: Patrice Chotard Thanks

Re: [PATCH 04/17] pinctrl: st: constify pinconf_ops, pinctrl_ops, and pinmux_ops structures

2017-08-10 Thread Patrice CHOTARD
truct pinctrl_dev > *pctldev, > ST_PINCONF_UNPACK_RT_DELAY(config)); > } > > -static struct pinconf_ops st_confops = { > +static const struct pinconf_ops st_confops = { > .pin_config_get = st_pinconf_get, > .pin_config_set = st_pinconf_set, > .pin_config_dbg_show= st_pinconf_dbg_show, > Acked-by: Patrice Chotard Thanks

Re: [RFC PATCH v2] ARM: dts: stm32: change pinctrl bindings definition

2017-07-26 Thread Patrice CHOTARD
Hi Alex On 07/21/2017 02:34 PM, Alexandre Torgue wrote: > Initially each pin was declared in "include/dt-bindings/stm32f429-pinfunc.h" > and each definition contained SOC names (ex: STM32F429_PA9_FUNC_USART1_TX). > Since this approach was approved, the number of supported MCU has > increased (STM3

Re: [PATCH] cpufreq: Convert to using %pOF instead of full_name

2017-07-19 Thread Patrice CHOTARD
Wysocki" > Cc: Viresh Kumar > Cc: Benjamin Herrenschmidt > Cc: Paul Mackerras > Cc: Michael Ellerman > Cc: Patrice Chotard > Cc: linux...@vger.kernel.org > Cc: linuxppc-...@lists.ozlabs.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: ker...@stl

Re: [PATCH] pinctrl: Convert to using %pOF instead of full_name

2017-07-19 Thread Patrice CHOTARD
j > Cc: Lee Jones > Cc: Eric Anholt > Cc: Stefan Wahren > Cc: Florian Fainelli > Cc: Ray Jui > Cc: Scott Branden > Cc: bcm-kernel-feedback-l...@broadcom.com > Cc: Ludovic Desroches > Cc: Patrice Chotard > Cc: Tomasz Figa > Cc: Krzysztof Kozlowski > Cc: S

Re: [PATCH] pinctrl: st: constify gpio_chip structure

2017-07-12 Thread Patrice CHOTARD
static const struct gpio_chip st_gpio_template = { > .request= gpiochip_generic_request, > .free = gpiochip_generic_free, > .get= st_gpio_get, > Acked-by: Patrice Chotard Thanks

Re: [PATCH 3/3] rtc: st-lpc: make it robust against y2038/2106 bug

2017-06-19 Thread Patrice CHOTARD
ow_secs; > + unsigned long long alarm_secs; > unsigned long long lpa; > > st_rtc_read_time(dev, &now); > - rtc_tm_to_time(&now, &now_secs); > - rtc_tm_to_time(&t->time, &alarm_secs); > + now_secs = rtc_tm_to_time64(&now); > + alarm_secs = rtc_tm_to_time64(&t->time); > > /* Invalid alarm time */ > if (now_secs > alarm_secs) > Acked-by: Patrice Chotard Thanks

Re: [PATCH 0/3] Add stm32h743i-disco board

2017-06-07 Thread Patrice CHOTARD
Hi Alex On 06/07/2017 09:04 AM, Alexandre Torgue wrote: > Hi Patrice > > On 06/06/2017 06:13 PM, patrice.chot...@st.com wrote: >> From: Patrice Chotard >> >> This series adds basic support for the stm32h743i discovery board >> based on stm32h7 MCU. >>

Re: [PATCH 1/1] ARM: compressed/head.S: fix __nop macro rept number for ARMv7M

2017-05-19 Thread Patrice CHOTARD
Hi Ard On 05/18/2017 01:16 PM, Ard Biesheuvel wrote: > On 12 May 2017 at 08:22, Patrice CHOTARD wrote: >> Hi all >> >> It's a gentle reminder to not forgot this patch. >> >> Thanks >> >> Patrice >> >> On 04/25/2017 11:01 AM, patri

Re: [PATCH 1/1] ARM: compressed/head.S: fix __nop macro rept number for ARMv7M

2017-05-12 Thread Patrice CHOTARD
Hi all It's a gentle reminder to not forgot this patch. Thanks Patrice On 04/25/2017 11:01 AM, patrice.chot...@st.com wrote: > From: Patrice Chotard > > In zImage, _magic_sig is located at offset 0x24. > But for ARMv7M architecture which enable CPU_THUMBONLY > and !

Re: [PATCH] mfd: stmpe: Fix bit clearing on STMPE1600

2017-03-31 Thread Patrice CHOTARD
_GPCR_CSB]= STMPE1600_REG_GPSR_MSB, > [STMPE_IDX_GPDR_LSB]= STMPE1600_REG_GPDR_LSB, > [STMPE_IDX_GPDR_CSB]= STMPE1600_REG_GPDR_MSB, > [STMPE_IDX_IEGPIOR_LSB] = STMPE1600_REG_IEGPIOR_LSB, > Reviewed-by: Patrice Chotard Thanks

  1   2   3   >