From: Krzysztof Kozlowski <krzk@kernel.org>
To: Mike Looijmans <mike.looijmans@topic.nl>,
Conor Dooley <conor@kernel.org>
Cc: dri-devel@lists.freedesktop.org,
Andrzej Hajda <andrzej.hajda@intel.com>,
Conor Dooley <conor+dt@kernel.org>,
David Airlie <airlied@gmail.com>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Jonas Karlman <jonas@kwiboo.se>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Neil Armstrong <neil.armstrong@linaro.org>,
Rob Herring <robh@kernel.org>, Robert Foss <rfoss@kernel.org>,
Simona Vetter <simona@ffwll.ch>,
Thomas Zimmermann <tzimmermann@suse.de>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: drm/bridge: ti-tmds181: Add TI TMDS181 and SN65DP159 bindings
Date: Tue, 19 Aug 2025 09:51:06 +0200 [thread overview]
Message-ID: <9fba4917-a24f-4fee-8f1a-7509a0bc542e@kernel.org> (raw)
In-Reply-To: <f4ec7690-322e-493a-b346-7b9560ac0616@topic.nl>
On 19/08/2025 09:46, Mike Looijmans wrote:
>>> +
>>> +properties:
>>> + compatible:
>>> + enum:
>>> + - ti,tmds181
>>> + - ti,sn65dp159
>> The driver contains:
>> + { .compatible = "ti,tmds181", },
>> + { .compatible = "ti,sn65dp159", },
>> + {}
>> so why is a fallback compatible not suitable here?
>
> I don't understand the question. The two are slightly different chips,
Your driver says they are compatible. No one said the same, but compatible.
> so it makes sense to describe that in the DT.
Compatible devices should use fallback. There is plenty of examples (90%
of all binding files?) including example-schema describing this.
>
Best regards,
Krzysztof
next prev parent reply other threads:[~2025-08-19 7:51 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1b153bce-a66a-45ee-a5c6-963ea6fb1c82.949ef384-8293-46b8-903f-40a477c056ae.c8608b8f-6aa9-4650-b701-3b3ffa0b2b1d@emailsignatures365.codetwo.com>
2025-08-12 14:51 ` [PATCH 0/2] drm: bridge: Add TI tmds181 and sn65dp159 driver (RFC) Mike Looijmans
2025-08-12 14:51 ` [PATCH 1/2] dt-bindings: drm/bridge: ti-tmds181: Add TI TMDS181 and SN65DP159 bindings Mike Looijmans
2025-08-12 17:58 ` Conor Dooley
2025-08-19 7:46 ` Mike Looijmans
2025-08-19 7:51 ` Krzysztof Kozlowski [this message]
2025-08-19 8:26 ` Mike Looijmans
2025-08-19 17:22 ` Conor Dooley
2025-08-20 6:44 ` Krzysztof Kozlowski
2025-08-20 9:37 ` Mike Looijmans
2025-08-20 11:35 ` Krzysztof Kozlowski
2025-08-20 12:10 ` Mike Looijmans
2025-08-20 18:20 ` Conor Dooley
2025-08-12 14:51 ` [PATCH 2/2] drm: bridge: Add TI tmds181 and sn65dp159 driver Mike Looijmans
2025-08-13 7:39 ` kernel test robot
2025-08-19 10:26 ` Dmitry Baryshkov
2025-08-19 11:19 ` Mike Looijmans
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=9fba4917-a24f-4fee-8f1a-7509a0bc542e@kernel.org \
--to=krzk@kernel.org \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=conor+dt@kernel.org \
--cc=conor@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mike.looijmans@topic.nl \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=robh@kernel.org \
--cc=simona@ffwll.ch \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.