2018-12-09, 22:49:07 +0100, Andrew Lunn wrote:
> On Sun, Dec 09, 2018 at 10:33:10PM +0100, Heiner Kallweit wrote:
> > On 09.12.2018 22:11, Andrew Lunn wrote:
> > > On Mon, Dec 10, 2018 at 08:00:45AM +1100, Stephen Rothwell wrote:
> > >> Hi all,
> > >>
> > >> Commits
> > >>
> > >> dc9d38cec71c ("net: phy: mdio-gpio: Add phy_ignore_ta_mask to platform
> > >> data")
> > >> 04fa26bab06d ("net: phy: mdio-gpio: Add platform_data support for
> > >> phy_mask")
> > >>
> > >> are missing a Signed-off-by from their author.
> > >
> > > Hi David.
> > >
> > > Any idea what happened here? The version in my git repo has SOB.
> > >
> > > https://patchwork.ozlabs.org/patch/1009811/ also has my SOB.
> > >
> > Instead of
> > v2
> > --
> > int -> u32 in platform data structure
> >
> > Signed-off-by: Andrew Lunn <and...@lunn.ch>
> >
> > shouldn't it be
> >
> > Signed-off-by: Andrew Lunn <and...@lunn.ch>
> > ---
> > v2
> > - int -> u32 in platform data structure
>
> Hi Heiner
>
> David said he wanted to see the version history. So i deliberately put
> it above the ---.
>
> I'm just wondering if -- was enough to trigger something in David's
> scripts? Or patchwork. The -- has disappeared in the commit which made
> it into net-next.
>
> Andrew
If you fetch the mbox from patchwork at
https://patchwork.ozlabs.org/patch/1009811/mbox/
it contains:
-------- 8< --------
v2
Reviewed-by: Florian Fainelli <f.faine...@gmail.com>
---
int -> u32 in platform data structure
Signed-off-by: Andrew Lunn <and...@lunn.ch>
---
[diffstat]
-------- 8< --------
That's 3 dashes instead of the 2 from your mail. If you "git am" that,
the sign-off and actual history will get chopped off.
Maybe Stephen's script to detect those missing sign-offs could be run
as a commit/apply hook by David? This happens regularly, sometimes
dropping more than just a sign-off.
--
Sabrina
