From patchwork Mon Jun 1 06:33:04 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ricardo_Ca=C3=B1uelo?= X-Patchwork-Id: 11581451 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id E54B014F6 for ; Mon, 1 Jun 2020 06:34:10 +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 C31502074B for ; Mon, 1 Jun 2020 06:34:10 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="mWU88gRa" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C31502074B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=collabora.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:To :From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=PS4Q7/HHZOS9Bnl9qxu0TxHItPZEn/FiJeBoDVo3or4=; b=mWU88gRaHjqtnU V6/xoiops9ukf/3ff1e3zgCp+AGzHhx2QH9FqCtl+vHu7N6FGwfSrnYYX+MGsdZfjd/hJ0yzt/jBr fQ9RbJq6CyDs4YCnRsVqr4WeT08/vAWLvIrko0kACCzMleMfSCD4bI28ic1IgdIARRpJBREENiPAq DrAmysG6a7dcJP02Ier5dkOWGqTXqqIi06quG5F+Yqaj8JDrAtSULI8hcU/QMS4IH35PgNrTf4IZy TWvno7taIN1D4jAtKboQFSePGNAE3MsL771kXU7Ak1zuhYr6hFXosdbrqeBSW+0jg8Lgqn52TGau+ q2F2w0pe1xAWZt5mlFbg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jfe1L-0008Kz-TB; Mon, 01 Jun 2020 06:34:03 +0000 Received: from bhuna.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e3e3]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jfe19-0008AO-Gq for linux-arm-kernel@lists.infradead.org; Mon, 01 Jun 2020 06:33:53 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: rcn) with ESMTPSA id A70C82A050B From: =?utf-8?q?Ricardo_Ca=C3=B1uelo?= To: laurent.pinchart@ideasonboard.com Subject: [PATCH v3 0/4] Convert adi,adv7511.txt DT bindings to yaml Date: Mon, 1 Jun 2020 08:33:04 +0200 Message-Id: <20200601063308.13045-1-ricardo.canuelo@collabora.com> X-Mailer: git-send-email 2.18.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200531_233351_686811_261D59DA X-CRM114-Status: GOOD ( 14.23 ) X-Spam-Score: -0.0 (/) X-Spam-Report: SpamAssassin version 3.4.4 on bombadil.infradead.org summary: Content analysis details: (-0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 SPF_HELO_PASS SPF: HELO matches SPF record -0.0 SPF_PASS SPF: sender matches SPF record 0.0 UNPARSEABLE_RELAY Informational: message has unparseable relay lines X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: marex@denx.de, devicetree@vger.kernel.org, michal.simek@xilinx.com, xuwei5@hisilicon.com, robh+dt@kernel.org, mcoquelin.stm32@gmail.com, kernel@collabora.com, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org Hi, This series convert the adi,adv7511.txt DT bindings to json-schema. As a result of the conversion some dts files needed to be updated. The changes to the dts files are of three types: - Reordering of the I2C slave addresses list of the ADV75xx node. The addresses in the 'reg' property and the matching names in 'reg-names' for an I2C slave don't need to be in any particular order, but the DT schema defines these properties as a cell array and a string array respectively, which are ordered, so the definitions in the dts files must match the order in the binding. - Filling the minimum binding requirements. Most of the time this means creating a 'ports' node in the boards that don't define them. Note, however, that the purpose of this is simply to make the definition compliant with the binding. I didn't define any endpoints for the ports. - Removing unneeded properties. About the binding conversion: - The original binding covered five different devices: ADV7511, ADV7511W, ADV7513, ADV7533 and ADV7535. They all share a common set of properties but ADV7533 and ADV7535 have enough differences from the rest to warrant their own binding file. In v1 I modelled all the properties constraints for all five devices in a single file but it turned out a bit too complex. Splitting the binding into one for ADV7511/11W/13 and another for ADV7533/35 makes them much easier to read and maintain. Patches 1/4 to 3/4 contain the dts changes. Patch 4/4 contains the binding conversion. NOTE: the bindings have been tested with: make dt_binding_check ARCH= DT_SCHEMA_FILES=<...adi,adv7511.yaml> make dt_binding_check ARCH= DT_SCHEMA_FILES=<...adi,adv7533.yaml> make dtbs_check ARCH= DT_SCHEMA_FILES=<...adi,adv7511.yaml> make dtbs_check ARCH= DT_SCHEMA_FILES=<...adi,adv7533.yaml> for = arm and arm64. dts changes haven't been tested in hardware. Some existing DTs are expected to fail after this conversion. Changes in v3: - Removed from the patch series (already in mainline): - arm64: dts: renesas: make hdmi encoder nodes compliant with DT bindings - ARM: dts: renesas: make hdmi encoder nodes compliant with DT bindings - ARM: dts: iwg20d-q7-dbcm-ca: remove unneeded properties in hdmi@39 - Additional DTs fixes: - boot/dts/stm32mp15xx-dhcor-avenger96.dtsi. - [Laurent] adi,adv7511.yaml and adi,adv7533.yaml. - Documentation fixes and typos. - Removed unnecessary allOf's. - adi,embedded-sync data type changed to boolean. - Power supplies defined as required. - Examples updated. Ricardo Cañuelo (4): ARM: dts: zynq: add port definitions to hdmi-tx@39 arm64: dts: hisilicon: hikey: fixes to comply with adi,adv7533 DT binding ARM: dts: stm32: make hdmi-transmitter node compliant with DT bindings dt-bindings: drm: bridge: adi,adv7511.txt: convert to yaml .../bindings/display/bridge/adi,adv7511.txt | 143 ----------- .../bindings/display/bridge/adi,adv7511.yaml | 231 ++++++++++++++++++ .../bindings/display/bridge/adi,adv7533.yaml | 175 +++++++++++++ .../boot/dts/stm32mp15xx-dhcor-avenger96.dtsi | 6 +- arch/arm/boot/dts/zynq-zc702.dts | 10 + arch/arm/boot/dts/zynq-zc706.dts | 10 + .../boot/dts/hisilicon/hi3660-hikey960.dts | 11 + .../arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 +- 8 files changed, 440 insertions(+), 148 deletions(-) delete mode 100644 Documentation/devicetree/bindings/display/bridge/adi,adv7511.txt create mode 100644 Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml create mode 100644 Documentation/devicetree/bindings/display/bridge/adi,adv7533.yaml