[meta-freescale] [meta-fsl-arm-extra][PATCH] linux-boundary, u-boot-script-boundary: Set compatible machine

[Otavio Salvador]

Hello Eric,

On Wed, Feb 19, 2014 at 1:05 PM, Eric Nelson
<eric.nelson at boundarydevices.com> wrote:
> Hi Otavio,
>
>
> On 02/19/2014 04:43 AM, Otavio Salvador wrote:
>>
>> On Tue, Feb 18, 2014 at 11:14 PM, Eric Nelson
>> <eric.nelson at boundarydevices.com> wrote:
>>>
>>> Thanks Otavio,
>>>
>>>
>>> On 02/10/2014 04:12 AM, Otavio Salvador wrote:
>>>>
>>>>
>>>> The compatible machine filter is required to avoid build warnings when
>>>> building for other boards, as:
>>>>
>>>> ,----[ Build warning ]
>>>> | WARNING: Unable to get checksum for linux-boundary
>>>> |     SRC_URI entry defconfig: file could not be found
>>>> `----
>>>>
>>>> Change-Id: I2f55697fbef29c35b332bfdcc5023c5f70bc9e4b
>>>> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
>>>> ---
>>>>    recipes-bsp/u-boot/u-boot-script-boundary_git.bb | 2 +-
>>>>    recipes-kernel/linux/linux-boundary_3.0.35.bb    | 1 +
>>>>    recipes-kernel/linux/linux-boundary_3.10.17.bb   | 3 ++-
>>>>    3 files changed, 4 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
>>>> b/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
>>>> index 345294e..380a785 100644
>>>> --- a/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
>>>> +++ b/recipes-bsp/u-boot/u-boot-script-boundary_git.bb
>>>> @@ -1,6 +1,5 @@
>>>>    LICENSE = "GPLv2+"
>>>>    LIC_FILES_CHKSUM =
>>>> "file://Licenses/README;md5=025bf9f768cbcb1a165dbe1a110babfb"
>>>> -COMPATIBLE_MACHINE = "(imx6qsabrelite|nitrogen6x)"
>>>>    DEPENDS = "u-boot-mkimage-native"
>>>>
>>>>    PV = "v2013.10+git${SRCPV}"
>>>> @@ -66,3 +65,4 @@ do_install[noexec] = "1"
>>>>    do_populate_sysroot[noexec] = "1"
>>>>
>>>>    PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>
>>>
>>>
>>> Note that this is different from the removed line above
>>> (Doesn't include imx6qsabrelite).
>>>
>>> My understanding is also that the clauses in COMPATIBLE_MACHINE are
>>> sub-strings or regex's of some sort, such that 'nitrogen6x-lite'
>>> is automatically allowed by 'nitrogen6x'.
>>
>>
>> Not really; In
>> http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/classes/base.bbclass?id=9bb5eb86c12ba6598fe317f449171fdf2310b933
>> I added support for it to use the MACHINEOVERRIDE. This is the
>> background mechanism used by SOC_FAMILY which does make the connection
>> between them ;-)
>
>
> Thanks for clarifying.
>
> Don't we still need imx6qsabrelite in the clause though?
>
> We keep seeing references that customers aren't able to build
> without pulling in PREFERRED_provider for our U-Boot and kernel.
>
> I usually recommend that they just use MACHINE=nitrogen6x,
> which also fixes the problem.

The override is added in:

https://github.com/Freescale/meta-fsl-arm-extra/blob/master/conf/machine/imx6qsabrelite.conf#L12

So this should be enough for it to work. Could you elaborate how it
failing at your customers? I did try it when I introduce this change
in linux-boundary and it worked fine so I may be missing something
which we may really need to fix.

Regards,

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750