From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1C28EC433ED for ; Fri, 7 May 2021 17:26:48 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 72292613ED for ; Fri, 7 May 2021 17:26:47 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 72292613ED Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:CC:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=bXg/s8GlZvhcJAWdrsuupZ9oopvx3wylpd/7X7PaTBw=; b=mDRsOmvKKPPozM/7y6mbCIkJI Zej4odR0lefNr2TfQyUuDARfCUeOKHTJ9wIkRPDnmFXZL2p4DV9MBJfP4N240wiDIFyILnwGil5ix We3i/OZW9g4nIX4tsI0io9q5ZDgeFYoH9cWG6OdnqSCeUi+lBXWOlJq+LzvkoYCy7xnAU7+BGPv5F U1b3ZTnLTPBjMjrLkF7boI04zCLlTaAK8tLJ2XqlzcQ+FHiYsyWR0D3Xls/wsitjqWhgU3QkUrFOS zKQidvzIcobUzTVPXK83ByWB3GenuDBL+cWoewajr+jUeUsBGgEpzED5ZbQrtg4Lo1LlMmbsTIGbD d0D4ffubw==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lf4E7-007fi9-Il; Fri, 07 May 2021 17:25:23 +0000 Received: from bombadil.infradead.org ([2607:7c80:54:e::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lf4E4-007fht-K8 for linux-arm-kernel@desiato.infradead.org; Fri, 07 May 2021 17:25:20 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:In-Reply-To:MIME-Version:Date:Message-ID:From:References:CC:To: Subject:Sender:Reply-To:Content-ID:Content-Description; bh=Mfe3VzxO5eVTgpcrjkbuWFbFY3AOvc+A6TkDDBQODfw=; b=GHrsRfwunUoFPkfweLnKQHNuSt HbXLTbKLV3Pa+iPvldBVqVoSH8eIv8IT5odzAAum5dvFq/OYP349znfKovF1bw4XNTmrjU1u7llSo A3MTqYMN0J7Wxvv/XxB68JjsI10cJnbGuoFLtPoep/K2tpliQQdtLM5LSmXbjlBQNoOI05mtendX9 AK498eZuGFR1TI9qnzavrUvOBIRS+jIqGi9BCRHSWXYyOOpUKbwGtnEb/tgbdsegPmjpQQ/TP6c3y vRqrejxzUZtC8rAYOyOwL9pV9bix0VvYc3E45pQS/8NZCK6IwtU1Yg5J6AX5BHZHL+RSvGaCB8rSQ cg9/LJ6g==; Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lf4E1-0072T1-U2 for linux-arm-kernel@lists.infradead.org; Fri, 07 May 2021 17:25:19 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 147HPA4M066732; Fri, 7 May 2021 12:25:10 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1620408310; bh=Mfe3VzxO5eVTgpcrjkbuWFbFY3AOvc+A6TkDDBQODfw=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=wxkOJ9i3ToJ/P/tUmQgTIBndTsX7BGuoRHm4aSOLU+yK2W8nJ4qHCB3Z4w3SHGjFB 0QSBLPalZP6KnIZyQ/qHWY/8vl2ZPezMEGJ65dJtpqQuwphIBOQrWcTbk6VqXqSRc2 rWc8kFZ+sULQooEDGjtm3FepemCYpNf7wVlRKckU= Received: from DFLE113.ent.ti.com (dfle113.ent.ti.com [10.64.6.34]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 147HPAlJ042114 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 7 May 2021 12:25:10 -0500 Received: from DFLE101.ent.ti.com (10.64.6.22) by DFLE113.ent.ti.com (10.64.6.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Fri, 7 May 2021 12:25:10 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE101.ent.ti.com (10.64.6.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2 via Frontend Transport; Fri, 7 May 2021 12:25:09 -0500 Received: from [10.250.100.73] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 147HP62T106353; Fri, 7 May 2021 12:25:06 -0500 Subject: Re: [PATCH v2] dt-bindings: i2c: Move i2c-omap.txt to YAML format To: Andreas Kemnade , Vignesh Raghavendra CC: Rob Herring , , , , , Linux ARM Mailing List , Tony Lindgren , Nishanth Menon References: <20210506140026.31254-1-vigneshr@ti.com> <429a740a-c2b9-1cf8-ed2b-0fb7b1bea422@ti.com> <20210507163602.219894f4@aktux> From: Grygorii Strashko Message-ID: <1ef076ac-e0de-a0df-a918-aeb8ed6c5956@ti.com> Date: Fri, 7 May 2021 20:24:59 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210507163602.219894f4@aktux> Content-Language: en-US X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210507_102518_079318_51AAEC09 X-CRM114-Status: GOOD ( 19.41 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 07/05/2021 17:36, Andreas Kemnade wrote: > On Fri, 7 May 2021 19:45:45 +0530 > Vignesh Raghavendra wrote: > >> On 5/7/21 12:24 PM, Grygorii Strashko wrote: >>> >>> >>> On 06/05/2021 17:00, Vignesh Raghavendra wrote: >>>> Convert i2c-omap.txt to YAML schema for better checks and documentation. >>>> >>>> Following properties were used in DT but were not documented in txt >>>> bindings and has been included in YAML schema: >>>> 1. Include ti,am4372-i2c compatible >>>> 2. Include dmas property used in few OMAP dts files >>> >>> The DMA is not supported by i2c-omap driver, so wouldn't be better to >>> just drop dmas from DTBs to avoid confusions? >>> It can be added later. >>> >> >> Will do.. I will also send patches dropping dmas from dts that currently >> have them populated. >> > hmm, we have > - DO attempt to make bindings complete even if a driver doesn't support some > features. For example, if a device has an interrupt, then include the > 'interrupts' property even if the driver is only polled mode. > > in Documentation/devicetree/bindings/writing-bindings.rst > Shouln't the dma stay there if the hardware supports it? Devicetree > should describe the hardware not the driver if I understood things > right. True. But my above statement is also valid - it introduces confusion from user point of view. More over, 'dmas' is not part of original binding and were randomly added to some SoCs. And it's much more easy to extend binding (in the future) then remove something after. I leave it to Vignesh, Tony to decide. -- Best regards, grygorii _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel