Re: [PATCH v2 0/3] pwm: add pwm AO on meson gx

From: Thierry Reding
Date: Thu Jul 06 2017 - 05:17:52 EST


On Thu, Jul 06, 2017 at 11:02:20AM +0200, Jerome Brunet wrote:
> On Thu, 2017-07-06 at 08:59 +0200, Thierry Reding wrote:
> > On Thu, Jun 08, 2017 at 02:24:13PM +0200, Jerome Brunet wrote:
> > > The pwms found in the AO domain of the meson gx family have different
> > > clock sources than the ones found in the EE domain. The AO pwms only
> > > have 2 connected clock input which are clk81 and the crytal.
> > >
> > > This patchset adds the necessary compatible and the clock data for it.
> > >
> > > Changes since v1: [0]
> > > * Correct clock source order for gxbb pwm ao. Documenation shows xtal as
> > > Â source #1 while it is actually source #0
> > > * Add patch 3 to fix pwm calculation. Issue while testing pwm with clk81
> > > Â as clock source.
> > >
> > > [0]:ÂÂhttps://lkml.kernel.org/r/20170607133013.4766-1-jbrunet@xxxxxxxxxxxx
> > >
> > > Jerome Brunet (3):
> > > Â dt-bindings: pwm: meson: add comptabible for gxbb ao pwms
> > > Â pwm: meson: add compatible for the gxbb ao pwms
> > > Â pwm: meson: improve pwm calculation precision.
> > >
> > > Â.../devicetree/bindings/pwm/pwm-meson.txtÂÂÂÂÂÂÂÂÂÂ|ÂÂ4 +-
> > > Âdrivers/pwm/pwm-meson.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 47 +++++++++++++++++
> > > -----
> > > Â2 files changed, 41 insertions(+), 10 deletions(-)
> >
> > For some reason the Author field keeps getting set to:
> >
> > jbrunet <jbrunet@xxxxxxxxxxxx>
> >
> > It seems like the headers are correct, so maybe this is patchwork
> > messing things up?
>
> And it strikes again ... :(
> I already had the issue with https://patchwork.kernel.org a few month ago.
> I have contacted the kernel.org helpdesk about it and here is the explanation:
>
> "Patchwork treats users and patch submitters slightly differently, so once your
> entry is created upon first patch submission, it sticks forever. I've adjusted
> your entry manually per your request."
>
> So my very first patch on the ML must have been malformed somehow.
>
> >
> > Anyway, I've applied the series and manually fixed up the Author field.
>
> Thanks a lot for this !
> Any idea who I may contact to get this "manual adjustment" done on https://patch
> work.ozlabs.org/ ?

I /think/ Jeremy Kerr (To'ed) would be a good person to contact about
this.

Jeremy, anything you can do about this?

Thanks,
Thierry

Attachment: signature.asc
Description: PGP signature