From: Paul Kocialkowski <paulk@sys-base.io>
To: "Kuba Szczodrzyński" <kuba@szczodrzynski.pl>
Cc: Maxime Ripard <mripard@kernel.org>,
Samuel Holland <samuel@sholland.org>,
Chen-Yu Tsai <wens@csie.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Thomas Zimmermann <tzimmermann@suse.de>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
linux-arm-kernel@lists.infradead.org,
linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, linux-phy@lists.infradead.org,
devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 2/5] drm/sun4i: Support LVDS using MIPI DSI combo D-PHY
Date: Wed, 25 Jun 2025 10:36:28 +0200 [thread overview]
Message-ID: <aFu1DC4eGtYmAuSf@shepard> (raw)
In-Reply-To: <aFutWCScsMNgXUAu@shepard>
[-- Attachment #1: Type: text/plain, Size: 974 bytes --]
On Wed 25 Jun 25, 10:03, Paul Kocialkowski wrote:
> On Fri 21 Feb 25, 17:17, Kuba Szczodrzyński wrote:
> > @@ -183,6 +184,8 @@ static void sun4i_tcon_lvds_set_status(struct sun4i_tcon *tcon,
> > } else {
> > regmap_update_bits(tcon->regs, SUN4I_TCON0_LVDS_IF_REG,
> > SUN4I_TCON0_LVDS_IF_EN, 0);
>
> Does the T113-S3/D1 also have LVDS support in TCON regsiters?
> My understanding is that it only has the combo-phy, so this register write
> should not take place.
>
> I think it would be easier to just move this to a common callback for users of
> the tcon0 lvds phy and always call
Nevermind this comment, the LVDS_IF writes are actually necessary and
independent from the LVDS PHY configuration.
All the best,
Paul
--
Paul Kocialkowski,
Independent contractor - sys-base - https://www.sys-base.io/
Free software developer - https://www.paulk.fr/
Expert in multimedia, graphics and embedded hardware support with Linux.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Paul Kocialkowski <paulk@sys-base.io>
To: "Kuba Szczodrzyński" <kuba@szczodrzynski.pl>
Cc: Maxime Ripard <mripard@kernel.org>,
Samuel Holland <samuel@sholland.org>,
Chen-Yu Tsai <wens@csie.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Thomas Zimmermann <tzimmermann@suse.de>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
linux-arm-kernel@lists.infradead.org,
linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, linux-phy@lists.infradead.org,
devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 2/5] drm/sun4i: Support LVDS using MIPI DSI combo D-PHY
Date: Wed, 25 Jun 2025 10:36:28 +0200 [thread overview]
Message-ID: <aFu1DC4eGtYmAuSf@shepard> (raw)
In-Reply-To: <aFutWCScsMNgXUAu@shepard>
[-- Attachment #1.1: Type: text/plain, Size: 974 bytes --]
On Wed 25 Jun 25, 10:03, Paul Kocialkowski wrote:
> On Fri 21 Feb 25, 17:17, Kuba Szczodrzyński wrote:
> > @@ -183,6 +184,8 @@ static void sun4i_tcon_lvds_set_status(struct sun4i_tcon *tcon,
> > } else {
> > regmap_update_bits(tcon->regs, SUN4I_TCON0_LVDS_IF_REG,
> > SUN4I_TCON0_LVDS_IF_EN, 0);
>
> Does the T113-S3/D1 also have LVDS support in TCON regsiters?
> My understanding is that it only has the combo-phy, so this register write
> should not take place.
>
> I think it would be easier to just move this to a common callback for users of
> the tcon0 lvds phy and always call
Nevermind this comment, the LVDS_IF writes are actually necessary and
independent from the LVDS PHY configuration.
All the best,
Paul
--
Paul Kocialkowski,
Independent contractor - sys-base - https://www.sys-base.io/
Free software developer - https://www.paulk.fr/
Expert in multimedia, graphics and embedded hardware support with Linux.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 112 bytes --]
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
WARNING: multiple messages have this Message-ID (diff)
From: Paul Kocialkowski <paulk@sys-base.io>
To: "Kuba Szczodrzyński" <kuba@szczodrzynski.pl>
Cc: Maxime Ripard <mripard@kernel.org>,
Samuel Holland <samuel@sholland.org>,
Chen-Yu Tsai <wens@csie.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Thomas Zimmermann <tzimmermann@suse.de>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
linux-arm-kernel@lists.infradead.org,
linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org,
linux-riscv@lists.infradead.org, linux-phy@lists.infradead.org,
devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 2/5] drm/sun4i: Support LVDS using MIPI DSI combo D-PHY
Date: Wed, 25 Jun 2025 10:36:28 +0200 [thread overview]
Message-ID: <aFu1DC4eGtYmAuSf@shepard> (raw)
In-Reply-To: <aFutWCScsMNgXUAu@shepard>
[-- Attachment #1.1: Type: text/plain, Size: 974 bytes --]
On Wed 25 Jun 25, 10:03, Paul Kocialkowski wrote:
> On Fri 21 Feb 25, 17:17, Kuba Szczodrzyński wrote:
> > @@ -183,6 +184,8 @@ static void sun4i_tcon_lvds_set_status(struct sun4i_tcon *tcon,
> > } else {
> > regmap_update_bits(tcon->regs, SUN4I_TCON0_LVDS_IF_REG,
> > SUN4I_TCON0_LVDS_IF_EN, 0);
>
> Does the T113-S3/D1 also have LVDS support in TCON regsiters?
> My understanding is that it only has the combo-phy, so this register write
> should not take place.
>
> I think it would be easier to just move this to a common callback for users of
> the tcon0 lvds phy and always call
Nevermind this comment, the LVDS_IF writes are actually necessary and
independent from the LVDS PHY configuration.
All the best,
Paul
--
Paul Kocialkowski,
Independent contractor - sys-base - https://www.sys-base.io/
Free software developer - https://www.paulk.fr/
Expert in multimedia, graphics and embedded hardware support with Linux.
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 161 bytes --]
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2025-06-25 9:19 UTC|newest]
Thread overview: 129+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-21 16:17 [PATCH 0/5] drm/sun4i: Support LVDS on D1s/T113 combo D-PHY Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` [PATCH 1/5] phy: allwinner: phy-sun6i-mipi-dphy: Support LVDS in " Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-06-25 7:56 ` Paul Kocialkowski
2025-06-25 7:56 ` Paul Kocialkowski
2025-06-25 7:56 ` Paul Kocialkowski
2025-02-21 16:17 ` [PATCH 2/5] drm/sun4i: Support LVDS using MIPI DSI " Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-06-25 8:03 ` Paul Kocialkowski
2025-06-25 8:03 ` Paul Kocialkowski
2025-06-25 8:03 ` Paul Kocialkowski
2025-06-25 8:36 ` Paul Kocialkowski [this message]
2025-06-25 8:36 ` Paul Kocialkowski
2025-06-25 8:36 ` Paul Kocialkowski
2025-02-21 16:17 ` [PATCH 3/5] drm/sun4i: Enable LVDS output on sun20i D1s/T113 Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-06-25 8:06 ` Paul Kocialkowski
2025-06-25 8:06 ` Paul Kocialkowski
2025-06-25 8:06 ` Paul Kocialkowski
2025-02-21 16:17 ` [PATCH 4/5] riscv: dts: allwinner: d1s-t113: Add D-PHY to TCON LCD0 Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-06-25 8:09 ` Paul Kocialkowski
2025-06-25 8:09 ` Paul Kocialkowski
2025-06-25 8:09 ` Paul Kocialkowski
2025-02-21 16:17 ` [PATCH 5/5] riscv: dts: allwinner: d1s-t113: Add LVDS0 pins Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-02-21 16:17 ` Kuba Szczodrzyński
2025-06-25 8:24 ` Paul Kocialkowski
2025-06-25 8:24 ` Paul Kocialkowski
2025-06-25 8:24 ` Paul Kocialkowski
2025-06-25 8:35 ` [PATCH 0/5] drm/sun4i: Support LVDS on D1s/T113 combo D-PHY Paul Kocialkowski
2025-06-25 8:35 ` Paul Kocialkowski
2025-06-25 8:35 ` Paul Kocialkowski
2025-11-08 16:53 ` Parthiban
2025-11-08 16:53 ` Parthiban
2025-11-08 16:53 ` Parthiban
2025-11-16 13:18 ` [PATCH v2 0/6] " Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` [PATCH v2 1/6] phy: allwinner: phy-sun6i-mipi-dphy: Support LVDS in " Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` [PATCH v2 2/6] drm/sun4i: Support LVDS using MIPI DSI " Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` [PATCH v2 3/6] drm/sun4i: Enable LVDS output on sun20i D1s/T113 Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` [PATCH v2 4/6] dt-bindings: display: sun4i: Add D1s/T113 combo D-PHY bindings Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:18 ` Kuba Szczodrzyński
2025-11-16 13:25 ` [PATCH v2 5/6] riscv: dts: allwinner: d1s-t113: Add D-PHY to TCON LCD0 Kuba Szczodrzyński
2025-11-16 13:25 ` Kuba Szczodrzyński
2025-11-16 13:25 ` Kuba Szczodrzyński
2025-11-16 13:25 ` [PATCH v2 6/6] riscv: dts: allwinner: d1s-t113: Add LVDS0 pins Kuba Szczodrzyński
2025-11-16 13:25 ` Kuba Szczodrzyński
2025-11-16 13:25 ` Kuba Szczodrzyński
2025-11-16 13:46 ` [PATCH v3 0/6] drm/sun4i: Support LVDS on D1s/T113 combo D-PHY Kuba Szczodrzyński
2025-11-16 13:46 ` Kuba Szczodrzyński
2025-11-16 13:46 ` Kuba Szczodrzyński
2025-11-16 13:47 ` [PATCH v3 1/6] phy: allwinner: phy-sun6i-mipi-dphy: Support LVDS in " Kuba Szczodrzyński
2025-11-16 13:47 ` Kuba Szczodrzyński
2025-11-16 13:47 ` Kuba Szczodrzyński
2025-11-20 6:24 ` Parthiban
2025-11-20 6:24 ` Parthiban
2025-11-20 6:24 ` Parthiban
2025-11-22 11:27 ` Kuba Szczodrzyński
2025-11-22 11:27 ` Kuba Szczodrzyński
2025-11-22 11:27 ` Kuba Szczodrzyński
2026-01-22 10:15 ` Parthiban
2026-01-22 10:15 ` Parthiban
2026-01-22 10:15 ` Parthiban
2025-11-16 13:48 ` [PATCH v3 2/6] drm/sun4i: Support LVDS using MIPI DSI " Kuba Szczodrzyński
2025-11-16 13:48 ` Kuba Szczodrzyński
2025-11-16 13:48 ` Kuba Szczodrzyński
2025-11-16 13:48 ` [PATCH v3 3/6] drm/sun4i: Enable LVDS output on sun20i D1s/T113 Kuba Szczodrzyński
2025-11-16 13:48 ` Kuba Szczodrzyński
2025-11-16 13:48 ` Kuba Szczodrzyński
2026-01-22 10:22 ` Parthiban
2026-01-22 10:22 ` Parthiban
2026-01-22 10:22 ` Parthiban
2025-11-16 13:49 ` [PATCH v3 4/6] dt-bindings: display: sun4i: Add D1s/T113 combo D-PHY bindings Kuba Szczodrzyński
2025-11-16 13:49 ` Kuba Szczodrzyński
2025-11-16 13:49 ` Kuba Szczodrzyński
2025-11-17 7:05 ` Krzysztof Kozlowski
2025-11-17 7:05 ` Krzysztof Kozlowski
2025-11-17 7:05 ` Krzysztof Kozlowski
2025-11-17 11:08 ` Kuba Szczodrzyński
2025-11-17 11:08 ` Kuba Szczodrzyński
2025-11-17 11:08 ` Kuba Szczodrzyński
2025-11-17 11:53 ` Krzysztof Kozlowski
2025-11-17 11:53 ` Krzysztof Kozlowski
2025-11-17 11:53 ` Krzysztof Kozlowski
2025-11-17 12:39 ` Kuba Szczodrzyński
2025-11-17 12:39 ` Kuba Szczodrzyński
2025-11-17 12:39 ` Kuba Szczodrzyński
2025-11-16 13:49 ` [PATCH v3 5/6] riscv: dts: allwinner: d1s-t113: Add D-PHY to TCON LCD0 Kuba Szczodrzyński
2025-11-16 13:49 ` Kuba Szczodrzyński
2025-11-16 13:49 ` Kuba Szczodrzyński
2025-11-16 13:50 ` [PATCH v3 6/6] riscv: dts: allwinner: d1s-t113: Add LVDS0 pins Kuba Szczodrzyński
2025-11-16 13:50 ` Kuba Szczodrzyński
2025-11-16 13:50 ` Kuba Szczodrzyński
2025-11-16 14:03 ` [PATCH v3 0/6] drm/sun4i: Support LVDS on D1s/T113 combo D-PHY Chen-Yu Tsai
2025-11-16 14:03 ` Chen-Yu Tsai
2025-11-16 14:03 ` Chen-Yu Tsai
2025-11-16 14:31 ` Kuba Szczodrzyński
2025-11-16 14:31 ` Kuba Szczodrzyński
2025-11-16 14:31 ` Kuba Szczodrzyński
2025-11-16 14:37 ` Chen-Yu Tsai
2025-11-16 14:37 ` Chen-Yu Tsai
2025-11-16 14:37 ` Chen-Yu Tsai
2025-11-16 16:48 ` Kuba Szczodrzyński
2025-11-16 16:48 ` Kuba Szczodrzyński
2025-11-16 16:48 ` Kuba Szczodrzyński
2026-02-07 13:34 ` Parthiban
2026-02-07 13:34 ` Parthiban
2026-02-07 13:34 ` Parthiban
2026-04-01 8:39 ` Parthiban
2026-04-01 8:39 ` Parthiban
2026-04-01 8:39 ` Parthiban
2026-04-01 15:04 ` Kuba Szczodrzyński
2026-04-01 15:04 ` Kuba Szczodrzyński
2026-04-01 15:04 ` Kuba Szczodrzyński
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=aFu1DC4eGtYmAuSf@shepard \
--to=paulk@sys-base.io \
--cc=airlied@gmail.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@szczodrzynski.pl \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=linux-riscv@lists.infradead.org \
--cc=linux-sunxi@lists.linux.dev \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=robh@kernel.org \
--cc=samuel@sholland.org \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
--cc=wens@csie.org \
/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.