Hi Uwe, On Thu, 5 May 2011 11:17:41 +0200 Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> wrote: > Commit > > 09ba0de (USB: fsl_udc_core: prepare for SoCs with BE registers and descriptors) > > introduced two function pointers _fsl_readl and _fsl_writel in an #ifdef > CONFIG_PPC32 block and used then unconditionally in fsl_udc_probe. > To make the driver compile again this use has to be protected by an > #ifdef, too. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > --- > drivers/usb/gadget/fsl_udc_core.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) Thanks. It turns out that there is another issue when compiling the driver for ARM in linux-next tree. Can you please add following fix to your patch and resubmit. Thanks! diff --git a/drivers/usb/gadget/fsl_udc_core.c b/drivers/usb/gadget/fsl_udc_core.c index b200d3e..47152e0 100644 --- a/drivers/usb/gadget/fsl_udc_core.c +++ b/drivers/usb/gadget/fsl_udc_core.c @@ -1333,8 +1333,10 @@ static void ch9getstatus(struct fsl_udc *udc, u8 request_type, u16 value, /* Fill in the reqest structure */ *((u16 *) req->req.buf) = cpu_to_le16(tmp); +#ifdef CONFIG_PPC32 /* flush cache for the req buffer */ flush_dcache_range((u32)req->req.buf, (u32)req->req.buf + 8); +#endif req->ep = ep; req->req.length = 2; -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html