mbox series

[0/4] qoriq: Add platform dependencies

Message ID 20200507112955.23520-1-geert+renesas@glider.be (mailing list archive)
Headers show
Series qoriq: Add platform dependencies | expand

Message

Geert Uytterhoeven May 7, 2020, 11:29 a.m. UTC
Hi all,

Several QorIQ blocks are only present on Freescale or NXP SoCs.
This series adds platform dependencies to the corresponding config
ymbols, to avoid asking the user about them when configuring a kernel
without support for these SoCs.

Most patches in this series are independent, but the third patch may
cause some Kconfig warnings when applied before the second patch, and
enabling the QorIQ CPU frequency scaling driver in a non-Layerscape
kernel.

Thanks for your comments!

Geert Uytterhoeven (4):
  ahci: qoriq: Add platform dependencies
  cpufreq: qoriq: Add platform dependencies
  clk: qoriq: Add platform dependencies
  thermal: qoriq: Add platform dependencies

 drivers/ata/Kconfig     | 1 +
 drivers/clk/Kconfig     | 3 ++-
 drivers/cpufreq/Kconfig | 3 ++-
 drivers/thermal/Kconfig | 4 ++--
 4 files changed, 7 insertions(+), 4 deletions(-)

Comments

Arnd Bergmann May 7, 2020, 1:24 p.m. UTC | #1
On Thu, May 7, 2020 at 1:30 PM Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
>
>         Hi all,
>
> Several QorIQ blocks are only present on Freescale or NXP SoCs.
> This series adds platform dependencies to the corresponding config
> ymbols, to avoid asking the user about them when configuring a kernel
> without support for these SoCs.
>
> Most patches in this series are independent, but the third patch may
> cause some Kconfig warnings when applied before the second patch, and
> enabling the QorIQ CPU frequency scaling driver in a non-Layerscape
> kernel.
>
> Thanks for your comments!

These all look good to me, thank you for the improvements!

Acked-by: Arnd Bergmann <arnd@arndb.de>
Leo Li May 7, 2020, 9:57 p.m. UTC | #2
On Thu, May 7, 2020 at 6:31 AM Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
>
>         Hi all,
>
> Several QorIQ blocks are only present on Freescale or NXP SoCs.
> This series adds platform dependencies to the corresponding config
> ymbols, to avoid asking the user about them when configuring a kernel
> without support for these SoCs.
>
> Most patches in this series are independent, but the third patch may
> cause some Kconfig warnings when applied before the second patch, and
> enabling the QorIQ CPU frequency scaling driver in a non-Layerscape
> kernel.
>
> Thanks for your comments!

Thanks.  The series looks good to me.

Are we trying to merge them through the various driver subsystems or I
can also pull them in through the fsl-soc tree.  If we want to go
through driver subsystems:

Acked-by: Li Yang <leoyang.li@nxp.com>

>
> Geert Uytterhoeven (4):
>   ahci: qoriq: Add platform dependencies
>   cpufreq: qoriq: Add platform dependencies
>   clk: qoriq: Add platform dependencies
>   thermal: qoriq: Add platform dependencies
>
>  drivers/ata/Kconfig     | 1 +
>  drivers/clk/Kconfig     | 3 ++-
>  drivers/cpufreq/Kconfig | 3 ++-
>  drivers/thermal/Kconfig | 4 ++--
>  4 files changed, 7 insertions(+), 4 deletions(-)
>
> --
> 2.17.1
>
> Gr{oetje,eeting}s,
>
>                                                 Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> 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
Geert Uytterhoeven May 8, 2020, 9:50 a.m. UTC | #3
Hi Li,

On Thu, May 7, 2020 at 11:58 PM Li Yang <leoyang.li@nxp.com> wrote:
> On Thu, May 7, 2020 at 6:31 AM Geert Uytterhoeven
> <geert+renesas@glider.be> wrote:
> > Several QorIQ blocks are only present on Freescale or NXP SoCs.
> > This series adds platform dependencies to the corresponding config
> > ymbols, to avoid asking the user about them when configuring a kernel
> > without support for these SoCs.
> >
> > Most patches in this series are independent, but the third patch may
> > cause some Kconfig warnings when applied before the second patch, and
> > enabling the QorIQ CPU frequency scaling driver in a non-Layerscape
> > kernel.
> >
> > Thanks for your comments!
>
> Thanks.  The series looks good to me.
>
> Are we trying to merge them through the various driver subsystems or I
> can also pull them in through the fsl-soc tree.  If we want to go
> through driver subsystems:

"fsl-soc" is safest, if maintainers agree, due to the weak dependency the
randconfig people may notice.
Note that Viresh already applied the 2nd patch to the cpufreq tree.

> Acked-by: Li Yang <leoyang.li@nxp.com>

Thanks!

Gr{oetje,eeting}s,

                        Geert