From: Sergey Organov <sorga...@gmail.com> Sent: Wednesday, July 8, 2020 4:49 PM > Andy Duan <fugang.d...@nxp.com> writes: > > > From: Sergey Organov <sorga...@gmail.com> Sent: Tuesday, July 7, 2020 > > 10:43 PM > >> Andy Duan <fugang.d...@nxp.com> writes: > >> > >> > From: Sergey Organov <sorga...@gmail.com> Sent: Monday, July 6, > >> > 2020 > >> 10:26 PM > >> >> Code of the form "if(x) x = 0" replaced with "x = 0". > >> >> > >> >> Code of the form "if(x == a) x = a" removed. > >> >> > >> >> Signed-off-by: Sergey Organov <sorga...@gmail.com> > >> >> --- > >> >> drivers/net/ethernet/freescale/fec_ptp.c | 4 +--- > >> >> 1 file changed, 1 insertion(+), 3 deletions(-) > >> >> > >> >> diff --git a/drivers/net/ethernet/freescale/fec_ptp.c > >> >> b/drivers/net/ethernet/freescale/fec_ptp.c > >> >> index e455343..4152cae 100644 > >> >> --- a/drivers/net/ethernet/freescale/fec_ptp.c > >> >> +++ b/drivers/net/ethernet/freescale/fec_ptp.c > >> >> @@ -485,9 +485,7 @@ int fec_ptp_set(struct net_device *ndev, > >> >> struct > >> ifreq > >> >> *ifr) > >> >> > >> >> switch (config.rx_filter) { > >> >> case HWTSTAMP_FILTER_NONE: > >> >> - if (fep->hwts_rx_en) > >> >> - fep->hwts_rx_en = 0; > >> >> - config.rx_filter = HWTSTAMP_FILTER_NONE; The original patch seems fine. Thanks!
For the patch: Acked-by: Fugang Duan <fugang.d...@nxp.com>