[alsa-devel] [PATCH] ALSA: hda/tegra: iomem fixups for sparse warnings

Takashi Iwai tiwai at suse.de
Wed Jun 22 08:02:30 CEST 2016


On Tue, 21 Jun 2016 20:18:32 +0200,
Ben Dooks wrote:
> 
> The readl/writel are not being passed __iomem annotated
> variables, so fix the following sparse warnings by adding
> __iomem in:
> 
> sound/pci/hda/hda_tegra.c:120:9: warning: incorrect type in argument 2 (different address spaces)
> sound/pci/hda/hda_tegra.c:120:9:    expected void volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:120:9:    got unsigned int [usertype] *addr
> sound/pci/hda/hda_tegra.c:125:16: warning: incorrect type in argument 1 (different address spaces)
> sound/pci/hda/hda_tegra.c:125:16:    expected void const volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:125:16:    got unsigned int [usertype] *addr
> sound/pci/hda/hda_tegra.c:134:13: warning: incorrect type in argument 1 (different address spaces)
> sound/pci/hda/hda_tegra.c:134:13:    expected void const volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:134:13:    got void *dword_addr
> sound/pci/hda/hda_tegra.c:137:9: warning: incorrect type in argument 2 (different address spaces)
> sound/pci/hda/hda_tegra.c:137:9:    expected void volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:137:9:    got void *dword_addr
> sound/pci/hda/hda_tegra.c:146:13: warning: incorrect type in argument 1 (different address spaces)
> sound/pci/hda/hda_tegra.c:146:13:    expected void const volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:146:13:    got void *dword_addr
> sound/pci/hda/hda_tegra.c:156:13: warning: incorrect type in argument 1 (different address spaces)
> sound/pci/hda/hda_tegra.c:156:13:    expected void const volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:156:13:    got void *dword_addr
> sound/pci/hda/hda_tegra.c:159:9: warning: incorrect type in argument 2 (different address spaces)
> sound/pci/hda/hda_tegra.c:159:9:    expected void volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:159:9:    got void *dword_addr
> sound/pci/hda/hda_tegra.c:168:13: warning: incorrect type in argument 1 (different address spaces)
> sound/pci/hda/hda_tegra.c:168:13:    expected void const volatile [noderef] <asn:2>*addr
> sound/pci/hda/hda_tegra.c:168:13:    got void *dword_addr
> sound/pci/hda/hda_tegra.c:173:23: warning: incorrect type in initializer (incompatible argument 2 (different address spaces))
> sound/pci/hda/hda_tegra.c:173:23:    expected void ( *reg_writel )( ... )
> sound/pci/hda/hda_tegra.c:173:23:    got void ( static [toplevel] *<noident> )( ... )
> sound/pci/hda/hda_tegra.c:174:22: warning: incorrect type in initializer (incompatible argument 1 (different address spaces))
> sound/pci/hda/hda_tegra.c:174:22:    expected unsigned int ( *reg_readl )( ... )
> sound/pci/hda/hda_tegra.c:174:22:    got unsigned int ( static [toplevel] *<noident> )( ... )
> sound/pci/hda/hda_tegra.c:175:23: warning: incorrect type in initializer (incompatible argument 2 (different address spaces))
> sound/pci/hda/hda_tegra.c:175:23:    expected void ( *reg_writew )( ... )
> sound/pci/hda/hda_tegra.c:175:23:    got void ( static [toplevel] *<noident> )( ... )
> sound/pci/hda/hda_tegra.c:176:22: warning: incorrect type in initializer (incompatible argument 1 (different address spaces))
> sound/pci/hda/hda_tegra.c:176:22:    expected unsigned short ( *reg_readw )( ... )
> sound/pci/hda/hda_tegra.c:176:22:    got unsigned short ( static [toplevel] *<noident> )( ... )
> sound/pci/hda/hda_tegra.c:177:23: warning: incorrect type in initializer (incompatible argument 2 (different address spaces))
> sound/pci/hda/hda_tegra.c:177:23:    expected void ( *reg_writeb )( ... )
> sound/pci/hda/hda_tegra.c:177:23:    got void ( static [toplevel] *<noident> )( ... )
> sound/pci/hda/hda_tegra.c:178:22: warning: incorrect type in initializer (incompatible argument 1 (different address spaces))
> sound/pci/hda/hda_tegra.c:178:22:    expected unsigned char ( *reg_readb )( ... )
> sound/pci/hda/hda_tegra.c:178:22:    got unsigned char ( static [toplevel] *<noident> )( ... )
> 
> Signed-off-by: Ben Dooks <ben.dooks at codethink.co.uk>

Applied, thanks.


Takashi


More information about the Alsa-devel mailing list