Skip to contentSkip to conversation

Patch "net: bcmgenet: soft reset 40nm EPHYs before MAC init" has been added to the 5.3-stable tree

DISCUSSION Greg Kroah-Hartman 2 participants 3 messages Nov 6, 2019 – Nov 7, 2019 3 messages not in the archive Stable branch

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

A message that is not in the archive (sent off-list or not yet received): CA+PBWz0ukwrZw6hmjeJ4xDgPZ0=YaCaVYTAXdUcXqfnpsTrjjw@mail.gmail.com

Greg Kroah-Hartman

author maintainer
#2 lore raw
Quoted 2 lines · Doug Berger

Please do not apply this patch to the stable trees. It has been superceded by a recent patch set and reverted upstream.

So drop this and keep the one after this? That would be 25382b991d25 ("net: bcmgenet: reset 40nm EPHY on energy detect")

thanks,

greg k-h

A message that is not in the archive (sent off-list or not yet received): CA+PBWz092XryavGihP3uTBoiDRpGSAJh=apEiTHYSKBCjZAhzA@mail.gmail.com

#4 lore raw

For clarity, this commit should be handled the same way for all stable trees (not just 5.3), and I am replying again since my phone used HTML which was blocked by the mailing lists :(.

My preference is to omit this commit rather than reverting it in the near future, but if you would prefer to add this one and then revert it because it is easier for you I will let you make that call.

Thanks,
Doug

Quoted 31 lines · Doug Berger

Yes, that was my intention.

I was trying to reduce some code churn and associated merge conflicts in the backports.

It is not a functional problem to backport this and then backport the recent reversion, but I would think it would simplify things to just not backport this one in the first place. Perhaps it's too late to simplify things:).

Sorry for the confusion,
    Doug

On Wed, Nov 6, 2019, 6:56 AM Greg Kroah-Hartman <[email protected] <mailto:[email protected]>> wrote:

On Wed, Nov 06, 2019 at 06:43:37AM -0800, Doug Berger wrote:
> Please do not apply this patch to the stable trees. It has been superceded
> by a recent patch set and reverted upstream.

So drop this and keep the one after this?  That would be 25382b991d25 ("net: bcmgenet: reset 40nm EPHY on energy detect")

thanks,

greg k-h

author maintainer
#5 lore raw
Quoted 7 lines · Doug Berger

For clarity, this commit should be handled the same way for all stable trees (not just 5.3), and I am replying again since my phone used HTML which was blocked by the mailing lists :(.

My preference is to omit this commit rather than reverting it in the near future, but if you would prefer to add this one and then revert it because it is easier for you I will let you make that call.

Not a problem, I've dropped it now from the 5.3.y queue, thanks!

greg k-h