Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1591185 > unrolled thread

Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-03-02 16:20 +0100
    Re: [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for  PRCM CCUs "maxime.ripard@free-electrons.com" <maxime.ripard@free-electrons.com> - 2017-03-03 16:00 +0100

#1591185 — Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs

[Multipart message — attachments visible in raw view] — view raw

On Wed, Mar 01, 2017 at 08:22:13PM +0800, Icenowy Zheng wrote:
> > I'm a bit worried by that to be honest. You claim to support the A31, 
> > yet jugdging by the current state of that code you never actually 
> > tested it on that SoC. 
> >
> > What makes you say that the PRCM clocks are the same for the H3 and 
> > A64? We have to be sure, otherwise we might not be able to get the DT 
> > binding right from the very beginning, and we might not be able to fix 
> > it later. 
> 
> In fact, if we worry about this, we shouldn't make r-ccu, as
> dedicated clocks are more easy to fix.
> 
> For newer SoCs' PRCM, we never have enough documents, and Allwinner
> have said that they cannot provide it. (I asked them for this.)
> 
> The best solution is to implement mature enough dedicated clocks
> before we convert to ccu.

What do you mean by dedicated clocks?

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [next] | [standalone]

#1592027 — Re: [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs

[Multipart message — attachments visible in raw view] — view raw

On Thu, Mar 02, 2017 at 11:13:53PM +0800, Icenowy Zheng wrote:
> 
> 
> 02.03.2017, 22:09, "Maxime Ripard" <maxime.ripard@free-electrons.com>:
> > On Wed, Mar 01, 2017 at 08:22:13PM +0800, Icenowy Zheng wrote:
> >>  > I'm a bit worried by that to be honest. You claim to support the A31,
> >>  > yet jugdging by the current state of that code you never actually
> >>  > tested it on that SoC.
> >>  >
> >>  > What makes you say that the PRCM clocks are the same for the H3 and
> >>  > A64? We have to be sure, otherwise we might not be able to get the DT
> >>  > binding right from the very beginning, and we might not be able to fix
> >>  > it later.
> >>
> >>  In fact, if we worry about this, we shouldn't make r-ccu, as
> >>  dedicated clocks are more easy to fix.
> >>
> >>  For newer SoCs' PRCM, we never have enough documents, and Allwinner
> >>  have said that they cannot provide it. (I asked them for this.)
> >>
> >>  The best solution is to implement mature enough dedicated clocks
> >>  before we convert to ccu.
> >
> > What do you mean by dedicated clocks?
> 
> The legacy form of clocks.

Which itself creates another form of issues. What happens if we get
something wrong on those clocks (as it is likely to happen)?

We potentially can't fix it at all, that's what happens.

And that's leaving aside the DT and clocks maintainers regular
complaints that we should get away from those.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web