Re: [PATCH v2 12/13] treewide: convert rdmsrq() from a macro to an inline function

Damien Le Moal <[email protected]> Thu, 20 Aug 2026 15:06:10 +0900

Newsgroups	gmane.linux.kernel.hwmon,gmane.linux.kernel,gmane.linux.kernel.perf.user,gmane.comp.emulators.kvm.devel,gmane.linux.kernel.pci,gmane.linux.power-management.general,gmane.linux.acpi.devel,gmane.linux.ide,gmane.comp.video.dri.devel,gmane.linux.kernel.cryptoapi,gmane.linux.kernel.gpio,gmane.linux.drivers.mtd,gmane.linux.drivers.platform.x86.devel,gmane.comp.emulators.xen.devel,gmane.linux.ports.geode
Organization	Western Digital Research
Message-ID	<[email protected]>

On 8/19/26 19:23, Juergen Gross wrote:
> Today rdmsrq() is a macro using its second parameter as the target for
> storing the read MSR value.
> 
> Convert rdmsrq() to an inline function returning the MSR value.
> 
> The users have been converted using the following semantic patch:
> 
>   // Options: --include-headers
> 
>   virtual patch
>   virtual report
> 
>   @@
>   expression msr, val;
>   @@
>   (
>   - rdmsrq(msr,val)
>   + val = rdmsrq(msr)
>   )
> 
> Signed-off-by: Juergen Gross <[email protected]>

For the ata bits,

Acked-by: Damien Le Moal <[email protected]>

-- 
Damien Le Moal
Western Digital Research