Hi Simon,
On Mon, Feb 16, 2015 at 2:57 PM, Simon Horman <horms@xxxxxxxxxxxx> wrote:
> On Mon, Feb 16, 2015 at 10:20:10AM +0100, Geert Uytterhoeven wrote:
>> Hi Kaneko-san, Mizuguchi-san,
>>
>> On Sun, Feb 15, 2015 at 3:49 PM, Yoshihiro Kaneko <ykaneko0929@xxxxxxxxx> wrote:
>> > From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx>
>> >
>> > HSUSB uses hp_clk for H/W register access.
>>
>> That indeed seems to be true, and that's what this patch is changing.
>>
>> > HSUSB uses mp_clk for USB-DMAC access.
>>
>> However, page 64-2 doesn't mention the MP clock for USB-DMAC:
>> "AXI bus clock (ZSφ: 260 MHz), HPB bus clock (HPφ: 130 MHz).
>> USB local bus clock (48 MHz)"
>> So your statement doesn't seem to be true?
>
> Thanks, I see that too.
>
>> Note that mstp3_clks in dtsi has hp_clk as the parent for both usbdmac0 and
>> usbdmac1, so if your statement would be true, that has to changed, too?
>>
>> The same is true for other members of the R-Car Gen2 family, e.g. your patch
>> for r8a7791.
>
> It seems to me that we could:
>
> 1. Apply this patch with the changelog updated and;
> 2. Follow up on the DMAC clocks
>
> Does that sound reasonable to you?
Yes, that sounds fine to me.
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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
