Subject: Re: [PATCH 01/10] cpm2: Infrastructure code cleanup.

From: Arnd Bergmann <[email-masked]>
Date: Sat, 8 Sep 2007 16:48:53 +0200

On Wednesday 05 September 2007, Scott Wood wrote:
>=20
> =A0void *cpm_dpram_addr(unsigned long offset)
> =A0{
> -=A0=A0=A0=A0=A0=A0=A0return (void *)(im_dprambase + offset);
> +=A0=A0=A0=A0=A0=A0=A0return (void __force *)(im_dprambase + offset);
> =A0}
> =A0EXPORT_SYMBOL(cpm_dpram_addr);

This does not look like a legitimate use of __force. I think it's better
to leave the warning in place for cases like this, so that someone else
can clean up the remaining users of this.

In this particular case, it seems that the function should either
be removed or changed to return an __iomem pointer so you get the
warning in the mac-fcc driver using it.

The rest of your patch looks great.

	Arnd <><