[U-Boot] [U-Boot, V3] Convert CONFIG_SYS_DV_CLKMODE et al to Kconfig

[Tom Rini]

On Mon, Jan 22, 2018 at 04:07:44PM -0600, Adam Ford wrote:
> On Mon, Jan 22, 2018 at 3:30 PM, Tom Rini <trini at konsulko.com> wrote:
> > On Fri, Jan 12, 2018 at 07:26:06AM -0600, Adam Ford wrote:
> >
> >> This converts the following to Kconfig:
> >>    CONFIG_SYS_DV_CLKMODE
> >>    CONFIG_SYS_DA850_PLL0_POSTDIV
> >>    CONFIG_SYS_DA850_PLL0_PLLDIV1
> >>    CONFIG_SYS_DA850_PLL0_PLLDIV2
> >>    CONFIG_SYS_DA850_PLL0_PLLDIV3
> >>    CONFIG_SYS_DA850_PLL0_PLLDIV4
> >>    CONFIG_SYS_DA850_PLL0_PLLDIV5
> >>    CONFIG_SYS_DA850_PLL0_PLLDIV7
> >>    CONFIG_SYS_DA850_PLL1_POSTDIV
> >>    CONFIG_SYS_DA850_PLL1_PLLDIV1
> >>    CONFIG_SYS_DA850_PLL1_PLLDIV2
> >>    CONFIG_SYS_DA850_PLL1_PLLDIV3
> >>
> >> Signed-off-by: Adam Ford <aford173 at gmail.com>
> >> Reviewed-by: David Lechner <david at lechnology.com>
> >> ---
> >> Changes in V3:
> >>       Using the L138 Technical Reference manual, I changed the names to match
> >>         the descriptions of the registers as listed in Table 8-2 and 8-3 of the
> >>         PLL Control 0 and PLL Control 1 Registers.
> >> V2:
> >>         Expand Kconfig help definitions to give a small explanation of options
> >> V1:
> >> This patch is a continuation of Convert CONFIG_SOC_DA8XX et al
> >> This also showed warnings when used with the legoev3_defconfig, however
> >> it seems like the ev3 board was defining things in the header it didn't need
> >> since it doesn't seem to be building da850_lowlevel.c.  I don't have the
> >> hardware to test that board.
> >>
> >>  arch/arm/mach-davinci/Kconfig   | 85 ++++++++++++++++++++++++++++++++++++++++-
> >>  configs/omapl138_lcdk_defconfig |  1 +
> >>  include/configs/calimain.h      | 13 -------
> >>  include/configs/da850evm.h      | 13 -------
> >>  include/configs/ipam390.h       | 13 -------
> >>  include/configs/legoev3.h       | 13 -------
> >>  include/configs/omapl138_lcdk.h | 13 -------
> >>  scripts/config_whitelist.txt    | 12 ------
> >>  8 files changed, 85 insertions(+), 78 deletions(-)
> >
> > [patchwork link is https://patchwork.ozlabs.org/patch/859919/]
> >
> > First, this should also cover CONFIG_SYS_DA850_PLL0_PLLM which shouldn't
> > be too bad.
> >
> > Second, this needs to deal with calimain, which has both
> > CONFIG_SYS_DA850_PLL1_PLLM and CONFIG_SYS_DA850_PLL0_PLLM as dynamic.
> > Ideas?  Thanks!
> >
> Can we push the low-hanging fruit and come back and revisit the rest
> at a later date?   My goal was just try and clean up a bunch of config
> files.

Yeah, please respin a v4 that drops out CONFIG_SYS_DA850_PLL1_PLLM for
now, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180122/e639d5c3/attachment-0001.sig>