Re: [PATCH v9 5/5] i2c: aspeed: added slave support for Aspeed I2C driver

2017-06-02 Thread Brendan Higgins
On Fri, Jun 2, 2017 at 5:04 AM, Benjamin Herrenschmidt wrote: > On Fri, 2017-06-02 at 01:46 -0700, Brendan Higgins wrote: >> Added slave support for Aspeed I2C controller. Supports fourteen busses >> present in AST24XX and AST25XX BMC SoCs by Aspeed. > > (Not an issue

Re: [PATCH v9 5/5] i2c: aspeed: added slave support for Aspeed I2C driver

2017-06-02 Thread Brendan Higgins
On Fri, Jun 2, 2017 at 5:04 AM, Benjamin Herrenschmidt wrote: > On Fri, 2017-06-02 at 01:46 -0700, Brendan Higgins wrote: >> Added slave support for Aspeed I2C controller. Supports fourteen busses >> present in AST24XX and AST25XX BMC SoCs by Aspeed. > > (Not an issue for merging) > > Have you

Re: [PATCH v9 5/5] i2c: aspeed: added slave support for Aspeed I2C driver

2017-06-02 Thread Benjamin Herrenschmidt
On Fri, 2017-06-02 at 01:46 -0700, Brendan Higgins wrote: > Added slave support for Aspeed I2C controller. Supports fourteen busses > present in AST24XX and AST25XX BMC SoCs by Aspeed. (Not an issue for merging) Have you looked at a "mode" by which you implement just enough of the slave support

Re: [PATCH v9 5/5] i2c: aspeed: added slave support for Aspeed I2C driver

2017-06-02 Thread Benjamin Herrenschmidt
On Fri, 2017-06-02 at 01:46 -0700, Brendan Higgins wrote: > Added slave support for Aspeed I2C controller. Supports fourteen busses > present in AST24XX and AST25XX BMC SoCs by Aspeed. (Not an issue for merging) Have you looked at a "mode" by which you implement just enough of the slave support

[PATCH v9 5/5] i2c: aspeed: added slave support for Aspeed I2C driver

2017-06-02 Thread Brendan Higgins
Added slave support for Aspeed I2C controller. Supports fourteen busses present in AST24XX and AST25XX BMC SoCs by Aspeed. Signed-off-by: Brendan Higgins --- Added in v6: - Pulled slave support out of initial driver commit into its own commit. - No longer

[PATCH v9 5/5] i2c: aspeed: added slave support for Aspeed I2C driver

2017-06-02 Thread Brendan Higgins
Added slave support for Aspeed I2C controller. Supports fourteen busses present in AST24XX and AST25XX BMC SoCs by Aspeed. Signed-off-by: Brendan Higgins --- Added in v6: - Pulled slave support out of initial driver commit into its own commit. - No longer arbitrarily restrict bus to be slave