From: Frieder Schrempf <firstname.lastname@example.org> To: Marek Vasut <email@example.com>, Adam Ford <firstname.lastname@example.org> Cc: "Dong Aisheng" <email@example.com>, devicetree <firstname.lastname@example.org>, "Abel Vesa" <email@example.com>, "Fabio Estevam" <firstname.lastname@example.org>, "Guido Günther" <email@example.com>, "Rob Herring" <firstname.lastname@example.org>, "NXP Linux Team" <email@example.com>, "Shawn Guo" <firstname.lastname@example.org>, arm-soc <email@example.com>, "Lucas Stach" <firstname.lastname@example.org> Subject: Re: [PATCH 1/5] Documentation: bindings: clk: Add bindings for i.MX8MM BLK_CTL Date: Mon, 30 Nov 2020 12:47:12 +0100 [thread overview] Message-ID: <email@example.com> (raw) In-Reply-To: <firstname.lastname@example.org> 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 <email@example.com> wrote: >>> >>> On Wed, Oct 7, 2020 at 3:03 PM Marek Vasut <firstname.lastname@example.org> wrote: >>>> >>>> On 10/7/20 9:52 PM, Adam Ford wrote: >>>>> On Sun, Oct 4, 2020 at 12:53 AM Marek Vasut <email@example.com> 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? Thanks, Frieder
next prev parent reply other threads:[~2020-11-30 11:48 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 [this message] 2020-11-30 15:43 ` Adam Ford 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 \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --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).