* [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs [not found] <20170301152236.MUXmRgOe@smtp1m.mail.yandex.net> @ 2017-03-02 14:09 ` Maxime Ripard 2017-03-02 15:13 ` [linux-sunxi] " Icenowy Zheng 0 siblings, 1 reply; 6+ messages in thread From: Maxime Ripard @ 2017-03-02 14:09 UTC (permalink / raw) To: linux-arm-kernel 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170302/08d1f681/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs 2017-03-02 14:09 ` [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs Maxime Ripard @ 2017-03-02 15:13 ` Icenowy Zheng 2017-03-03 14:51 ` maxime.ripard at free-electrons.com 0 siblings, 1 reply; 6+ messages in thread From: Icenowy Zheng @ 2017-03-02 15:13 UTC (permalink / raw) To: linux-arm-kernel 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. > > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com > > -- > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs 2017-03-02 15:13 ` [linux-sunxi] " Icenowy Zheng @ 2017-03-03 14:51 ` maxime.ripard at free-electrons.com 0 siblings, 0 replies; 6+ messages in thread From: maxime.ripard at free-electrons.com @ 2017-03-03 14:51 UTC (permalink / raw) To: linux-arm-kernel 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170303/a049eba7/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <20170301152220.MEgmqtKw@smtp2p.mail.yandex.net>]
* [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs [not found] <20170301152220.MEgmqtKw@smtp2p.mail.yandex.net> @ 2017-03-02 14:08 ` Maxime Ripard 2017-03-02 15:13 ` Icenowy Zheng 0 siblings, 1 reply; 6+ messages in thread From: Maxime Ripard @ 2017-03-02 14:08 UTC (permalink / raw) To: linux-arm-kernel On Wed, Mar 01, 2017 at 08:17:39PM +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. > > I only claim to reserve possibility to extend the driver to support > A31, but I didn't claim it currently support A31. Well, you did call it sun6i. Please rename it, and we will consolidate in due time. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170302/1a730f25/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs 2017-03-02 14:08 ` Maxime Ripard @ 2017-03-02 15:13 ` Icenowy Zheng 2017-03-03 14:48 ` Maxime Ripard 0 siblings, 1 reply; 6+ messages in thread From: Icenowy Zheng @ 2017-03-02 15:13 UTC (permalink / raw) To: linux-arm-kernel 02.03.2017, 22:09, "Maxime Ripard" <maxime.ripard@free-electrons.com>: > On Wed, Mar 01, 2017 at 08:17:39PM +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. >> >> ?I only claim to reserve possibility to extend the driver to support >> ?A31, but I didn't claim it currently support A31. > > Well, you did call it sun6i. > > Please rename it, and we will consolidate in due time. Then rename it back when it really gains support for A31? > > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com ^ permalink raw reply [flat|nested] 6+ messages in thread
* [linux-sunxi] Re: [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs 2017-03-02 15:13 ` Icenowy Zheng @ 2017-03-03 14:48 ` Maxime Ripard 0 siblings, 0 replies; 6+ messages in thread From: Maxime Ripard @ 2017-03-03 14:48 UTC (permalink / raw) To: linux-arm-kernel On Thu, Mar 02, 2017 at 11:13:18PM +0800, Icenowy Zheng wrote: > > > 02.03.2017, 22:09, "Maxime Ripard" <maxime.ripard@free-electrons.com>: > > On Wed, Mar 01, 2017 at 08:17:39PM +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. > >> > >> ?I only claim to reserve possibility to extend the driver to support > >> ?A31, but I didn't claim it currently support A31. > > > > Well, you did call it sun6i. > > > > Please rename it, and we will consolidate in due time. > > Then rename it back when it really gains support for A31? Yes. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170303/b1f2dd6f/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-03-03 14:51 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20170301152236.MUXmRgOe@smtp1m.mail.yandex.net>
2017-03-02 14:09 ` [PATCH 2/3] clk: sunxi-ng: add support for PRCM CCUs Maxime Ripard
2017-03-02 15:13 ` [linux-sunxi] " Icenowy Zheng
2017-03-03 14:51 ` maxime.ripard at free-electrons.com
[not found] <20170301152220.MEgmqtKw@smtp2p.mail.yandex.net>
2017-03-02 14:08 ` Maxime Ripard
2017-03-02 15:13 ` Icenowy Zheng
2017-03-03 14:48 ` Maxime Ripard
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox