[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdY1E9rLwG-aXWSB7d32rhvmng+4QbVX7E0WR6aqP0oFaw@mail.gmail.com>
Date: Sat, 13 Feb 2016 15:48:11 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Maxime Coquelin <maxime.coquelin@...com>
Cc: Maxime Coquelin <mcoquelin.stm32@...il.com>,
Arnd Bergmann <arnd@...db.de>,
Mark Rutland <mark.rutland@....com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
Daniel Thompson <daniel.thompson@...aro.org>,
Bruno Herrera <bruherrera@...il.com>,
Patrice CHOTARD <patrice.chotard@...com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>,
Rob Herring <robh+dt@...nel.org>,
Andreas Färber <afaerber@...e.de>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v4 4/9] pinctrl: Add STM32 MCUs support
On Mon, Feb 1, 2016 at 1:09 PM, Maxime Coquelin <maxime.coquelin@...com> wrote:
> On 02/01/2016 11:39 AM, Maxime Coquelin wrote:
>> 2016-02-01 11:21 GMT+01:00 Arnd Bergmann <arnd@...db.de>:
>>> 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?
I guess the other fixes removes the need to do this.
Sorry for not staying on top of this issue, I was swamped with mail
and had a hard time prioritizing :(
Yours,
Linus Walleij
Powered by blists - more mailing lists