Re: [meta-fsl-arm-extra][PATCH 4/4] cfa10036: Add cfa10036 machine configuration

[Alexandre Belloni <alexandre.belloni@free-electrons.com>]

On 22/08/2013 19:06, Otavio Salvador wrote:
> On Thu, Aug 22, 2013 at 11:51 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
>> This configuration supports the following Crystalfontz boards:
>>  - cfa-10036
>>  - cfa-10037
>>  - cfa-10049
>>  - cfa-10055
>>  - cfa-10056
>>  - cfa-10057
>>  - cfa-10058
>>
>> which are all based on the cfa-10036.
>>
>> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>> ---
>>  conf/machine/cfa10036.conf | 40 ++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 40 insertions(+)
>>  create mode 100644 conf/machine/cfa10036.conf
>>
>> diff --git a/conf/machine/cfa10036.conf b/conf/machine/cfa10036.conf
>> new file mode 100644
>> index 0000000..7f2fe35
>> --- /dev/null
>> +++ b/conf/machine/cfa10036.conf
>> @@ -0,0 +1,40 @@
>> +#@TYPE: Machine
>> +#@NAME: Crystalfontz CFA-10036
>> +#@SOC: i.MX28
>> +#@DESCRIPTION: Machine configuration for CFA-10036
>> +
>> +include conf/machine/include/mxs-base.inc
>> +
>> +SOC_FAMILY = "mxs:mx28"
>> +
>> +PREFERRED_PROVIDER_virtual/kernel = "linux-cfa"
>> +PREFERRED_VERSION_imx-bootlets = "cfa"
>> +
>> +BAREBOX_MACHINE = "cfa10036_defconfig"
>> +IMAGE_BOOTLOADER = "barebox"
>> +BAREBOX_BINARY = "barebox"
>> +
>> +IMXBOOTLETS_MACHINE = "cfa10036"
>> +BOOT_SPACE="32768"
>> +
>> +KERNEL_IMAGETYPE = "zImage"
>> +KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx28-cfa10036.dts"
> imx28-cfa10036.dtb ...
>
> No full path.

Oh yeah, and I saw that in linux-dtb.inc but didn't react on it. I would
be expecting a warning there with:

bbwarn "${DTB} contains the full path to the the dts file, but only the
dtb name should be used."

but didn't get any.



-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com