From: Francesco Dolcini <francesco@dolcini.it>
To: Alexandru Ardelean <alex@shruggie.ro>
Cc: Francesco Dolcini <francesco@dolcini.it>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
devicetree@vger.kernel.org, adrien.grassein@gmail.com,
andrzej.hajda@intel.com, neil.armstrong@linaro.org,
rfoss@kernel.org, Laurent.pinchart@ideasonboard.com,
jonas@kwiboo.se, jernej.skrabec@gmail.com, airlied@gmail.com,
daniel@ffwll.ch, maarten.lankhorst@linux.intel.com,
mripard@kernel.org, tzimmermann@suse.de, robh@kernel.org,
krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
stefan.eichenberger@toradex.com, francesco.dolcini@toradex.com,
marius.muresan@mxt.ro, irina.muresan@mxt.ro
Subject: Re: [PATCH 1/2] drm/bridge: lt8912b: add support for P/N pin swap
Date: Wed, 3 Apr 2024 08:52:58 +0200 [thread overview]
Message-ID: <Zgz8yvj2gSNk-0KU@gaggiata.pivistrello.it> (raw)
In-Reply-To: <CAH3L5Qr-sT+Q9ZvNSxHKwVMr8-3fU5WPvvaccEWWH49x7oWMyQ@mail.gmail.com>
On Wed, Apr 03, 2024 at 09:32:41AM +0300, Alexandru Ardelean wrote:
> I did it like this, because I don't have a board with the P/N in the
You use this 'P/N' both here and in the binding document, to me this is
just too generic and confusing.
Just use some wording that people familiar with the topic can easily undestand,
the Lontium datasheet uses MIPI RX DP/DN, MIPI DSI DP/DN would also work fine
for me, or at least DP/DN that is the working used on some MIPI documentation.
This comment applies to both the changes in the driver and the binding.
Thanks,
Francesco
prev parent reply other threads:[~2024-04-03 6:53 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-02 10:59 [PATCH 1/2] drm/bridge: lt8912b: add support for P/N pin swap Alexandru Ardelean
2024-04-02 10:59 ` [PATCH 2/2] dt-bindings: display: bridge: lt8912b: document 'lontium,pn-swap' property Alexandru Ardelean
2024-04-02 18:06 ` Conor Dooley
2024-04-03 6:16 ` Alexandru Ardelean
2024-04-03 16:02 ` Conor Dooley
2024-04-07 20:31 ` [PATCH 2/2] dt-bindings: display: bridge: lt8912b: document 'lontium, pn-swap' property Dmitry Baryshkov
2024-04-11 7:23 ` Alexandru Ardelean
2024-04-02 16:53 ` [PATCH 1/2] drm/bridge: lt8912b: add support for P/N pin swap Francesco Dolcini
2024-04-03 6:32 ` Alexandru Ardelean
2024-04-03 6:52 ` Francesco Dolcini [this message]
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=Zgz8yvj2gSNk-0KU@gaggiata.pivistrello.it \
--to=francesco@dolcini.it \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=adrien.grassein@gmail.com \
--cc=airlied@gmail.com \
--cc=alex@shruggie.ro \
--cc=andrzej.hajda@intel.com \
--cc=conor+dt@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=francesco.dolcini@toradex.com \
--cc=irina.muresan@mxt.ro \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=marius.muresan@mxt.ro \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=robh@kernel.org \
--cc=stefan.eichenberger@toradex.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