[PATCH 1/2] MAINTAINERS: Add myself to imx-pip as reviewer

[PATCH 1/2] MAINTAINERS: Add myself to imx-pip as reviewer

[bod]

From: Bryan O'Donoghue <bod@kernel.org>

As discussed @ the media-summit in Nice this year I'd like to volunteer to
do some review on this driver, mostly as a learning exercise and because
more eyeballs equals hopefully less bugs. Mostly for the learning though.

Signed-off-by: Bryan O'Donoghue <bod@kernel.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index efbf808063e50..ec0743b41a463 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -16027,6 +16027,7 @@ F:	include/uapi/linux/media.h
 
 MEDIA DRIVER FOR FREESCALE IMX PXP
 M:	Philipp Zabel <p.zabel@pengutronix.de>
+R:	Bryan O'Donoghue <bod@kernel.org>
 L:	linux-media@vger.kernel.org
 S:	Maintained
 T:	git git://linuxtv.org/media.git
-- 
2.54.0

[PATCH 2/2] MAINTAINERS: Add myself to dw100 as reviewer

[bod]

From: Bryan O'Donoghue <bod@kernel.org>

To facilitate cross-pollination between one arch and another I'd like to
add myself as reviewer to the dw100. We discussed @ Nice it would be
beneficial to have different maintainers reading/reviewing things not
directly inside of their remit. This driver was mentioned, I'd be happy to
join in.

Signed-off-by: Bryan O'Donoghue <bod@kernel.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index ec0743b41a463..35368c2ff0ca6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -19243,6 +19243,7 @@ F:	drivers/media/platform/nxp/imx8-isi/
 
 NXP i.MX 8MP DW100 V4L2 DRIVER
 M:	Xavier Roumegue <xavier.roumegue@oss.nxp.com>
+R:	Bryan O'Donoghue <bod@kernel.org>
 L:	linux-media@vger.kernel.org
 S:	Maintained
 F:	Documentation/devicetree/bindings/media/nxp,dw100.yaml
-- 
2.54.0

Re: [PATCH 2/2] MAINTAINERS: Add myself to dw100 as reviewer

[Frank Li]

On Mon, Jun 01, 2026 at 04:50:59PM +0100, bod@kernel.org wrote:
> From: Bryan O'Donoghue <bod@kernel.org>
>
> To facilitate cross-pollination between one arch and another I'd like to
> add myself as reviewer to the dw100. We discussed @ Nice it would be
> beneficial to have different maintainers reading/reviewing things not
> directly inside of their remit. This driver was mentioned, I'd be happy to
> join in.
>
> Signed-off-by: Bryan O'Donoghue <bod@kernel.org>

Thanks quick response. See previous patch's comments.

> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ec0743b41a463..35368c2ff0ca6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19243,6 +19243,7 @@ F:	drivers/media/platform/nxp/imx8-isi/
>
>  NXP i.MX 8MP DW100 V4L2 DRIVER
>  M:	Xavier Roumegue <xavier.roumegue@oss.nxp.com>
> +R:	Bryan O'Donoghue <bod@kernel.org>
>  L:	linux-media@vger.kernel.org

Please also add imx@lists.linux.dev

Frank

>  S:	Maintained
>  F:	Documentation/devicetree/bindings/media/nxp,dw100.yaml
> --
> 2.54.0
>

Re: [PATCH 2/2] MAINTAINERS: Add myself to dw100 as reviewer

[Laurent Pinchart]

Hi Bryan,

On Mon, Jun 01, 2026 at 04:50:59PM +0100, bod@kernel.org wrote:
> From: Bryan O'Donoghue <bod@kernel.org>
> 
> To facilitate cross-pollination between one arch and another I'd like to
> add myself as reviewer to the dw100. We discussed @ Nice it would be
> beneficial to have different maintainers reading/reviewing things not
> directly inside of their remit. This driver was mentioned, I'd be happy to
> join in.

Thanks for volunteering :-)

This may however not be the best candidate. The dw100 driver is well
maintained today, by Xavier as well as Stefan, as Ideas on Board makes
active use of it (Stefan, you could also volunteer as a reviewer).

Drivers that I think would most benefit from your reviews would be
imx8mq-mipi-csi2, dw-mipi-csi2rx (technically a Synopsys driver, but
used in NXP SoCs), and the new imx95-csi-formatter driver ([1]).

Other candidates could be imx7-media-csi and imx-mipi-csis, but those
see little activity as they support older hardware. The imx media
staging drivers could also do with some love, but that would be active
development and not just review.

[1] https://lore.kernel.org/20260525-csi_formatter-v8-0-6b646231224b@oss.nxp.com

> Signed-off-by: Bryan O'Donoghue <bod@kernel.org>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ec0743b41a463..35368c2ff0ca6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19243,6 +19243,7 @@ F:	drivers/media/platform/nxp/imx8-isi/
>  
>  NXP i.MX 8MP DW100 V4L2 DRIVER
>  M:	Xavier Roumegue <xavier.roumegue@oss.nxp.com>
> +R:	Bryan O'Donoghue <bod@kernel.org>
>  L:	linux-media@vger.kernel.org
>  S:	Maintained
>  F:	Documentation/devicetree/bindings/media/nxp,dw100.yaml

-- 
Regards,

Laurent Pinchart

Re: [PATCH 2/2] MAINTAINERS: Add myself to dw100 as reviewer

[Bryan O'Donoghue]

On 02/06/2026 09:55, Laurent Pinchart wrote:
> Thanks for volunteering 🙂
> 
> This may however not be the best candidate. The dw100 driver is well
> maintained today, by Xavier as well as Stefan, as Ideas on Board makes
> active use of it (Stefan, you could also volunteer as a reviewer).
> 
> Drivers that I think would most benefit from your reviews would be
> imx8mq-mipi-csi2, dw-mipi-csi2rx (technically a Synopsys driver, but
> used in NXP SoCs), and the new imx95-csi-formatter driver ([1]).
> 
> Other candidates could be imx7-media-csi and imx-mipi-csis, but those
> see little activity as they support older hardware. The imx media
> staging drivers could also do with some love, but that would be active
> development and not just review.
> 
> [1]https://lore.kernel.org/20260525-csi_formatter-v8-0-6b646231224b@oss.nxp.com
> 
>> Signed-off-by: Bryan O'Donoghue<bod@kernel.org>
>> ---
>>   MAINTAINERS | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index ec0743b41a463..35368c2ff0ca6 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -19243,6 +19243,7 @@ F:	drivers/media/platform/nxp/imx8-isi/
>>
>>   NXP i.MX 8MP DW100 V4L2 DRIVER
>>   M:	Xavier Roumegue<xavier.roumegue@oss.nxp.com>
>> +R:	Bryan O'Donoghue<bod@kernel.org>
>>   L:	linux-media@vger.kernel.org
>>   S:	Maintained
>>   F:	Documentation/devicetree/bindings/media/nxp,dw100.yaml
> --
> Regards,
> 
> Laurent Pinchart

Ah, wrong set.

No problem, I'll v2 for the above instead.

---
bod

Re: [PATCH 2/2] MAINTAINERS: Add myself to dw100 as reviewer

[Laurent Pinchart]

On Tue, Jun 02, 2026 at 11:10:24AM +0100, Bryan O'Donoghue wrote:
> On 02/06/2026 09:55, Laurent Pinchart wrote:
> > Thanks for volunteering 🙂
> > 
> > This may however not be the best candidate. The dw100 driver is well
> > maintained today, by Xavier as well as Stefan, as Ideas on Board makes
> > active use of it (Stefan, you could also volunteer as a reviewer).
> > 
> > Drivers that I think would most benefit from your reviews would be
> > imx8mq-mipi-csi2, dw-mipi-csi2rx (technically a Synopsys driver, but
> > used in NXP SoCs), and the new imx95-csi-formatter driver ([1]).
> > 
> > Other candidates could be imx7-media-csi and imx-mipi-csis, but those
> > see little activity as they support older hardware. The imx media
> > staging drivers could also do with some love, but that would be active
> > development and not just review.
> > 
> > [1]https://lore.kernel.org/20260525-csi_formatter-v8-0-6b646231224b@oss.nxp.com
> > 
> >> Signed-off-by: Bryan O'Donoghue<bod@kernel.org>
> >> ---
> >>   MAINTAINERS | 1 +
> >>   1 file changed, 1 insertion(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index ec0743b41a463..35368c2ff0ca6 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -19243,6 +19243,7 @@ F:	drivers/media/platform/nxp/imx8-isi/
> >>
> >>   NXP i.MX 8MP DW100 V4L2 DRIVER
> >>   M:	Xavier Roumegue<xavier.roumegue@oss.nxp.com>
> >> +R:	Bryan O'Donoghue<bod@kernel.org>
> >>   L:	linux-media@vger.kernel.org
> >>   S:	Maintained
> >>   F:	Documentation/devicetree/bindings/media/nxp,dw100.yaml
> > --
> > Regards,
> > 
> > Laurent Pinchart
> 
> Ah, wrong set.
> 
> No problem, I'll v2 for the above instead.

Thank you. Don't get me wrong, your reviews for the dw100 driver would
also be appreciated, but if you have limited time (and I assume we all
do), the above drivers would benefit more from your comments.

-- 
Regards,

Laurent Pinchart

Re: [PATCH 2/2] MAINTAINERS: Add myself to dw100 as reviewer

[Bryan O'Donoghue]

On 02/06/2026 15:26, Laurent Pinchart wrote:
> On Tue, Jun 02, 2026 at 11:10:24AM +0100, Bryan O'Donoghue wrote:
>> On 02/06/2026 09:55, Laurent Pinchart wrote:
>>> Thanks for volunteering 🙂
>>>
>>> This may however not be the best candidate. The dw100 driver is well
>>> maintained today, by Xavier as well as Stefan, as Ideas on Board makes
>>> active use of it (Stefan, you could also volunteer as a reviewer).
>>>
>>> Drivers that I think would most benefit from your reviews would be
>>> imx8mq-mipi-csi2, dw-mipi-csi2rx (technically a Synopsys driver, but
>>> used in NXP SoCs), and the new imx95-csi-formatter driver ([1]).
>>>
>>> Other candidates could be imx7-media-csi and imx-mipi-csis, but those
>>> see little activity as they support older hardware. The imx media
>>> staging drivers could also do with some love, but that would be active
>>> development and not just review.
>>>
>>> [1]https://lore.kernel.org/20260525-csi_formatter-v8-0-6b646231224b@oss.nxp.com
>>>
>>>> Signed-off-by: Bryan O'Donoghue<bod@kernel.org>
>>>> ---
>>>>    MAINTAINERS | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index ec0743b41a463..35368c2ff0ca6 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -19243,6 +19243,7 @@ F:	drivers/media/platform/nxp/imx8-isi/
>>>>
>>>>    NXP i.MX 8MP DW100 V4L2 DRIVER
>>>>    M:	Xavier Roumegue<xavier.roumegue@oss.nxp.com>
>>>> +R:	Bryan O'Donoghue<bod@kernel.org>
>>>>    L:	linux-media@vger.kernel.org
>>>>    S:	Maintained
>>>>    F:	Documentation/devicetree/bindings/media/nxp,dw100.yaml
>>> --
>>> Regards,
>>>
>>> Laurent Pinchart
>>
>> Ah, wrong set.
>>
>> No problem, I'll v2 for the above instead.
> 
> Thank you. Don't get me wrong, your reviews for the dw100 driver would
> also be appreciated, but if you have limited time (and I assume we all
> do), the above drivers would benefit more from your comments.
> 
> --
> Regards,
> 
> Laurent Pinchart

Eh, not a bit of it.

You flagged we should do some cross review. I'm happy to be shown to 
whichever drivers/files it is you mean.

---
bod

Re: [PATCH 1/2] MAINTAINERS: Add myself to imx-pip as reviewer

[Frank Li]

On Mon, Jun 01, 2026 at 04:50:58PM +0100, bod@kernel.org wrote:
> From: Bryan O'Donoghue <bod@kernel.org>

Subjest:
	Add Bryan O'Donoghue to imx-pxp as reviewer
>
> As discussed @ the media-summit in Nice this year I'd like to volunteer to

As discussed at the 2026 media-summit in Nice. I'd ....

> do some review on this driver.

> , mostly as a learning exercise and because
> more eyeballs equals hopefully less bugs. Mostly for the learning though.

Needn't this part.

>
> Signed-off-by: Bryan O'Donoghue <bod@kernel.org>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index efbf808063e50..ec0743b41a463 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -16027,6 +16027,7 @@ F:	include/uapi/linux/media.h
>
>  MEDIA DRIVER FOR FREESCALE IMX PXP
>  M:	Philipp Zabel <p.zabel@pengutronix.de>
> +R:	Bryan O'Donoghue <bod@kernel.org>
>  L:	linux-media@vger.kernel.org

You touch this part, please add also

L:	imx@lists.linux.dev

Frank
>  S:	Maintained
>  T:	git git://linuxtv.org/media.git
> --
> 2.54.0
>