>-----Original Message----- >From: Nguyen, Anthony L <[email protected]> >Sent: Tuesday, January 27, 2026 7:51 PM >To: Kwapulinski, Piotr <[email protected]>; Keller, Jacob E ><[email protected]>; Simon Horman <[email protected]> >Cc: [email protected]; [email protected]; >[email protected]; [email protected]; Loktionov, Aleksandr ><[email protected]> >Subject: Re: [PATCH iwl-next v2 1/2] ixgbe: e610: Convert ACI descriptor >buffer to little endian > > > >On 1/27/2026 3:31 AM, Kwapulinski, Piotr wrote: > >... > >>>>> @@ -145,7 +146,7 @@ static int ixgbe_aci_send_cmd_execute(struct >>>> ixgbe_hw *hw, >>>>> if ((hicr & IXGBE_PF_HICR_SV)) { >>>>> for (i = 0; i < IXGBE_ACI_DESC_SIZE_IN_DWORDS; i++) { >>>>> raw_desc[i] = IXGBE_READ_REG(hw, >>>> IXGBE_PF_HIDA(i)); >>>>> - raw_desc[i] = raw_desc[i]; >>>> >>>> I'm also curious to know what the intent (if any) of the line above was/is. > >... >> will drop this patch, > >Could we remove the raw_desc redundant assignment though? I think there's one >other instance nearby. You can send it separately and I'll make it part of the >same series as the other patch when it goes to netdev. That's the plan, will do. Piotr
>Thanks, >Tony
