[U-Boot] [PATCH] usb: composite: Move bitmap related operations to ./include/linux/bitmap.h
Stefan Agner
stefan at agner.ch
Mon Dec 3 15:31:14 UTC 2018
On 02.12.2018 09:49, Lukasz Majewski wrote:
> The BITMAP related operations can now be moved to ./include/linux/bitmap.h
> file to mimic the Linux kernel directory tree.
>
> This change also allows removing the lin_gadget_compat.h header file
> (which is a legacy code only for composite U-boot layer).
> It was also possible to remove #includes from several USB gadget drivers
> as relevant bit operations have been mored out earlier from this file.
s/mored/moved
Otherwise looks good to me:
Reviewed-by: Stefan Agner <stefan.agner at toradex.com>
--
Stefan
>
> Signed-off-by: Lukasz Majewski <lukma at denx.de>
>
> ---
>
> drivers/usb/dwc3/gadget.c | 1 -
> drivers/usb/dwc3/ti_usb_phy.c | 1 -
> drivers/usb/eth/r8152.c | 1 -
> drivers/usb/gadget/at91_udc.c | 1 -
> drivers/usb/gadget/atmel_usba_udc.c | 1 -
> drivers/usb/gadget/dwc2_udc_otg.c | 1 -
> drivers/usb/gadget/dwc2_udc_otg_phy.c | 1 -
> drivers/usb/gadget/dwc2_udc_otg_priv.h | 1 -
> drivers/usb/gadget/f_mass_storage.c | 2 +-
> drivers/usb/gadget/pxa25x_udc.c | 1 -
> include/linux/bitmap.h | 23 ++++++++++++++++++++++
> include/linux/types.h | 3 +++
> include/linux/usb/composite.h | 2 +-
> include/usb/lin_gadget_compat.h | 35 ----------------------------------
> 14 files changed, 28 insertions(+), 46 deletions(-)
> create mode 100644 include/linux/bitmap.h
> delete mode 100644 include/usb/lin_gadget_compat.h
>
> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> index e340cb268f..085f7b8968 100644
> --- a/drivers/usb/dwc3/gadget.c
> +++ b/drivers/usb/dwc3/gadget.c
> @@ -16,7 +16,6 @@
> #include <common.h>
> #include <malloc.h>
> #include <asm/dma-mapping.h>
> -#include <usb/lin_gadget_compat.h>
> #include <linux/bug.h>
> #include <linux/list.h>
>
> diff --git a/drivers/usb/dwc3/ti_usb_phy.c b/drivers/usb/dwc3/ti_usb_phy.c
> index d168e868e3..e7ea12c163 100644
> --- a/drivers/usb/dwc3/ti_usb_phy.c
> +++ b/drivers/usb/dwc3/ti_usb_phy.c
> @@ -19,7 +19,6 @@
> #include <common.h>
> #include <malloc.h>
> #include <ti-usb-phy-uboot.h>
> -#include <usb/lin_gadget_compat.h>
> #include <linux/ioport.h>
> #include <asm/io.h>
> #include <asm/arch/sys_proto.h>
> diff --git a/drivers/usb/eth/r8152.c b/drivers/usb/eth/r8152.c
> index 941158abdd..e5f73e3d4c 100644
> --- a/drivers/usb/eth/r8152.c
> +++ b/drivers/usb/eth/r8152.c
> @@ -10,7 +10,6 @@
> #include <malloc.h>
> #include <memalign.h>
> #include <usb.h>
> -#include <usb/lin_gadget_compat.h>
> #include <linux/mii.h>
> #include <linux/bitops.h>
> #include "usb_ether.h"
> diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c
> index 39ea87005c..2a6626b443 100644
> --- a/drivers/usb/gadget/at91_udc.c
> +++ b/drivers/usb/gadget/at91_udc.c
> @@ -24,7 +24,6 @@
> #include <linux/usb/gadget.h>
> #include <linux/usb/at91_udc.h>
> #include <malloc.h>
> -#include <usb/lin_gadget_compat.h>
>
> #include "at91_udc.h"
>
> diff --git a/drivers/usb/gadget/atmel_usba_udc.c
> b/drivers/usb/gadget/atmel_usba_udc.c
> index c9d483318f..dffa5117f9 100644
> --- a/drivers/usb/gadget/atmel_usba_udc.c
> +++ b/drivers/usb/gadget/atmel_usba_udc.c
> @@ -16,7 +16,6 @@
> #include <linux/usb/gadget.h>
> #include <linux/usb/atmel_usba_udc.h>
> #include <malloc.h>
> -#include <usb/lin_gadget_compat.h>
>
> #include "atmel_usba_udc.h"
>
> diff --git a/drivers/usb/gadget/dwc2_udc_otg.c
> b/drivers/usb/gadget/dwc2_udc_otg.c
> index e3edd10e29..3c7ad033e3 100644
> --- a/drivers/usb/gadget/dwc2_udc_otg.c
> +++ b/drivers/usb/gadget/dwc2_udc_otg.c
> @@ -33,7 +33,6 @@
>
> #include "dwc2_udc_otg_regs.h"
> #include "dwc2_udc_otg_priv.h"
> -#include <usb/lin_gadget_compat.h>
>
> /***********************************************************/
>
> diff --git a/drivers/usb/gadget/dwc2_udc_otg_phy.c
> b/drivers/usb/gadget/dwc2_udc_otg_phy.c
> index 47aa78ae94..c4338af0d7 100644
> --- a/drivers/usb/gadget/dwc2_udc_otg_phy.c
> +++ b/drivers/usb/gadget/dwc2_udc_otg_phy.c
> @@ -33,7 +33,6 @@
>
> #include "dwc2_udc_otg_regs.h"
> #include "dwc2_udc_otg_priv.h"
> -#include <usb/lin_gadget_compat.h>
>
> #include <usb/dwc2_udc.h>
>
> diff --git a/drivers/usb/gadget/dwc2_udc_otg_priv.h
> b/drivers/usb/gadget/dwc2_udc_otg_priv.h
> index b64e222b6d..aaa90187fb 100644
> --- a/drivers/usb/gadget/dwc2_udc_otg_priv.h
> +++ b/drivers/usb/gadget/dwc2_udc_otg_priv.h
> @@ -12,7 +12,6 @@
> #include <linux/usb/ch9.h>
> #include <linux/usb/gadget.h>
> #include <linux/list.h>
> -#include <usb/lin_gadget_compat.h>
> #include <usb/dwc2_udc.h>
>
> /*-------------------------------------------------------------------------*/
> diff --git a/drivers/usb/gadget/f_mass_storage.c
> b/drivers/usb/gadget/f_mass_storage.c
> index a3101afa0d..45c7b58eed 100644
> --- a/drivers/usb/gadget/f_mass_storage.c
> +++ b/drivers/usb/gadget/f_mass_storage.c
> @@ -256,7 +256,7 @@
> #include <linux/usb/gadget.h>
> #include <linux/usb/gadget.h>
> #include <linux/usb/composite.h>
> -#include <usb/lin_gadget_compat.h>
> +#include <linux/bitmap.h>
> #include <g_dnl.h>
>
> /*------------------------------------------------------------------------*/
> diff --git a/drivers/usb/gadget/pxa25x_udc.c b/drivers/usb/gadget/pxa25x_udc.c
> index 44092df25b..09c0a30b2b 100644
> --- a/drivers/usb/gadget/pxa25x_udc.c
> +++ b/drivers/usb/gadget/pxa25x_udc.c
> @@ -29,7 +29,6 @@
>
> #include <linux/usb/ch9.h>
> #include <linux/usb/gadget.h>
> -#include <usb/lin_gadget_compat.h>
> #include <asm/arch/pxa-regs.h>
>
> #include "pxa25x_udc.h"
> diff --git a/include/linux/bitmap.h b/include/linux/bitmap.h
> new file mode 100644
> index 0000000000..4a54ae0509
> --- /dev/null
> +++ b/include/linux/bitmap.h
> @@ -0,0 +1,23 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +#ifndef __LINUX_BITMAP_H
> +#define __LINUX_BITMAP_H
> +
> +#include <asm/types.h>
> +#include <linux/types.h>
> +#include <linux/bitops.h>
> +
> +#define small_const_nbits(nbits) \
> + (__builtin_constant_p(nbits) && (nbits) <= BITS_PER_LONG)
> +
> +static inline void bitmap_zero(unsigned long *dst, int nbits)
> +{
> + if (small_const_nbits(nbits)) {
> + *dst = 0UL;
> + } else {
> + int len = BITS_TO_LONGS(nbits) * sizeof(unsigned long);
> +
> + memset(dst, 0, len);
> + }
> +}
> +
> +#endif /* __LINUX_BITMAP_H */
> diff --git a/include/linux/types.h b/include/linux/types.h
> index 1f3cd63b8f..cc6f7cb39e 100644
> --- a/include/linux/types.h
> +++ b/include/linux/types.h
> @@ -158,4 +158,7 @@ struct ustat {
> char f_fpack[6];
> };
>
> +#define DECLARE_BITMAP(name, bits) \
> + unsigned long name[BITS_TO_LONGS(bits)]
> +
> #endif /* _LINUX_TYPES_H */
> diff --git a/include/linux/usb/composite.h b/include/linux/usb/composite.h
> index 30c464ce39..a49a66f2f8 100644
> --- a/include/linux/usb/composite.h
> +++ b/include/linux/usb/composite.h
> @@ -24,7 +24,7 @@
> #include <common.h>
> #include <linux/usb/ch9.h>
> #include <linux/usb/gadget.h>
> -#include <usb/lin_gadget_compat.h>
> +#include <linux/bitmap.h>
>
> /*
> * USB function drivers should return USB_GADGET_DELAYED_STATUS if they
> diff --git a/include/usb/lin_gadget_compat.h b/include/usb/lin_gadget_compat.h
> deleted file mode 100644
> index e5dba473b7..0000000000
> --- a/include/usb/lin_gadget_compat.h
> +++ /dev/null
> @@ -1,35 +0,0 @@
> -/* SPDX-License-Identifier: GPL-2.0+ */
> -/*
> - * Copyright (c) 2011 Samsung Electronics
> - * Lukasz Majewski <l.majewski at samsung.com>
> - *
> - * This is a Linux kernel compatibility layer for USB Gadget
> - */
> -
> -#ifndef __LIN_COMPAT_H__
> -#define __LIN_COMPAT_H__
> -
> -#include <linux/bitops.h>
> -#include <linux/compat.h>
> -
> -/* common */
> -#define DECLARE_BITMAP(name, bits) \
> - unsigned long name[BITS_TO_LONGS(bits)]
> -
> -#define small_const_nbits(nbits) \
> - (__builtin_constant_p(nbits) && (nbits) <= BITS_PER_LONG)
> -
> -static inline void bitmap_zero(unsigned long *dst, int nbits)
> -{
> - if (small_const_nbits(nbits))
> - *dst = 0UL;
> - else {
> - int len = BITS_TO_LONGS(nbits) * sizeof(unsigned long);
> - memset(dst, 0, len);
> - }
> -}
> -
> -#define dma_cache_maint(addr, size, mode) cache_flush()
> -void cache_flush(void);
> -
> -#endif /* __LIN_COMPAT_H__ */
More information about the U-Boot
mailing list