Re: [PATCH 02/12] ASoC: amd: add Vangogh ACP PCI driver
From: Mark Brown
Date: Wed Jul 07 2021 - 12:18:04 EST
On Wed, Jul 07, 2021 at 11:26:13AM +0530, Vijendar Mukunda wrote:
> +static inline u32 acp_readl(void __iomem *base_addr)
> +{
> + return readl(base_addr - ACP5x_PHY_BASE_ADDRESS);
> +}
I see this was the same for Renoir but it's weird that the read and
write functions are substracting rather than adding the base address
here. A comment might be good.
Attachment:
signature.asc
Description: PGP signature