[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YgWNE05eVK+LijL/@kunai>
Date:   Thu, 10 Feb 2022 23:09:23 +0100
From:   Wolfram Sang <wsa@...nel.org>
To:     Jan Dabros <jsd@...ihalf.com>
Cc:     linux-kernel@...r.kernel.org, linux-i2c@...r.kernel.org,
        jarkko.nikula@...ux.intel.com, andriy.shevchenko@...ux.intel.com,
        mika.westerberg@...ux.intel.com, hdegoede@...hat.com,
        rrangel@...omium.org, mw@...ihalf.com, jaz@...ihalf.com,
        upstream@...ihalf.com, thomas.lendacky@....com,
        alexander.deucher@....com, Nimesh.Easow@....com,
        mario.limonciello@....com
Subject: Re: [PATCH v4 2/2] i2c: designware: Add AMD PSP I2C bus support
On Tue, Feb 08, 2022 at 03:12:18PM +0100, Jan Dabros wrote:
> Implement an I2C controller sharing mechanism between the host (kernel)
> and PSP co-processor on some platforms equipped with AMD Cezanne SoC.
> 
> On these platforms we need to implement "software" i2c arbitration.
> Default arbitration owner is PSP and kernel asks for acquire as well
> as inform about release of the i2c bus via mailbox mechanism.
> 
>             +---------+
>  <- ACQUIRE |         |
>   +---------|   CPU   |\
>   |         |         | \      +----------+  SDA
Applied to for-next, thanks! I fixed the following checkpatch warnings:
CHECK: Please don't use multiple blank lines
#232: FILE: drivers/i2c/busses/i2c-designware-amdpsp.c:92:
+
+
WARNING: braces {} are not necessary for single statement blocks
#361: FILE: drivers/i2c/busses/i2c-designware-amdpsp.c:221:
+	if (ret) {
+		goto cleanup;
+	}
Please also use checkpatch next time.
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists
 
