[U-Boot] [PATCH 13/13] mips: jz47xx: Add Creator CI20 platform

[Marek Vasut]

On 02/12/2017 12:52 PM, Andreas Färber wrote:
> Hi Marek,

Hi!

> Am 01.12.2016 um 02:06 schrieb Marek Vasut:
>> From: Paul Burton <paul.burton at imgtec.com>
>>
>> Add support for the Creator CI20 platform based on the JZ4780 SoC.
>> The DTS file comes from Linux 4.6 as of revision
>> 78800558d104e003f9ae92e0107f1de39cf9de9f
>>
>> So far, there are still a few details which will have to be fixed
>> once they are fleshed out in Linux:
>> - pinmux: Thus far, this board just pokes the pinmux registers to
>>           set the pinmux. For MMC in SPL, this will have to stay.
>> 	  But for full u-boot a proper pinmux driver will have to
>> 	  be added once the pinmux semantics in DT are in mainline
>> 	  Linux.
>> - ethernet,efuse: DT bindings are missing from mainline Linux.
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> Cc: Daniel Schwierzeck <daniel.schwierzeck at gmail.com>
>> Cc: Paul Burton <paul.burton at imgtec.com>
>> ---
>>  arch/mips/dts/Makefile        |   1 +
>>  arch/mips/dts/ci20.dts        | 114 ++++++++++++++
>>  arch/mips/mach-jz47xx/Kconfig |  11 ++
>>  board/imgtec/ci20/Kconfig     |  35 +++++
>>  board/imgtec/ci20/Makefile    |   5 +
>>  board/imgtec/ci20/README      |  10 ++
>>  board/imgtec/ci20/ci20.c      | 354 ++++++++++++++++++++++++++++++++++++++++++
>>  configs/ci20_defconfig        |  28 ++++
>>  include/configs/ci20.h        | 105 +++++++++++++
>>  9 files changed, 663 insertions(+)
>>  create mode 100644 arch/mips/dts/ci20.dts
>>  create mode 100644 board/imgtec/ci20/Kconfig
>>  create mode 100644 board/imgtec/ci20/Makefile
>>  create mode 100644 board/imgtec/ci20/README
>>  create mode 100644 board/imgtec/ci20/ci20.c
>>  create mode 100644 configs/ci20_defconfig
>>  create mode 100644 include/configs/ci20.h
> 
> I've looked into testing the remainder of this patchset, not seeing a
> newer version. You can find my branch here:
> 
> https://github.com/afaerber/u-boot/commits/ci20
> 
> In particular I fixed the MMC set_ios signature to silence a warning
> about the int vs. void return type, which I intend to clean up and submit.
> 
> The code compiled okay after some defconfig tweaks, save for a few
> unused-variable SPL-only warnings, but testing did not give any output.
> Investigating that, it seemed to me
> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y was missing for
> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR. Some other include options
> could be moved into defconfig, too.

Hm, so it broke again ? That's kinda sad ...

> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR=y leads to this error:
> 
>   LD      spl/u-boot-spl
> mipsel-suse-linux-ld.bfd: u-boot-spl section `.data' will not fit in
> region `.sram'
> mipsel-suse-linux-ld.bfd: region `.sram' overflowed by 288 bytes
> ../scripts/Makefile.spl:304: recipe for target 'spl/u-boot-spl' failed
> make[2]: *** [spl/u-boot-spl] Error 1
> /home/andreas/OBS/u-boot/Makefile:1342: recipe for target
> 'spl/u-boot-spl' failed
> make[1]: *** [spl/u-boot-spl] Error 2
> make[1]: Leaving directory '/home/andreas/OBS/u-boot/ci20'
> Makefile:150: recipe for target 'sub-make' failed
> make: *** [sub-make] Error 2
> 
> I've reviewed all SPL Kconfig options and found three seemingly unneeded
> options defaulting to y, but I did not find a way to get this number
> down even a single byte with my GCC 6.3.1, and the recommended 4.8.1 was
> even worse (~748).

The recommended one was gcc 6.x , again, when I submitted it, I had no
problem. So it seems U-Boot again gained bloat, oh well ...

> I also tried combining the downstream 4.8.1-built SPL
> with the upstream U-Boot (from without
> CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR), but still no output.
> 
> Is this the reason this series is not respun?

Eh ? No, the reason is I lack time :) The SPL will give you no output
though, since it is stripped down to bare minimum.

> Regards,
> Andreas
> 


-- 
Best regards,
Marek Vasut