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

[Eric Nelson <eric.nelson@boundarydevices.com>]

Thanks Otavio,

On 02/19/2014 09:45 AM, Otavio Salvador wrote:
> Hello Eric,
>
> On Wed, Feb 19, 2014 at 1:05 PM, Eric Nelson
> <eric.nelson@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@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@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
>

Cool.

> 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.
>

I'm not 100% certain it's our customer. The failure was posed most
recently on i.MX Community:
	https://community.freescale.com/thread/319716

It's also not clear that the problem didn't stem from the use
of another repository (freesclae-release-bsp).

Regards,


Eric