Re: [U-Boot] [PATCH v2] mx6: add support of multi-processor command

Hello Gabriel,

On 22-06-14 01:55, Gabriel Huau wrote:

This allows u-boot to load different OS or Bare Metal application on the different cores of the i.MX6DQ. For example: we can run Android on cpu0 and a RT OS like QNX/FreeRTOS on cpu1.

Signed-off-by: Gabriel Huau contact@huau-gabriel.fr

funny..

+int is_core_valid(unsigned int nr) +{

• if (nr < 0 || nr >= CONFIG_NUM_CPUS)

• return 0;
  

• return 1;

+}

I am not sure if u-boot has rules for it, but personally I would prefer not to test values which are obvious unsigned for negativity, since it causes unnecessary noise when compiling u-boot with W=1, W=2.

Regards, Jeroen