Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1436162

Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF)

Show all headers | View raw

On Mon, Jun 27, 2016 at 8:15 PM, Randy Dunlap <rdunlap@infradead.org> wrote:

> On 06/26/16 23:39, Stephen Rothwell wrote:
>> Hi all,
>>
>> Changes since 20160624:
>>
>
> on i386, when CONFIG_OF is not enabled ...
> but OF_GPIO is enabled due to this in drivers/gpio/Kconfig:
>
> config OF_GPIO
>         def_bool y
>         depends on OF || COMPILE_TEST
>
> (above from commit 1e4a80640338924b9f9fd7a121ac31d08134410a
> from Alexander Stein <alexanders83@web.de>)
>
>
> ../drivers/pinctrl/bcm/pinctrl-iproc-gpio.c:381:20: error: 'pinconf_generic_dt_node_to_map_pin' undeclared here (not in a function)
> ../drivers/pinctrl/bcm/pinctrl-cygnus-mux.c:739:20: error: 'pinconf_generic_dt_node_to_map_group' undeclared here (not in a function)
> ../drivers/pinctrl/bcm/pinctrl-nsp-gpio.c:365:20: error: 'pinconf_generic_dt_node_to_map_pin' undeclared here (not in a function)
>
> because that function is only present when CONFIG_OF is enabled.
>
>
> Also, why does that commit (1e4a80640338924b9f9fd7a121ac31d08134410a)
> not have any other S-O-B lines in it?  like whoever merged it?

I merged it I think, Alex made a long series enabling compile
testing and I started to cherry-pick the first commits to let
them trickle in.

I was worried about it because some of the patches caused
severe build problems on some archs.

It's a  bit tricky to know what to do here: we want compile
coverage to get proper testing, when we turn it on we get regressions,
so trying to improve things make things break and it becomes a
vicious circle of trouble. I don't know what the biggest pain is ...

I don't really see the conclusion of this discussion thread, whether
I should revert the patch or not? For fixes or next?

Yours,
Linus Walleij

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

linux-next: Tree for Jun 27 Stephen Rothwell <sfr@canb.auug.org.au> - 2016-06-27 08:40 +0200
  Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Randy Dunlap <rdunlap@infradead.org> - 2016-06-27 20:20 +0200
    Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Arnd Bergmann <arnd@arndb.de> - 2016-06-27 23:40 +0200
      Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Alexander Stein <alexanders83@web.de> - 2016-06-28 09:10 +0200
        Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Randy Dunlap <rdunlap@infradead.org> - 2016-06-28 10:30 +0200
        Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Arnd Bergmann <arnd@arndb.de> - 2016-06-28 10:40 +0200
          Re: Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Alexander Stein <alexanders83@web.de> - 2016-06-29 08:10 +0200
    Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Linus Walleij <linus.walleij@linaro.org> - 2016-07-04 11:50 +0200
      Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Randy Dunlap <rdunlap@infradead.org> - 2016-07-04 18:20 +0200
        Re: linux-next: Tree for Jun 27 (pinctrl && !CONFIG_OF) Linus Walleij <linus.walleij@linaro.org> - 2016-07-05 08:50 +0200

csiph-web