All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael Walle" <mwalle@kernel.org>
To: "AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Alexandre Mergnat" <amergnat@baylibre.com>,
	<chunkuang.hu@kernel.org>
Cc: <robh@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
	<conor+dt@kernel.org>, <p.zabel@pengutronix.de>,
	<airlied@gmail.com>, <daniel@ffwll.ch>,
	<maarten.lankhorst@linux.intel.com>, <mripard@kernel.org>,
	<tzimmermann@suse.de>, <matthias.bgg@gmail.com>,
	<shawn.sung@mediatek.com>, <yu-chang.lee@mediatek.com>,
	<ck.hu@mediatek.com>, <jitao.shi@mediatek.com>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<dri-devel@lists.freedesktop.org>,
	<linux-mediatek@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>, <wenst@chromium.org>,
	<kernel@collabora.com>
Subject: Re: [PATCH v2 0/3] drm/mediatek: Add support for OF graphs
Date: Mon, 06 May 2024 12:02:16 +0200	[thread overview]
Message-ID: <D12H4GDJJEUF.1Y91H9RMUIX20@kernel.org> (raw)
In-Reply-To: <608fdbde-ad06-45ec-9771-18aa9f002f2d@collabora.com>

Hi Angelo,

On Tue Apr 30, 2024 at 1:33 PM CEST, AngeloGioacchino Del Regno wrote:
> >> This series was tested on MT8195 Cherry Tomato and on MT8395 Radxa
> >> NIO-12L with both hardcoded paths, OF graph support and partially
> >> hardcoded paths (meaning main display through OF graph and external
> >> display hardcoded, because of OVL_ADAPTOR).
> > 
> > Is that make sense for you to add the DTS changes of these boards into this serie ?
> > I asked because, IMHO, that could help to understand the serie.
> > 
>
> Yes and no... but I imagine that you're asking this because you're trying to
> prepare something with a different SoC+board(s) combination :-)
>
> In that case, I'm preventively sorry because what follows here is not 100%
> perfectly tidy yet as I didn't mean to send the devicetree commits upstream
> before this series got picked....
>
> ... but there you go - I'm sure that you won't mind and that the example will
> be more than good enough for you.

I've tested this series with the DSI0 output and it works. Nice! No
need for my DSI0 patch for the MT8395 anymore.

But I can't get it to work with the DisplayPort output, that is the
dp_intf1/dp_tx interface. I don' know how the pipeline have to look
like. The functional spec seems to be ambiguous on this. The text
seem to refer to the second vdosys but there is also a diagram where
you can use the first vdosys and dsc0. If you have any pointers for
me, I'm all ears :)

-michael


WARNING: multiple messages have this Message-ID (diff)
From: "Michael Walle" <mwalle@kernel.org>
To: "AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Alexandre Mergnat" <amergnat@baylibre.com>,
	<chunkuang.hu@kernel.org>
Cc: <robh@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
	<conor+dt@kernel.org>, <p.zabel@pengutronix.de>,
	<airlied@gmail.com>, <daniel@ffwll.ch>,
	<maarten.lankhorst@linux.intel.com>, <mripard@kernel.org>,
	<tzimmermann@suse.de>, <matthias.bgg@gmail.com>,
	<shawn.sung@mediatek.com>, <yu-chang.lee@mediatek.com>,
	<ck.hu@mediatek.com>, <jitao.shi@mediatek.com>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<dri-devel@lists.freedesktop.org>,
	<linux-mediatek@lists.infradead.org>,
	<linux-arm-kernel@lists.infradead.org>, <wenst@chromium.org>,
	<kernel@collabora.com>
Subject: Re: [PATCH v2 0/3] drm/mediatek: Add support for OF graphs
Date: Mon, 06 May 2024 12:02:16 +0200	[thread overview]
Message-ID: <D12H4GDJJEUF.1Y91H9RMUIX20@kernel.org> (raw)
In-Reply-To: <608fdbde-ad06-45ec-9771-18aa9f002f2d@collabora.com>

Hi Angelo,

On Tue Apr 30, 2024 at 1:33 PM CEST, AngeloGioacchino Del Regno wrote:
> >> This series was tested on MT8195 Cherry Tomato and on MT8395 Radxa
> >> NIO-12L with both hardcoded paths, OF graph support and partially
> >> hardcoded paths (meaning main display through OF graph and external
> >> display hardcoded, because of OVL_ADAPTOR).
> > 
> > Is that make sense for you to add the DTS changes of these boards into this serie ?
> > I asked because, IMHO, that could help to understand the serie.
> > 
>
> Yes and no... but I imagine that you're asking this because you're trying to
> prepare something with a different SoC+board(s) combination :-)
>
> In that case, I'm preventively sorry because what follows here is not 100%
> perfectly tidy yet as I didn't mean to send the devicetree commits upstream
> before this series got picked....
>
> ... but there you go - I'm sure that you won't mind and that the example will
> be more than good enough for you.

I've tested this series with the DSI0 output and it works. Nice! No
need for my DSI0 patch for the MT8395 anymore.

But I can't get it to work with the DisplayPort output, that is the
dp_intf1/dp_tx interface. I don' know how the pipeline have to look
like. The functional spec seems to be ambiguous on this. The text
seem to refer to the second vdosys but there is also a diagram where
you can use the first vdosys and dsc0. If you have any pointers for
me, I'm all ears :)

-michael

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2024-05-06 10:02 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-09 12:02 [PATCH v2 0/3] drm/mediatek: Add support for OF graphs AngeloGioacchino Del Regno
2024-04-09 12:02 ` AngeloGioacchino Del Regno
2024-04-09 12:02 ` [PATCH v2 1/3] dt-bindings: display: mediatek: Add OF graph support for board path AngeloGioacchino Del Regno
2024-04-09 12:02   ` AngeloGioacchino Del Regno
2024-04-09 15:20   ` Dmitry Baryshkov
2024-04-09 15:20     ` Dmitry Baryshkov
2024-04-09 15:41     ` AngeloGioacchino Del Regno
2024-04-09 15:41       ` AngeloGioacchino Del Regno
2024-04-09 15:45       ` Dmitry Baryshkov
2024-04-09 15:45         ` Dmitry Baryshkov
2024-04-19  7:40         ` AngeloGioacchino Del Regno
2024-04-19  7:40           ` AngeloGioacchino Del Regno
2024-04-10 19:03   ` Rob Herring
2024-04-10 19:03     ` Rob Herring
2024-04-19  7:44     ` AngeloGioacchino Del Regno
2024-04-19  7:44       ` AngeloGioacchino Del Regno
2024-04-09 12:02 ` [PATCH v2 2/3] dt-bindings: arm: mediatek: mmsys: " AngeloGioacchino Del Regno
2024-04-09 12:02   ` AngeloGioacchino Del Regno
2024-04-10 19:15   ` Rob Herring
2024-04-10 19:15     ` Rob Herring
2024-04-19  7:54     ` AngeloGioacchino Del Regno
2024-04-19  7:54       ` AngeloGioacchino Del Regno
2024-04-25  2:23   ` CK Hu (胡俊光)
2024-04-25  2:23     ` CK Hu (胡俊光)
2024-04-25  2:23     ` CK Hu (胡俊光)
2024-05-02  8:50     ` AngeloGioacchino Del Regno
2024-05-02  8:50       ` AngeloGioacchino Del Regno
2024-05-07  6:59       ` CK Hu (胡俊光)
2024-05-07  6:59         ` CK Hu (胡俊光)
2024-05-07  6:59         ` CK Hu (胡俊光)
2024-05-07 14:07         ` AngeloGioacchino Del Regno
2024-05-07 14:07           ` AngeloGioacchino Del Regno
2024-05-08  7:19           ` CK Hu (胡俊光)
2024-05-08  7:19             ` CK Hu (胡俊光)
2024-05-08  7:19             ` CK Hu (胡俊光)
2024-05-08 13:03             ` AngeloGioacchino Del Regno
2024-05-08 13:03               ` AngeloGioacchino Del Regno
2024-05-09  5:42               ` CK Hu (胡俊光)
2024-05-09  5:42                 ` CK Hu (胡俊光)
2024-05-09  5:42                 ` CK Hu (胡俊光)
2024-05-09  9:27                 ` AngeloGioacchino Del Regno
2024-05-09  9:27                   ` AngeloGioacchino Del Regno
2024-05-10  9:34                   ` CK Hu (胡俊光)
2024-05-10  9:34                     ` CK Hu (胡俊光)
2024-05-10  9:34                     ` CK Hu (胡俊光)
2024-05-10 10:14                     ` AngeloGioacchino Del Regno
2024-05-10 10:14                       ` AngeloGioacchino Del Regno
2024-05-13  6:15                       ` CK Hu (胡俊光)
2024-05-13  6:15                         ` CK Hu (胡俊光)
2024-05-13  6:15                         ` CK Hu (胡俊光)
2024-05-13 13:44                         ` AngeloGioacchino Del Regno
2024-05-13 13:44                           ` AngeloGioacchino Del Regno
2024-05-13 16:28                           ` Conor Dooley
2024-05-13 16:28                             ` Conor Dooley
2024-05-16  7:27                         ` AngeloGioacchino Del Regno
2024-05-16  7:27                           ` AngeloGioacchino Del Regno
2024-04-09 12:02 ` [PATCH v2 3/3] drm/mediatek: Implement OF graphs support for display paths AngeloGioacchino Del Regno
2024-04-09 12:02   ` AngeloGioacchino Del Regno
2024-05-06  9:11   ` Michael Walle
2024-05-06  9:11     ` Michael Walle
2024-05-06 12:58     ` AngeloGioacchino Del Regno
2024-05-06 12:58       ` AngeloGioacchino Del Regno
2024-04-30 10:17 ` [PATCH v2 0/3] drm/mediatek: Add support for OF graphs Alexandre Mergnat
2024-04-30 10:17   ` Alexandre Mergnat
2024-04-30 11:33   ` AngeloGioacchino Del Regno
2024-04-30 11:33     ` AngeloGioacchino Del Regno
2024-05-02 16:53     ` Alexandre Mergnat
2024-05-02 16:53       ` Alexandre Mergnat
2024-05-14  9:46       ` Alexandre Mergnat
2024-05-14  9:46         ` Alexandre Mergnat
2024-05-14 12:00         ` AngeloGioacchino Del Regno
2024-05-14 12:00           ` AngeloGioacchino Del Regno
2024-05-06 10:02     ` Michael Walle [this message]
2024-05-06 10:02       ` Michael Walle
2024-05-06 10:56       ` AngeloGioacchino Del Regno
2024-05-06 10:56         ` AngeloGioacchino Del Regno
2024-05-06 11:22         ` AngeloGioacchino Del Regno
2024-05-06 11:22           ` AngeloGioacchino Del Regno
2024-05-06 13:17           ` Michael Walle
2024-05-06 13:17             ` Michael Walle
2024-05-06 14:18             ` AngeloGioacchino Del Regno
2024-05-06 14:18               ` AngeloGioacchino Del Regno

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=D12H4GDJJEUF.1Y91H9RMUIX20@kernel.org \
    --to=mwalle@kernel.org \
    --cc=airlied@gmail.com \
    --cc=amergnat@baylibre.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=chunkuang.hu@kernel.org \
    --cc=ck.hu@mediatek.com \
    --cc=conor+dt@kernel.org \
    --cc=daniel@ffwll.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jitao.shi@mediatek.com \
    --cc=kernel@collabora.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mripard@kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=robh@kernel.org \
    --cc=shawn.sung@mediatek.com \
    --cc=tzimmermann@suse.de \
    --cc=wenst@chromium.org \
    --cc=yu-chang.lee@mediatek.com \
    /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.