> -----Original Message----- > From: Fabio Estevam [mailto:festevam@xxxxxxxxx] > Sent: Tuesday, July 14, 2015 6:57 PM > To: Manoil Claudiu-B08782 > Cc: devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux- > kernel; Shawn Guo; Wang Huan-B18965 > Subject: Re: [PATCH v2] dts: ls1021a: Add dts nodes for eTSEC0, eTSEC1 and > eTSEC2 > > On Tue, Jul 14, 2015 at 12:46 PM, Claudiu Manoil > <claudiu.manoil@xxxxxxxxxxxxx> wrote: > > This patch adds generic dts nodes for eTSEC0, eTSEC1 and eTSEC2. > > > > Signed-off-by: Alison Wang <alison.wang@xxxxxxxxxxxxx> > > > > Enable support for the second interrupt group register block > > and the corresponding Rx/Tx/Err interrupt sources, for each > > eTSEC node. DT binding documentation updates. > > > > Signed-off-by: Claudiu Manoil <claudiu.manoil@xxxxxxxxxxxxx> > > It seems you missed to put Alison in the From field. > > You should also put his Signed-off-by tag just before yours. Is this a rule invented by you? Last time I checked this was a valid patch format. Also, this time I sent the (updated) patch, not Alison, which should explain the "From" part. Thanks. ��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f