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