On Mon, Jan 22, 2018 at 04:07:44PM -0600, Adam Ford wrote: > On Mon, Jan 22, 2018 at 3:30 PM, Tom Rini <tr...@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 <aford...@gmail.com> > >> Reviewed-by: David Lechner <da...@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
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
