On 2012-11-19 11:15, Russell King - ARM Linux wrote:
On Mon, Nov 19, 2012 at 10:51:08AM +0530, Archit Taneja wrote:
quoted
On Friday 16 November 2012 10:14 PM, H Hartley Sweeten wrote:
quoted
On Friday, November 16, 2012 2:16 AM, Archit Taneja wrote:
quoted
This removes the sparse warnings on arm platforms:
warning: cast removes address space of expression
Signed-off-by: Archit Taneja <redacted>
I submitted the same patch around early March 2012. So FWIW:
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Thanks.
Florian,
Could you queue this for 3.8 merge window?
Actually no. Has anyone checked whether this has any impact for the BE
ARM platforms?
Probably not. I can't say anything to that matter, but I wonder if this
patch is just going around the problem that we get sparse warnings when
falling into the else ifdef block in fb.h.
The macros in the else block are defined as:
#define fb_readb(addr) (*(volatile u8 *) (addr))
And fb code passes a pointer to __iomem. So shouldn't the cast be to
(volatile u8 __iomem *)?
Tomi