>-----Original Message-----
>From: Nguyen, Anthony L <anthony.l.ngu...@intel.com>
>Sent: Tuesday, November 19, 2024 12:03 AM
>To: Michal Swiatkowski <michal.swiatkow...@linux.intel.com>;
>intel-wired-...@lists.osuosl.org; Kwapulinski, Piotr
><piotr.kwapulin...@intel.com>
>Cc: Kitszel, Przemyslaw <przemyslaw.kits...@intel.com>
>Subject: Re: [iwl-next v1] ixgbe: use little endian type in admin commands
>
>
>
>On 11/14/2024 12:12 AM, Michal Swiatkowski wrote:
>> Firmware is always using little endian. Mark it using __le prefixed
>> type in all admin commands structures.
>>
>> Do a proper conversion whenever the structures are used.
>>
>> Reviewed-by: Przemek Kitszel <przemyslaw.kits...@intel.com>
>> Signed-off-by: Michal Swiatkowski <michal.swiatkow...@linux.intel.com>
>> ---
>> To be squashed with commit ea007e0cd2dc
>> ("ixgbe: Add support for E610 FW Admin Command Interface")
>
>These changes look to involve at least two commits from the original series
>[1] i.e. does not squash with the mentioned patch.
>
>It's probably going to work best if these changes get rolled into the
>respective patches of the series as a new version.
Will apply,
Thanks,
Piotr
