From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Tony Lindgren <tony@atomide.com>
Cc: Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <rfoss@kernel.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>, Simha BN <simhavcs@gmail.com>,
Sam Ravnborg <sam@ravnborg.org>,
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Michael Walle <mwalle@kernel.org>,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v3 03/10] dt-bindings: display: bridge: tc358775: Add support for tc358765
Date: Mon, 12 Feb 2024 12:30:12 +0100 [thread overview]
Message-ID: <d6ecc1aa-419d-4f3c-8ba2-b124e5423293@linaro.org> (raw)
In-Reply-To: <20240212081744.GE5299@atomide.com>
On 12/02/2024 09:17, Tony Lindgren wrote:
> * Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [240212 08:06]:
>> On 11/02/2024 10:51, Tony Lindgren wrote:
>>> The tc358765 is similar to tc358775. The tc358765 just an earlier version
>>> of the hardware, and it's pin and register compatible with tc358775 for
>>> most part.
>>>
>>> From the binding point of view the only difference is that the tc358765
>>> does not have stdby-gpios.
>>>
>>> Signed-off-by: Tony Lindgren <tony@atomide.com>
>>> ---
>>
>> It does not look like you tested the bindings, at least after quick
>> look. Please run `make dt_binding_check` (see
>> Documentation/devicetree/bindings/writing-schema.rst for instructions).
>> Maybe you need to update your dtschema and yamllint.
>
> I did.. But I did not notice that this no longer works:
>
> $ make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> LINT Documentation/devicetree/bindings
> usage: yamllint [-h] [-] [-c CONFIG_FILE | -d CONFIG_DATA] [--list-files] [-f {parsable,standard,colored,github,auto}] [-s] [--no-warnings] [-v] [FILE_OR_DIR ...]
> yamllint: error: one of the arguments FILE_OR_DIR - is required
> CHKDT Documentation/devicetree/bindings/processed-schema.json
> SCHEMA Documentation/devicetree/bindings/processed-schema.json
>
> After removing the ">2&1" from the Makefile, there's some more info:
>
> yamllint: error: one of the arguments FILE_OR_DIR - is required
>
> Where DT_SCHEMA_FILES ends up empty. I guess dt_binding_check needs
> to be now run with just:
>
> $ make dt_binding_check DT_SCHEMA_FILES=toshiba,tc358775.yaml
>
Yes, since June last year. Rob later brought (in July) backwards
compatible way, but apparently something changed now in Makefile.
Anyway full Linux path should be considered deprecated. The same for
dtbs_check.
Best regards,
Krzysztof
next prev parent reply other threads:[~2024-02-12 11:30 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-11 9:51 [PATCH v3 00/10] Improvments for tc358775 with support for tc358765 Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 01/10] dt-bindings: display: bridge: tc358775: make stby gpio optional Tony Lindgren
2024-02-12 8:04 ` Krzysztof Kozlowski
2024-02-11 9:51 ` [PATCH v3 02/10] dt-bindings: display: bridge: tc358775: Add data-lanes Tony Lindgren
2024-02-12 8:05 ` Krzysztof Kozlowski
2024-02-11 9:51 ` [PATCH v3 03/10] dt-bindings: display: bridge: tc358775: Add support for tc358765 Tony Lindgren
2024-02-11 14:31 ` Rob Herring
2024-02-12 3:36 ` kernel test robot
2024-02-12 8:06 ` Krzysztof Kozlowski
2024-02-12 8:17 ` Tony Lindgren
2024-02-12 11:30 ` Krzysztof Kozlowski [this message]
2024-02-12 13:51 ` Rob Herring
2024-02-13 6:13 ` Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 04/10] drm/bridge: tc358775: fix support for jeida-18 and jeida-24 Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 05/10] drm/bridge: tc358775: make standby GPIO optional Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 06/10] drm/bridge: tc358775: Get bridge data lanes instead of the DSI host lanes Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 07/10] drm/bridge: tc358775: Add burst and low-power modes Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 08/10] drm/bridge: tc358775: Enable pre_enable_prev_first flag Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 09/10] drm/bridge: tc358775: Add support for tc358765 Tony Lindgren
2024-02-11 9:51 ` [PATCH v3 10/10] drm/bridge: tc358775: Configure hs_rate and lp_rate Tony Lindgren
2024-02-12 11:26 ` Michael Walle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d6ecc1aa-419d-4f3c-8ba2-b124e5423293@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=conor+dt@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=mwalle@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=robh+dt@kernel.org \
--cc=sam@ravnborg.org \
--cc=simhavcs@gmail.com \
--cc=tony@atomide.com \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).