Thread (20 messages) flat view 20 messages, 3 authors, 2019-12-21

Re: [PATCH V7 net-next 01/11] net: phy: Introduce helper functions for time stamping support.

From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2019-12-20 22:24:57
Also in: linux-devicetree

On 12/20/19 10:15 AM, Richard Cochran wrote:
Some parts of the networking stack and at least one driver test fields
within the 'struct phy_device' in order to query time stamping
capabilities and to invoke time stamping methods.  This patch adds a
functional interface around the time stamping fields.  This will allow
insulating the callers from future changes to the details of the time
stamping implemenation.

Signed-off-by: Richard Cochran <richardcochran@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>

Ideally we may even add locking annotations to make sure that there
there is no use after free in that the test is true, you de-reference
the PHY device and/or driver method, and boom. All callers appear to be
under RTNL right now.
-- 
Florian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help