[U-Boot] [PATCH v3 1/2] usb: net: Add support for Microchip LAN75xx and LAN78xx

Joe Hershberger joe.hershberger at ni.com
Thu Aug 10 18:44:32 UTC 2017


On Thu, Aug 10, 2017 at 1:13 PM,  <Yuiko.Oshino at microchip.com> wrote:
> Hi Marek,
>
>>-----Original Message-----
>>From: Marek Vasut [mailto:marex at denx.de]
>>Sent: Wednesday, August 9, 2017 1:12 PM
>>To: Yuiko Oshino - C18177; u-boot at lists.denx.de
>>Cc: Joe Hershberger
>>Subject: Re: [PATCH v3 1/2] usb: net: Add support for Microchip LAN75xx and
>>LAN78xx
>>
>>On 08/09/2017 06:25 PM, yuiko.oshino at microchip.com wrote:
>>> From: Yuiko Oshino <yuiko.oshino at microchip.com>
>>>
>>> Series-Changes: 3
>>
>>FYI, this will end in the commit message when applied, remove it or move it
>>below the --- . Also commit message is missing.
>
> I did my best to follow the patman instructions and I added "commit-notes:" tag, but I guess it wasn't good enough.

You need to not use commit notes here. That's for things that you want
to be a note in the email, but not end up in the commit log.

> Should I always manually edit the patch before sending email in the patman?

No, you shouldn't have to. The reason this failed to be removed is
that you capitalized the 'C' in changes, and it didn't see it.

Also, these tags are parsed and included as notes. You don't want to
put these in a commit notes tag. That's only needed for other random
commit notes.

> Also, when I am ready to update this patch again, should I do a series or just this patch?

Do the series. Also, you should have the dependency patch first, and
the dependent patch second (swap them around).

> How can I update the [PATCH v] number? If just his patch, then will it be [PATCH v4]?

Go ahead and use v4. So use:

Series-version: 4

>>
>>>    - All #ifdef CONFIG_DM_ETH and #endif are removed.
>>>    - The lan7x_eth_recv() is modifed to correctly support the Driver Model
>>>      and returns packet_en.
>>>    - Add mii_resolve_flowctrl_fdx() patch in the series.
>>>
>>> Series-Changes: 2
>>>    - The wait_for_bit functions copy the real one.
>>>    - Uses phylib
>>>    - Unnecessary variables are removed
>>>    - All return values are checked
>>>    - Uses mii_resolve_flowctrl_fdx() from linux/mii.h
>>>
>>> Signed-off-by: Yuiko Oshino <yuiko.oshino at microchip.com>
>>> ---
>>> Add support for Microchip LAN7500, LAN7800 and LAN7850, USB to
>>> 10/100/1000 Ethernet Controllers.

The fact that this is in the notes means that you put it under
"Commit-notes:" ... "END"

>>>
>>>
>>>  drivers/usb/Kconfig       |   2 +
>>>  drivers/usb/eth/Kconfig   |  17 ++
>>>  drivers/usb/eth/Makefile  |   2 +
>>>  drivers/usb/eth/lan75xx.c | 318 +++++++++++++++++++++++++++++
>>> drivers/usb/eth/lan78xx.c | 477
>>++++++++++++++++++++++++++++++++++++++++++++
>>>  drivers/usb/eth/lan7x.c   | 495
>>++++++++++++++++++++++++++++++++++++++++++++++
>>>  drivers/usb/eth/lan7x.h   | 230 +++++++++++++++++++++
>>>  7 files changed, 1541 insertions(+)

-Joe


More information about the U-Boot mailing list