Subject: Re: [PATCH v5 0/6] drivers/i2c: Add FSI-attached I2C master algorithm
To: Christopher Bostic <cbostic@linux.vnet.ibm.com>,
        Joel Stanley <joel@jms.id.au>
Cc: Eddie James <eajames@linux.vnet.ibm.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-i2c@vger.kernel.org, devicetree <devicetree@vger.kernel.org>,
        Wolfram Sang <wsa@the-dreams.de>, Rob Herring <robh+dt@kernel.org>,
        Jeremy Kerr <jk@ozlabs.org>, Andrew Jeffery <andrew@aj.id.au>,
        "Edward A. James" <eajames@us.ibm.com>
References: <1501089198-10356-1-git-send-email-eajames@linux.vnet.ibm.com>
 <f324da3b-7c9e-3dc1-0cdc-91576c7a50b8@axentia.se>
 <CACPK8XexFdz63t4y5uO1GzoWrx=wnkqBxbYKGtuFq0JupeLNHQ@mail.gmail.com>
 <59175354-07ba-2bbb-962e-ac1de36dfa4a@linux.vnet.ibm.com>
From: Peter Rosin <peda@axentia.se>
Organization: Axentia Technologies AB
Message-ID: <538605fe-f109-b84c-cc21-f1b9824ad5d0@axentia.se>
Date: Tue, 15 Aug 2017 19:35:47 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <59175354-07ba-2bbb-962e-ac1de36dfa4a@linux.vnet.ibm.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Aug 2017 17:35:52.6677 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0202MB2556
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2575
Lines: 49

On 2017-08-15 18:28, Christopher Bostic wrote:
> On 8/15/17 3:10 AM, Joel Stanley wrote:
>> On Tue, Aug 15, 2017 at 4:06 PM, Peter Rosin <peda@axentia.se> wrote:
>>> On 2017-07-26 19:13, Eddie James wrote:
>>>> From: "Edward A. James" <eajames@us.ibm.com>
>>>>
>>>> This series adds an algorithm for an I2C master physically located on an FSI
>>>> slave device. The I2C master has multiple ports, each of which may be connected
>>>> to an I2C slave. Access to the I2C master registers is achieved over FSI bus.
>>>>
>>>> Due to the multi-port nature of the I2C master, the driver instantiates a new
>>>> I2C adapter for each port connected to a slave. The connected ports should be
>>>> defined in the device tree under the I2C master device.
>>> Hmmm, AFAIU fsi is a bus, and on this bus you have some "client" device that
>>> happens to be an i2c master, and this is a driver for that "client". Is it
>>> totally inconceivable to have some other client device in the future that is
>>> implementing an i2c master differently, but still using the fsi bus?
>>>
>>> With that in mind, is it wise to pick the driver name from the bus that the
>>> device is connected to, and nothing else without further qualification?
>>>
>>> I don't see any "i2c-usb" driver, but I think there are a couple of i2c master
>>> drivers that communicate via usb.
>> You make a fair point. When I did a prototype of this driver I called
>> it i2c-cfam, as it is part of the CFAM hardware unit inside of the
>> Power8/Power9 processors.
>>
>> The documentation does call it FSI_I2CM, so that's an argument for the
>> current name.
>>
>> I'm not sure how accurate that name is. Chris, Eddie, do you have any
>> other suggestions?
> The I2C engine up to now has been always accessed via the FSI bus so 
> historically I assume that's why its labelled as FSI_I2CM in the p8/p9 
> specs.  There isn't any reason this I2C device couldn't be implemented 
> in some other topology independent of FSI / CFAMs.  In other words there 
> are no FSI details internal to this I2C engine, an argument for removing 
> the 'FSI' tag.

Note that I wasn't primarily concerned with this i2c engine growing some other
non-fsi interface (like many devices have both i2c and spi interface versions).
I was more concerned with some future and totally different i2c engine that
naturally sports a totally different register map but still uses the fsi bus.

But you have a point. If this i2c engine evolves and ends up supporting some
other interface, then that too would be cause to regret the i2c-fsi name.

Cheers,
peda