Subject: Re: [PATCH v4 net-next 2/2] net: phy: smsc: add LAN8742 phy support.

From: Andrew Lunn <[email-masked]>
Date: Sat, 7 May 2022 15:48:42 +0200

On Fri, May 06, 2022 at 03:45:13PM -0700, Jakub Kicinski wrote:
> On Thu, 5 May 2022 21:29:13 +0200 Andrew Lunn wrote:
> > On Thu, May 05, 2022 at 11:12:52AM -0700, Yuiko Oshino wrote:
> > > The current phy IDs on the available hardware.
> > >         LAN8742 0x0007C130, 0x0007C131
> > > 
> > > Signed-off-by: Yuiko Oshino <[email-masked]>  
> > 
> > Reviewed-by: Andrew Lunn <[email-masked]>
> 
> The comments which I think were requested in the review of v2 and
> appeared in v3 are now gone, again. Is that okay?

Ah, i had not noticed. Thanks for pointing it out.

Those comments are important, since these mask are odd, somebody is
either going to ask about them, or try to 'fix' them. Some robot will
fall over them, etc.

     Andrew