linux-kernel - Re: [PATCH v13 01/10] dt-bindings: mediatek,dp: Add Display Port binding

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20220712180632.GH1823936-robh@kernel.org>
Date:   Tue, 12 Jul 2022 12:06:32 -0600
From:   Rob Herring <robh@...nel.org>
To:     Rex-BC Chen <rex-bc.chen@...iatek.com>
Cc:     chunkuang.hu@...nel.org, p.zabel@...gutronix.de, daniel@...ll.ch,
        krzysztof.kozlowski+dt@...aro.org, mripard@...nel.org,
        tzimmermann@...e.de, matthias.bgg@...il.com, deller@....de,
        airlied@...ux.ie, msp@...libre.com, granquet@...libre.com,
        jitao.shi@...iatek.com, wenst@...omium.org,
        angelogioacchino.delregno@...labora.com, ck.hu@...iatek.com,
        liangxu.xu@...iatek.com, dri-devel@...ts.freedesktop.org,
        linux-mediatek@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-fbdev@...r.kernel.org,
        Project_Global_Chrome_Upstream_Group@...iatek.com
Subject: Re: [PATCH v13 01/10] dt-bindings: mediatek,dp: Add Display Port
 binding

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@...libre.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