Hi Morimoto-san,
On Tue, Oct 10, 2017 at 2:07 AM, Kuninori Morimoto
<kuninori.morimoto.gx@xxxxxxxxxxx> wrote:
>> > - audio_clka
>> > - audio_clkb
>> > + audio_clk_a
>> > + audio_clk_b
>>
>> It's debatable.
>> The datasheet uses "CLKA" and "CLKOUT", while the pinctrl drivers tend
>> to use "clk_a" (with underscore) and "clkout" (without underscore).
>>
>> Except for r8a7794, which uses the "clka" form. Oops...
>> As this is DT ABI, we cannot easily change it. But if you want, we can add
>> aliases.
>
> I have no strong opinion about this "clka" vs "clk_a".
> I will do nothing if you are OK on this current patch (= "clka"),
> I will post v2 patch if you say "clk_a" is good.
I think it's best to follow what's done in the other R-Car Gen3 SoCs.
Else we may end up in trouble if a board shows up that can support both
R-Car D3 and another R-Car Gen3 SoC (cfr. Salvator-X(S) and ULCB
with R-Car H3 and M3-W).
Do you agree?
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
