Re: [PATCH net-next 1/2] net_tstamp: add new flag HWTSTAMP_FLAGS_UNSTABLE_PHC

On Thu, Dec 09, 2021 at 12:31:30PM +0800, Hangbin Liu wrote:

On Wed, Dec 08, 2021 at 07:20:22AM -0800, Richard Cochran wrote:

quoted

I guess that the original intent of hwtstamp_config.flags was for user
space to SET flags that it wanted. 
Now this has become a place for drivers to return values back.

I think it's a flag that when uses want phc index of bond.
There is no affect for other drivers. It only affect bond interfaces.
When this flag set, it means users want to get the info from bond.

Do I missed something?

No, I simply mean that the input/output direction of the bit in the
flags should be clear.

- User space will not set this bit, only read it.
- Drivers may set this bit, but if user sets it, it is an error.

Thanks,
Richard