Re: [EXT] Re: [v11 2/2] clk: ls1028a: Add clock driver for Display output interface

From: Michael Walle
Date: Fri Dec 20 2019 - 09:42:07 EST


Am 2019-12-17 08:08, schrieb Wen He:
-----Original Message-----
From: Stephen Boyd <sboyd@xxxxxxxxxx>
Sent: 2019å12æ17æ 1:56
To: Leo Li <leoyang.li@xxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>;
Michael Turquette <mturquette@xxxxxxxxxxxx>; Michael Walle
<michael@xxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Wen He
<wen.he_1@xxxxxxx>; devicetree@xxxxxxxxxxxxxxx; linux-clk@xxxxxxxxxxxxxxx;
linux-kernel@xxxxxxxxxxxxxxx
Subject: [EXT] Re: [v11 2/2] clk: ls1028a: Add clock driver for Display output
interface

Caution: EXT Email

Quoting Michael Walle (2019-12-12 16:06:16)
> Am 12. Dezember 2019 23:18:16 MEZ schrieb Stephen Boyd
<sboyd@xxxxxxxxxx>:
> >Quoting Wen He (2019-12-04 23:26:53)
> >> Add clock driver for QorIQ LS1028A Display output interfaces(LCD,
> >DPHY),
> >> as implemented in TSMC CLN28HPM PLL, this PLL supports the
> >programmable
> >> integer division and range of the display output pixel clock's
> >27-594MHz.
> >>
> >> Signed-off-by: Wen He <wen.he_1@xxxxxxx>
> >> Signed-off-by: Michael Walle <michael@xxxxxxxx>
> >
> >Is Michael the author? SoB chain is backwards here.
>
> the original driver was from Wen. I've just supplied some code and the
> vco frequency stuff. so its basically a sob of us both.
>
> -michael

Ok. That's a Co-developed-by: tag then. Thanks for letting us know.

The v12 version patch was sent out.
Does need I change the author sign to below?

I guess you mean the MODULE_AUTHOR()? I don't care if you add me or not,
so do as you like.

Co-developed-by: Wen He <wen.he_1@xxxxxxx>
Signed-off-by: Wen He <wen.he_1@xxxxxxx>
Co-developed-by: Michael Walle <michael.@xxxxxxxx>
Signed-off-by: Michael Walle <michael.@xxxxxxxx>
Signed-off-by: Wen He <wen.he_1@xxxxxxx>

Judging by the other Co-developed-by: lines it should be

Co-developed-by: Michael Walle <michael.@xxxxxxxx>
Signed-off-by: Michael Walle <michael.@xxxxxxxx>
Signed-off-by: Wen He <wen.he_1@xxxxxxx>

-michael