From: Rob Herring <robh@kernel.org>
To: Rex-BC Chen <rex-bc.chen@mediatek.com>
Cc: chunkuang.hu@kernel.org, p.zabel@pengutronix.de, daniel@ffwll.ch,
krzysztof.kozlowski+dt@linaro.org, mripard@kernel.org,
tzimmermann@suse.de, matthias.bgg@gmail.com, deller@gmx.de,
airlied@linux.ie, msp@baylibre.com, granquet@baylibre.com,
jitao.shi@mediatek.com, wenst@chromium.org,
angelogioacchino.delregno@collabora.com, ck.hu@mediatek.com,
liangxu.xu@mediatek.com, dri-devel@lists.freedesktop.org,
linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-fbdev@vger.kernel.org,
Project_Global_Chrome_Upstream_Group@mediatek.com
Subject: Re: [PATCH v13 01/10] dt-bindings: mediatek,dp: Add Display Port binding
Date: Tue, 12 Jul 2022 12:06:32 -0600 [thread overview]
Message-ID: <20220712180632.GH1823936-robh@kernel.org> (raw)
In-Reply-To: <d8b6f7d9b79608c9a533042f714869219ad067be.camel@mediatek.com>
On Tue, Jul 05, 2022 at 05:40:27PM +0800, Rex-BC Chen wrote:
> On Fri, 2022-07-01 at 14:29 -0600, Rob Herring wrote:
> > On Fri, Jul 01, 2022 at 02:27:59PM +0800, Bo-Chen Chen wrote:
> > > From: Markus Schneider-Pargmann <msp@baylibre.com>
> > >
> > > This controller is present on several mediatek hardware. Currently
> > > mt8195 and mt8395 have this controller without a functional
> > > difference,
> > > so only one compatible field is added.
> > >
> > > The controller can have two forms, as a normal display port and as
> > > an
> > > embedded display port.
> >
> > I'm sure you answered this before, but I'll keep asking until the
> > information is contained within this patch. Otherwise, I won't
> > remember.
> > Is there a h/w difference in the 2 blocks? Different registers? Why
> > can't you just look at what the output is connected to?
> >
>
> Hello Rob,
>
> Thanks for your review.
> Yes, it's two different hw for edp and dp and they have different
> registers base address.
Multiple instances *always* have different base addresses. That's not a
difference.
I probably asked before on what the difference is between edp and dp
variants if anything? If the blocks have some difference then, different
compatibles makes sense. But you need to explain the difference or I'll
just be asking again...
Rob
next prev parent reply other threads:[~2022-07-12 18:06 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-01 6:27 [PATCH v13 00/10] drm/mediatek: Add MT8195 DisplayPort driver Bo-Chen Chen
2022-07-01 6:27 ` [PATCH v13 01/10] dt-bindings: mediatek,dp: Add Display Port binding Bo-Chen Chen
2022-07-01 20:29 ` Rob Herring
2022-07-05 9:40 ` Rex-BC Chen
2022-07-12 18:06 ` Rob Herring [this message]
2022-07-01 6:28 ` [PATCH v13 02/10] drm/edid: Convert cea_sad helper struct to kernelDoc Bo-Chen Chen
2022-07-01 6:28 ` [PATCH v13 03/10] drm/edid: Add cea_sad helpers for freq/length Bo-Chen Chen
2022-07-01 6:28 ` [PATCH v13 04/10] video/hdmi: Add audio_infoframe packing for DP Bo-Chen Chen
2022-07-01 6:28 ` [PATCH v13 05/10] drm/mediatek: Add MT8195 Embedded DisplayPort driver Bo-Chen Chen
2022-07-01 8:09 ` AngeloGioacchino Del Regno
2022-07-07 2:21 ` CK Hu
2022-07-12 8:10 ` Rex-BC Chen
2022-07-07 2:35 ` CK Hu
2022-07-07 4:11 ` CK Hu
2022-07-07 5:07 ` CK Hu
2022-07-07 5:14 ` CK Hu
2022-07-12 6:57 ` Rex-BC Chen
2022-07-07 6:04 ` CK Hu
2022-07-07 6:14 ` CK Hu
2022-07-07 6:26 ` CK Hu
2022-07-07 7:50 ` CK Hu
2022-07-07 7:57 ` CK Hu
2022-07-07 8:00 ` CK Hu
2022-07-12 6:49 ` Rex-BC Chen
2022-07-01 6:28 ` [PATCH v13 06/10] drm/mediatek: Add MT8195 External DisplayPort support Bo-Chen Chen
2022-07-01 8:14 ` AngeloGioacchino Del Regno
2022-07-05 9:51 ` Rex-BC Chen
2022-07-01 6:28 ` [PATCH v13 07/10] drm/mediatek: add hpd debounce Bo-Chen Chen
2022-07-01 6:28 ` [PATCH v13 08/10] drm/mediatek: set monitor to DP_SET_POWER_D3 to avoid garbage Bo-Chen Chen
2022-07-01 6:28 ` [PATCH v13 09/10] drm/mediatek: DP audio support for MT8195 Bo-Chen Chen
2022-07-01 8:19 ` AngeloGioacchino Del Regno
2022-07-01 6:28 ` [PATCH v13 10/10] drm/mediatek: fix no audio when resolution change Bo-Chen Chen
2022-07-01 8:24 ` 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=20220712180632.GH1823936-robh@kernel.org \
--to=robh@kernel.org \
--cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
--cc=airlied@linux.ie \
--cc=angelogioacchino.delregno@collabora.com \
--cc=chunkuang.hu@kernel.org \
--cc=ck.hu@mediatek.com \
--cc=daniel@ffwll.ch \
--cc=deller@gmx.de \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=granquet@baylibre.com \
--cc=jitao.shi@mediatek.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=liangxu.xu@mediatek.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=mripard@kernel.org \
--cc=msp@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=rex-bc.chen@mediatek.com \
--cc=tzimmermann@suse.de \
--cc=wenst@chromium.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 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).