mbox series

[v3,0/4] Add Fast Mode Plus and other fixes

Message ID 1525968837-13381-1-git-send-email-austinwc@codeaurora.org
Headers show
Series Add Fast Mode Plus and other fixes | expand

Message

Christ, Austin May 10, 2018, 4:13 p.m. UTC
Add support for Fast Mode Plus (1MHz) to the I2C QUP driver and correct
duty cycle for Fast Mode and Fast Mode Plus.

Add configuration in probe for Centriq devices.

Add a command line module parameter to allow override of SCL frequency.

[v3]
- correct probe patch to branch within QUPv2 segment.

[v2]
- whitespace corrections

Austin Christ (4):
  i2c: qup: add probe path for Centriq ACPI devices
  i2c: qup: Add support for Fast Mode Plus
  i2c: qup: Correct duty cycle for FM and FM+
  i2c: qup: Add command-line parameter to override SCL frequency

 drivers/i2c/busses/i2c-qup.c | 61 ++++++++++++++++++++++++++++++--------------
 1 file changed, 42 insertions(+), 19 deletions(-)

Comments

Timur Tabi May 23, 2018, 4:11 p.m. UTC | #1
On Thu, May 10, 2018 at 11:13 AM, Austin Christ <austinwc@codeaurora.org> wrote:
> Add support for Fast Mode Plus (1MHz) to the I2C QUP driver and correct
> duty cycle for Fast Mode and Fast Mode Plus.
>
> Add configuration in probe for Centriq devices.
>
> Add a command line module parameter to allow override of SCL frequency.
>
> [v3]
> - correct probe patch to branch within QUPv2 segment.
>
> [v2]
> - whitespace corrections
>
> Austin Christ (4):
>   i2c: qup: add probe path for Centriq ACPI devices
>   i2c: qup: Add support for Fast Mode Plus
>   i2c: qup: Correct duty cycle for FM and FM+
>   i2c: qup: Add command-line parameter to override SCL frequency

Wolfram,

Is this patch set acceptable?  We'd like to see it in 4.18, but it's
not in your i2c/for-next branch.
Wolfram Sang May 23, 2018, 4:19 p.m. UTC | #2
On Wed, May 23, 2018 at 11:11:40AM -0500, Timur Tabi wrote:
> On Thu, May 10, 2018 at 11:13 AM, Austin Christ <austinwc@codeaurora.org> wrote:
> > Add support for Fast Mode Plus (1MHz) to the I2C QUP driver and correct
> > duty cycle for Fast Mode and Fast Mode Plus.
> >
> > Add configuration in probe for Centriq devices.
> >
> > Add a command line module parameter to allow override of SCL frequency.
> >
> > [v3]
> > - correct probe patch to branch within QUPv2 segment.
> >
> > [v2]
> > - whitespace corrections
> >
> > Austin Christ (4):
> >   i2c: qup: add probe path for Centriq ACPI devices
> >   i2c: qup: Add support for Fast Mode Plus
> >   i2c: qup: Correct duty cycle for FM and FM+
> >   i2c: qup: Add command-line parameter to override SCL frequency
> 
> Wolfram,
> 
> Is this patch set acceptable?  We'd like to see it in 4.18, but it's
> not in your i2c/for-next branch.

I am just waiting here for the maintainers response, too. Andy? David?
Andy Gross May 25, 2018, 8:31 p.m. UTC | #3
On Wed, May 23, 2018 at 06:19:45PM +0200, Wolfram Sang wrote:
> On Wed, May 23, 2018 at 11:11:40AM -0500, Timur Tabi wrote:
> > On Thu, May 10, 2018 at 11:13 AM, Austin Christ <austinwc@codeaurora.org> wrote:
> > > Add support for Fast Mode Plus (1MHz) to the I2C QUP driver and correct
> > > duty cycle for Fast Mode and Fast Mode Plus.
> > >
> > > Add configuration in probe for Centriq devices.
> > >
> > > Add a command line module parameter to allow override of SCL frequency.
> > >
> > > [v3]
> > > - correct probe patch to branch within QUPv2 segment.
> > >
> > > [v2]
> > > - whitespace corrections
> > >
> > > Austin Christ (4):
> > >   i2c: qup: add probe path for Centriq ACPI devices
> > >   i2c: qup: Add support for Fast Mode Plus
> > >   i2c: qup: Correct duty cycle for FM and FM+
> > >   i2c: qup: Add command-line parameter to override SCL frequency
> > 
> > Wolfram,
> > 
> > Is this patch set acceptable?  We'd like to see it in 4.18, but it's
> > not in your i2c/for-next branch.
> 
> I am just waiting here for the maintainers response, too. Andy? David?

Wolfram,

I finally got around to looking at these.  They all look good and rdy to go.

Andy
Wolfram Sang May 29, 2018, 6:05 p.m. UTC | #4
On Thu, May 10, 2018 at 10:13:53AM -0600, Austin Christ wrote:
> Add support for Fast Mode Plus (1MHz) to the I2C QUP driver and correct
> duty cycle for Fast Mode and Fast Mode Plus.
> 
> Add configuration in probe for Centriq devices.
> 
> Add a command line module parameter to allow override of SCL frequency.
> 
> [v3]
> - correct probe patch to branch within QUPv2 segment.
> 
> [v2]
> - whitespace corrections
> 
> Austin Christ (4):
>   i2c: qup: add probe path for Centriq ACPI devices
>   i2c: qup: Add support for Fast Mode Plus
>   i2c: qup: Correct duty cycle for FM and FM+
>   i2c: qup: Add command-line parameter to override SCL frequency

Applied to for-next, thanks!