From: Nicolas Ferre <nicolas.ferre@microchip.com> To: Emil Velikov <emil.l.velikov@gmail.com>, Ezequiel Garcia <ezequiel@collabora.com>, devicetree <devicetree@vger.kernel.org>, "Rob Herring" <robh+dt@kernel.org>, Ludovic Desroches <ludovic.desroches@microchip.com>, Alexandre Belloni <alexandre.belloni@bootlin.com> Cc: Philipp Zabel <p.zabel@pengutronix.de>, <linux-media@vger.kernel.org>, linux-rockchip <linux-rockchip@lists.infradead.org> Subject: Re: [PATCH v2 00/10] Microship SAMA5D4 VPU support et al Date: Wed, 24 Mar 2021 14:44:14 +0100 [thread overview] Message-ID: <50b3b4b3-6c5b-3f1e-3499-c88574ce9f74@microchip.com> (raw) In-Reply-To: <CACvgo51uNyQgzGdW=f-0wxvjv-+OD1p7E4DJXRzu1GvnAHbcCQ@mail.gmail.com> Emil, On 24/03/2021 at 13:49, Emil Velikov wrote: > On Tue, 16 Mar 2021 at 17:23, Ezequiel Garcia <ezequiel@collabora.com> wrote: > >> On Thu, 2021-03-11 at 15:40 +0000, Emil Velikov wrote: >>> Emil Velikov (10): >>> media: hantro: use G1_REG_INTERRUPT directly for the mpeg2 >>> media: hantro: imx: reuse MB_DIM define >>> media: hantro: imx: remove duplicate dec_base init >>> media: hantro: imx: remove unused include >>> media: hantro: introduce hantro_g1.c for common API >> >> For patches 1-5: >> >> Reviewed-by: Ezequiel Garcia <ezequiel@collabora.com> >> >>> media: dt-bindings: Document SAMA5D4 VDEC bindings >> >> This one need to be reviewed by DT maintainers, I think. >> > Rob can you help with this one? > >>> media: hantro: add initial SAMA5D4 support >> >> For patch 7: >> >> Reviewed-by: Ezequiel Garcia <ezequiel@collabora.com> >> >>> ARM: dts: sama5d4: enable Hantro G1 VDEC >>> ARM: configs: at91: sama5: update with savedefconfig >>> ARM: configs: at91: sama5: enable the Hantro G1 engine >>> >> >> These need review from Microchip maintainers. >> > Alexandre, Ludovic, Nicolas > Do you have any input of the patches or series as a whole? The patch series looks good to me. If needed on patches that we don't take ourselves, you can add my: Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com> Now, when we have the tag from Rob, how to coordinate these different pieces? Will it go through the media git tree? Will we benefit from a stable branch to share or will we just have to wait for the driver to hit Mainline before adding the defconfig and DT patches? > If you prefer we can drop the last two patches for the defconfig. I've > included those for posterity. No strong opinion on my side, except that defconfig stuff might be better handled in at91 + arm-soc trees because we'll have other changes to queue for 5.13. > Thanks for the review Eze. > Would you recommend that I resend the series with your R-B or it's > better to wait for feedback from others? Thanks a lot for this nice work. Best regards, -- Nicolas Ferre
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@microchip.com> To: Emil Velikov <emil.l.velikov@gmail.com>, Ezequiel Garcia <ezequiel@collabora.com>, devicetree <devicetree@vger.kernel.org>, "Rob Herring" <robh+dt@kernel.org>, Ludovic Desroches <ludovic.desroches@microchip.com>, Alexandre Belloni <alexandre.belloni@bootlin.com> Cc: Philipp Zabel <p.zabel@pengutronix.de>, <linux-media@vger.kernel.org>, linux-rockchip <linux-rockchip@lists.infradead.org> Subject: Re: [PATCH v2 00/10] Microship SAMA5D4 VPU support et al Date: Wed, 24 Mar 2021 14:44:14 +0100 [thread overview] Message-ID: <50b3b4b3-6c5b-3f1e-3499-c88574ce9f74@microchip.com> (raw) In-Reply-To: <CACvgo51uNyQgzGdW=f-0wxvjv-+OD1p7E4DJXRzu1GvnAHbcCQ@mail.gmail.com> Emil, On 24/03/2021 at 13:49, Emil Velikov wrote: > On Tue, 16 Mar 2021 at 17:23, Ezequiel Garcia <ezequiel@collabora.com> wrote: > >> On Thu, 2021-03-11 at 15:40 +0000, Emil Velikov wrote: >>> Emil Velikov (10): >>> media: hantro: use G1_REG_INTERRUPT directly for the mpeg2 >>> media: hantro: imx: reuse MB_DIM define >>> media: hantro: imx: remove duplicate dec_base init >>> media: hantro: imx: remove unused include >>> media: hantro: introduce hantro_g1.c for common API >> >> For patches 1-5: >> >> Reviewed-by: Ezequiel Garcia <ezequiel@collabora.com> >> >>> media: dt-bindings: Document SAMA5D4 VDEC bindings >> >> This one need to be reviewed by DT maintainers, I think. >> > Rob can you help with this one? > >>> media: hantro: add initial SAMA5D4 support >> >> For patch 7: >> >> Reviewed-by: Ezequiel Garcia <ezequiel@collabora.com> >> >>> ARM: dts: sama5d4: enable Hantro G1 VDEC >>> ARM: configs: at91: sama5: update with savedefconfig >>> ARM: configs: at91: sama5: enable the Hantro G1 engine >>> >> >> These need review from Microchip maintainers. >> > Alexandre, Ludovic, Nicolas > Do you have any input of the patches or series as a whole? The patch series looks good to me. If needed on patches that we don't take ourselves, you can add my: Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com> Now, when we have the tag from Rob, how to coordinate these different pieces? Will it go through the media git tree? Will we benefit from a stable branch to share or will we just have to wait for the driver to hit Mainline before adding the defconfig and DT patches? > If you prefer we can drop the last two patches for the defconfig. I've > included those for posterity. No strong opinion on my side, except that defconfig stuff might be better handled in at91 + arm-soc trees because we'll have other changes to queue for 5.13. > Thanks for the review Eze. > Would you recommend that I resend the series with your R-B or it's > better to wait for feedback from others? Thanks a lot for this nice work. Best regards, -- Nicolas Ferre _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip
next prev parent reply other threads:[~2021-03-24 13:47 UTC|newest] Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-03-11 15:40 [PATCH v2 00/10] Microship SAMA5D4 VPU support et al Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 01/10] media: hantro: use G1_REG_INTERRUPT directly for the mpeg2 Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 02/10] media: hantro: imx: reuse MB_DIM define Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 03/10] media: hantro: imx: remove duplicate dec_base init Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 04/10] media: hantro: imx: remove unused include Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 05/10] media: hantro: introduce hantro_g1.c for common API Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 06/10] media: dt-bindings: Document SAMA5D4 VDEC bindings Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-24 15:17 ` Rob Herring 2021-03-24 15:17 ` Rob Herring 2021-03-25 14:14 ` Emil Velikov 2021-03-25 14:14 ` Emil Velikov 2021-03-26 14:28 ` Ezequiel Garcia 2021-03-26 14:28 ` Ezequiel Garcia 2021-03-31 17:42 ` Emil Velikov 2021-03-31 17:42 ` Emil Velikov 2021-03-26 14:33 ` Ezequiel Garcia 2021-03-26 14:33 ` Ezequiel Garcia 2021-03-11 15:40 ` [PATCH v2 07/10] media: hantro: add initial SAMA5D4 support Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 08/10] ARM: dts: sama5d4: enable Hantro G1 VDEC Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 09/10] ARM: configs: at91: sama5: update with savedefconfig Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` [PATCH v2 10/10] ARM: configs: at91: sama5: enable the Hantro G1 engine Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-11 15:40 ` Emil Velikov 2021-03-16 17:23 ` [PATCH v2 00/10] Microship SAMA5D4 VPU support et al Ezequiel Garcia 2021-03-16 17:23 ` Ezequiel Garcia 2021-03-24 12:49 ` Emil Velikov 2021-03-24 12:49 ` Emil Velikov 2021-03-24 13:44 ` Nicolas Ferre [this message] 2021-03-24 13:44 ` Nicolas Ferre 2021-03-25 8:48 ` Alexandre Belloni 2021-03-25 8:48 ` Alexandre Belloni 2021-03-25 14:22 ` Emil Velikov 2021-03-25 14:22 ` Emil Velikov 2021-03-29 9:53 ` Nicolas Ferre 2021-03-29 9:53 ` Nicolas Ferre 2021-03-31 17:36 ` Emil Velikov 2021-03-31 17:36 ` Emil Velikov 2021-03-25 15:50 ` Nicolas Ferre 2021-03-25 15:50 ` Nicolas Ferre 2021-03-24 19:58 ` Nicolas Dufresne 2021-03-24 19:58 ` Nicolas Dufresne
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=50b3b4b3-6c5b-3f1e-3499-c88574ce9f74@microchip.com \ --to=nicolas.ferre@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=devicetree@vger.kernel.org \ --cc=emil.l.velikov@gmail.com \ --cc=ezequiel@collabora.com \ --cc=linux-media@vger.kernel.org \ --cc=linux-rockchip@lists.infradead.org \ --cc=ludovic.desroches@microchip.com \ --cc=p.zabel@pengutronix.de \ --cc=robh+dt@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: linkBe 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.