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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D2DDDC433FE for ; Fri, 8 Oct 2021 22:21:55 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 A05BA6103C for ; Fri, 8 Oct 2021 22:21:55 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org A05BA6103C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=68Ug4HQRrT/cQcfiZbM+qmArh2cq/7AQmQyeDisvy7w=; b=rj05CCAN021zTr 01//XWjhJtSor8RoZFh7V5C7Yy43oHH9ITE3PJA30PXHPKO+MkTqIWmJO+dIscGDZmvkzl2epN5iJ pxQeUSGyNFgKIwYcSb+oqd/1wKPQu/ltSD+QyTCn+Kv0VyKL0ZCqSHdHJheGTyoUI9LQSOGY/AYca qEX0dNkhO9ol2EdUhfgKUGx44NgXxwmZlaj2hhOgaFRcEMhvS6o9t4trpn5HewG0jJ96PxIdNmR35 lQ245LqTWJswkQk/NSzZBAagKBfn9zAhjaSlqDbaekvYA4QcVDYkbpzloH8EKOJ8DEaqv+KOht40x 52hFvhzlMKeIXMfxK1wQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mYyDp-004Hil-33; Fri, 08 Oct 2021 22:20:09 +0000 Received: from mail-oi1-f181.google.com ([209.85.167.181]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mYyDl-004Hi7-L1 for linux-arm-kernel@lists.infradead.org; Fri, 08 Oct 2021 22:20:06 +0000 Received: by mail-oi1-f181.google.com with SMTP id n64so15621813oih.2 for ; Fri, 08 Oct 2021 15:20:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=UU6kbMIJ6P0+5LUEJL0eEAN6Y3jPevBlPgH2FjrwHnA=; b=a5f7CHpywFPqFLzzjP+O9VlhCnB3SzRcmVvd6fKilo3J3NL/mnxtIdZsP26BqullAS 5gnhAXQFHDEvfSmuDQcA87pWoiqQHXhznio0/fZrWxxVnUqva5lxT1qPv6rXJW8No8Ru B1BB8FaEfP/QK6VTp8VA5xo2hPtvne7CuqEwxTnf1PoNQ8CmROpd8x6mClWcwCQjmqlu VvI+rJBK5bZX8yAcXEOgBROmDfjy7ogbZwG6ZrLl4l+mBHM4OHwM9gMcbXIJeW0pLlSP TOuPRr6ug4KsHW9ECj7k8bJRDe9esEJ0MrEc1qeeV36Ku91MuwO+eInPu2T7ILlrayPg lLUA== X-Gm-Message-State: AOAM5335pdevzUqJZ6cu+//Tx8PbLLbtUklf22aDErvjICfVFpOmHcaF fQv8hPjaKmTDHDf6+2PAuw== X-Google-Smtp-Source: ABdhPJzy7BWI36byfDiSz/sSJ/LcU68rMB1Z+VoTQBSiuLbVB1C7jFa2dQsUtAsjty74pAZ7+8mnig== X-Received: by 2002:a05:6808:292:: with SMTP id z18mr9290273oic.61.1633731604863; Fri, 08 Oct 2021 15:20:04 -0700 (PDT) Received: from robh.at.kernel.org (66-90-148-213.dyn.grandenetworks.net. [66.90.148.213]) by smtp.gmail.com with ESMTPSA id a13sm159169oiy.9.2021.10.08.15.20.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Oct 2021 15:20:02 -0700 (PDT) Received: (nullmailer pid 3419237 invoked by uid 1000); Fri, 08 Oct 2021 22:20:01 -0000 Date: Fri, 8 Oct 2021 17:20:01 -0500 From: Rob Herring To: Li Yang Cc: Shawn Guo , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Oleksij Rempel , linux-arm-kernel , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v2 02/16] dt-bindings: i2c: imx: update schema to align with original txt binding Message-ID: References: <20211001000417.15334-1-leoyang.li@nxp.com> <20211001000417.15334-3-leoyang.li@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211008_152005_720684_7AFFF62D X-CRM114-Status: GOOD ( 30.60 ) 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-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Oct 01, 2021 at 12:37:54PM -0500, Li Yang wrote: > On Fri, Oct 1, 2021 at 8:24 AM Rob Herring wrote: > > > > On Thu, Sep 30, 2021 at 7:04 PM Li Yang wrote: > > > > > > When the binding was converted from txt to yaml, it actually added more > > > constrains than the original txt binding which was already used in many > > > in-tree DTSes. Some of the newly added constrains are either not valid > > > or not neccessary. > > > > IMO, both of these should be fixed in the dts files. > > > > > Not all SoCs use ipg as the clock name for i2c. There is no point in > > > having SoC integration information defined in i2c binding. Remove the > > > clock name requirement in the schema. > > > > Any name you want is not fine. Your choices are remove clock-names, > > add all the names used, or change everyone to use 'ipg'. > > I understand that the name should be important as clocks are > referenced by name. But since the i2c controller only has one clock , > the name is never referenced in the driver. Then just remove 'clock-names' from the dts file. > If we really want to define the name, IMO, it should be from the > perspective of the i2c controller like "clkin" or "i2c" instead of the > "ipg" from the perspective of SoC integration which could be changing > with a different integration. I can list both "i2c" and "ipg" for now > as a workaround though. $modulename for $foo-names always looks made up and pointless to me. > > > > > > The original txt binding didn't require the order of tx and rx for > > > dmas/dma-names. Many in tree DTSes are already using the other order. > > > Both orders should just work fine. Update the schema to allow both. > > > > Doesn't sound like a case where defining the order is challenging. > > No, it is not challenging. But as dma channel is only referenced by > name instead of index. I don't see too much benefit in enforcing the > order other than easier to create the schema. Easier is nice, and that's the 'DT way' is the other reason. Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel 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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 92B80C433F5 for ; Fri, 8 Oct 2021 22:20:09 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6F5316103C for ; Fri, 8 Oct 2021 22:20:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243563AbhJHWWC (ORCPT ); Fri, 8 Oct 2021 18:22:02 -0400 Received: from mail-oi1-f177.google.com ([209.85.167.177]:38511 "EHLO mail-oi1-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231830AbhJHWWB (ORCPT ); Fri, 8 Oct 2021 18:22:01 -0400 Received: by mail-oi1-f177.google.com with SMTP id t4so13753807oie.5; Fri, 08 Oct 2021 15:20:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=UU6kbMIJ6P0+5LUEJL0eEAN6Y3jPevBlPgH2FjrwHnA=; b=6m7l58ZqzCgBjCZUu6prLMLynijDqQh6An/+WJgR8By9uGRpyMwt+k91NWovCLMoZr E+a5MLXRoIx10N1Fwlf9sVcaujEUIAGc7JIXVnHa89VC1pbtQ+Pmy7WFKuHX3sSCGtjO a7DEz1k1Bvqfo3mtv99Y5KXpxJn97612riizPVLHShjtpHmrOR06GFaZ3voKvbfjGPQI BLlLxBQPBhXWIs9m9OCb0fQe4Oh3wMJpn3626srOmzdpBowwUXE64P2jsm7vflrz25Y/ aVu8yI+YtKqS4UpaMosYDi5GRist2LpsI9rXqFmU77XDS7IuZIC4wpVUTF1xdhavu/0e 0DJw== X-Gm-Message-State: AOAM5316Yq2g5i7cWBqNSsHysvCLbzLwvrcCsR4K3qMeudfHjdXDPPUw jlJaq/7zkBx2K0EWZnji3NuSkvU/Kg== X-Google-Smtp-Source: ABdhPJzy7BWI36byfDiSz/sSJ/LcU68rMB1Z+VoTQBSiuLbVB1C7jFa2dQsUtAsjty74pAZ7+8mnig== X-Received: by 2002:a05:6808:292:: with SMTP id z18mr9290273oic.61.1633731604863; Fri, 08 Oct 2021 15:20:04 -0700 (PDT) Received: from robh.at.kernel.org (66-90-148-213.dyn.grandenetworks.net. [66.90.148.213]) by smtp.gmail.com with ESMTPSA id a13sm159169oiy.9.2021.10.08.15.20.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Oct 2021 15:20:02 -0700 (PDT) Received: (nullmailer pid 3419237 invoked by uid 1000); Fri, 08 Oct 2021 22:20:01 -0000 Date: Fri, 8 Oct 2021 17:20:01 -0500 From: Rob Herring To: Li Yang Cc: Shawn Guo , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Oleksij Rempel , linux-arm-kernel , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v2 02/16] dt-bindings: i2c: imx: update schema to align with original txt binding Message-ID: References: <20211001000417.15334-1-leoyang.li@nxp.com> <20211001000417.15334-3-leoyang.li@nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Fri, Oct 01, 2021 at 12:37:54PM -0500, Li Yang wrote: > On Fri, Oct 1, 2021 at 8:24 AM Rob Herring wrote: > > > > On Thu, Sep 30, 2021 at 7:04 PM Li Yang wrote: > > > > > > When the binding was converted from txt to yaml, it actually added more > > > constrains than the original txt binding which was already used in many > > > in-tree DTSes. Some of the newly added constrains are either not valid > > > or not neccessary. > > > > IMO, both of these should be fixed in the dts files. > > > > > Not all SoCs use ipg as the clock name for i2c. There is no point in > > > having SoC integration information defined in i2c binding. Remove the > > > clock name requirement in the schema. > > > > Any name you want is not fine. Your choices are remove clock-names, > > add all the names used, or change everyone to use 'ipg'. > > I understand that the name should be important as clocks are > referenced by name. But since the i2c controller only has one clock , > the name is never referenced in the driver. Then just remove 'clock-names' from the dts file. > If we really want to define the name, IMO, it should be from the > perspective of the i2c controller like "clkin" or "i2c" instead of the > "ipg" from the perspective of SoC integration which could be changing > with a different integration. I can list both "i2c" and "ipg" for now > as a workaround though. $modulename for $foo-names always looks made up and pointless to me. > > > > > > The original txt binding didn't require the order of tx and rx for > > > dmas/dma-names. Many in tree DTSes are already using the other order. > > > Both orders should just work fine. Update the schema to allow both. > > > > Doesn't sound like a case where defining the order is challenging. > > No, it is not challenging. But as dma channel is only referenced by > name instead of index. I don't see too much benefit in enforcing the > order other than easier to create the schema. Easier is nice, and that's the 'DT way' is the other reason. Rob