[meta-freescale] [meta-fsl-arm][PATCH 08/14] u-boot-imx: Upgrade to 2014.04

[Otavio Salvador]

On Fri, Aug 29, 2014 at 10:53 AM, Daiane Angolini <daiane.list at gmail.com> wrote:
> On Wed, Aug 27, 2014 at 4:07 PM, Lauren Post <lauren.post at freescale.com> wrote:
>> - Add mx6sx support
>> - General fixes for mx6slevk and other mx6
>> - Support to use zImage
>
> I just realized the patch order is wrong. You must first add the uboot
> support and _then_ add the new machine support which depends on that
> u-boot specific version.
>
> Can you, please, change the order when you send v2?

Agreed.

>>
>> Signed-off-by: Lauren Post <lauren.post at freescale.com>
>> ---
>>  ...ol_2013.04.bb => u-boot-imx-mfgtool_2014.04.bb} |    0
>>  ...u-boot-imx_2013.04.bb => u-boot-imx_2014.04.bb} |    8 ++++----
>>  2 files changed, 4 insertions(+), 4 deletions(-)
>>  rename recipes-bsp/u-boot/{u-boot-imx-mfgtool_2013.04.bb => u-boot-imx-mfgtool_2014.04.bb} (100%)
>>  rename recipes-bsp/u-boot/{u-boot-imx_2013.04.bb => u-boot-imx_2014.04.bb} (57%)
>
> You are updating both u-boot-imx and u-boot-imx-mfgtool in the same patch.
>
> I would split it, but I´m not sure what Otavio say about this.
>
> If you keep both I would ask you to update commit subject and commit
> log, as you are updating 2 recipes and say you are updating only 1.
>
> You can say something like:
>
>  - Add mx6sx support
>  - General fixes for mx6slevk and other mx6
>  - Support to use zImage
> - Bump u-boot-imx-mfgtools version from 2013.04 to 2014.04 with no change.

You need to bump them together as the mfgtools one includes the regular recipe.

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