Hello Linus, I have sent modified patch for this bag as v2. Please refer to the following.
[PATCH v2] https://patchwork.kernel.org/patch/10533929/ [Changes since v1] diff --git a/drivers/pinctrl/uniphier/pinctrl-uniphier-sld8.c b/drivers/pinctrl/uniphier/pinctrl-uniphier-sld8.c -index 0f921a6..f912999 100644 +index 0f921a6..f086083 100644 --- a/drivers/pinctrl/uniphier/pinctrl-uniphier-sld8.c +++ b/drivers/pinctrl/uniphier/pinctrl-uniphier-sld8.c @@ -504,6 +504,8 @@ static const unsigned nand_cs1_pins[] = {22, 23}; @@ -390,7 +393,7 @@ static const char * const i2c3_groups[] = {"i2c3"}; static const char * const nand_groups[] = {"nand", "nand_cs1"}; static const char * const sd_groups[] = {"sd"}; -+static const char * const spi_groups[] = {"spi0"}; ++static const char * const spi0_groups[] = {"spi0"}; static const char * const system_bus_groups[] = {"system_bus", "system_bus_cs1", "system_bus_cs2", I mistook "spi0_groups" with " spi_groups", so there was an undeclared error. ----------------- Best Regards, Keiji Hayashibara > -----Original Message----- > From: Linus Walleij [mailto:linus.wall...@linaro.org] > Sent: Monday, July 30, 2018 5:47 AM > To: Hayashibara, Keiji/林原 啓二 <hayashibara.ke...@socionext.com> > Cc: Yamada, Masahiro/山田 真弘 <yamada.masah...@socionext.com>; open list:GPIO > SUBSYSTEM > <linux-g...@vger.kernel.org>; Linux ARM > <linux-arm-ker...@lists.infradead.org>; Masami Hiramatsu > <masami.hirama...@linaro.org>; Jassi Brar <jaswinder.si...@linaro.org>; > linux-kernel@vger.kernel.org; Hayashi, > Kunihiko/林 邦彦 <hayashi.kunih...@socionext.com> > Subject: Re: [PATCH] pinctrl: uniphier: add spi pin-mux settings > > On Thu, Jul 19, 2018 at 8:21 AM Keiji Hayashibara > <hayashibara.ke...@socionext.com> wrote: > > > From: Kunihiko Hayashi <hayashi.kunih...@socionext.com> > > > > Add pin-mux settings for spi controller. > > > > Signed-off-by: Kunihiko Hayashi <hayashi.kunih...@socionext.com> > > Signed-off-by: Keiji Hayashibara <hayashibara.ke...@socionext.com> > > There is some problem with the test robot, can you comment/followup? > > Yours, > Linus Walleij
