Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Min Li <min.li.xe@renesas.com>
To:     Min Li <min.li.xe@renesas.com>,
        "richardcochran@gmail.com" <richardcochran@gmail.com>
CC:     "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH net 3/4] ptp: ptp_idt82p33: use do_aux_work for delay work
Thread-Topic: [PATCH net 3/4] ptp: ptp_idt82p33: use do_aux_work for delay
 work
Thread-Index: AQHWbNOYY6MaRrcEtUGauhEGkbdWIqmZebYg
Date:   Thu, 15 Oct 2020 19:30:51 +0000
Message-ID: <OSAPR01MB1780713D7A363ADDB13C59DBBA020@OSAPR01MB1780.jpnprd01.prod.outlook.com>
References: <1596815885-11094-1-git-send-email-min.li.xe@renesas.com>
In-Reply-To: <1596815885-11094-1-git-send-email-min.li.xe@renesas.com>
Accept-Language: en-CA, en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 2b46f421-3601-4eb3-805c-08d87140d3e4
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Oct 2020 19:30:51.8709
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 53d82571-da19-47e4-9cb4-625a166a4a2a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: p34Fgb4lu8tomDPwNv1gjkk9VBV1ODlRmfUQzPL6TL17XUgUL7L6ZlYZlvMB8dTc/H1Y3jv5EF0AhZY25axj0Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSBPR01MB2583
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi David/Richard

When you have time, can you take a look at this change? Thanks

Min

-----Original Message-----
From: min.li.xe@renesas.com <min.li.xe@renesas.com>=20
Sent: August 7, 2020 11:58 AM
To: richardcochran@gmail.com
Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Min Li <min.li.xe=
@renesas.com>
Subject: [PATCH net 3/4] ptp: ptp_idt82p33: use do_aux_work for delay work

From: Min Li <min.li.xe@renesas.com>

Instead of declaring its own delay_work, use ptp_clock provided do_aux_work=
 to configure sync_tod.

Signed-off-by: Min Li <min.li.xe@renesas.com>
---
 drivers/ptp/ptp_idt82p33.c | 24 ++++++++++++------------  drivers/ptp/ptp_=
idt82p33.h |  2 --
 2 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/drivers/ptp/ptp_idt82p33.c b/drivers/ptp/ptp_idt82p33.c index =
189bb81..2d62aed 100644
--- a/drivers/ptp/ptp_idt82p33.c
+++ b/drivers/ptp/ptp_idt82p33.c
@@ -531,8 +531,8 @@ static int idt82p33_sync_tod(struct idt82p33_channel *c=
hannel, bool enable)
=20
 	if (enable =3D=3D channel->sync_tod_on) {
 		if (enable && sync_tod_timeout) {
-			mod_delayed_work(system_wq, &channel->sync_tod_work,
-					 sync_tod_timeout * HZ);
+			ptp_schedule_worker(channel->ptp_clock,
+					    sync_tod_timeout * HZ);
 		}
 		return 0;
 	}
@@ -555,24 +555,27 @@ static int idt82p33_sync_tod(struct idt82p33_channel =
*channel, bool enable)
 	channel->sync_tod_on =3D enable;
=20
 	if (enable && sync_tod_timeout) {
-		mod_delayed_work(system_wq, &channel->sync_tod_work,
-				 sync_tod_timeout * HZ);
+		ptp_schedule_worker(channel->ptp_clock,
+				    sync_tod_timeout * HZ);
 	}
=20
 	return 0;
 }
=20
-static void idt82p33_sync_tod_work_handler(struct work_struct *work)
+static long idt82p33_sync_tod_work_handler(struct ptp_clock_info *ptp)
 {
 	struct idt82p33_channel *channel =3D
-		container_of(work, struct idt82p33_channel, sync_tod_work.work);
+			container_of(ptp, struct idt82p33_channel, caps);
 	struct idt82p33 *idt82p33 =3D channel->idt82p33;
+	int ret;
=20
 	mutex_lock(&idt82p33->reg_lock);
=20
-	(void)idt82p33_sync_tod(channel, false);
+	ret =3D idt82p33_sync_tod(channel, false);
=20
 	mutex_unlock(&idt82p33->reg_lock);
+
+	return ret;
 }
=20
 static int idt82p33_pps_enable(struct idt82p33_channel *channel, bool enab=
le) @@ -659,10 +662,8 @@ static void idt82p33_ptp_clock_unregister_all(stru=
ct idt82p33 *idt82p33)
=20
 		channel =3D &idt82p33->channel[i];
=20
-		if (channel->ptp_clock) {
+		if (channel->ptp_clock)
 			ptp_clock_unregister(channel->ptp_clock);
-			cancel_delayed_work_sync(&channel->sync_tod_work);
-		}
 	}
 }
=20
@@ -862,8 +863,6 @@ static int idt82p33_channel_init(struct idt82p33_channe=
l *channel, int index)
 		return -EINVAL;
 	}
=20
-	INIT_DELAYED_WORK(&channel->sync_tod_work,
-			  idt82p33_sync_tod_work_handler);
 	channel->sync_tod_on =3D false;
 	channel->current_freq_ppb =3D 0;
=20
@@ -881,6 +880,7 @@ static void idt82p33_caps_init(struct ptp_clock_info *c=
aps)
 	caps->gettime64 =3D idt82p33_gettime;
 	caps->settime64 =3D idt82p33_settime;
 	caps->enable =3D idt82p33_enable;
+	caps->do_aux_work =3D idt82p33_sync_tod_work_handler;
 }
=20
 static int idt82p33_enable_channel(struct idt82p33 *idt82p33, u32 index) d=
iff --git a/drivers/ptp/ptp_idt82p33.h b/drivers/ptp/ptp_idt82p33.h index 9=
d46966..1dcd2c0 100644
--- a/drivers/ptp/ptp_idt82p33.h
+++ b/drivers/ptp/ptp_idt82p33.h
@@ -119,8 +119,6 @@ struct idt82p33_channel {
 	struct ptp_clock	*ptp_clock;
 	struct idt82p33	*idt82p33;
 	enum pll_mode	pll_mode;
-	/* task to turn off SYNC_TOD bit after pps sync */
-	struct delayed_work	sync_tod_work;
 	bool			sync_tod_on;
 	s32			current_freq_ppb;
 	u8			output_mask;
--
2.7.4