Subject: Re: [PATCH v5 01/11] x86/tsc: Add base clock properties in clocksource structure

From: Thomas Gleixner <[email-masked]>
Date: Wed, 20 Mar 2024 12:23:37 +0100

On Tue, Mar 19 2024 at 18:35, [email-masked] wrote:
> From: Lakshmi Sowjanya D <[email-masked]>
>
> Add base clock hardware abstraction in clocksource structure.
>
> Add clocksource ID for x86 ART(Always Running Timer).

This change log tells the WHAT but not the WHY. You have to add context
and explanation WHY this change is required and why it makes sense.

Also I think this should be ordered differently:

  1) Add the clocksource_base struct and provide the infrastructure in
     get_device_system_crosststamp()

  2) Make TSC/ART use it

  3) Add the realtime muck

Thanks,

        tglx