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