On Fri,  5 Feb 2021 20:12:02 +0100
Pali Rohár <p...@kernel.org> wrote:

> PERI_RXCSR is 16bit register so store its value into 16bit local
> variable.
> 
> Signed-off-by: Pali Rohár <p...@kernel.org>
> ---
>  drivers/usb/musb/musb_udc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/musb/musb_udc.c b/drivers/usb/musb/musb_udc.c
> index d901f8777c..67d1c56f9a 100644
> --- a/drivers/usb/musb/musb_udc.c
> +++ b/drivers/usb/musb/musb_udc.c
> @@ -629,7 +629,7 @@ static void musb_peri_ep0(void)
>  static void musb_peri_rx_ep(unsigned int ep)
>  {
>       u16 peri_rxcount;
> -     u8 peri_rxcsr = readw(&musbr->ep[ep].epN.rxcsr);
> +     u16 peri_rxcsr = readw(&musbr->ep[ep].epN.rxcsr);
>  
>       if (!(peri_rxcsr & MUSB_RXCSR_RXPKTRDY)) {
>               if (debug_level > 0)

Reviewed-by: Lukasz Majewski <lu...@denx.de>


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-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de

Attachment: pgp7_qrfeztP4.pgp
Description: OpenPGP digital signature

Reply via email to