* Re: [GIT PULL FOR v6.19] NXP media drivers changes
[not found] <20251103001640.GA9221@pendragon.ideasonboard.com>
@ 2025-11-03 8:08 ` Hans Verkuil
2025-11-03 10:04 ` Laurent Pinchart
0 siblings, 1 reply; 4+ messages in thread
From: Hans Verkuil @ 2025-11-03 8:08 UTC (permalink / raw)
To: Laurent Pinchart; +Cc: Linux Media Mailing List, Mauro Carvalho Chehab
Hi Laurent,
I noticed you forgot to CC to linux-media (I added that now), but also that all
these patches have a Link: tag. Linus only wants Link tags if there is additional
information through that link. That's not the case here.
I'm dropping the Link tags from the patches in this PR, let me know if you disagree.
Regards,
Hans
On 03/11/2025 01:16, Laurent Pinchart wrote:
> Hi Hans, Mauro,
>
> The following changes since commit 163917839c0eea3bdfe3620f27f617a55fd76302:
>
> MAINTAINERS: Update Daniel Scally's email address (2025-10-29 14:07:02 +0100)
>
> are available in the Git repository at:
>
> https://gitlab.freedesktop.org/linux-media/users/pinchartl.git tags/next-media-nxp-20251103
>
> for you to fetch changes up to a581a7c7d2c354131e184a8770f761c40168dda6:
>
> media: imx8mq-mipi-csi2: drop unused module alias (2025-11-03 01:23:28 +0200)
>
> ----------------------------------------------------------------
> NXP media drivers changes:
>
> - Add Frank Li as a reviewer for NXP media drivers
> - Improve buffer sequence in rkisp1
> - Add i.MX91 support and i.MX93 parallel input support to imx8-isi
> - Drop unused module aliases
>
> ----------------------------------------------------------------
> Alice Yuan (2):
> media: dt-bindings: nxp,imx8-isi: Add i.MX91 ISI compatible string
> media: nxp: imx8-isi: Add parallel camera input support for i.MX93
>
> Frank Li (1):
> MAINTAINERS: Add Frank Li as reviewer for NXP media drivers
>
> Guoniu Zhou (3):
> media: nxp: imx8-isi: Refine code by using helper macro
> media: nxp: imx8-isi: Reorder the platform data
> media: nxp: imx8-isi: Add ISI support for i.MX91
>
> Johan Hovold (3):
> media: imx-mipi-csis: drop unused module alias
> media: imx7-media-csi: drop unused module alias
> media: imx8mq-mipi-csi2: drop unused module alias
>
> Stefan Klug (1):
> media: rkisp1: Improve frame sequence correctness on stats and params buffers
>
> .../devicetree/bindings/media/nxp,imx8-isi.yaml | 13 ++++-
> MAINTAINERS | 4 ++
> drivers/media/platform/nxp/imx-mipi-csis.c | 1 -
> drivers/media/platform/nxp/imx7-media-csi.c | 1 -
> .../media/platform/nxp/imx8-isi/imx8-isi-core.c | 58 +++++++++++++---------
> .../media/platform/nxp/imx8-isi/imx8-isi-core.h | 1 +
> .../media/platform/nxp/imx8-isi/imx8-isi-gasket.c | 22 ++++++--
> drivers/media/platform/nxp/imx8mq-mipi-csi2.c | 1 -
> .../media/platform/rockchip/rkisp1/rkisp1-common.h | 1 +
> .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 27 ++++++++--
> 10 files changed, 92 insertions(+), 37 deletions(-)
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [GIT PULL FOR v6.19] NXP media drivers changes
2025-11-03 8:08 ` [GIT PULL FOR v6.19] NXP media drivers changes Hans Verkuil
@ 2025-11-03 10:04 ` Laurent Pinchart
2025-11-03 10:15 ` Hans Verkuil
0 siblings, 1 reply; 4+ messages in thread
From: Laurent Pinchart @ 2025-11-03 10:04 UTC (permalink / raw)
To: Hans Verkuil; +Cc: Linux Media Mailing List, Mauro Carvalho Chehab
On Mon, Nov 03, 2025 at 09:08:10AM +0100, Hans Verkuil wrote:
> Hi Laurent,
>
> I noticed you forgot to CC to linux-media (I added that now), but also that all
Oops sorry, my bad.
> these patches have a Link: tag. Linus only wants Link tags if there is additional
> information through that link. That's not the case here.
>
> I'm dropping the Link tags from the patches in this PR, let me know if you disagree.
You missed the last episode of the drama :-)
https://lore.kernel.org/all/CAHk-=wj5MATvT-FR8qNpXuuBGiJdjY1kRfhtzuyBSpTKR+=Vtw@mail.gmail.com/
We can use Link: trailers to reference the patch that was picked, and we
need to use the patch.msgid.link domain for that purpose. Adding
[b4]
linkmask = https://patch.msgid.link/%s
to your .gitconfig will instruct b4 to use that domain.
> On 03/11/2025 01:16, Laurent Pinchart wrote:
> > Hi Hans, Mauro,
> >
> > The following changes since commit 163917839c0eea3bdfe3620f27f617a55fd76302:
> >
> > MAINTAINERS: Update Daniel Scally's email address (2025-10-29 14:07:02 +0100)
> >
> > are available in the Git repository at:
> >
> > https://gitlab.freedesktop.org/linux-media/users/pinchartl.git tags/next-media-nxp-20251103
> >
> > for you to fetch changes up to a581a7c7d2c354131e184a8770f761c40168dda6:
> >
> > media: imx8mq-mipi-csi2: drop unused module alias (2025-11-03 01:23:28 +0200)
> >
> > ----------------------------------------------------------------
> > NXP media drivers changes:
> >
> > - Add Frank Li as a reviewer for NXP media drivers
> > - Improve buffer sequence in rkisp1
> > - Add i.MX91 support and i.MX93 parallel input support to imx8-isi
> > - Drop unused module aliases
> >
> > ----------------------------------------------------------------
> > Alice Yuan (2):
> > media: dt-bindings: nxp,imx8-isi: Add i.MX91 ISI compatible string
> > media: nxp: imx8-isi: Add parallel camera input support for i.MX93
> >
> > Frank Li (1):
> > MAINTAINERS: Add Frank Li as reviewer for NXP media drivers
> >
> > Guoniu Zhou (3):
> > media: nxp: imx8-isi: Refine code by using helper macro
> > media: nxp: imx8-isi: Reorder the platform data
> > media: nxp: imx8-isi: Add ISI support for i.MX91
> >
> > Johan Hovold (3):
> > media: imx-mipi-csis: drop unused module alias
> > media: imx7-media-csi: drop unused module alias
> > media: imx8mq-mipi-csi2: drop unused module alias
> >
> > Stefan Klug (1):
> > media: rkisp1: Improve frame sequence correctness on stats and params buffers
> >
> > .../devicetree/bindings/media/nxp,imx8-isi.yaml | 13 ++++-
> > MAINTAINERS | 4 ++
> > drivers/media/platform/nxp/imx-mipi-csis.c | 1 -
> > drivers/media/platform/nxp/imx7-media-csi.c | 1 -
> > .../media/platform/nxp/imx8-isi/imx8-isi-core.c | 58 +++++++++++++---------
> > .../media/platform/nxp/imx8-isi/imx8-isi-core.h | 1 +
> > .../media/platform/nxp/imx8-isi/imx8-isi-gasket.c | 22 ++++++--
> > drivers/media/platform/nxp/imx8mq-mipi-csi2.c | 1 -
> > .../media/platform/rockchip/rkisp1/rkisp1-common.h | 1 +
> > .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 27 ++++++++--
> > 10 files changed, 92 insertions(+), 37 deletions(-)
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [GIT PULL FOR v6.19] NXP media drivers changes
2025-11-03 10:04 ` Laurent Pinchart
@ 2025-11-03 10:15 ` Hans Verkuil
2025-11-03 10:50 ` Laurent Pinchart
0 siblings, 1 reply; 4+ messages in thread
From: Hans Verkuil @ 2025-11-03 10:15 UTC (permalink / raw)
To: Laurent Pinchart; +Cc: Linux Media Mailing List, Mauro Carvalho Chehab
On 03/11/2025 11:04, Laurent Pinchart wrote:
> On Mon, Nov 03, 2025 at 09:08:10AM +0100, Hans Verkuil wrote:
>> Hi Laurent,
>>
>> I noticed you forgot to CC to linux-media (I added that now), but also that all
>
> Oops sorry, my bad.
>
>> these patches have a Link: tag. Linus only wants Link tags if there is additional
>> information through that link. That's not the case here.
>>
>> I'm dropping the Link tags from the patches in this PR, let me know if you disagree.
>
> You missed the last episode of the drama :-)
>
> https://lore.kernel.org/all/CAHk-=wj5MATvT-FR8qNpXuuBGiJdjY1kRfhtzuyBSpTKR+=Vtw@mail.gmail.com/
>
> We can use Link: trailers to reference the patch that was picked, and we
> need to use the patch.msgid.link domain for that purpose. Adding
OK. Good to know. I'm fixing my check for Link: tags to test for that domain.
Regards,
Hans
>
> [b4]
> linkmask = https://patch.msgid.link/%s
>
> to your .gitconfig will instruct b4 to use that domain.
>
>> On 03/11/2025 01:16, Laurent Pinchart wrote:
>>> Hi Hans, Mauro,
>>>
>>> The following changes since commit 163917839c0eea3bdfe3620f27f617a55fd76302:
>>>
>>> MAINTAINERS: Update Daniel Scally's email address (2025-10-29 14:07:02 +0100)
>>>
>>> are available in the Git repository at:
>>>
>>> https://gitlab.freedesktop.org/linux-media/users/pinchartl.git tags/next-media-nxp-20251103
>>>
>>> for you to fetch changes up to a581a7c7d2c354131e184a8770f761c40168dda6:
>>>
>>> media: imx8mq-mipi-csi2: drop unused module alias (2025-11-03 01:23:28 +0200)
>>>
>>> ----------------------------------------------------------------
>>> NXP media drivers changes:
>>>
>>> - Add Frank Li as a reviewer for NXP media drivers
>>> - Improve buffer sequence in rkisp1
>>> - Add i.MX91 support and i.MX93 parallel input support to imx8-isi
>>> - Drop unused module aliases
>>>
>>> ----------------------------------------------------------------
>>> Alice Yuan (2):
>>> media: dt-bindings: nxp,imx8-isi: Add i.MX91 ISI compatible string
>>> media: nxp: imx8-isi: Add parallel camera input support for i.MX93
>>>
>>> Frank Li (1):
>>> MAINTAINERS: Add Frank Li as reviewer for NXP media drivers
>>>
>>> Guoniu Zhou (3):
>>> media: nxp: imx8-isi: Refine code by using helper macro
>>> media: nxp: imx8-isi: Reorder the platform data
>>> media: nxp: imx8-isi: Add ISI support for i.MX91
>>>
>>> Johan Hovold (3):
>>> media: imx-mipi-csis: drop unused module alias
>>> media: imx7-media-csi: drop unused module alias
>>> media: imx8mq-mipi-csi2: drop unused module alias
>>>
>>> Stefan Klug (1):
>>> media: rkisp1: Improve frame sequence correctness on stats and params buffers
>>>
>>> .../devicetree/bindings/media/nxp,imx8-isi.yaml | 13 ++++-
>>> MAINTAINERS | 4 ++
>>> drivers/media/platform/nxp/imx-mipi-csis.c | 1 -
>>> drivers/media/platform/nxp/imx7-media-csi.c | 1 -
>>> .../media/platform/nxp/imx8-isi/imx8-isi-core.c | 58 +++++++++++++---------
>>> .../media/platform/nxp/imx8-isi/imx8-isi-core.h | 1 +
>>> .../media/platform/nxp/imx8-isi/imx8-isi-gasket.c | 22 ++++++--
>>> drivers/media/platform/nxp/imx8mq-mipi-csi2.c | 1 -
>>> .../media/platform/rockchip/rkisp1/rkisp1-common.h | 1 +
>>> .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 27 ++++++++--
>>> 10 files changed, 92 insertions(+), 37 deletions(-)
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [GIT PULL FOR v6.19] NXP media drivers changes
2025-11-03 10:15 ` Hans Verkuil
@ 2025-11-03 10:50 ` Laurent Pinchart
0 siblings, 0 replies; 4+ messages in thread
From: Laurent Pinchart @ 2025-11-03 10:50 UTC (permalink / raw)
To: Hans Verkuil; +Cc: Linux Media Mailing List, Mauro Carvalho Chehab
On Mon, Nov 03, 2025 at 11:15:10AM +0100, Hans Verkuil wrote:
> On 03/11/2025 11:04, Laurent Pinchart wrote:
> > On Mon, Nov 03, 2025 at 09:08:10AM +0100, Hans Verkuil wrote:
> >> Hi Laurent,
> >>
> >> I noticed you forgot to CC to linux-media (I added that now), but also that all
> >
> > Oops sorry, my bad.
> >
> >> these patches have a Link: tag. Linus only wants Link tags if there is additional
> >> information through that link. That's not the case here.
> >>
> >> I'm dropping the Link tags from the patches in this PR, let me know if you disagree.
> >
> > You missed the last episode of the drama :-)
> >
> > https://lore.kernel.org/all/CAHk-=wj5MATvT-FR8qNpXuuBGiJdjY1kRfhtzuyBSpTKR+=Vtw@mail.gmail.com/
> >
> > We can use Link: trailers to reference the patch that was picked, and we
> > need to use the patch.msgid.link domain for that purpose. Adding
>
> OK. Good to know. I'm fixing my check for Link: tags to test for that domain.
We could add that to CI too, but we can't make it an error. Link
trailers with other domains are still allowed to point to discussions or
other relevant material. If you ask me I don't see why we should use
trailers for that, I find the inline reference to footnotes ([1]) better
for links that are relevant to the commit message, but it's a separate
matter.
[1] This would be a link footnote.
> > [b4]
> > linkmask = https://patch.msgid.link/%s
> >
> > to your .gitconfig will instruct b4 to use that domain.
> >
> >> On 03/11/2025 01:16, Laurent Pinchart wrote:
> >>> Hi Hans, Mauro,
> >>>
> >>> The following changes since commit 163917839c0eea3bdfe3620f27f617a55fd76302:
> >>>
> >>> MAINTAINERS: Update Daniel Scally's email address (2025-10-29 14:07:02 +0100)
> >>>
> >>> are available in the Git repository at:
> >>>
> >>> https://gitlab.freedesktop.org/linux-media/users/pinchartl.git tags/next-media-nxp-20251103
> >>>
> >>> for you to fetch changes up to a581a7c7d2c354131e184a8770f761c40168dda6:
> >>>
> >>> media: imx8mq-mipi-csi2: drop unused module alias (2025-11-03 01:23:28 +0200)
> >>>
> >>> ----------------------------------------------------------------
> >>> NXP media drivers changes:
> >>>
> >>> - Add Frank Li as a reviewer for NXP media drivers
> >>> - Improve buffer sequence in rkisp1
> >>> - Add i.MX91 support and i.MX93 parallel input support to imx8-isi
> >>> - Drop unused module aliases
> >>>
> >>> ----------------------------------------------------------------
> >>> Alice Yuan (2):
> >>> media: dt-bindings: nxp,imx8-isi: Add i.MX91 ISI compatible string
> >>> media: nxp: imx8-isi: Add parallel camera input support for i.MX93
> >>>
> >>> Frank Li (1):
> >>> MAINTAINERS: Add Frank Li as reviewer for NXP media drivers
> >>>
> >>> Guoniu Zhou (3):
> >>> media: nxp: imx8-isi: Refine code by using helper macro
> >>> media: nxp: imx8-isi: Reorder the platform data
> >>> media: nxp: imx8-isi: Add ISI support for i.MX91
> >>>
> >>> Johan Hovold (3):
> >>> media: imx-mipi-csis: drop unused module alias
> >>> media: imx7-media-csi: drop unused module alias
> >>> media: imx8mq-mipi-csi2: drop unused module alias
> >>>
> >>> Stefan Klug (1):
> >>> media: rkisp1: Improve frame sequence correctness on stats and params buffers
> >>>
> >>> .../devicetree/bindings/media/nxp,imx8-isi.yaml | 13 ++++-
> >>> MAINTAINERS | 4 ++
> >>> drivers/media/platform/nxp/imx-mipi-csis.c | 1 -
> >>> drivers/media/platform/nxp/imx7-media-csi.c | 1 -
> >>> .../media/platform/nxp/imx8-isi/imx8-isi-core.c | 58 +++++++++++++---------
> >>> .../media/platform/nxp/imx8-isi/imx8-isi-core.h | 1 +
> >>> .../media/platform/nxp/imx8-isi/imx8-isi-gasket.c | 22 ++++++--
> >>> drivers/media/platform/nxp/imx8mq-mipi-csi2.c | 1 -
> >>> .../media/platform/rockchip/rkisp1/rkisp1-common.h | 1 +
> >>> .../media/platform/rockchip/rkisp1/rkisp1-isp.c | 27 ++++++++--
> >>> 10 files changed, 92 insertions(+), 37 deletions(-)
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-11-03 10:50 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20251103001640.GA9221@pendragon.ideasonboard.com>
2025-11-03 8:08 ` [GIT PULL FOR v6.19] NXP media drivers changes Hans Verkuil
2025-11-03 10:04 ` Laurent Pinchart
2025-11-03 10:15 ` Hans Verkuil
2025-11-03 10:50 ` Laurent Pinchart
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).