Re: [PATCH v3] platform/chrome: cros_ec_sensorhub: Modify clock drift estimation
From: Satyam Gupta
Date: Wed Oct 14 2026 - 03:48:30 EST
Hi Tzung-Bi,
On Wed, Oct 14, 2026 at 06:48:58AM +0000, Tzung-Bi Shih wrote:
> I'm not convinced. I think they should still separate into 5 patches and
> form a series.
>
> > > On a related note, is [1] still relevant? Should we disregard it?
> > >
> > > [1] https://lore.kernel.org/all/20260720110405.2127011-1-sgsatyam@xxxxxxxxxx
> >
> > Yes, [1] ("platform/chrome: cros_ec_sensorhub: Fix AP/EC TS wrap") is still
> > relevant — it addresses 32-bit EC timestamp rollover across deep AP suspend.
>
> Please resend or include it in the same series.
Sounds good — I have separated the changes into 5 patches, included
"platform/chrome: cros_ec_sensorhub: Fix AP/EC TS wrap" as patch 1/6, and
squashed the MAX_DRIFT_PPM = 5000LL update into patch 3/6 in the v5 series.
Best regards,
Satyam Gupta