From: Nisar Sayed <redacted>
This series of patches are for lan78xx driver.
These patches fixes potential issues associated with lan78xx driver
Nisar Sayed (3):
Fix for eeprom read/write when device autosuspend
Allow EEPROM write for less than MAX_EEPROM_SIZE
Use default value loaded from EEPROM/OTP when resetting
drivers/net/usb/lan78xx.c | 27 ++++++++++++++++++++++-----
1 file changed, 22 insertions(+), 5 deletions(-)
--
1.9.1
On Thu, Sep 07, 2017 at 07:10:51AM +0000, Nisar.Sayed@microchip.com
wrote:
quoted
From: Nisar Sayed <redacted>
This series of patches are for lan78xx driver.
These patches fixes potential issues associated with lan78xx driver
Hi Nisar
So this is version 2? Please include v2 in the subject line.
Also, briefly list what is different from the previous version.
Hi Andrew,
Because Nisar dropped of non-mdio patch in the series,
I suggested to treat as new patch than version 2.
In this case, it is still considered as version 2 than new series?
- Woojung
From: Andrew Lunn <andrew@lunn.ch> Date: 2017-09-07 14:20:02
On Thu, Sep 07, 2017 at 02:03:38PM +0000, Woojung.Huh@microchip.com wrote:
quoted
On Thu, Sep 07, 2017 at 07:10:51AM +0000, Nisar.Sayed@microchip.com
wrote:
quoted
From: Nisar Sayed <redacted>
This series of patches are for lan78xx driver.
These patches fixes potential issues associated with lan78xx driver
Hi Nisar
So this is version 2? Please include v2 in the subject line.
Also, briefly list what is different from the previous version.
Hi Andrew,
Because Nisar dropped of non-mdio patch in the series,
I suggested to treat as new patch than version 2.
In this case, it is still considered as version 2 than new series?
Hi Woojung
So it is not a clear hard rule here. But we have seen these patches
before, and they have been modified based on my comments. So i would
say these are version 2. But a new series would also be O.K. What
really matters is that the cover note explains what is going on. That
some of the patches in the previous version have been dropped and will
be posted later, and what changes have been made to the remaining
patches.
Andrew
This series of patches are for lan78xx driver.
These patches fixes potential issues associated with lan78xx driver
Hi Nisar
So this is version 2? Please include v2 in the subject line.
Also, briefly list what is different from the previous version.
Hi Andrew,
Because Nisar dropped of non-mdio patch in the series,
I suggested to treat as new patch than version 2.
In this case, it is still considered as version 2 than new series?
Hi Woojung
So it is not a clear hard rule here. But we have seen these patches
before, and they have been modified based on my comments. So i would
say these are version 2. But a new series would also be O.K. What
really matters is that the cover note explains what is going on. That
some of the patches in the previous version have been dropped and will
be posted later, and what changes have been made to the remaining
patches.
Andrew,
Thanks for detail explanation and agree that cover page has more description.
Thanks.
Woojung