Re: [PATCH v2 1/5] usb: xhci: add firmware loader for uPD720201 and uPD720202 w/o ROM

From: Christian Lamparter
Date: Fri Jun 21 2019 - 15:59:28 EST


On Friday, June 21, 2019 10:59:09 AM CEST Vinod Koul wrote:
> + /*
> + * The Firmware's Data Format is describe in
> + * "6.3 Data Format" R19UH0078EJ0500 Rev.5.00 page 124
> + */
> +
> + /* "Each row is 8 bytes". => firmware size must be a multiple of 8. */
> + if (length % 8 != 0)
> + dev_warn(&dev->dev, "firmware size is not a multiple of 8.");

It doesn't look like this holds true for the newer K2026090.mem which
arguably fixes a lot of bugs over K2013080.mem. I think we should remove
this check and message.

Cheers,
Christian