From: Adam Ford <aford173@gmail.com>
To: Marek Vasut <marex@denx.de>
Cc: arm-soc <linux-arm-kernel@lists.infradead.org>,
"Dong Aisheng" <aisheng.dong@nxp.com>,
"Abel Vesa" <abel.vesa@nxp.com>,
devicetree <devicetree@vger.kernel.org>,
"Shawn Guo" <shawnguo@kernel.org>,
"Guido Günther" <agx@sigxcpu.org>,
"Rob Herring" <robh+dt@kernel.org>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Fabio Estevam" <festevam@gmail.com>,
"Lucas Stach" <l.stach@pengutronix.de>
Subject: Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL
Date: Wed, 7 Oct 2020 15:17:45 -0500 [thread overview]
Message-ID: <CAHCN7xKDP02QTNrxg84OvVF=Orcnh_gWj551ZV4bJphysfgu5w@mail.gmail.com> (raw)
In-Reply-To: <CAHCN7xJFsi-zfzHOYJ56s5cx-owRL5EPrXCqzBgfrhrveAfo2A@mail.gmail.com>
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.
adam
>
> adam
WARNING: multiple messages have this Message-ID (diff)
From: Adam Ford <aford173@gmail.com>
To: Marek Vasut <marex@denx.de>
Cc: "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: Wed, 7 Oct 2020 15:17:45 -0500 [thread overview]
Message-ID: <CAHCN7xKDP02QTNrxg84OvVF=Orcnh_gWj551ZV4bJphysfgu5w@mail.gmail.com> (raw)
In-Reply-To: <CAHCN7xJFsi-zfzHOYJ56s5cx-owRL5EPrXCqzBgfrhrveAfo2A@mail.gmail.com>
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.
adam
>
> adam
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-10-07 20:17 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-03 22:45 [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Marek Vasut
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-03 22:45 ` Marek Vasut
2020-10-06 21:12 ` Rob Herring
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-03 22:45 ` Marek Vasut
2020-10-06 21:12 ` Rob Herring
2020-10-06 21:12 ` Rob Herring
2020-10-03 22:45 ` [PATCH 4/5] clk: imx: Fix rewriting of hws by resets in generic blk-ctl driver Marek Vasut
2020-10-03 22:45 ` [PATCH 5/5] clk: imx: Add blk-ctl driver for i.MX8MM Marek Vasut
2020-10-06 21:12 ` [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Rob Herring
2020-10-06 21:12 ` Rob Herring
2020-10-07 19:52 ` Adam Ford
2020-10-07 19:52 ` Adam Ford
2020-10-07 20:01 ` Marek Vasut
2020-10-07 20:01 ` Marek Vasut
2020-10-07 20:08 ` Adam Ford
2020-10-07 20:08 ` Adam Ford
2020-10-07 20:17 ` Adam Ford [this message]
2020-10-07 20:17 ` Adam Ford
2020-10-07 20:50 ` Marek Vasut
2020-10-07 20:50 ` Marek Vasut
2020-11-30 11:47 ` Frieder Schrempf
2020-11-30 11:47 ` Frieder Schrempf
2020-11-30 15:43 ` Adam Ford
2020-11-30 15:43 ` Adam Ford
2020-12-10 15:14 ` Frieder Schrempf
2020-12-10 15:14 ` Frieder Schrempf
2020-12-16 21:24 ` Tim Harvey
2020-12-16 21:24 ` Tim Harvey
2020-12-22 9:07 ` Frieder Schrempf
2020-12-22 9:07 ` Frieder Schrempf
2021-02-04 12:46 ` Adam Ford
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='CAHCN7xKDP02QTNrxg84OvVF=Orcnh_gWj551ZV4bJphysfgu5w@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=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 \
/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.