[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8e85648d-fc88-45dc-a07f-b80f9c2a5fd8@amperemail.onmicrosoft.com>
Date: Mon, 8 Jul 2024 18:16:06 -0400
From: Adam Young <admiyo@...eremail.onmicrosoft.com>
To: Jeremy Kerr <jk@...econstruct.com.au>, admiyo@...amperecomputing.com,
Matt Johnston <matt@...econstruct.com.au>,
"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
Sudeep Holla <sudeep.holla@....com>,
Jonathan Cameron <Jonathan.Cameron@...wei.com>,
Huisong Li <lihuisong@...wei.com>
Subject: Re: [PATCH v4 3/3] mctp pcc: Implement MCTP over PCC Transport
On 7/4/24 06:23, Jeremy Kerr wrote:
>> +struct mctp_pcc_hdr {
>> + u32 signature;
>> + u32 flags;
>> + u32 length;
>> + char mctp_signature[4];
>> +};
> I see you've added the __le annotations that I mentioned, but to a
> different patch in the series. Was that intentional?
Yes. Since the header is used by the Mailbox implementation now, in
order to check the ACK flag.
It made sense to move the structure to a common location, but the change
was based on your last review.
Powered by blists - more mailing lists