Re: [PATCH v2 0/6] clk: axi-clk-gen: misc updates to the driver

From: Moritz Fischer
Date: Thu Sep 24 2020 - 00:53:18 EST


Hi Stephen,

On Wed, Sep 23, 2020 at 04:58:33PM -0700, Stephen Boyd wrote:
> Quoting Alexandru Ardelean (2020-09-22 23:22:33)
> > On Tue, Sep 22, 2020 at 10:42 PM Stephen Boyd <sboyd@xxxxxxxxxx> wrote:
> > >
> > > Quoting Moritz Fischer (2020-09-14 19:41:38)
> > > > On Mon, Sep 14, 2020 at 11:11:05AM +0300, Alexandru Ardelean wrote:
> > > > > On Mon, Aug 10, 2020 at 4:41 PM Alexandru Ardelean
> > > > > <alexandru.ardelean@xxxxxxxxxx> wrote:
> > > > > >
> > > > > > These patches synchronize the driver with the current state in the
> > > > > > Analog Devices Linux tree:
> > > > > > https://github.com/analogdevicesinc/linux/
> > > > > >
> > > > > > They have been in the tree for about 2-3, so they did receive some
> > > > > > testing.
> > > > >
> > > > > Ping on this series.
> > > > > Do I need to do a re-send?
> > >
> > > I got this patch series twice. Not sure why.
> >
> > My fault here.
> > Some Ctrl + R usage and not being attentive with the arguments.
> > I think I added "*.patch" twice on the send-mail command.
> > I did something similar [by accident] for some DMA patches.
> > Apologies.
> >
> > I can do a re-send for this, if it helps.
>
> Sure. Please resend it.
>
> >
> > >
> > > >
> > > > I've applied the FPGA one, the other ones should go through the clock
> > > > tree I think?
> > >
> > > Doesn't patch 6 rely on the FPGA patch? How can that driver build
> > > without the header file?
> >
> > Yes it does depend on the FPGA patch.
> > We can drop patch 6 for now, pending a merge to Linus' tree and then
> > wait for the trickle-down.
> > I don't mind waiting for these patches.
> > I have plenty of backlog that I want to run through, and cleanup and
> > then upstream.
> > So, there is no hurry.
>
> Can you send me a signed tag with that patch? I can base this patch
> series on top of that. Or I can just apply it to clk tree and if nobody
> changes it in the meantime merge should work out in linux-next and
> linus' tree upstream.

Long story short I messed up my pull-request to Greg and had to back out
the patch anyways. In retrospect I think the patch should have gone
through your tree anyways, so here's our chance to get it right.

Feel free to take it with the rest of the changes through your tree.

Note: When I applied the patch I fixed up the whitespace that checkpatch
complained about so you might want to do that (or ask Alexandru to
resend the patch).

Acked-by: Moritz Fischer <mdf@xxxxxxxxxx>

Sorry for the confusion and let me know if you still prefer a signed
tag.

- Moritz