[U-Boot] [PATCH v2 4/4] lin_gadget: use common linux/compat.h

Anatolij Gustschin agust at denx.de
Thu Apr 26 11:03:19 CEST 2012


Hi Lukasz,

On Wed, 25 Apr 2012 17:58:51 +0200
Lukasz Majewski <l.majewski at samsung.com> wrote:

> Hi Anatolij,
> 
> > From: Mike Frysinger <vapier at gentoo.org>
> > 
> > Merge our duplicate definitions with the common header.
> > Also fix drivers/usb/gadget/s3c_udc_otg_xfer_dma.c to
> > use min() instead of min_t() since we remove the latter
> > from compat.h.
> 
> Yes. the include/usb/lin_gadget_compat.h layer.
> Good idea to provide one compat file (as fair as I remember similar
> problem is with mtd/compat.h).
> 
> I'll look into the include/linux.h file.

Mike already moved mtd/compat.h to linux/compat.h, please see this
patch http://patchwork.ozlabs.org/patch/151500/

Now I see another issue. In this 4/4 patch we are removing kmalloc:

-#define kmalloc(size, type) memalign(CONFIG_SYS_CACHELINE_SIZE, size)

linux/compat.h defines kmalloc() to be malloc(), so the gadget driver
should probably use memalign() directly where it is needed?

Thanks,
Anatolij


More information about the U-Boot mailing list