Linux-Clk Archive on lore.kernel.org
 help / color / Atom feed
* clk: imx: clock driver for imx8qm?
@ 2020-02-13 15:31 Oliver Graute
  2020-02-13 16:18 ` Aisheng Dong
  0 siblings, 1 reply; 6+ messages in thread
From: Oliver Graute @ 2020-02-13 15:31 UTC (permalink / raw)
  To: festevam, Anson.Huang
  Cc: Aisheng Dong, leonard.crestez, peng.fan, linux-clk, linux-kernel

Hello Favio,
Hello Anson,

is someone working on clock driver for imx8qm? I miss at least a
clk-imx8qm.c in the drivers/imx/clk/ directory. I saw that you are
working in this area and perhaps you can give me some insights what is
needed here.

Best regards,

Oliver

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: clk: imx: clock driver for imx8qm?
  2020-02-13 15:31 clk: imx: clock driver for imx8qm? Oliver Graute
@ 2020-02-13 16:18 ` Aisheng Dong
  2020-02-13 17:42   ` Oliver Graute
  0 siblings, 1 reply; 6+ messages in thread
From: Aisheng Dong @ 2020-02-13 16:18 UTC (permalink / raw)
  To: Oliver Graute, festevam, Anson Huang, Stephen Boyd
  Cc: Leonard Crestez, Peng Fan, linux-clk, linux-kernel

Hi Oliver,

> From: Oliver Graute <oliver.graute@gmail.com>
> Sent: Thursday, February 13, 2020 11:32 PM
>
> Hello Favio,
> Hello Anson,
> 
> is someone working on clock driver for imx8qm? I miss at least a clk-imx8qm.c
> in the drivers/imx/clk/ directory. I saw that you are working in this area and
> perhaps you can give me some insights what is needed here.
> 

MX8QM/QXP are using the same clock driver clk-imx8qxp.c

[PATCH RESEND V5 00/11] clk: imx8: add new clock binding for better pm support
https://www.spinics.net/lists/arm-kernel/msg781687.html
The review of that patch series is pending for a couple of months.

Stephen,
Would you help it?

Regards
Aisheng

> Best regards,
> 
> Oliver

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: clk: imx: clock driver for imx8qm?
  2020-02-13 16:18 ` Aisheng Dong
@ 2020-02-13 17:42   ` Oliver Graute
  2020-02-14  5:50     ` Aisheng Dong
  0 siblings, 1 reply; 6+ messages in thread
From: Oliver Graute @ 2020-02-13 17:42 UTC (permalink / raw)
  To: Aisheng Dong
  Cc: festevam, Anson Huang, Stephen Boyd, Leonard Crestez, Peng Fan,
	linux-clk, linux-kernel

On 13/02/20, Aisheng Dong wrote:
> Hi Oliver,
> 
> > 
> > is someone working on clock driver for imx8qm? I miss at least a clk-imx8qm.c
> > in the drivers/imx/clk/ directory. I saw that you are working in this area and
> > perhaps you can give me some insights what is needed here.
> > 
> 
> MX8QM/QXP are using the same clock driver clk-imx8qxp.c

ok thx, for that clarification.

> 
> [PATCH RESEND V5 00/11] clk: imx8: add new clock binding for better pm support
> https://www.spinics.net/lists/arm-kernel/msg781687.html
> The review of that patch series is pending for a couple of months.

yes that is what I currently use. So further imx8qm development can
happen if this is integrated?

Best regards,

Oliver

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: clk: imx: clock driver for imx8qm?
  2020-02-13 17:42   ` Oliver Graute
@ 2020-02-14  5:50     ` Aisheng Dong
  2020-02-17  7:04       ` Shawn Guo
  0 siblings, 1 reply; 6+ messages in thread
From: Aisheng Dong @ 2020-02-14  5:50 UTC (permalink / raw)
  To: Oliver Graute, Shawn Guo
  Cc: festevam, Anson Huang, Stephen Boyd, Leonard Crestez, Peng Fan,
	linux-clk, linux-kernel

> From: Oliver Graute <oliver.graute@gmail.com>
> Sent: Friday, February 14, 2020 1:42 AM
> 
> On 13/02/20, Aisheng Dong wrote:
> > Hi Oliver,
> >
> > >
> > > is someone working on clock driver for imx8qm? I miss at least a
> > > clk-imx8qm.c in the drivers/imx/clk/ directory. I saw that you are
> > > working in this area and perhaps you can give me some insights what is
> needed here.
> > >
> >
> > MX8QM/QXP are using the same clock driver clk-imx8qxp.c
> 
> ok thx, for that clarification.
> 
> >
> > [PATCH RESEND V5 00/11] clk: imx8: add new clock binding for better pm
> > The review of that patch series is pending for a couple of months.
> 
> yes that is what I currently use. So further imx8qm development can happen if
> this is integrated?

Yes, it blocks the most following MX8QXP/QM work.
Let me loop in Shawn to see if any suggestions.

Shawn,
Any ideas?

Regards
Aisheng

> 
> Best regards,
> 
> Oliver

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: clk: imx: clock driver for imx8qm?
  2020-02-14  5:50     ` Aisheng Dong
@ 2020-02-17  7:04       ` Shawn Guo
  2020-02-17  7:31         ` Aisheng Dong
  0 siblings, 1 reply; 6+ messages in thread
From: Shawn Guo @ 2020-02-17  7:04 UTC (permalink / raw)
  To: Aisheng Dong, Stephen Boyd
  Cc: Oliver Graute, festevam, Anson Huang, Leonard Crestez, Peng Fan,
	linux-clk, linux-kernel

On Fri, Feb 14, 2020 at 05:50:48AM +0000, Aisheng Dong wrote:
> > From: Oliver Graute <oliver.graute@gmail.com>
> > Sent: Friday, February 14, 2020 1:42 AM
> > 
> > On 13/02/20, Aisheng Dong wrote:
> > > Hi Oliver,
> > >
> > > >
> > > > is someone working on clock driver for imx8qm? I miss at least a
> > > > clk-imx8qm.c in the drivers/imx/clk/ directory. I saw that you are
> > > > working in this area and perhaps you can give me some insights what is
> > needed here.
> > > >
> > >
> > > MX8QM/QXP are using the same clock driver clk-imx8qxp.c
> > 
> > ok thx, for that clarification.
> > 
> > >
> > > [PATCH RESEND V5 00/11] clk: imx8: add new clock binding for better pm
> > > The review of that patch series is pending for a couple of months.
> > 
> > yes that is what I currently use. So further imx8qm development can happen if
> > this is integrated?
> 
> Yes, it blocks the most following MX8QXP/QM work.
> Let me loop in Shawn to see if any suggestions.
> 
> Shawn,
> Any ideas?

I guess there are still something Stephen is unhappy about, as we
haven't got a nod from him.  Do we really addressed all his comments
and concerns already?

Shawn

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: clk: imx: clock driver for imx8qm?
  2020-02-17  7:04       ` Shawn Guo
@ 2020-02-17  7:31         ` Aisheng Dong
  0 siblings, 0 replies; 6+ messages in thread
From: Aisheng Dong @ 2020-02-17  7:31 UTC (permalink / raw)
  To: Shawn Guo, Stephen Boyd
  Cc: Oliver Graute, festevam, Anson Huang, Leonard Crestez, Peng Fan,
	linux-clk, linux-kernel

> From: Shawn Guo <shawnguo@kernel.org>
> Sent: Monday, February 17, 2020 3:05 PM
> 
> On Fri, Feb 14, 2020 at 05:50:48AM +0000, Aisheng Dong wrote:
> > > From: Oliver Graute <oliver.graute@gmail.com>
> > > Sent: Friday, February 14, 2020 1:42 AM
> > >
> > > On 13/02/20, Aisheng Dong wrote:
> > > > Hi Oliver,
> > > >
> > > > >
> > > > > is someone working on clock driver for imx8qm? I miss at least a
> > > > > clk-imx8qm.c in the drivers/imx/clk/ directory. I saw that you
> > > > > are working in this area and perhaps you can give me some
> > > > > insights what is
> > > needed here.
> > > > >
> > > >
> > > > MX8QM/QXP are using the same clock driver clk-imx8qxp.c
> > >
> > > ok thx, for that clarification.
> > >
> > > >
> > > > [PATCH RESEND V5 00/11] clk: imx8: add new clock binding for
> > > > better pm The review of that patch series is pending for a couple of
> months.
> > >
> > > yes that is what I currently use. So further imx8qm development can
> > > happen if this is integrated?
> >
> > Yes, it blocks the most following MX8QXP/QM work.
> > Let me loop in Shawn to see if any suggestions.
> >
> > Shawn,
> > Any ideas?
> 
> I guess there are still something Stephen is unhappy about, as we haven't got a
> nod from him.  Do we really addressed all his comments and concerns already?
> 

I remember I addressed all Stephen's comments in the last round of review
which is marked in the V5 resend change log.
https://www.spinics.net/lists/arm-kernel/msg769365.html

But I can double check it.
Should I re-send after a checking or wait for Stephen's feedback?

As this does block a lot of following qm/qxp work for a long time, hopefully we
can get some feedback/guidance from Stephen.

BTW, this patchset is already used in NXP internal 5.4 release although it's missed
5.4 LTS kernel.

Regards
Aisheng

> Shawn

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, back to index

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-13 15:31 clk: imx: clock driver for imx8qm? Oliver Graute
2020-02-13 16:18 ` Aisheng Dong
2020-02-13 17:42   ` Oliver Graute
2020-02-14  5:50     ` Aisheng Dong
2020-02-17  7:04       ` Shawn Guo
2020-02-17  7:31         ` Aisheng Dong

Linux-Clk Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-clk/0 linux-clk/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-clk linux-clk/ https://lore.kernel.org/linux-clk \
		linux-clk@vger.kernel.org
	public-inbox-index linux-clk

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-clk


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git