[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f3e65225-0560-4940-9342-7ba12807cff8@amperemail.onmicrosoft.com>
Date: Tue, 26 Aug 2025 18:37:52 -0400
From: Adam Young <admiyo@...eremail.onmicrosoft.com>
To: Jeremy Kerr <jk@...econstruct.com.au>, admiyo@...amperecomputing.com,
Matt Johnston <matt@...econstruct.com.au>,
Andrew Lunn <andrew+netdev@...n.ch>, "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 net-next v25 1/1] mctp pcc: Implement MCTP over PCC
Transport
On 8/22/25 04:21, Jeremy Kerr wrote:
>> +static int mctp_pcc_ndo_open(struct net_device *ndev)
>> +{
>> + struct mctp_pcc_ndev *mctp_pcc_ndev =
>> + netdev_priv(ndev);
>> + struct mctp_pcc_mailbox *outbox =
>> + &mctp_pcc_ndev->outbox;
>> + struct mctp_pcc_mailbox *inbox =
>> + &mctp_pcc_ndev->inbox;
> Minor: I don't think these need wrapping?
The outbox and inbox lines are longer than the mctp_pcc_ndev line, and
they depend on it. This ordering and wrapping passes the xmas tree
check and keeps assignment with declaration.
Powered by blists - more mailing lists