lkml.org 
[lkml]   [2023]   [May]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH net-next v3 2/6] net: phy: microchip_t1s: replace read-modify-write code with phy_modify_mmd
> This change also invalidates most of the comment. I think this should be
> reduced to something along the lines of:
> /* HW quirk: Microchip states in the application note (AN1699) for the phy
> * that a set of read-modify-write (rmw) operations has to be performed
> * on a set of seemingly magic registers.
> * The result of these operations is just described as 'optimal performance'
> * Microchip gives no explanation as to what these mmd regs do,
> * in fact they are marked as reserved in the datasheet.*/

I agree the comments should be reviewed in light of these changes.

>
> Additionally I don't mind it if you change the tone of the comment. This was brought
> up in the sitdown we had, where it was explained from Microchip that
> documenting what the reg operations actually does would expose to much
> of the internal workings of the chip.

They cannot care too much, or the firmware in the PHY would do this
where it is all hidden away.

Andrew

\
 
 \ /
  Last update: 2023-05-25 17:52    [W:0.352 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site