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

[Alexandru Ardelean]

On Thu, Sep 24, 2020 at 7:53 AM Moritz Fischer <mdf@xxxxxxxxxx> wrote:
>
> 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).
>

I'll fixup the checkpatch stuff, re-send as a V3, and add your Acked-by.
Thanks & apologies for the mess-up on my part.

> Acked-by: Moritz Fischer <mdf@xxxxxxxxxx>
>
> Sorry for the confusion and let me know if you still prefer a signed
> tag.
>
> - Moritz