On Fri, Aug 15, 2008 at 12:53:08PM +0100, Russell King wrote: > Shrug. Everything is not a PC. I don't see anything odd about the above. > > We basically implement a standard set of IO macros (the __raw_*) stuff > which is used to implement the standard set of IO support (inl, insl) > and provide a set of extensions for our platforms (readsl to complement > insl - named precisely the same way because that's what they are) because > the generic stuff doesn't cover all our needs. > > > I assume that "bytelen" should have been "bytecount" or similar. > > bytelen and bytecount mean the same thing to me. Thanks Russel for the clarification. So I think until we have a better solution we'll need the attached patch to musb_io.h Paul, is it ok for you ? -- balbi
>From f3cba688c0447504f3812c570cf37732e1f9493b Mon Sep 17 00:00:00 2001 From: Paul Mundt <lethal@xxxxxxxxxxxx> Date: Fri, 15 Aug 2008 10:51:56 +0300 Subject: [PATCH] musb: superh also provides read/write friends do not redefine them unless we're not building for scuch archs. Signed-off-by: Paul Mundt <lethal@xxxxxxxxxxxx> Signed-off-by: Felipe Balbi <felipe.balbi@xxxxxxxxx> --- drivers/usb/musb/musb_io.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/usb/musb/musb_io.h b/drivers/usb/musb/musb_io.h index 6bbedae..d0f812a 100644 --- a/drivers/usb/musb/musb_io.h +++ b/drivers/usb/musb/musb_io.h @@ -37,7 +37,7 @@ #include <linux/io.h> -#ifndef CONFIG_ARM +#if !defined(CONFIG_ARM) && !defined(CONFIG_SUPERH) static inline void readsl(const void __iomem *addr, void *buf, int len) { insl((unsigned long)addr, buf, len); } static inline void readsw(const void __iomem *addr, void *buf, int len) -- 1.6.0.rc3.6.ga0653