Re: [PATCH v4 4/9] pinctrl: Add STM32 MCUs support

From: Maxime Coquelin
Date: Mon Feb 01 2016 - 07:09:58 EST



On 02/01/2016 11:39 AM, Maxime Coquelin wrote:
2016-02-01 11:21 GMT+01:00 Arnd Bergmann <arnd@xxxxxxxx>:
On Wednesday 27 January 2016 14:16:32 Linus Walleij wrote:
On Thu, Jan 14, 2016 at 1:16 PM, Maxime Coquelin
<mcoquelin.stm32@xxxxxxxxx> wrote:

This patch adds pinctrl and GPIO support to STMicroelectronic's STM32
family of MCUs.

While it only supports STM32F429 for now, it has been designed to enable
support of other MCUs of the family (e.g. STM32F746).

Acked-by: Patrice Chotard <patrice.chotard@xxxxxx>
Signed-off-by: Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>
This patch applied for kernel v4.6.

This breaks the kernel build, please revert:

drivers/pinctrl/stm32/pinctrl-stm32.c:26:47: fatal error: dt-bindings/pinctrl/pinctrl-stm32.h: No such file or directory
Right, I missed to notify this dependency to Linus, sorry about that.
This dependency is no more needed, so I could just move the macros to
the driver side.

Note that I also noticed a breakage at build time due to field
renaming in gpio_chip structure (dev -> parent).

I propose to rebase the series and move the defines into the driver.
Is it ok for you?

I have rebased the series, taking care to fix the mentioned problems.

Linus, do you confirm you will revert STM32 pinctrl patches from your tree?

Thanks,
Maxime