[Buildroot] [PATCH 2/2 v3] xtensa: support configurable processor configurations

Chris Zankel chris at zankel.net
Mon Nov 12 23:45:47 UTC 2012


On 11/12/2012 01:33 PM, Arnout Vandecappelle wrote:
>
>    Perhaps it's better to move the condition to a
> if BR2_xtensa
> ...
> endif
> around the entire file.  Thomas has a patch lined up to move all
> the conditions outside of the Config.in.*
>> It seems Xtensa would be the only one that uses that scheme, so maybe
>> wait for the generic patch from Thomas?
>   I don't think so. Then it means more work for Thomas to refresh his
> patch.  There will be a merge conflict anyway between the two, but
> with the condition outside the file the conflict will be small.
>
>   That said, Peter hasn't merged Thomas's patch even though I acked
> it, so maybe he has some fundamental objection to the principle.
> Or maybe it's just because I didn't test it.
I guess I don't understand what that patch does, i.e. where is the
'if...endif'? Could you maybe point to the patch?

Thanks,
-Chris



More information about the buildroot mailing list