On 17/04/2020 04:05, Tom Rini wrote: (adding Masahiro for Kconfig)
> On Mon, Apr 06, 2020 at 05:58:19PM +0530, Amit Singh Tomar wrote: > >> This adds Cubieboard7[1] support based on Action Semi's S700 SoC[2], It's >> Quad-core ARMv8 SoC >> with Cortex-A53 cores. Peripheral like UART seems to be compatible with S900 >> SoC(basic support >> for it is alreay present in u-boot). >> >> This series(v10) takes care the commments provided by Mani and patches >> 04/12, 07/12 and 12/12 >> has been changed to address those comments. >> >> Previous series(v9) fixes a Bug that breaks bubblegum96 board boot(reported >> by Mani). It was >> due to fact that driver data read is not proper in the clock driver. There >> are changes in >> patch 06/12 to fix it. >> >> Series(v8) removes the SoC specific include instead just uses owl-common. >> For this >> patch 01/12 and 09/12 changes a bit. >> >> Series(v7) fixes a serious Bug that breaks S900, it was there since >> v5.Thanks to Andre >> for pointing it out. >> >> Series(v6)[3] does following changes: >> >> * [PATCH v5 06/11] becomes [PATCH v6 03/11] >> * [PATCH v5 03/11] becomes [PATCH v6 04/11] >> * Introduce a new patch to move defconfig options to Kconfig which is [PATCH >> v6 10/12] >> >> Series(v5)[4] just re-orders the patches so that U-BOOT(with >> bubblegum96_defconfig) builds >> after every patch of the series(suggested by Andre). >> >> S700 support is tested[5] on Cubieboard7-lite board and S900 support is just >> compiled tested. >> >> This patch series can be tested using below tree: >> https://github.com/Atomar25/u-boot/commits/s700_v10 >> >> [1]: http://www.cubietech.com/product-detail/cubieboard7/ >> [2]: http://www.actions-semi.com/en/productview.aspx?id=225 >> [3]: >> http://u-boot.10912.n7.nabble.com/PATCH-v6-00-12-Actions-S700-SoC-support-td403562.html#a403567 >> [4]: >> http://u-boot.10912.n7.nabble.com/PATCH-v5-00-11-Actions-S700-SoC-support-td402752.html#a402762 >> [5]: https://paste.ubuntu.com/p/TbBtk5dPGS/ >> >> Amit Singh Tomar (12): >> arm: actions: Add common framework for Actions Owl Semi SoCs >> arm: actions: rename sysmap-s900 to sysmap-owl >> serial: actions: add compatible string >> arm: dts: sync dts for Action Semi S900 >> arm: dts: actions: s900: add u-boot specific dtsi file >> clk: actions: Add common clock driver >> arm: actions: add S700 SoC device tree >> arm: dts: actions: s700: add u-boot specific dtsi file >> arm: add support Actions Semi S700 >> actions: Move defconfig options to Kconfig >> arm: add Cubieboard7 board support >> doc: boards: add Cubieboard7 documentation > > A few problems. First, "actions: Move defconfig options to Kconfig" > breaks a large number of boards including p2371-2180 in one way and > libretech_all_h5_cc_h5 (along with lots of other sunxi platforms) in a > different but related way. (Masahiro: it's about this patch: https://lists.denx.de/pipermail/u-boot/2020-April/405672.html) Tom, many thanks for the heads up, I can confirm the problem, but am totally clueless as of *why* this happens: The changes in this patch add options to arch/arm/mach-owl/Kconfig, and are totally contained inside an "if ARCH_OWL .. endif" clamp, so how could this even affect other platforms (which are clearly not defining ARCH_OWL)? Cheers, Andre > Second, the new defconfig isn't listed under > any MAINTAINERS file and needs to be. I had fixed this up while testing > but the other problem is more severe and needs looking in to. Thanks! >