Patchwork [meta-fsl-arm-extra,5/6] wandboard-dual: Use U-Boot mainline

login
register
mail settings
Submitter Otavio Salvador
Date April 9, 2013, 5:04 p.m.
Message ID <1365527093-12139-5-git-send-email-otavio@ossystems.com.br>
Download mbox | patch
Permalink /patch/47755/
State Changes Requested
Delegated to: Otavio Salvador
Headers show

Comments

Otavio Salvador - April 9, 2013, 5:04 p.m.
The U-Boot 2013.04 has support for Wandboard so we use it, from now on.

Change-Id: I9bdf3ce688d13f8d07d83643df097a1ce6341b6c
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 conf/machine/wandboard-dual.conf | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
John Weber - April 9, 2013, 5:39 p.m.
> diff --git a/conf/machine/wandboard-dual.conf b/conf/machine/wandboard-dual.conf
> index 02d63b7..f19b1a4 100644
> --- a/conf/machine/wandboard-dual.conf
> +++ b/conf/machine/wandboard-dual.conf
> @@ -9,12 +9,9 @@ SOC_FAMILY = "wandboard:mx6dl:mx6"
>
>   KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx6dl-wandboard.dts"

Same question as in the solo recipe patch.  We're not using dts, so where does 
this file come from?
Otavio Salvador - April 9, 2013, 5:53 p.m.
On Tue, Apr 9, 2013 at 2:39 PM, John Weber <rjohnweber@gmail.com> wrote:
>> diff --git a/conf/machine/wandboard-dual.conf
>> b/conf/machine/wandboard-dual.conf
>> index 02d63b7..f19b1a4 100644
>> --- a/conf/machine/wandboard-dual.conf
>> +++ b/conf/machine/wandboard-dual.conf
>> @@ -9,12 +9,9 @@ SOC_FAMILY = "wandboard:mx6dl:mx6"
>>
>>   KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx6dl-wandboard.dts"
>
>
> Same question as in the solo recipe patch.  We're not using dts, so where
> does this file come from?
>

This will be used by mainline kernel; however I did not add it for the
dual and it came on first board patch. I think it does not hurt
anyway.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
Daiane Angolini - April 9, 2013, 5:56 p.m.
On 04/09/2013 02:53 PM, Otavio Salvador wrote:
> On Tue, Apr 9, 2013 at 2:39 PM, John Weber <rjohnweber@gmail.com> wrote:
>>> diff --git a/conf/machine/wandboard-dual.conf
>>> b/conf/machine/wandboard-dual.conf
>>> index 02d63b7..f19b1a4 100644
>>> --- a/conf/machine/wandboard-dual.conf
>>> +++ b/conf/machine/wandboard-dual.conf
>>> @@ -9,12 +9,9 @@ SOC_FAMILY = "wandboard:mx6dl:mx6"
>>>
>>>    KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx6dl-wandboard.dts"
>>
>>
>> Same question as in the solo recipe patch.  We're not using dts, so where
>> does this file come from?
>>
>
> This will be used by mainline kernel; however I did not add it for the
> dual and it came on first board patch. I think it does not hurt
> anyway.

It did not hurt, but I decided to not include this "useless" source code 
when I created the imx6dlsabresd and imx6slevk.

It may lead user to an error.

Daiane
Otavio Salvador - April 9, 2013, 6 p.m.
On Tue, Apr 9, 2013 at 2:56 PM, Daiane Angolini
<daiane.angolini@freescale.com> wrote:
> On 04/09/2013 02:53 PM, Otavio Salvador wrote:
>>
>> On Tue, Apr 9, 2013 at 2:39 PM, John Weber <rjohnweber@gmail.com> wrote:
>>>>
>>>> diff --git a/conf/machine/wandboard-dual.conf
>>>> b/conf/machine/wandboard-dual.conf
>>>> index 02d63b7..f19b1a4 100644
>>>> --- a/conf/machine/wandboard-dual.conf
>>>> +++ b/conf/machine/wandboard-dual.conf
>>>> @@ -9,12 +9,9 @@ SOC_FAMILY = "wandboard:mx6dl:mx6"
>>>>
>>>>    KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx6dl-wandboard.dts"
>>>
>>>
>>>
>>> Same question as in the solo recipe patch.  We're not using dts, so where
>>> does this file come from?
>>>
>>
>> This will be used by mainline kernel; however I did not add it for the
>> dual and it came on first board patch. I think it does not hurt
>> anyway.
>
>
> It did not hurt, but I decided to not include this "useless" source code
> when I created the imx6dlsabresd and imx6slevk.
>
> It may lead user to an error.

I can remove this from solo addition, and add a patch to drop the dual one.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Patch

diff --git a/conf/machine/wandboard-dual.conf b/conf/machine/wandboard-dual.conf
index 02d63b7..f19b1a4 100644
--- a/conf/machine/wandboard-dual.conf
+++ b/conf/machine/wandboard-dual.conf
@@ -9,12 +9,9 @@  SOC_FAMILY = "wandboard:mx6dl:mx6"
 
 KERNEL_DEVICETREE = "${S}/arch/arm/boot/dts/imx6dl-wandboard.dts"
 
-PREFERRED_PROVIDER_u-boot = "u-boot-imx" 
 PREFERRED_PROVIDER_virtual/kernel = "linux-wandboard"
 
-UBOOT_MACHINE = "wandboard_config"
-UBOOT_SUFFIX = "bin"
-UBOOT_PADDING = "2"
+UBOOT_MACHINE = "wandboard_dl_config"
 
 SERIAL_CONSOLE = "115200 ttymxc0"