devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Adam Ford <aford173@gmail.com>
To: Frieder Schrempf <frieder.schrempf@kontron.de>
Cc: "Marek Vasut" <marex@denx.de>,
	"Dong Aisheng" <aisheng.dong@nxp.com>,
	devicetree <devicetree@vger.kernel.org>,
	"Abel Vesa" <abel.vesa@nxp.com>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Guido Günther" <agx@sigxcpu.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"NXP Linux Team" <linux-imx@nxp.com>,
	"Shawn Guo" <shawnguo@kernel.org>,
	arm-soc <linux-arm-kernel@lists.infradead.org>,
	"Lucas Stach" <l.stach@pengutronix.de>
Subject: Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL
Date: Mon, 30 Nov 2020 09:43:01 -0600	[thread overview]
Message-ID: <CAHCN7x+oz9hoNfNYytGD++EbtAkceer7q359qoOP=pwUvaD9mQ@mail.gmail.com> (raw)
In-Reply-To: <aa5e86be-07a8-878c-54ff-2c2c7cdd9ef7@kontron.de>

On Mon, Nov 30, 2020 at 5:47 AM Frieder Schrempf
<frieder.schrempf@kontron.de> wrote:
>
> Hi,
>
> On 07.10.20 22:50, Marek Vasut wrote:
> > On 10/7/20 10:17 PM, Adam Ford wrote:
> >> On Wed, Oct 7, 2020 at 3:08 PM Adam Ford <aford173@gmail.com> wrote:
> >>>
> >>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <marex@denx.de> wrote:
> >>>>
> >>>> On 10/7/20 9:52 PM, Adam Ford wrote:
> >>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <marex@denx.de> wrote:
> >>>>>>
> >>>>>> Add the i.MX8MM BLK_CTL compatible string to the list.
> >>>> [...]
> >>>>>> ---
> >>>>>>   Documentation/devicetree/bindings/clock/fsl,imx-blk-ctl.yaml | 1 +
> >>>>>>   1 file changed, 1 insertion(+)
> >>>>>>
> >>>>>
> >>>>> Is there a DTSI change part of this patch?  I was going to try to test
> >>>>> it, but  I am not seeing a change to the imx8mm.dtsi, and I am not
> >>>>> sure where to put the node.
> >>>>
> >>>> There are in fact quite a few other pieces you need to have in place,
> >>>> this patchset in itself is not particularly useful, it is just infra for
> >>>> the LCDIF and MIPI DSIM block control. You might want to wait until they
> >>>> all land in next and test that result.
> >>>
> >>> I have several patches in place, the GPCv2, this block driver,
> >>> enabling GPU DT node, I'm also working on the DSIM patch you posted.
> >>> I was hoping to test them all together and reply to the various
> >>> threads with tested-by.  I also want to get my device tree stuff ready
> >>> on the beacon boards so when everything lands, I can post DTS updates
> >>> to enable the LCDIF, DSI, and the HDMI bridge.
> >>>
> >>> If you have a repo somewhere that has all these combined, I can just
> >>> work on the final layer to enable the device tree plumbing on my
> >>> board.  I just need the imx8mm.dtsi changes for this, DSIM, and the
> >>> LCDIF so I can finish the task.
> >>
> >> On that note, I also have a i.MX8M Nano board which is similar to my
> >> 8MM.  If I understood the 8MM clock block driver better, I hope to
> >> adapt your changes for the Nano too.  Once the GPCv2 driver is
> >> accepted, I was also going to look at updating it to support the Nano
> >> as well which also has the same DSIM and LCDIF as the 8MM as well and
> >> a better GPU than the Mini but lacking the VPU.
> >
> > I don't have a branch, but I sent you the collected patches off-list.
> >
>
> I would also be interested in the patch collection for BLK_CTL, DSIM,
> etc. Marek, would you mind sending me those, too?
>
> Adam, did you already set up a branch and do some tests with the full stack?

Frieder,

I have been monitoring some of the activity on the BLK_CTL.  It seems
like there is some disagreement on how to connect the power domain
controller with the BLK_CTL.  Someone reported that it causes a hang
on the 8MP, so until that gets resolved, I doubt we'll be able to use
the display system.  Some of the DSIM changes happening are being
pushed back for further changes, so it seems like having the full
integration might be a while.

adam
>
> Thanks,
> Frieder

  reply	other threads:[~2020-11-30 15:44 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-03 22:45 Marek Vasut
2020-10-03 22:45 ` [PATCH 2/5] dt-bindings: clock: imx8mm: Add media blk_ctl clock IDs Marek Vasut
2020-10-06 21:12   ` Rob Herring
2020-10-03 22:45 ` [PATCH 3/5] dt-bindings: reset: imx8mm: Add media blk_ctl reset IDs Marek Vasut
2020-10-06 21:12   ` Rob Herring
2020-10-06 21:12 ` [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Rob Herring
2020-10-07 19:52 ` Adam Ford
2020-10-07 20:01   ` Marek Vasut
2020-10-07 20:08     ` Adam Ford
2020-10-07 20:17       ` Adam Ford
2020-10-07 20:50         ` Marek Vasut
2020-11-30 11:47           ` Frieder Schrempf
2020-11-30 15:43             ` Adam Ford [this message]
2020-12-10 15:14               ` Frieder Schrempf
2020-12-16 21:24                 ` Tim Harvey
2020-12-22  9:07                   ` Frieder Schrempf
2021-02-04 12:46 ` Adam Ford

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='CAHCN7x+oz9hoNfNYytGD++EbtAkceer7q359qoOP=pwUvaD9mQ@mail.gmail.com' \
    --to=aford173@gmail.com \
    --cc=abel.vesa@nxp.com \
    --cc=agx@sigxcpu.org \
    --cc=aisheng.dong@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=l.stach@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=marex@denx.de \
    --cc=robh+dt@kernel.org \
    --cc=shawnguo@kernel.org \
    --subject='Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL' \
    /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

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).