Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
From:   Avri Altman <Avri.Altman@wdc.com>
To:     Bean Huo <beanhuo@iokpp.de>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "mani@kernel.org" <mani@kernel.org>,
        "quic_cang@quicinc.com" <quic_cang@quicinc.com>,
        "quic_asutoshd@quicinc.com" <quic_asutoshd@quicinc.com>,
        "beanhuo@micron.com" <beanhuo@micron.com>,
        "thomas@t-8ch.de" <thomas@t-8ch.de>
CC:     "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "mikebi@micron.com" <mikebi@micron.com>,
        "lporzio@micron.com" <lporzio@micron.com>
Subject: RE: [PATCH v3 2/3] scsi: ufs: core: Add UFS RTC support
Thread-Topic: [PATCH v3 2/3] scsi: ufs: core: Add UFS RTC support
Thread-Index: AQHaJTkKGJCMMiwoCEWzpUFDJD5+pLCcrOBA
Date:   Wed, 6 Dec 2023 19:38:26 +0000
Message-ID: <DM6PR04MB6575C233D314AF011D5008C0FC84A@DM6PR04MB6575.namprd04.prod.outlook.com>
References: <20231202160227.766529-1-beanhuo@iokpp.de>
 <20231202160227.766529-3-beanhuo@iokpp.de>
In-Reply-To: <20231202160227.766529-3-beanhuo@iokpp.de>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?qhfSC3RazOl2RZETSzTEtsoOcMKkla/DMy4+IUNz/cz370R2DTJJkXLcL3TD?=
 =?us-ascii?Q?fip8ttJ1kJ8RhFW3vv5QvbnoYaY+s3uRlDKNeReZofMSNb+eivgGlWrBMrHS?=
 =?us-ascii?Q?lWOaCksm9RZ8F1Hbww/kO4UvoCZdw7s+/cWBlPsn259vZ4P6PSJZnVk7FXwS?=
 =?us-ascii?Q?u9tj59NfDvdyTm2ka1VSjkVu1GmkffhWDr0C1mVCTS9k0FiLmy61YkHyWD5Y?=
 =?us-ascii?Q?m4fLuQptFc6fkODUdazaepRwbzqnLGwR+fe6JTlpajpWtlwA9ovr+FRGP7eZ?=
 =?us-ascii?Q?F/WS7Kar+pAuj40xgzXU79akIJ8F5OMcz+TW7mmLBpd7G5SlFOFDzznEJmbn?=
 =?us-ascii?Q?hgGDKS1eAwTCY9kJgaz8FT2UxkEZR49N9HD2EVOz5jVKzplP/28Ops7Zr4RW?=
 =?us-ascii?Q?SgZoJwEKuZkjXQGLG3uQCGktDqFmH9hF6aqGnY8NLXT0X9SnVyn36g8Ig3d2?=
 =?us-ascii?Q?HsDDs5jj5imgzc64hYGII3Kne9pJmxkzquPcqaqw/NZYZ1GCs5WyCZ8dIbvk?=
 =?us-ascii?Q?iY2DVcSyyXv/dFJ/72yltDj/fZ2iAekNdDtoDqUsWFopvDsDbZjJ5WfumDam?=
 =?us-ascii?Q?nyDivWoWnrqGNJtATv3ibVJ5YhSv6f8zXxyTs4sn+7mlUmzjO1zJlM7zciPO?=
 =?us-ascii?Q?gX6ZonOjITG9qUiHw7pAqf9q+88LJzf/I2ewDwaUHxgnXjERUrZyeHyyp7fP?=
 =?us-ascii?Q?2nGAUTYyikeu5EqGnIGkF0k45BsYPPFK6IAGXdPva93SOweMIlpwsyC6NE9H?=
 =?us-ascii?Q?w6zNHQxUtrDDGWDk5j11ZeSDrqH9BaRFb7RZKVeaZMnxLKXTW4hBsfnWq0ta?=
 =?us-ascii?Q?jr5LqgJlfrBCzFoTvJG+fdvG9oVUPrSADL2/77WmG66x3uFlinlesvJ1f4ba?=
 =?us-ascii?Q?LjxAyXdte85ioxLPvLcfy0sVjlsEUBsZkUMwav+sP5IZ0H+zV6gsvGSSpBoZ?=
 =?us-ascii?Q?2xBzAGy/QE+9ltuGl7pZhpiMSDl2MUsDRuA2FdB+edjVRQe2DTA59T7AN2dh?=
 =?us-ascii?Q?BtePp7GSnY2S9HiPqqEvB2aNg8c4+fhpAtz2TFmHMP4zqJCY0/VEK2dV9MxS?=
 =?us-ascii?Q?ZVRRZYj+r2IFeN6INnYcamTVXk07gUSLn1P1hJFev2cTVYZ61usmePsI6lJe?=
 =?us-ascii?Q?6TxJ0A23iX7KXDGCj75Bt5P65pt3f/yjAOodewe1/m14wujmERyEsc++Nl7t?=
 =?us-ascii?Q?e6tOCLYuGI9/c+I/olczr3CahlKVaxfig/JIcyDIvJtD20AyIEOuOu+rt2r4?=
 =?us-ascii?Q?Z34WO2FtG3wAQXnDuDAi58n82c1ZbWmeZxuUJsElW6/hhqhD4ygAP/q+8e6f?=
 =?us-ascii?Q?Lo+lYffuADz46vFUmu8ANT927t7oFJwrWsE5BHixpmX9Ubtw6vmb5AQMAl91?=
 =?us-ascii?Q?LQNGcT2F20W/G95tUThTkI6Npcfu1Jf+WfQu64ZI/Qf9ZcCtRuPpYWYM10hu?=
 =?us-ascii?Q?3KBUNfQrDPZmfyRKjR/INAorx0LEDVl0CTEc1H7o5sWOp24FMBgexrVLBZqv?=
 =?us-ascii?Q?0Vp7I0QkQNNytbTgz1I0bPwoxBic8pKD74HnbzlAyqoqCB6HpOCyxU5u6btL?=
 =?us-ascii?Q?mWfTmT/iSCJtuyCik4V0ZVh1hdhFgCErJLmb6sdP?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?fIXzK0+WP1UDHWADwAxYcQ9rgFPBA+23p5RK90/JCVXPJMT7hSCV7sVNqTP8?=
 =?us-ascii?Q?mq4sH5X+IKf3ONJ90hsM2nh5Okwc5OZlKd3IE7aJoTw2NfF1JoXS8ZjUy6qi?=
 =?us-ascii?Q?j0GJqEPM7EyaNN5LIWE8NrVcTTggwQVS650ZJHGD7zjR1TMMtYieAdajnfRk?=
 =?us-ascii?Q?n7buTk0f+nIhJ2Sf28KLw97lCq0wKpvKm52jG9AcWGqhGZtLWWfEufs7on/v?=
 =?us-ascii?Q?1+kjQmC4AVyamYxPOCiwxdwAZwmyQxatI/TuHt48QnyxFvAMODmzb8kD/Cu5?=
 =?us-ascii?Q?UFBJDzaxhZ+aPrb7bGPvWuU7W03sTyvVj47o3uZlYgHJ5myDATKwyK3o/brQ?=
 =?us-ascii?Q?nIH4SrovoMpFNwpiO90KE7Ji0jqzzElNjpgcRDtoH5q60byJuRU5e5ySvpuZ?=
 =?us-ascii?Q?JKkhCX035Ud/1OOyOYyyDbzXA5rXPQ5FpaPqQHg2l3WYRQddoURo823ajgOA?=
 =?us-ascii?Q?WsA4vQ/Xu5spm9etQrdSr5QRxRjzTPmEMiQdRvnR8+UnLMi544Qyb0izQ1Aa?=
 =?us-ascii?Q?2rIaU4if/cL4VUP4FCI1chRMhO7MpaLH5e/CzLpd8PYmfGj/aZSX8+CijTWa?=
 =?us-ascii?Q?VH66LNZua2vtCphS2d1mKP+qdYEYH8FYfrZnme4kHXtawLXS473daZp6CDiX?=
 =?us-ascii?Q?qkOD0Qja+kh2GsMfGgeWwNF+qwvgHl63xj0r1aJqshWAaOXt2utzhC0ywVgM?=
 =?us-ascii?Q?GO66TVdpQP0lqSxRTSD0GobffpG/F6xiS1yJGf9RWVgiVId+9hFEjMCasiLP?=
 =?us-ascii?Q?vhb17GED7J6syquE1Idt6zjNVZJExQfMJpOOBjJyWuQPxjUFK8vvixmDmwSc?=
 =?us-ascii?Q?1Abl2Yz0esZvnHrpI19tQNr1cX72xEIYy3Pf3lFlRmmfN2FVLwr4XXK+lsBF?=
 =?us-ascii?Q?K2v14xUoBktdo5WJQTKqIxvWM9G8pJQtJPr556KV4eeL39SbYdOqd9dBVma6?=
 =?us-ascii?Q?X0BgxGKDfzjbyE3YMH5oPlUvMeM9FMBPDj3V9/b/qJ2peyq23ZS7yc/DzTFv?=
 =?us-ascii?Q?9L3Ik8dR9/OC8MFYz8zkTY6Kk9cJ6iI7BOTs0xuA2tcyCW0=3D?=
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB6575.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4015a4b5-aaaa-446d-1b20-08dbf692ea6b
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Dec 2023 19:38:26.1537
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: jPZNEt1RJqBiEdD9FBP3guSxxZBPnq3vj6l43033Fr6BAy+Qk4tZSZrRa0QvAbaVFRjtgK/8CBP/Q6SvtQya7w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR04MB7140
X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,
	SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Wed, 06 Dec 2023 11:38:43 -0800 (PST)

> From: Bean Huo <beanhuo@micron.com>
>=20
> Add Real Time Clock (RTC) support for UFS device. This enhancement is
> crucial for the internal maintenance operations of the UFS device. The pa=
tch
> enables the device to handle both absolute and relative time information.
> Furthermore, it includes periodic task to update the RTC in accordance wi=
th
> the UFS Spec, ensuring the accuracy of RTC information for the device's
> internal processes.
>=20
> RTC and qTimestamp serve distinct purposes. The RTC provides a coarse
> level of granularity with, at best, approximate single-second resolution.=
 This
> makes the RTC well-suited for the device to determine the approximate age
> of programmed blocks after being updated by the host. On the other hand,
> qTimestamp offers nanosecond granularity and is specifically designed for
> synchronizing Device Error Log entries with corresponding host-side logs.
>=20
> Given that the RTC has been a standard feature since UFS Spec 2.0, and
> qTimestamp was introduced in UFS Spec 4.0, the majority of UFS devices
> currently on the market rely on RTC. Therefore, it is advisable to contin=
ue
> supporting RTC in the Linux kernel. This ensures compatibility with the
> prevailing UFS device implementations and facilitates seamless integratio=
n
> with existing hardware.
> By maintaining support for RTC, we ensure broad compatibility and avoid
> potential issues arising from deviations in device specifications across
> different UFS versions.
>=20
> Signed-off-by: Bean Huo <beanhuo@micron.com>
> Signed-off-by: Mike Bi <mikebi@micron.com>
> Signed-off-by: Luca Porzio <lporzio@micron.com>
Acked-by: Avri Altman <avri.altman@wdc.com>