Received: by 2002:a25:c205:0:0:0:0:0 with SMTP id s5csp1103623ybf; Fri, 28 Feb 2020 14:14:46 -0800 (PST) X-Google-Smtp-Source: APXvYqz6bYgwLi/z7kevpozMMuCH5ioKH+r93F2RyEtxiHn13HA+u/ZTwCv5T7IPyp4IhyA6A/gU X-Received: by 2002:aca:3857:: with SMTP id f84mr4449742oia.150.1582928086114; Fri, 28 Feb 2020 14:14:46 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1582928086; cv=none; d=google.com; s=arc-20160816; b=VBcN5Rs6datEIn5qtKRzJFT3TNMaYWKwqDg4wOYx7+5lAbWZfB1fwiMJVQx13ePTeG A4CNmYVQBihT2hFXif/HFE7X/E7h1qoycYCHNG+AexqEmz6jXvjANuPG48S6VhWlm7O6 XnYyiOKXs7awoNWem23pDXlxK1vBzplAeGfFKZSU25W7ZC+BOHmqu43MOWfmJRhCbC41 STGILlqcqJo6lY0x70sWVzs5sRZoAgCvej/fojiq/7+kr6vLEVwJiZN8J84i04hfT/E9 2XYvl6Z8FtpcyftluMIowuvIN9upTcanN9h0eLB42XD1e6jWHHETq439rwe1DvSo8U2T pFWQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=Ns84jhjwh80AF+NgTsU5G8wYY3x4+c5K/NhAZhbbc6Q=; b=rCilHpNfLyaeKEptLxas3FYFtAN17ZqacrTq/4NoMJXF1OCGE4MAQkmm0BXGcN+6TV MgrfXeXmeCvG4HJAE8K3D/oaSd1tlA/O9ctQLZ5u7s4R4sG8e53M4tmHuMSW4fNvl9OB ep73QY1DMzyaPduQ1Dcr7/2wee9HapwgFn4sXSYO1e3CdPmIV4SrE/ZpX2x4VnM4XtG3 G5mifxbjanvKqrdMsol4+7iLdFMXnrOYhdWSrV4dxQzFCaL2MpMkYlw9AEJtWHAkMGp5 C5MTSgQvyXqmkenoi1AEaX+cIqsLkg2v2RZzE2GCacOxXw2s/K3UBSUjSA6tQxo89DDf eExQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=UGTTPcGE; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v25si2356428otq.93.2020.02.28.14.14.33; Fri, 28 Feb 2020 14:14:46 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=UGTTPcGE; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726658AbgB1WNS (ORCPT + 99 others); Fri, 28 Feb 2020 17:13:18 -0500 Received: from mail-lf1-f68.google.com ([209.85.167.68]:36168 "EHLO mail-lf1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726603AbgB1WNS (ORCPT ); Fri, 28 Feb 2020 17:13:18 -0500 Received: by mail-lf1-f68.google.com with SMTP id s1so2727663lfd.3 for ; Fri, 28 Feb 2020 14:13:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Ns84jhjwh80AF+NgTsU5G8wYY3x4+c5K/NhAZhbbc6Q=; b=UGTTPcGEoBQyMt2ffSmRLLgjAyPIUt+r0iIa8o1P49YLGdoYVKEbB6QakZAN00mAhm zy9v/0hSmLVI9UU5NBqAMSlsMEbwzjXJUYrFOUUGuxdKU48PBR44wBrg3ACsxrb/zaWM Or4tdMTMz2yn0WZyUSohIWbPxmV6UCTiz32JYGddmb92FTjXa/6SjWPkUvU+rdOtCmfO xQ2M/T2sSSDc2DNlZwW4KMcPv9JPGtOTigyxCycxvrdmkRaiIjVU8c1ZwVNZ6OvQgbcv EhDIU4HLC/IeZT9Q78lwgZ26eC8OIKVeUBXJL6rcX9IfF0d5WMo1ny5So2z7fU+ebfXj wg5A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Ns84jhjwh80AF+NgTsU5G8wYY3x4+c5K/NhAZhbbc6Q=; b=HaT+ekETVhmW9oAPfOZE7akt3O5BUbt5rKqCsdaZXTt1a9zwNsGoGMZftgs55HN/6L REQT/DQmpf7p/empmnSnqgNTRpMlR+p+a1DHBGAPE29rLEIDBWtJjs6XBs9wJqfhItKJ fvCTXZowaNREhcRRismN+kirYF+41ToQ+E4jBGK+GIu5DuC1BZf8XhetDndxefi8E+bH jVAx6MtKs1O7PGJo+8vurEpwbVXMCL1P4Yn7oGrGGXDdwx7u1MSHGofdBr2AKR56Yz4s IyX5n8uWcNNTpFwjpdPGmhBkTgG1oL6pqtksWNj2lISCe+5OzHELbxo8YXv1zk5JuHdy u4GA== X-Gm-Message-State: ANhLgQ2b9q6mwgDcsOaul9M+MDMijQLKXPiznEf2f50i+otPvMVxQG4G sds04rN1r476bMpLSirKk35CmekklCUxGFXqvh7BIQ== X-Received: by 2002:ac2:44a5:: with SMTP id c5mr3541720lfm.4.1582927994221; Fri, 28 Feb 2020 14:13:14 -0800 (PST) MIME-Version: 1.0 References: <20200221021002.18795-1-yamada.masahiro@socionext.com> <20200221021002.18795-3-yamada.masahiro@socionext.com> In-Reply-To: <20200221021002.18795-3-yamada.masahiro@socionext.com> From: Linus Walleij Date: Fri, 28 Feb 2020 23:13:03 +0100 Message-ID: Subject: Re: [PATCH 3/3] dt-bingings: pinctrl: Convert UniPhier pin controller to json-schema To: Masahiro Yamada Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Rob Herring , Frank Rowand , "David S. Miller" , Greg Kroah-Hartman , Jonathan Cameron , Mark Rutland , Mauro Carvalho Chehab , Linux ARM , "open list:GPIO SUBSYSTEM" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Feb 21, 2020 at 3:10 AM Masahiro Yamada wrote: > Convert the UniPhier pin controller binding to DT schema format. > > Signed-off-by: Masahiro Yamada Nice! > In the original .txt file, there is a description: > The UniPhier pinctrl should be a subnode of a "syscon" compatible node > > I did not figure out how to represent (or check) it in dt-schema. > I just moved it to a comment line in 'examples'. > If there is a better way, please let me know. There is no way to do that AFAICT, we are checking nodes from one node and downwards, never upwards. The syscon needs to have its own binding file: if it has another specific compatible such as compatible = "foo", "syscon"; then for the DT bindings for foo, make sure to add this subnode as optional/compulsory if you want to tie up the whole thing. > -Required properties: > -- compatible: should be one of the following: > - "socionext,uniphier-ld4-pinctrl" - for LD4 SoC > - "socionext,uniphier-pro4-pinctrl" - for Pro4 SoC > - "socionext,uniphier-sld8-pinctrl" - for sLD8 SoC > - "socionext,uniphier-pro5-pinctrl" - for Pro5 SoC > - "socionext,uniphier-pxs2-pinctrl" - for PXs2 SoC > - "socionext,uniphier-ld6b-pinctrl" - for LD6b SoC > - "socionext,uniphier-ld11-pinctrl" - for LD11 SoC > - "socionext,uniphier-ld20-pinctrl" - for LD20 SoC > - "socionext,uniphier-pxs3-pinctrl" - for PXs3 SoC But: > + soc-glue@5f800000 { > + compatible = "socionext,uniphier-pro4-soc-glue", "simple-mfd", "syscon"; > + reg = <0x5f800000 0x2000>; > + > + pinctrl: pinctrl { > + compatible = "socionext,uniphier-pro4-pinctrl"; > + }; > + }; It looks like you want to check also for "simple-mfd" and "syscon" following after the enum (two consts) It seems you want to check that reg is there. It seems the subnode pinctrl is also compulsory. All of this have examples in example-schema.yaml IIRC. Yours, Linus Walleij