Re: [RFC PATCH v3 5/5] usb: typec: tcpm: Support for Alternate Modes

From: Andy Shevchenko
Date: Sat May 12 2018 - 17:31:20 EST


On Fri, May 11, 2018 at 4:18 PM, Heikki Krogerus
<heikki.krogerus@xxxxxxxxxxxxxxx> wrote:
> This adds more complete handling of VDMs and registration of
> partner alternate modes, and introduces callbacks for
> alternate mode operations.
>
> Only DFP role is supported for now.

> + for (i = 0; i < cnt; i++)
> + p[i] = le32_to_cpu(payload[i]);

I would recommend to consider to use le32_to_cpu_array().

Though, actually we have slightly different API for BE and LE cases.
For LE existing would be renamed to rather le32_to_cpus_array() and
establishing the former one in the similar way how be32_to_cpu_array()
is implemented.


--
With Best Regards,
Andy Shevchenko