>>>>> "Igor" == Igor Mammedov <imamm...@redhat.com> writes:
Igor> If user provided non-sense RAM size, board will complain and Igor> continue running with max RAM size supported. Also RAM is going Igor> to be allocated by generic code, so it won't be possible for Igor> board to fix things up for user. Igor> Make it error message and exit to force user fix CLI, instead of Igor> accepting non-sense CLI values. I think this comment needs rewording a little. Maybe: If the user provided too large a RAM size, the code used to complain and trim it to the max size. Now tht RAM is allocated by generic code, that's no longer possible, so generate an error and exit instead. Igor> /* Check the amount of memory is compatible with the SOC */ Igor> if (machine->ram_size > (FSL_IMX31_SDRAM0_SIZE + Igor> FSL_IMX31_SDRAM1_SIZE)) { Igor> - warn_report("RAM size " RAM_ADDR_FMT " above max supported, " Igor> + error_report("RAM size " RAM_ADDR_FMT " above max supported, " Igor> "reduced to %x", machine->ram_size, Igor> FSL_IMX31_SDRAM0_SIZE + FSL_IMX31_SDRAM1_SIZE); This needs to be changed so it doesn't say that the RAM size is reduced, just what the maximum is. Maybe: error_report("RAM size " RAM_ADDR_FMT " above max (%x) supported.", machine->ram_size, FSL_IMX31_SDRAM0_SIZE + FSL_IMX31_SDRAM1_SIZE); Peter C