Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754338AbdC1Jul (ORCPT ); Tue, 28 Mar 2017 05:50:41 -0400 Received: from mailgw01.mediatek.com ([218.249.47.110]:41712 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1753702AbdC1Jui (ORCPT ); Tue, 28 Mar 2017 05:50:38 -0400 Message-ID: <1490694612.18582.11.camel@mhfsdcap03> Subject: Re: [RESEND PATCH v3 1/2] dt-bindings: i2c: Add Mediatek MT2701 i2c binding From: Jun Gao To: Wolfram Sang , Matthias Brugger , CC: Matthias Brugger , , , , , , , Date: Tue, 28 Mar 2017 17:50:12 +0800 In-Reply-To: <20170322090543.sxacnddn2o7stjnq@ninjato> References: <1489029185-21447-1-git-send-email-jun.gao@mediatek.com> <1489029185-21447-2-git-send-email-jun.gao@mediatek.com> <20170322090543.sxacnddn2o7stjnq@ninjato> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2158 Lines: 52 On Wed, 2017-03-22 at 10:05 +0100, Wolfram Sang wrote: > On Thu, Mar 09, 2017 at 11:13:04AM +0800, Jun Gao wrote: > > From: Jun Gao > > > > Add MT2701 i2c binding to i2c-mt6577.txt and there is no need to > > modify i2c driver. > > > > Signed-off-by: Jun Gao > > There is still the issue which Matthias Brugger pointed out on January, > 24th: you need to describe the fallback compatibles. > IMHO the value under compatible are just inputs which can be accepted. It's little strange to add fallback information in binding. Some other bindings do not describe so detailedly. Is it OK to make binding as minimum standard? If we describe it very detailedly, we will have to modify binding if there are some changes for mt2701. Rob, could you give some suggestions? Thanks! > > --- > > .../devicetree/bindings/i2c/i2c-mt6577.txt | 11 ++++++----- > > 1 file changed, 6 insertions(+), 5 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/i2c/i2c-mt6577.txt b/Documentation/devicetree/bindings/i2c/i2c-mt6577.txt > > index 0ce6fa3..27dbbf9 100644 > > --- a/Documentation/devicetree/bindings/i2c/i2c-mt6577.txt > > +++ b/Documentation/devicetree/bindings/i2c/i2c-mt6577.txt > > @@ -4,11 +4,12 @@ The Mediatek's I2C controller is used to interface with I2C devices. > > > > Required properties: > > - compatible: value should be either of the following. > > - (a) "mediatek,mt6577-i2c", for i2c compatible with mt6577 i2c. > > - (b) "mediatek,mt6589-i2c", for i2c compatible with mt6589 i2c. > > - (c) "mediatek,mt8127-i2c", for i2c compatible with mt8127 i2c. > > - (d) "mediatek,mt8135-i2c", for i2c compatible with mt8135 i2c. > > - (e) "mediatek,mt8173-i2c", for i2c compatible with mt8173 i2c. > > + "mediatek,mt2701-i2c" > > + "mediatek,mt6577-i2c" > > + "mediatek,mt6589-i2c" > > + "mediatek,mt8127-i2c" > > + "mediatek,mt8135-i2c" > > + "mediatek,mt8173-i2c" > > - reg: physical base address of the controller and dma base, length of memory > > mapped region. > > - interrupts: interrupt number to the cpu. > > -- > > 1.7.9.5 > >