Re: [PATCH 2/6] thunderbolt: property: Unify format_dwdata()/parse_dwdata()

From: Andy Shevchenko

Date: Tue Jun 02 2026 - 20:19:45 EST


On Sun, May 24, 2026 at 03:06:44PM +0200, Thomas Weißschuh wrote:
> Both function do the same. They are about to change. To make that
> easier, implement one in terms of the other.

...

> static inline void format_dwdata(void *dst, const void *src, size_t dwords)
> {
> - cpu_to_be32_array(dst, src, dwords);
> + return parse_dwdata(dst, src, dwords);

Why return?

> }

Can't you simply kill this and use parse_dwdata everywhere?

--
With Best Regards,
Andy Shevchenko