[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b35f756d-08e7-6eed-5992-ad63ee419e1f@os.amperecomputing.com>
Date: Mon, 3 Oct 2022 09:06:46 +0700
From: Quan Nguyen <quan@...amperecomputing.com>
To: Wolfram Sang <wsa@...nel.org>,
Randy Dunlap <rdunlap@...radead.org>,
Corey Minyard <minyard@....org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Brendan Higgins <brendan.higgins@...ux.dev>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Joel Stanley <joel@....id.au>,
Andrew Jeffery <andrew@...id.au>,
openipmi-developer@...ts.sourceforge.net,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-i2c@...r.kernel.org, openbmc@...ts.ozlabs.org,
linux-arm-kernel@...ts.infradead.org,
linux-aspeed@...ts.ozlabs.org,
Open Source Submission <patches@...erecomputing.com>,
Phong Vo <phong@...amperecomputing.com>,
thang@...amperecomputing.com
Subject: Re: [PATCH v9 3/3] i2c: aspeed: Assert NAK when slave is busy
On 02/10/2022 05:55, Wolfram Sang wrote:
>
>> + if (ret == -EBUSY)
>
> Since we documented this:
>
> "+ 'ret': 0 if the backend is ready, otherwise some errno"
>
> the code above should be '(ret < 0)'
>
Will update.
Thanks a lot for the review.
- Quan
Powered by blists - more mailing lists