From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S968061AbcA0U7h (ORCPT ); Wed, 27 Jan 2016 15:59:37 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:36188 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965371AbcA0U7c (ORCPT ); Wed, 27 Jan 2016 15:59:32 -0500 Subject: Re: [PATCH v4 0/9] Add STM32 pinctrl/GPIO driver To: Linus Walleij References: <1452773795-24216-1-git-send-email-mcoquelin.stm32@gmail.com> Cc: Patrice CHOTARD , Mark Rutland , Rob Herring , "linux-gpio@vger.kernel.org" , Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , =?UTF-8?Q?Andreas_F=c3=a4rber?= , "devicetree@vger.kernel.org" , Daniel Thompson , Bruno Herrera From: Maxime Coquelin Message-ID: <56A92FB0.7080907@gmail.com> Date: Wed, 27 Jan 2016 21:59:28 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/27/2016 02:20 PM, Linus Walleij wrote: > On Thu, Jan 14, 2016 at 1:16 PM, Maxime Coquelin > wrote: > >> >This is the fourth round of STM32 pinctrl series, which takes into >> >account comments made by Patrice on patch 4. > I have applied patch 2 and 4 to the pinctrl tree. Just tried to compile your devel branch, and it breaks: CC drivers/pinctrl/stm32/pinctrl-stm32f429.o drivers/pinctrl/stm32/pinctrl-stm32.c: In function 'stm32_gpiolib_register_bank': drivers/pinctrl/stm32/pinctrl-stm32.c:692:17: error: 'struct gpio_chip' has no member named 'dev' bank->gpio_chip.dev = dev; ^ make[3]: *** [drivers/pinctrl/stm32/pinctrl-stm32.o] Error 1 This is due to the field renaming you did in gpio_chip struct: 58383c7 gpio: change member .dev to .parent How do you prefer to proceed? Should I send a fix on top of your devel branch, or you will fix it directly? Thanks, Maxime