[U-Boot] [PATCH 2/3] ARM: am335x: Add support for chiliSOM

Marcin Niestroj m.niestroj at grinn-global.com
Mon Jan 23 17:27:09 CET 2017


Hi,
Thanks for review! See my comment below.

On 23.01.2017 16:56, Tom Rini wrote:
> On Mon, Jan 23, 2017 at 02:39:15PM +0100, Marcin Niestroj wrote:
>
>> chiliSOM is a System On Module (http://http://grinn-global.com/chilisom/).
>> It can't exists on its own, but will be used as part of other boards.
>>
>> Hardware specification:
>>  * TI AM335x processor
>>  * 128M, 256M or 512M DDR3 memory
>>  * up to 256M NAND
>>
>> Here we treat SOM similar to SOC, so we place it inside arch/arm/mach-*
>> directory and make it possible to reuse initialization code (i.e. DDR,
>> NAND init) for all boards that use it. This approach is similar as for
>> liteSOM module.
>>
>> Signed-off-by: Marcin Niestroj <m.niestroj at grinn-global.com>
>> ---
>>  arch/arm/Kconfig                               |   2 +
>>  arch/arm/Makefile                              |   1 +
>>  arch/arm/mach-chilisom/Kconfig                 |   4 +
>>  arch/arm/mach-chilisom/Makefile                |   6 +
>>  arch/arm/mach-chilisom/chilisom.c              | 185 +++++++++++++++++++++++++
>>  arch/arm/mach-chilisom/include/mach/chilisom.h |  15 ++
>>  6 files changed, 213 insertions(+)
>
> These don't belong in a new mach directory, this should end up in
> board/grinn/common/ instead, thanks!
>

This will work for all grinn boards. But the idea is that some other
vendor can make a board that will use chilisom as it's base. And in
that case sources from board/grinn/common/ directory won't compile.
Do you have any idea how to bypass this restriction?

-- 
Marcin Niestroj


More information about the U-Boot mailing list