The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Frank Li <Frank.li@nxp.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: slongerbeam@gmail.com, p.zabel@pengutronix.de, rmfrfs@gmail.com,
	martin.kepplinger@puri.sm, kernel@puri.sm,
	linux-media@vger.kernel.org, imx@lists.linux.dev,
	linux-kernel@vger.kernel.org, mchehab@kernel.org,
	Hans Verkuil <hans@jjverkuil.nl>
Subject: Re: [PATCH 1/1] MAINTAINERS: media: nxp: Add Frank Li as reviewer for nxp SoC
Date: Fri, 5 Sep 2025 14:11:02 -0400	[thread overview]
Message-ID: <aLsntkMstrdDBjXi@lizhi-Precision-Tower-5810> (raw)
In-Reply-To: <20250905163903.GB21773@pendragon.ideasonboard.com>

On Fri, Sep 05, 2025 at 06:39:03PM +0200, Laurent Pinchart wrote:
> On Fri, Sep 05, 2025 at 12:35:33PM -0400, Frank Li wrote:
> > On Fri, Sep 05, 2025 at 06:04:53PM +0200, Laurent Pinchart wrote:
> > > On Thu, Sep 04, 2025 at 12:17:20PM -0400, Frank Li wrote:
> > > > Add Frank Li as reviewer for nxp SoC.
> > >
> > > Great idea, and thank you for volunteering ! I really appreciate your
> > > help with reviews so far, and I have already mentioned it in a
> > > conversation with another SoC vendor as a model they should follow.
> > >
> > > I'd mention media drivers here though, unless you want to volunteer to
> > > review patches for the whole SoC :-) Maybe something along the lines of
> > >
> > > ----
> > > MAINTAINERS: Add Frank Li as reviewer for NXP media drivers
> > >
> > > Frank has been reviewing NXP media drivers for a while, and is
> > > volunteering to continue. Add him as a reviewer for the IMX media
> > > drivers.
> > >
> > > Also add the NXP i.MX platform mailing list (imx@lists.linux.dev).
> > > ----
> > >
> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> >
> > Do I need repost it with updated commit message? Or you can handle it when
> > pick patche?
>
> No need to repost if you're fine with the proposal. I've applied the
> patch to my tree already to avoid forgetting about it. It would be
> easier if Hans or Mauro (on CC) could pick it though, to avoid sending a
> pull request with a single patch. Otherwise I'll send a pull request.

Thanks, I am fine with you suggested commit message.

Frank

>
> > > > Add nxp open source mail list imx@lists.linux.dev.
> > > >
> > > > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > > > ---
> > > > I am work on kernel community for the while. And upstreamed/reviewed:
> > > > cee06ca7a6748 media: imx8mq-mipi-csi2: Fix error code in imx8mq_mipi_csi_parse_dt()
> > > > 51ad3b570ea7b media: imx-jpeg: Account for data_offset when getting image address
> > > > 642b70d526ab8 media: imx8mq-mipi-csi2: Add support for i.MX8QXP
> > > > 382d53e9cefb3 media: imx8mq-mipi-csi2: Add imx8mq_plat_data for different compatible strings
> > > > 859278460faa4 media: imx8-isi: Add support for i.MX8QM and i.MX8QXP
> > > > dee8521f69874 media: nxp: imx8-isi: Use dev_err_probe() to simplify code
> > > > 73a40554f979e media: nxp: imx8-isi: Remove redundant check for dma_set_mask_and_coherent()
> > > > 60b8de2b9b4be media: nxp: imx8-isi: Use devm_clk_bulk_get_all() to fetch clocks
> > > > 66ede6d71d4e8 media: nxp: imx8-isi: Remove unused offset in mxc_isi_reg and use BIT() macro for mask
> > > > 2021b8d51cdb5 media: nxp: imx8-isi: Allow num_sources to be greater than num_sink
> > > > 038d27acf987c media: imx-mipi-csis: Use CSI-2 data type macros from mipi-csi2.h
> > > > fd5b6cd730676 media: imx-jpeg: Check decoding is ongoing for motion-jpeg
> > > > f65fbf8c3d671 media: imx-jpeg: Change the pattern size to 128x64
> > > > 7500bb9cf164e media: imx-jpeg: Cleanup after an allocation error
> > > > faa8051b128f4 media: imx-jpeg: Reset slot data pointers when freed
> > > > 46e9c092f850b media: imx-jpeg: Move mxc_jpeg_free_slot_data() ahea
> > > >
> > > > On going thread:
> > > > 5 patches, https://lore.kernel.org/imx/20250729-imx8qxp_pcam-v4-1-4dfca4ed2f87@nxp.com/
> > > > 32 patches, https://lore.kernel.org/imx/20250808-95_cam-v2-0-4b29fa6919a7@nxp.com/
> > > > 2 patches, https://lore.kernel.org/imx/aLbdRdS3TmLHjW+Q@lizhi-Precision-Tower-5810/
> > > >
> > > > I am volunteer to review media related patches so add myself to MAINTIANER
> > > > so I can get notified when related patches post.
> > > > ---
> > > >  MAINTAINERS | 4 ++++
> > > >  1 file changed, 4 insertions(+)
> > > >
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index b3ca0ba848879..2884a5216b427 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -15327,6 +15327,8 @@ F:	drivers/media/pci/ddbridge/*
> > > >  MEDIA DRIVERS FOR FREESCALE IMX
> > > >  M:	Steve Longerbeam <slongerbeam@gmail.com>
> > > >  M:	Philipp Zabel <p.zabel@pengutronix.de>
> > > > +R:	Frank Li <Frank.Li@nxp.com>
> > > > +L:	imx@lists.linux.dev
> > > >  L:	linux-media@vger.kernel.org
> > > >  S:	Maintained
> > > >  T:	git git://linuxtv.org/media.git
> > > > @@ -15341,6 +15343,8 @@ M:	Rui Miguel Silva <rmfrfs@gmail.com>
> > > >  M:	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > >  M:	Martin Kepplinger <martin.kepplinger@puri.sm>
> > > >  R:	Purism Kernel Team <kernel@puri.sm>
> > > > +R:	Frank Li <Frank.Li@nxp.com>
> > > > +L:	imx@lists.linux.dev
> > > >  L:	linux-media@vger.kernel.org
> > > >  S:	Maintained
> > > >  T:	git git://linuxtv.org/media.git
>
> --
> Regards,
>
> Laurent Pinchart

      reply	other threads:[~2025-09-05 18:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-04 16:17 [PATCH 1/1] MAINTAINERS: media: nxp: Add Frank Li as reviewer for nxp SoC Frank Li
2025-09-05 15:11 ` Philipp Zabel
2025-09-05 16:04 ` Laurent Pinchart
2025-09-05 16:35   ` Frank Li
2025-09-05 16:39     ` Laurent Pinchart
2025-09-05 18:11       ` Frank Li [this message]

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=aLsntkMstrdDBjXi@lizhi-Precision-Tower-5810 \
    --to=frank.li@nxp.com \
    --cc=hans@jjverkuil.nl \
    --cc=imx@lists.linux.dev \
    --cc=kernel@puri.sm \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=martin.kepplinger@puri.sm \
    --cc=mchehab@kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=rmfrfs@gmail.com \
    --cc=slongerbeam@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox