> On May 27, 2015, at 11:48 AM, Daiane Angolini <[email protected]> wrote:
> 
> On Wed, May 27, 2015 at 3:17 PM, Otavio Salvador
> <[email protected]> wrote:
>> On Wed, May 27, 2015 at 12:33 PM, Khem Raj <[email protected]> wrote:
>>> These SoCs are capable of thmub2 ISA as well as VFP
>>> unit is available on them. This patch makes the default
>>> tunes aware of that. The final ISA still remains as it is currently but
>>> it offers a choice to distros to choose an alternative thumb2 or a mix
>>> if they want to.
>>> 
>>> Signed-off-by: Khem Raj <[email protected]>
>> 
>> Your patch seems nicer; it would be good though if you could add the
>> discussion link included at:
>> 
>> http://patchwork.openembedded.org/patch/94025/
> 
> And please, improve your commit log a little bit. I think it's confusing.
> 
> "These SoCs are capable of thmub2 ISA as well as VFP
> unit is available on them. This patch makes the default
> tunes aware of that. The final ISA still remains as it is currently but
> it offers a choice to distros to choose an alternative thumb2 or a mix
> if they want to."
> 
> Instead of "these socs" please say "The mx6 SoCs”

sure

> 
> typo thmub2 -> thumb2

yes

> 
> soc are capable of thumb2 and VFP unit is available?
> 
> "The final ISA still remains as it is currently"
> What does it means? The default for ISA is the same?

It means that code generation still remains as it was before unless thumb2 is 
chosen
explicitly.

> 
> And please, avoid choice and choose in the same sentence. And I'm not
> sure what you mean by "thumb2 or a mix, if they want to" (who is they?
> distros doesn't make choices )

what ? Distro make choices about default ISA before the machines, they got to 
choose from the options that are offered by a machine.

> 
> Maybe something like:
> 
> "
> mx6 SoC can be used with both thumb2 ISA or VFP enabled.
> 
> Change the default tunes in order to enable both always.
> 
> In addition, it offers more options to external distros.
> “

its a machine layer so all distros are external to it.

> 
> At least it was what I understood from your commit log. This way you
> can double check if my understanding is what you meant.
> 

Thanks for your feedback, I will send a v2 shortly.

> 
> Daiane
> 
> 
>> 
>> --
>> 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
>> --
>> _______________________________________________
>> meta-freescale mailing list
>> [email protected]
>> https://lists.yoctoproject.org/listinfo/meta-freescale

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

-- 
_______________________________________________
meta-freescale mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-freescale

Reply via email to