From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f53.google.com (mail-wr1-f53.google.com [209.85.221.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 34670157465 for ; Tue, 8 Oct 2024 07:32:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728372730; cv=none; b=higHAKpj/pJxVPLSPT5dksVUmG9bdj+TmC6i3LmXFFLA4n38zvEea4GXQ29XasBejsr7TWSPMzcB2yoMeaW3ZgQvkE1a4aaPI7ncS1F1Stf+VK9V0xilAnFGq7oz2UZ+dujeMsq7CdQ4McrNzM0tdltvKSVfLk12NtOj0yMQRuI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728372730; c=relaxed/simple; bh=taT2GnH8mvuDeg7IRInmxni6+Y04ymstiVvMzK9X3vw=; h=Message-ID:Date:MIME-Version:From:Subject:To:Cc:References: In-Reply-To:Content-Type; b=oCfYCxNNUuQpJ/KJG8dR6xDqpOmdyBk8MeaMJNl58C6AFUYQ99z1wudgoFsu9GaKz1EWl1VGpCbktdgPF8dpavP9Hf1VhcbVWTfeqkAMaWF+NQ/9IicNyC59IhGBOhCGkWXy4jwlTWZmHZZDg1JHYdd9m1yeQqTg+LtvCc0KGQU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=xpzl1cI6; arc=none smtp.client-ip=209.85.221.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="xpzl1cI6" Received: by mail-wr1-f53.google.com with SMTP id ffacd0b85a97d-37ccfbbd467so3754719f8f.0 for ; Tue, 08 Oct 2024 00:32:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1728372726; x=1728977526; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:organization:autocrypt :content-language:references:cc:to:subject:reply-to:from:user-agent :mime-version:date:message-id:from:to:cc:subject:date:message-id :reply-to; bh=eQ3e9GN5oT4dEQ5C0dmBdYvJDkmnHPy8M4gnBpGOCuk=; b=xpzl1cI6GRSDqXKB1QsonGZGElqv6x0ENQjxlp9xGRKoMaoUJnOWAaVm1lNhMX1rh4 DwG/aVENxSY+eXFg7/VrrojkUP4j2NszUEBxcjRvaIO9bN/ZDyS6FXHPslYqOC5bxGJ1 HYkcO9HVmp8D4Wj2UytYAK5EzAOWHKosrSV33UDDAwdZWRMsT/yb/poycLeYekbsFmRe 9pfshomRVtQU2hymOtw04JNBBjkoN+EAmJuJTOUU344WyNFIHBDNC7ZpIIXo7mrW07wZ Ml42sJsAl1vKyWfLkE6niSzxtBp3qXfhMTXAj6BgFou+cqoir07vKER7sti5volXsPOO 5G1w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728372726; x=1728977526; h=content-transfer-encoding:in-reply-to:organization:autocrypt :content-language:references:cc:to:subject:reply-to:from:user-agent :mime-version:date:message-id:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to; bh=eQ3e9GN5oT4dEQ5C0dmBdYvJDkmnHPy8M4gnBpGOCuk=; b=Hk+XbJvii2NV+jSReSdb4a7RrdrReYEFthtsrZNzY2iEyexCwY7OEYLsPv1ObkqakN waBAGW9rBHRX62ID8RP2dWtP82TO+QHSI/815d0RHLZ0wO4To0W8udiJB0iI1OCqgk6S 4pYFTtTdyYDovxrf4DBM+sSYTqfsU6KQyyD2pyovLq99pik1sT9pw7AvRt44mot3H8NJ mqEwcc7m6T/82N15lykSaI1Zmb8wzIyHtcXGEp6cHgvFgzIyC5H3OGhlXSAB/Bcj3b2c aqtXwf4N3NcpHcsY1AsOTyhjf6zv75IewQ1qcZa2YtzkwJtZzTL5oMvPzfYFJy6RsIfE t4XA== X-Forwarded-Encrypted: i=1; AJvYcCWeBHkHG9XRJFdKwHBZoASQ9UEgd9tz6Tq5RdPGPiHGmuajH978R2nMGtnjMU/lBtXX3o3SJobfajEp@vger.kernel.org X-Gm-Message-State: AOJu0YymSURc9mWxoO0IKlEugSUvj9I3wvp2Z1WXGtxqc60Pk/FU1IBC cPek6+PKw/PKCw22t+H8L+4vNexjYr/2gNLlPS9gtLV6t5eNMX2KW//Uw1e26SQ= X-Google-Smtp-Source: AGHT+IFw4OU9sE129MTmVQZwtzejQ0K8o1Icfq6ZclA5XHOehCJfST7Ts2N4ivxajAWQ0s1XGFzZrg== X-Received: by 2002:a05:6000:186f:b0:371:8eb3:603a with SMTP id ffacd0b85a97d-37d0e77a468mr11072487f8f.27.1728372726453; Tue, 08 Oct 2024 00:32:06 -0700 (PDT) Received: from ?IPV6:2a01:e0a:982:cbb0:4595:23ef:4ba2:2d19? ([2a01:e0a:982:cbb0:4595:23ef:4ba2:2d19]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-37d169733c5sm7380859f8f.110.2024.10.08.00.32.05 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 08 Oct 2024 00:32:06 -0700 (PDT) Message-ID: Date: Tue, 8 Oct 2024 09:32:05 +0200 Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: Neil Armstrong Reply-To: neil.armstrong@linaro.org Subject: Re: [PATCH 1/2] dt-bindings: pinctrl: amlogic,meson8-pinctrl-aobus: add reg for aobus pinctrl nodes To: "Rob Herring (Arm)" Cc: linux-arm-kernel@lists.infradead.org, Jerome Brunet , linux-amlogic@lists.infradead.org, Conor Dooley , Linus Walleij , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Krzysztof Kozlowski , Heiner Kallweit , Kevin Hilman , linux-gpio@vger.kernel.org, Martin Blumenstingl References: <20241007-topic-amlogic-arm32-upstream-bindings-fixes-meson8-pinctrl-v1-0-d3d838540033@linaro.org> <20241007-topic-amlogic-arm32-upstream-bindings-fixes-meson8-pinctrl-v1-1-d3d838540033@linaro.org> <172831694232.937294.6889037408108312834.robh@kernel.org> Content-Language: en-US, fr Autocrypt: addr=neil.armstrong@linaro.org; keydata= xsBNBE1ZBs8BCAD78xVLsXPwV/2qQx2FaO/7mhWL0Qodw8UcQJnkrWmgTFRobtTWxuRx8WWP GTjuhvbleoQ5Cxjr+v+1ARGCH46MxFP5DwauzPekwJUD5QKZlaw/bURTLmS2id5wWi3lqVH4 BVF2WzvGyyeV1o4RTCYDnZ9VLLylJ9bneEaIs/7cjCEbipGGFlfIML3sfqnIvMAxIMZrvcl9 qPV2k+KQ7q+aXavU5W+yLNn7QtXUB530Zlk/d2ETgzQ5FLYYnUDAaRl+8JUTjc0CNOTpCeik 80TZcE6f8M76Xa6yU8VcNko94Ck7iB4vj70q76P/J7kt98hklrr85/3NU3oti3nrIHmHABEB AAHNKk5laWwgQXJtc3Ryb25nIDxuZWlsLmFybXN0cm9uZ0BsaW5hcm8ub3JnPsLAkQQTAQoA OwIbIwULCQgHAwUVCgkICwUWAgMBAAIeAQIXgBYhBInsPQWERiF0UPIoSBaat7Gkz/iuBQJk Q5wSAhkBAAoJEBaat7Gkz/iuyhMIANiD94qDtUTJRfEW6GwXmtKWwl/mvqQtaTtZID2dos04 YqBbshiJbejgVJjy+HODcNUIKBB3PSLaln4ltdsV73SBcwUNdzebfKspAQunCM22Mn6FBIxQ GizsMLcP/0FX4en9NaKGfK6ZdKK6kN1GR9YffMJd2P08EO8mHowmSRe/ExAODhAs9W7XXExw UNCY4pVJyRPpEhv373vvff60bHxc1k/FF9WaPscMt7hlkbFLUs85kHtQAmr8pV5Hy9ezsSRa GzJmiVclkPc2BY592IGBXRDQ38urXeM4nfhhvqA50b/nAEXc6FzqgXqDkEIwR66/Gbp0t3+r yQzpKRyQif3OwE0ETVkGzwEIALyKDN/OGURaHBVzwjgYq+ZtifvekdrSNl8TIDH8g1xicBYp QTbPn6bbSZbdvfeQPNCcD4/EhXZuhQXMcoJsQQQnO4vwVULmPGgtGf8PVc7dxKOeta+qUh6+ SRh3vIcAUFHDT3f/Zdspz+e2E0hPV2hiSvICLk11qO6cyJE13zeNFoeY3ggrKY+IzbFomIZY 4yG6xI99NIPEVE9lNBXBKIlewIyVlkOaYvJWSV+p5gdJXOvScNN1epm5YHmf9aE2ZjnqZGoM Mtsyw18YoX9BqMFInxqYQQ3j/HpVgTSvmo5ea5qQDDUaCsaTf8UeDcwYOtgI8iL4oHcsGtUX oUk33HEAEQEAAcLAXwQYAQIACQUCTVkGzwIbDAAKCRAWmrexpM/4rrXiB/sGbkQ6itMrAIfn M7IbRuiSZS1unlySUVYu3SD6YBYnNi3G5EpbwfBNuT3H8//rVvtOFK4OD8cRYkxXRQmTvqa3 3eDIHu/zr1HMKErm+2SD6PO9umRef8V82o2oaCLvf4WeIssFjwB0b6a12opuRP7yo3E3gTCS KmbUuLv1CtxKQF+fUV1cVaTPMyT25Od+RC1K+iOR0F54oUJvJeq7fUzbn/KdlhA8XPGzwGRy 4zcsPWvwnXgfe5tk680fEKZVwOZKIEuJC3v+/yZpQzDvGYJvbyix0lHnrCzq43WefRHI5XTT QbM0WUIBIcGmq38+OgUsMYu4NzLu7uZFAcmp6h8g Organization: Linaro In-Reply-To: <172831694232.937294.6889037408108312834.robh@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 07/10/2024 18:02, Rob Herring (Arm) wrote: > > On Mon, 07 Oct 2024 16:16:25 +0200, Neil Armstrong wrote: >> While converting the Amlogic Meson pinctrl binding to dt-schema, the mandatory >> reg property for Meson8/b aobus pinctrl node was ommited, fix this to finally >> pass all bindings checks. >> >> Fixes: 94df03a45276 ("dt-bindings: pinctrl: Convert Amlogic Meson pinctrl binding") >> Signed-off-by: Neil Armstrong >> --- >> .../pinctrl/amlogic,meson8-pinctrl-aobus.yaml | 22 +++++++++++++++++++--- >> 1 file changed, 19 insertions(+), 3 deletions(-) >> > > My bot found errors running 'make dt_binding_check' on your patch: > > yamllint warnings/errors: > > dtschema/dtc warnings/errors: > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/pinctrl/amlogic,meson8-pinctrl-aobus.example.dtb: pinctrl: 'reg' is a required property > from schema $id: http://devicetree.org/schemas/pinctrl/amlogic,meson8-pinctrl-aobus.yaml# > > doc reference errors (make refcheckdocs): > > See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20241007-topic-amlogic-arm32-upstream-bindings-fixes-meson8-pinctrl-v1-1-d3d838540033@linaro.org > > The base for the series is generally the latest rc1. A different dependency > should be noted in *this* patch. > > If you already ran 'make dt_binding_check' and didn't see the above > error(s), then make sure 'yamllint' is installed and dt-schema is up to > date: > > pip3 install dtschema --upgrade > > Please check and re-submit after running the above command yourself. Note > that DT_SCHEMA_FILES can be set to your schema file to speed up checking > your schema. However, it must be unset to test all examples with your schema. > Right, I thought I tested those bindings, but seems I missed something... Thanks, Neil