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 2709BC433F5 for ; Fri, 1 Oct 2021 13:19:41 +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 E4EB461A51 for ; Fri, 1 Oct 2021 13:19:40 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org E4EB461A51 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:MIME-Version:List-Subscribe:List-Help: List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:Subject: References:In-Reply-To:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=G8zYnjYHxIdq0tHeOr0+/MR7uORXCDjLw628BvAOZX0=; b=lsOKlseHg5Vbzm 1HursdgfcCWl0q5bb4J29x6wZMk//qs+2DDnrvA2gG3T3S4jcwtORSImo1p2Vb1OkKZBaq2U9c9dV lG4W8U2syN5ckB4i2e1eznmJSoCKki/ADRufsZpKEzEcQQGX5MNqCFa1GC6yjSc11paQYhkLpfZaY lfBgQxzHeeckZEHu/dVCK5UaqYp94hI7/fD+ItA6iKtr41es9ZePo5D7EqRaXbdN7HsVIOt5gohhW bqqVHYw15Slx/oCHlBvGFN6ICeJTTiEiVvC24ANOcZwq/r/w/cLMdSKDJmCX3ppfpRMOX7yt5Rxud kRNdrOx0Ym3C3JfhxDqQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mWIPm-000OzK-LM; Fri, 01 Oct 2021 13:17:26 +0000 Received: from mail-ot1-f54.google.com ([209.85.210.54]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mWIPS-000OxG-P0 for linux-arm-kernel@lists.infradead.org; Fri, 01 Oct 2021 13:17:08 +0000 Received: by mail-ot1-f54.google.com with SMTP id c6-20020a9d2786000000b005471981d559so11407246otb.5 for ; Fri, 01 Oct 2021 06:17:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:in-reply-to:references:subject:date :message-id; bh=GpaCFBluFudPZ/yH9IF+RzTInuanJkVWpBhaYtuvn70=; b=NYBzHEOAjCqnD83sSnlsjyq9VmkZEupOBP8rJ0U0rlmX05XkwjlMmtPhS1k4V0I/MZ jJ2PolYR6sef+KutHejnx/9VR+xopG7GgyhhHpaFWYjLn7vHc+cWlol4y8o9SMaL+rZR 7wIiiUJuD3FIqmU69qbSfVvMIPefqOm22TSTOUNr+eMV9BZpK5jFGkdRAgSzlbjpTnnW jNiBr1D48PVvIPSDGZDqtSTkQNiyfqTyFvnkXvruDO+TfKbOTzFCe4XHmsQ9Oc38lhjH BvGwbPHTYhf/21OcMnVFU9xuxEEXdsM4jHHhS5hmjEVPK4ys+hPYADI+J7w+xzlImBAO 880Q== X-Gm-Message-State: AOAM530UJ/ovv9Wk5fULxi4nb54fiUiClEVPm7cj8nIb4u0yAIWE/JV1 Fu7rk1XJHQx2/XLZOLLIcg== X-Google-Smtp-Source: ABdhPJzzeLKszTrXu+Szgav1S9FdZxShBPRvj5IXzS/1VS1vS3CRzG7P9PPuOKzQLyPVho+hlNMAMg== X-Received: by 2002:a05:6830:112:: with SMTP id i18mr10003411otp.186.1633094225354; Fri, 01 Oct 2021 06:17:05 -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 d7sm1169115ooa.36.2021.10.01.06.17.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Oct 2021 06:17:04 -0700 (PDT) Received: (nullmailer pid 3666446 invoked by uid 1000); Fri, 01 Oct 2021 13:16:57 -0000 From: Rob Herring To: Li Yang Cc: devicetree@vger.kernel.org, Oleksij Rempel , linux-kernel@vger.kernel.org, Rob Herring , linux-arm-kernel@lists.infradead.org, Shawn Guo In-Reply-To: <20211001000417.15334-3-leoyang.li@nxp.com> References: <20211001000417.15334-1-leoyang.li@nxp.com> <20211001000417.15334-3-leoyang.li@nxp.com> Subject: Re: [PATCH v2 02/16] dt-bindings: i2c: imx: update schema to align with original txt binding Date: Fri, 01 Oct 2021 08:16:57 -0500 Message-Id: <1633094217.893290.3666445.nullmailer@robh.at.kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211001_061706_834084_47C3DF50 X-CRM114-Status: GOOD ( 19.31 ) 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: , MIME-Version: 1.0 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 Thu, 30 Sep 2021 19:04:03 -0500, 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. > > 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. > > 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. > > Signed-off-by: Li Yang > --- > v2: > Updated the patch description > > Documentation/devicetree/bindings/i2c/i2c-imx.yaml | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings. Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future. Full log is available here: https://patchwork.ozlabs.org/patch/1535099 i2c@21a4000: clock-frequency:0:0: 50000 is not one of [100000, 400000] arch/arm/boot/dts/imx6dl-alti6p.dt.yaml i2c@21f8000: clock-frequency:0:0: 50000 is not one of [100000, 400000] arch/arm/boot/dts/imx6dl-alti6p.dt.yaml i2c@30a20000: clock-frequency:0:0: 387000 is not one of [100000, 400000] arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r3.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r4.dt.yaml i2c@30a30000: clock-frequency:0:0: 387000 is not one of [100000, 400000] arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r3.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r4.dt.yaml i2c@30a40000: clock-frequency:0:0: 387000 is not one of [100000, 400000] arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r3.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r4.dt.yaml i2c@30a50000: clock-frequency:0:0: 387000 is not one of [100000, 400000] arch/arm64/boot/dts/freescale/imx8mq-librem5-r2.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r3.dt.yaml arch/arm64/boot/dts/freescale/imx8mq-librem5-r4.dt.yaml _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel