[U-Boot] [PATCH v3 8/9] usb: f_mass_storage: Fix set_bit and clear_bit usage

Lukasz Majewski lukma at denx.de
Fri Apr 27 16:20:08 UTC 2018


Hi Bryan,

> Compiling the f_mass_storage driver for an x86 target results in a
> compilation error as set_bit and clear_bit are provided by bitops.h
> 
> Looking at the provenance of the current u-boot code and the git
> change history in the kernel, it looks like we have a local copy of
> set_bit and clear_bit as a hold-over from porting the Linux driver
> into u-boot.
> 
> These days __set_bit and __clear_bit are optionally provided by an
> arch and can be used as inputs to generic_bit_set and
> generic_bit_clear.
> 
> This patch switches over to generic_set_bit and generic_clear_bit to
> accommodate.
> 
> Tested on i.MX WaRP7 and Intel Edison

Just I'm a bit puzzled:

This is v3 8/9, followed by 9/9 (v1)?

And after some time you sent v2 8/9 and v2 9/9.

I suppose that the latter ones are correct?

Also it is very handy to have some kind of change log in patches - i.e.:

Changes for v2:
- Switching to generic_set|clear bit functions
- ......


The patch seems OK, but I need to test them on my machines.

> 
> Signed-off-by: Bryan O'Donoghue <pure.logic at nexus-software.ie>
> Cc: Lukasz Majewski <lukma at denx.de>
> Cc: Marek Vasut <marex at denx.de>
> ---
>  drivers/usb/gadget/f_mass_storage.c | 25 +++----------------------
>  1 file changed, 3 insertions(+), 22 deletions(-)
> 
> diff --git a/drivers/usb/gadget/f_mass_storage.c
> b/drivers/usb/gadget/f_mass_storage.c index 1ecb92ac6b..af42136e1d
> 100644 --- a/drivers/usb/gadget/f_mass_storage.c
> +++ b/drivers/usb/gadget/f_mass_storage.c
> @@ -252,6 +252,7 @@
>  #include <usb_mass_storage.h>
>  
>  #include <asm/unaligned.h>
> +#include <asm/bitops.h>
>  #include <linux/usb/gadget.h>
>  #include <linux/usb/gadget.h>
>  #include <linux/usb/composite.h>
> @@ -283,26 +284,6 @@ static const char fsg_string_interface[] = "Mass
> Storage"; struct kref {int x; };
>  struct completion {int x; };
>  
> -inline void set_bit(int nr, volatile void *addr)
> -{
> -	int	mask;
> -	unsigned int *a = (unsigned int *) addr;
> -
> -	a += nr >> 5;
> -	mask = 1 << (nr & 0x1f);
> -	*a |= mask;
> -}
> -
> -inline void clear_bit(int nr, volatile void *addr)
> -{
> -	int	mask;
> -	unsigned int *a = (unsigned int *) addr;
> -
> -	a += nr >> 5;
> -	mask = 1 << (nr & 0x1f);
> -	*a &= ~mask;
> -}
> -
>  struct fsg_dev;
>  struct fsg_common;
>  
> @@ -2086,7 +2067,7 @@ static int received_cbw(struct fsg_dev *fsg,
> struct fsg_buffhd *bh)
>  		 * we can simply accept and discard any data received
>  		 * until the next reset. */
>  		wedge_bulk_in_endpoint(fsg);
> -		set_bit(IGNORE_BULK_OUT, &fsg->atomic_bitflags);
> +		generic_set_bit(IGNORE_BULK_OUT,
> &fsg->atomic_bitflags); return -EINVAL;
>  	}
>  
> @@ -2250,7 +2231,7 @@ reset:
>  	fsg->bulk_out_enabled = 1;
>  	common->bulk_out_maxpacket =
>  				le16_to_cpu(get_unaligned(&d->wMaxPacketSize));
> -	clear_bit(IGNORE_BULK_OUT, &fsg->atomic_bitflags);
> +	generic_clear_bit(IGNORE_BULK_OUT, &fsg->atomic_bitflags);
>  
>  	/* Allocate the requests */
>  	for (i = 0; i < FSG_NUM_BUFFERS; ++i) {




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180427/9f121f33/attachment.sig>


More information about the U-Boot mailing list