From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751915AbeCUBsI (ORCPT ); Tue, 20 Mar 2018 21:48:08 -0400 Received: from mail-io0-f195.google.com ([209.85.223.195]:45714 "EHLO mail-io0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751891AbeCUBsD (ORCPT ); Tue, 20 Mar 2018 21:48:03 -0400 X-Google-Smtp-Source: AG47ELufoeQ7c6vPMWgp7Vd7YcJ6lpioYX4lvAUPeEZCU+kmdDviyfWS+FWvqSlMRS1hnlsdxMnebX0J21w/zATUtZo= MIME-Version: 1.0 In-Reply-To: References: <20180302083123.7259-1-ard.biesheuvel@linaro.org> From: Ard Biesheuvel Date: Wed, 21 Mar 2018 09:48:02 +0800 Message-ID: Subject: Re: [PATCH v5 0/2] add support for Socionext SynQuacer I2C controller To: Wolfram Sang , Rob Herring , Mark Rutland , linux-i2c , Devicetree List Cc: Linux Kernel Mailing List , linux-arm-kernel , Jassi Brar , Andy Shevchenko , Ard Biesheuvel Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 17 March 2018 at 00:19, Ard Biesheuvel wrote: > On 10 March 2018 at 15:01, Ard Biesheuvel wrote: >> On 2 March 2018 at 08:31, Ard Biesheuvel wrote: >>> Add a binding and a driver for the I2C IP found in the Socionext SynQuacer >>> SoC, which is essentially a rebranded version of the Fujitsu F_I2C controller. >>> >>> I think this is good to go now. Wolfram? >>> >> >> Ping? >> > > Ping? > Hello Wolfram, Do you intend to queue this for v4.17? Or are there any issues that need to be addressed first? Thanks, Ard. >>> v5: >>> - add Rob's ack to #1 >>> - drop unnecessary 'platform_set_drvdata(pdev, NULL)' in remove path (#2) >>> >>> v4: >>> - clarify binding that only a single interrupt specifier is expected (#1) >>> - check return value of clk_prepare_enable() on probe path (#2) >>> - add Andy's R-b to patch #2 >>> >>> v3: >>> - incorporate more of Andy's review comments (#2), especially regarding the >>> bus speed and clock source handling for ACPI >>> - patch #1 unchanged. >>> >>> v2: >>> - incorporate Andy's review comments (#2) >>> - patch #1 unchanged. >>> >>> Ard Biesheuvel (2): >>> dt-bindings: i2c: add binding for Socionext SynQuacer I2C >>> i2c: add support for Socionext SynQuacer I2C controller >>> >>> Documentation/devicetree/bindings/i2c/i2c-synquacer.txt | 29 + >>> drivers/i2c/busses/Kconfig | 10 + >>> drivers/i2c/busses/Makefile | 1 + >>> drivers/i2c/busses/i2c-synquacer.c | 791 ++++++++++++++++++++ >>> 4 files changed, 831 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/i2c/i2c-synquacer.txt >>> create mode 100644 drivers/i2c/busses/i2c-synquacer.c >>> >>> -- >>> 2.11.0 >>> From mboxrd@z Thu Jan 1 00:00:00 1970 From: ard.biesheuvel@linaro.org (Ard Biesheuvel) Date: Wed, 21 Mar 2018 09:48:02 +0800 Subject: [PATCH v5 0/2] add support for Socionext SynQuacer I2C controller In-Reply-To: References: <20180302083123.7259-1-ard.biesheuvel@linaro.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 17 March 2018 at 00:19, Ard Biesheuvel wrote: > On 10 March 2018 at 15:01, Ard Biesheuvel wrote: >> On 2 March 2018 at 08:31, Ard Biesheuvel wrote: >>> Add a binding and a driver for the I2C IP found in the Socionext SynQuacer >>> SoC, which is essentially a rebranded version of the Fujitsu F_I2C controller. >>> >>> I think this is good to go now. Wolfram? >>> >> >> Ping? >> > > Ping? > Hello Wolfram, Do you intend to queue this for v4.17? Or are there any issues that need to be addressed first? Thanks, Ard. >>> v5: >>> - add Rob's ack to #1 >>> - drop unnecessary 'platform_set_drvdata(pdev, NULL)' in remove path (#2) >>> >>> v4: >>> - clarify binding that only a single interrupt specifier is expected (#1) >>> - check return value of clk_prepare_enable() on probe path (#2) >>> - add Andy's R-b to patch #2 >>> >>> v3: >>> - incorporate more of Andy's review comments (#2), especially regarding the >>> bus speed and clock source handling for ACPI >>> - patch #1 unchanged. >>> >>> v2: >>> - incorporate Andy's review comments (#2) >>> - patch #1 unchanged. >>> >>> Ard Biesheuvel (2): >>> dt-bindings: i2c: add binding for Socionext SynQuacer I2C >>> i2c: add support for Socionext SynQuacer I2C controller >>> >>> Documentation/devicetree/bindings/i2c/i2c-synquacer.txt | 29 + >>> drivers/i2c/busses/Kconfig | 10 + >>> drivers/i2c/busses/Makefile | 1 + >>> drivers/i2c/busses/i2c-synquacer.c | 791 ++++++++++++++++++++ >>> 4 files changed, 831 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/i2c/i2c-synquacer.txt >>> create mode 100644 drivers/i2c/busses/i2c-synquacer.c >>> >>> -- >>> 2.11.0 >>>