As this change doesn't affect other users I thought it is safe to do it from PCI wrapper driver. But yes I agree that right way is to do it in PHY driver. I would like to drop this patch, please ignore only this patch from series.
>-----Original Message----- >From: Andrew Lunn <[email protected]> >Sent: Monday, June 17, 2019 8:35 PM >To: Parshuram Raju Thombare <[email protected]> >Cc: [email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >Rafal Ciepiela <[email protected]>; Anil Joy Varughese ><[email protected]>; Piotr Sroka <[email protected]> >Subject: Re: [PATCH 0/6] net: macb patch set cover letter > >EXTERNAL MAIL > > >> 3. 003-net-macb-add-PHY-configuration-in-MACB-PCI-wrapper.patch >> This patch is to configure TI PHY DP83867 in SGMII mode from >> our MAC PCI wrapper driver. >> With this change there is no need of PHY driver and dp83867 >> module must be disabled. Users wanting to setup DP83867 PHY >> in SGMII mode can disable dp83867.ko driver, else dp83867.ko >> overwrite this configuration and PHY is setup as per dp83867.ko. > >This sounds very wrong. Why not make the dp83867 driver support SGMII? > > Andrew Regards, Parshuram Thombare
