From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751552AbdBFLWi (ORCPT <rfc822;w@1wt.eu>);
        Mon, 6 Feb 2017 06:22:38 -0500
Received: from mx07-00178001.pphosted.com ([62.209.51.94]:22318 "EHLO
        mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK)
        by vger.kernel.org with ESMTP id S1750956AbdBFLWg (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 6 Feb 2017 06:22:36 -0500
Subject: Re: [PATCH 6/8] Documentation: dt: Add new compatible to STM32
 pinctrl driver bindings
To: Linus Walleij <linus.walleij@linaro.org>
References: <1485533721-9019-1-git-send-email-alexandre.torgue@st.com>
 <1485533721-9019-7-git-send-email-alexandre.torgue@st.com>
 <20170201150609.cbhwon76m6igmhyh@rob-hp-laptop>
 <abee01c1-da4f-97b4-258a-3ab937f620eb@st.com>
 <CACRpkdb=U+UMRqR2GKGwDzk9X_nLv0tf2dZ8LhDGceAD83LKag@mail.gmail.com>
CC: Rob Herring <robh@kernel.org>,
        Maxime Coquelin <mcoquelin.stm32@gmail.com>,
        Mark Rutland <mark.rutland@arm.com>, Arnd Bergmann <arnd@arndb.de>,
        Russell King <linux@armlinux.org.uk>, Olof Johansson <olof@lixom.net>,
        Patrice Chotard <patrice.chotard@st.com>,
        Lee Jones <lee.jones@linaro.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
From: Alexandre Torgue <alexandre.torgue@st.com>
Message-ID: <c1c5ca2b-3fc1-6077-87b3-4c815b7469d0@st.com>
Date: Mon, 6 Feb 2017 12:21:44 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.7.0
MIME-Version: 1.0
In-Reply-To: <CACRpkdb=U+UMRqR2GKGwDzk9X_nLv0tf2dZ8LhDGceAD83LKag@mail.gmail.com>
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Transfer-Encoding: 7bit
X-Originating-IP: [10.75.127.47]
X-ClientProxiedBy: SFHDAG4NODE2.st.com (10.75.127.11) To SFHDAG3NODE2.st.com
 (10.75.127.8)
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-02-06_04:,,
 signatures=0
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org



On 02/06/2017 09:46 AM, Linus Walleij wrote:
> On Wed, Feb 1, 2017 at 4:58 PM, Alexandre Torgue
> <alexandre.torgue@st.com> wrote:
>> On 02/01/2017 04:06 PM, Rob Herring wrote:
>>>
>>> On Fri, Jan 27, 2017 at 05:15:19PM +0100, Alexandre TORGUE wrote:
>>>>
>>>> Add new compatible for stm32f469 MCU.
>>>>
>>>> Signed-off-by: Alexandre TORGUE <alexandre.torgue@st.com>
>>>>
>>>> diff --git
>>>> a/Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.txt
>>>> b/Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.txt
>>>> index 25d9809..4f0487d 100644
>>>> --- a/Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.txt
>>>> +++ b/Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.txt
>>>> @@ -9,7 +9,8 @@ Pin controller node:
>>>>  Required properies:
>>>>   - compatible: value should be one of the following:
>>>>     (a) "st,stm32f429-pinctrl"
>>>> -   (b) "st,stm32f746-pinctrl"
>>>> +   (b) "st,stm32f469-pinctrl"
>>>> +   (c) "st,stm32f746-pinctrl"
>>>
>>>
>>> Drop the a,b,c. Otherwise, you'll be changing a bunch of lines every
>>> time.
>>
>>
>> I completely agree.
>>
>> Linus, Do i need to send a v2 for this patch ?
>
> Nopes, applied the amended v1 patch already with these
> changes, check the result.

Thanks Linus!

>
> Yours,
> Linus Walleij
>