{"id":1467052,"url":"http://patchwork.ozlabs.org/api/patches/1467052/?format=json","web_url":"http://patchwork.ozlabs.org/project/devicetree-bindings/patch/YHmOvvyxAyOY/fRL@Red/","project":{"id":37,"url":"http://patchwork.ozlabs.org/api/projects/37/?format=json","name":"Devicetree Bindings","link_name":"devicetree-bindings","list_id":"devicetree.vger.kernel.org","list_email":"devicetree@vger.kernel.org","web_url":"","scm_url":"","webscm_url":"","list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"","list_archive_url":null,"date":"2021-04-16T13:18:54","name":"Need help converting usb/faraday,fotg210.yaml to yaml","commit_ref":null,"pull_url":null,"state":"changes-requested","archived":true,"hash":"2f2916ee915e38011848ff60f4fbde837860787a","submitter":{"id":64152,"url":"http://patchwork.ozlabs.org/api/people/64152/?format=json","name":"Corentin Labbe","email":"clabbe.montjoie@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/devicetree-bindings/patch/YHmOvvyxAyOY/fRL@Red/mbox/","series":[{"id":239520,"url":"http://patchwork.ozlabs.org/api/series/239520/?format=json","web_url":"http://patchwork.ozlabs.org/project/devicetree-bindings/list/?series=239520","date":"2021-04-16T13:18:54","name":"Need help converting usb/faraday,fotg210.yaml to yaml","version":1,"mbox":"http://patchwork.ozlabs.org/series/239520/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/1467052/comments/","check":"fail","checks":"http://patchwork.ozlabs.org/api/patches/1467052/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=23.128.96.18; helo=vger.kernel.org;\n envelope-from=devicetree-owner@vger.kernel.org; receiver=)","ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20161025 header.b=amK1DS8U;\n\tdkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [23.128.96.18])\n\tby ozlabs.org (Postfix) with ESMTP id 4FMGyW3J0Dz9sCD\n\tfor ;\n Fri, 16 Apr 2021 23:19:31 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n id S244099AbhDPNTx (ORCPT );\n Fri, 16 Apr 2021 09:19:53 -0400","from lindbergh.monkeyblade.net ([23.128.96.19]:55680 \"EHLO\n lindbergh.monkeyblade.net\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n with ESMTP id S243838AbhDPNTY (ORCPT\n ); Fri, 16 Apr 2021 09:19:24 -0400","from mail-wr1-x436.google.com (mail-wr1-x436.google.com\n [IPv6:2a00:1450:4864:20::436])\n by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 35A18C061574;\n Fri, 16 Apr 2021 06:18:58 -0700 (PDT)","by mail-wr1-x436.google.com with SMTP id j5so25720134wrn.4;\n Fri, 16 Apr 2021 06:18:58 -0700 (PDT)","from Red ([2a01:cb1d:3d5:a100:264b:feff:fe03:2806])\n by smtp.googlemail.com with ESMTPSA id\n k22sm10512972wrh.5.2021.04.16.06.18.56\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 16 Apr 2021 06:18:56 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20161025;\n h=date:from:to:cc:subject:message-id:mime-version:content-disposition;\n bh=42+196C4PV/WgalEyYCCLfdIC87N4rvne8jso4N4z3M=;\n b=amK1DS8UnJHRBsTKWylu6z+VuhXAVU/mDKeVhFeNvZt+s4+L/GvpwAOrYjJqoqGHf/\n tj8usFiQsi/fmBRSsNK7uRTSyTOPVuocQQzuVRb7f2jeNBrNbns+fe4nMe0EmYU9t8Z9\n I8Sk7sngXBfBfBl18XI3QW7ddIqt0bdB7v+0M3mpm/87lAffzengXWpHreU82ghISPWR\n 9Mbo5bweohmIEg3k4G+ezvW7Cf5UScnmGhzAjrM8vIKr8Ofv2rkNgrd/kzpu0BJO3TPp\n hTLN4GRvU27PyMoPOuINjjphvKRcS3X7DR0cHL2okLh29xxE466/SXFWHI1ZypE78VFP\n UzkA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20161025;\n h=x-gm-message-state:date:from:to:cc:subject:message-id:mime-version\n :content-disposition;\n bh=42+196C4PV/WgalEyYCCLfdIC87N4rvne8jso4N4z3M=;\n b=VxQhkMqNky4EKd7tmhazFpfLZKfPgdzVks63SIRZrbnF8VivFHfyNzHbqesNwlzrP1\n i1LWJy5OqV/wt7dh+3MT1ZhLStJoO8N0VBmmN551W73qmQubSbSE2sLGZZjOLEHvpD0f\n EsjA94fqiS4OC5tVEXD4O2Ws71wkIiELLbSpOqaNNb3XHRhAyNEX5OyOcDuT/11p+JNu\n rvSwFyb6vtCRAJB2UZpUq4kSJflGGynCHTrKXYMyrP6RA/GadaAQQMsCtUtaykf79NAO\n sRQMlr2Vn4ti1qNzOrlSUgxbCLMJPgNhJXbVFD4Eiit9HHf0SXnix5grfTP84BF3M17n\n sKGA==","X-Gm-Message-State":"AOAM531vlMu0e0/AvGIgc7/j5Wyf3U6WQhMYPA2/DeZgMNiDojj3pv49\n S2RwOGcp8ES7dAAaf+UaRu3NiWKfgjw=","X-Google-Smtp-Source":"\n ABdhPJyAIzTQNakUUG3n72laAisHxLyY0sS4rYhhGhkKbfgjpqw/aAK1CQ+ChidZiP1dKAVM7MGWCQ==","X-Received":"by 2002:adf:dc8d:: with SMTP id r13mr8986466wrj.339.1618579137045;\n Fri, 16 Apr 2021 06:18:57 -0700 (PDT)","Date":"Fri, 16 Apr 2021 15:18:54 +0200","From":"Corentin Labbe ","To":"robh+dt@kernel.org, devicetree@vger.kernel.org","Cc":"linux-kernel@vger.kernel.org, linus.walleij@linaro.org","Subject":"Need help converting usb/faraday,fotg210.yaml to yaml","Message-ID":"","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"},"content":"Hello\n\nI am converting Documentation/devicetree/bindings/usb/faraday,fotg210.txt to yaml with the patch attached below.\nBut validating it give me:\nDocumentation/devicetree/bindings/usb/faraday,fotg210.example.dt.yaml: usb@68000000: 'syscon', 'wakeup-source' do not match any of the regexes: 'pinctrl-[0-9]+'\n\nI dont know from where came this pinctrl regex.\nI think this is perhaps due to the if not matched, but still didnt find any solution.\n\nThanks","diff":"--- a/Documentation/devicetree/bindings/usb/faraday,fotg210.txt\n+++ /dev/null\n@@ -1,35 +0,0 @@\n-Faraday FOTG Host controller\n-\n-This OTG-capable USB host controller is found in Cortina Systems\n-Gemini and other SoC products.\n-\n-Required properties:\n-- compatible: should be one of:\n- \"faraday,fotg210\"\n- \"cortina,gemini-usb\", \"faraday,fotg210\"\n-- reg: should contain one register range i.e. start and length\n-- interrupts: description of the interrupt line\n-\n-Optional properties:\n-- clocks: should contain the IP block clock\n-- clock-names: should be \"PCLK\" for the IP block clock\n-\n-Required properties for \"cortina,gemini-usb\" compatible:\n-- syscon: a phandle to the system controller to access PHY registers\n-\n-Optional properties for \"cortina,gemini-usb\" compatible:\n-- cortina,gemini-mini-b: boolean property that indicates that a Mini-B\n- OTG connector is in use\n-- wakeup-source: see power/wakeup-source.txt\n-\n-Example for Gemini:\n-\n-usb@68000000 {\n-\tcompatible = \"cortina,gemini-usb\", \"faraday,fotg210\";\n-\treg = <0x68000000 0x1000>;\n-\tinterrupts = <10 IRQ_TYPE_LEVEL_HIGH>;\n-\tclocks = <&cc 12>;\n-\tclock-names = \"PCLK\";\n-\tsyscon = <&syscon>;\n-\twakeup-source;\n-};\ndiff --git a/Documentation/devicetree/bindings/usb/faraday,fotg210.yaml b/Documentation/devicetree/bindings/usb/faraday,fotg210.yaml\nnew file mode 100644\nindex 000000000000..b5418f29745e\n--- /dev/null\n+++ b/Documentation/devicetree/bindings/usb/faraday,fotg210.yaml\n@@ -0,0 +1,78 @@\n+# SPDX-License-Identifier: GPL-2.0\n+%YAML 1.2\n+---\n+$id: http://devicetree.org/schemas/usb/faraday,fotg210.yaml#\n+$schema: http://devicetree.org/meta-schemas/core.yaml#\n+\n+title: Faraday FOTG Host controller\n+\n+maintainers:\n+ - Linus Walleij \n+\n+description: |\n+ This OTG-capable USB host controller is found in Cortina Systems\n+ Gemini and other SoC products.\n+\n+properties:\n+ compatible:\n+ oneOf:\n+ - const: faraday,fotg210\n+ - items:\n+ - const: cortina,gemini-usb\n+ - const: faraday,fotg210\n+\n+ reg:\n+ maxItems: 1\n+\n+ interrupts:\n+ maxItems: 1\n+\n+ clocks:\n+ maxItems: 1\n+\n+ clock-names:\n+ const: PCLK\n+\n+ resets:\n+ maxItems: 1\n+\n+required:\n+ - compatible\n+ - reg\n+ - interrupts\n+\n+if:\n+ properties:\n+ compatible:\n+ contains:\n+ const: cortina,gemini-usb\n+then:\n+ properties:\n+ syscon:\n+ maxItems: 1\n+ $ref: /schemas/types.yaml#/definitions/phandle\n+ description: Phandle to the system controller to access PHY registers\n+ cortina,gemini-mini-b:\n+ type: boolean\n+ description: boolean property that indicates that a Mini-B OTG connector is in use\n+ wakeup-source:\n+ type: boolean\n+ description: see power/wakeup-source.txt\n+ required:\n+ - syscon\n+\n+additionalProperties: false\n+\n+examples:\n+ - |\n+ #include \n+ usb@68000000 {\n+ compatible = \"cortina,gemini-usb\", \"faraday,fotg210\";\n+ reg = <0x68000000 0x1000>;\n+ interrupts = <10 IRQ_TYPE_LEVEL_HIGH>;\n+ clocks = <&cc 12>;\n+ clock-names = \"PCLK\";\n+ syscon = <&syscon>;\n+ wakeup-source;\n+ };\n+...\n","prefixes":[]}