From patchwork Fri May 1 08:32:22 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: 11522305 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 B1D22912 for ; Fri, 1 May 2020 08:33:25 +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 902B820787 for ; Fri, 1 May 2020 08:33:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="H8+/JPB5" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 902B820787 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=ckf7mkSt7drfpoV4kvU/O0PbyMLoyKbuAEXSqznACDw=; b=H8+/JPB5JqDVvR r4WKIIw1ve1M/p11OnMofzUGO8Fzmz4sUZnkGTUhCaN7htMgzeGCZ6l/k3PEcanU0UlZdWf6pIgZ+ wrfTG1v+ziFxs3T2jCYVuafdeo4HrhM5MNb3Q0JhufqKyLcJZrFj4N3S7X0D86JanJiNV2YESMLcB sOhzAfF5c+Y38tSxUA1ZY0m+/kbxzs34gD+jdcb5ugBhVhTO1eAzIsXiimH5jLUxmgsRNvqTAHS5G tmdgenUIk5okkZ7XnwgWcdQDtrpT5lxQ1AWKyQLTMGr5weEsU46vrx32GMX1VsjYEskL4gQxDBiM3 N0pIFpE8/ZY93dhFZFsQ==; 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 1jUR6n-0007bd-Fh; Fri, 01 May 2020 08:33:21 +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 1jUR6k-0007a6-E7 for linux-arm-kernel@lists.infradead.org; Fri, 01 May 2020 08:33:19 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: rcn) with ESMTPSA id 0D90E2A2C9F From: =?utf-8?q?Ricardo_Ca=C3=B1uelo?= To: laurent.pinchart@ideasonboard.com Subject: [RFT PATCH 0/5] Convert adi,adv7511.txt DT bindings to yaml Date: Fri, 1 May 2020 10:32:22 +0200 Message-Id: <20200501083227.10886-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-20200501_013318_603511_02B65AD0 X-CRM114-Status: GOOD ( 11.37 ) 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: devicetree@vger.kernel.org, geert+renesas@glider.be, xuwei5@hisilicon.com, robh+dt@kernel.org, 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 two 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. About the binding conversion: - The dependencies between properties are properly modelled but the result might look cluttered. If you find it hard to read or maintain, another option is to split the binding in two: one for the ADV7511/11w/13 and another one for the ADV7533/35. Patches 1/5 to 4/5 contain the dts changes. Patch 5/5 contains the binding conversion. NOTE: the bindings have been tested with: make dt_binding_check ARCH= DT_SCHEMA_FILES=<...adi,adv7511.yaml> make dtbs_check ARCH= DT_SCHEMA_FILES=<...adi,adv7511.yaml> for = arm and arm64. dts changes haven't been tested in hardware. Kind regards, Ricardo Ricardo CaƱuelo (5): arm64: dts: draak: Reorder hdmi-encoder@39 reg and reg-names properties ARM: dts: wheat: reorder reg and reg-names properties in hdmi bridges ARM: dts: zynq: add port definitions to hdmi-tx@39 arm64: dts: hisilicon: hikey960: add missing hdmi bridge properties dt-bindings: drm: bridge: adi,adv7511.txt: convert to yaml .../bindings/display/bridge/adi,adv7511.txt | 143 --------- .../bindings/display/bridge/adi,adv7511.yaml | 295 ++++++++++++++++++ arch/arm/boot/dts/r8a7792-wheat.dts | 8 +- 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/renesas/r8a77995-draak.dts | 4 +- 7 files changed, 332 insertions(+), 149 deletions(-) delete mode 100644 Documentation/devicetree/bindings/display/bridge/adi,adv7511.txt create mode 100644 Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml