[meta-freescale] [meta-fsl-arm PATCH v3 05/13] imx-base.inc: Disable firmware include for not fully supported BSPs

Otavio Salvador otavio at ossystems.com.br
Mon Apr 15 10:10:54 PDT 2013


On Mon, Apr 15, 2013 at 1:40 PM, Daiane Angolini
<daiane.angolini at freescale.com> wrote:
> On 04/15/2013 11:14 AM, Otavio Salvador wrote:
>>
>> On Mon, Apr 15, 2013 at 11:02 AM, Daiane Angolini
>> <daiane.angolini at freescale.com> wrote:
>>>
>>> On 04/15/2013 08:51 AM, Otavio Salvador wrote:
>>>>
>>>>
>>>> The i.MX25, i.MX27, i.MX31 and i.MX35 are not fully supported and we
>>>> don't have the firmware package for them so comment them out so it
>>>> doesn't break the build.
>>>
>>>
>>> Why to keep this code? I would prefer to remove the lines instead of
>>> comment
>>> them.
>>
>>
>> I can remove it but I kept it as it is the name of the generated files
>> and makes it easier to add it back (also has the FIXME). So if someone
>> grep the code looking for firmware for it, it'll find it commented and
>> with a FIXME on top of it.
>>
>> What do you think?
>
>
> I prefer to remove the code. But, it's not that hard preference.

And about the FIXME?

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



More information about the meta-freescale mailing list