Re: [PATCH v2 1/9] ARM: dts: consolidate aliases for Cygnus dt files

From: Ray Jui
Date: Thu Sep 24 2015 - 18:23:55 EST




On 9/23/2015 2:48 PM, Florian Fainelli wrote:
> On 23/09/15 14:46, Ray Jui wrote:
>>
>>
>> On 9/23/2015 2:31 PM, Arnd Bergmann wrote:
>>> On Friday 18 September 2015 14:44:54 Ray Jui wrote:
>>>> On 9/18/2015 2:27 PM, Arnd Bergmann wrote:
>>>>> On Friday 18 September 2015 14:24:06 Ray Jui wrote:
>>>>>
>>>>> The SoC has at least four uarts according to this, so it seems unlikely that
>>>>> each board really only uses only the fourth one of them and labels it '0'
>>>>> on the board. As soon as you get one board that has more than one uart wired
>>>>> up, you would need to undo this.
>>>>>
>>>>
>>>> I think Scott might have explained this in the past. uart3 is going to
>>>> be used on all Cygnus boards (including all future boards) because the
>>>> bootrom was designed to use uart3 as console and that won't change.
>>>>
>>>> Let me know if you still think I need to move this back to the dts.
>>>
>>> I would still like to see them stay in the .dts file, if only for
>>> consistency with other platforms.
>>>
>>> Also, even if you can guarantee that uart3 is always used for the
>>> console, that doesn't prevent board designers from adding more than
>>> one uart, right?
>>>
>>> Arnd
>>>
>>
>> Okay. Given that this patch series has been merged by Florian, I'll
>> submit another patch to move it back to .dts files.
>
> You could send me either a replacement patch series (all 9), or an
> individual patch to replace a previous version (e.g; replace v2 with a
> v3), or an incremental one, whatever works for you.
>
> Thanks!
>

Hi Florian, I just sent you an incremental patch to move the alias back
to .dts files.

Thanks,

Ray
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/