Re: [PATCH] pinctrl: ocelot: Add platform dependency

[Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>]

Hi  Alexandre,

On Mon, May 11, 2020 at 4:20 PM Alexandre Belloni
<alexandre.belloni@xxxxxxxxxxx> wrote:
> On 11/05/2020 13:39:42+0200, Geert Uytterhoeven wrote:
> > On Thu, May 7, 2020 at 2:47 PM Alexandre Belloni
> > <alexandre.belloni@xxxxxxxxxxx> wrote:
> > > On 07/05/2020 14:27:06+0200, Lars Povlsen wrote:
> > > > Alexandre Belloni writes:
> > > > > On 07/05/2020 13:40:15+0200, Geert Uytterhoeven wrote:
> > > > >> The Microsemi Ocelot pin controller is only present on Microsemi Ocelot
> > > > >> and Jaguar2 SoCs.  Add a platform dependency to the PINCTRL_OCELOT
> > > > >> config symbol, to avoid asking the user about it when configuring a
> > > > >> kernel without Ocelot or Jaguar2 support.
> > > > >>
> > > > >
> > > > > I have to NAK here because there are upcoming (hopefully this cycle)
> > > > > SoCs using this driver.
> > > >
> > > > Not only because of that, but also an arbitrary system connecting to
> > > > ocelot by PCI could be using the driver.
> > >
> > > Right, and that was why I removed the dependency when adding jaguar2
> >
> > That is far from obvious from the description of commit da801ab56ad87489
> > ("pinctrl: ocelot: add MSCC Jaguar2 support")....
> > BTW, probably PINCTRL_OCELOT should have got "default y if MSCC_OCELOT",
> > to avoid breaking old configs?
> >
> > > support I never booted the mips core of jaguar2, I used it using PCIe.
> >
> > Ah, you're using it as a PCIe endpoint. So what about ...
> >
> > > > >> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > > > >> ---
> > > > >>  drivers/pinctrl/Kconfig | 4 ++--
> > > > >>  1 file changed, 2 insertions(+), 2 deletions(-)
> > > > >>
> > > > >> diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
> > > > >> index f0ce4ce3e0f52456..bed67c08a0892240 100644
> > > > >> --- a/drivers/pinctrl/Kconfig
> > > > >> +++ b/drivers/pinctrl/Kconfig
> > > > >> @@ -394,8 +394,8 @@ config PINCTRL_RK805
> > > > >>
> > > > >>  config PINCTRL_OCELOT
> > > > >>       bool "Pinctrl driver for the Microsemi Ocelot and Jaguar2 SoCs"
> > > > >> -     depends on OF
> > > > >> -     depends on HAS_IOMEM
> > > > >> +     depends on OF && HAS_IOMEM
> > > > >> +     depends on MSCC_OCELOT || COMPILE_TEST
> >
> > ... "depends on MSCC_OCELOT || PCI || COMPILE_TEST" instead?
> >
> > If other SoCs gain support, the dependency list can be extended.
>
> Hum ok, note that it can also be connected as a SPI slave but I think
> this is a bit far fetched currently.

I can connect whatever other system using SPI, i2c, CAN, or Ethernet ;-)
That doesn't mean I will directly program the other system's pin controller
over that link, using the standard (MMIO) pinctrl driver.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds