Messages in this thread | | | Date | Sat, 14 Oct 2023 16:44:28 +0200 | From | Simon Horman <> | Subject | Re: [PATCH net-next 2/5] net: stmmac: fix PPS capture input index |
| |
On Thu, Oct 12, 2023 at 11:02:13AM +0200, Johannes Zink wrote: > The stmmac supports up to 4 auxiliary snapshots that can be enabled by > setting the appropriate bits in the PTP_ACR bitfield. > > Previously instead of setting the bits, a fixed value was written to > this bitfield instead of passing the appropriate bitmask. > > Now the correct bit is set according to the ptp_clock_request.extts_index > passed as a parameter to stmmac_enable(). > > Fixes: f4da56529da6 ("net: stmmac: Add support for external trigger timestamping") > Signed-off-by: Johannes Zink <j.zink@pengutronix.de>
Hi Johannes,
The fix language of the subject and presence of a fixes tag implies that this is a bug fix. But it's not clear to me that this is resolving bug that manifests as a problem.
If it is a bug fix then it should probably be targeted at 'net', creating a dependency for the remainder of this series.
On the other hand, if it is not a bug fix then perhaps it is best to update the subject and drop the Fixes tag.
I'm no expert on stmmac, but the rest of the series looks good to me.
...
| |