* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
[not found] ` <20201119161956.756455-2-jacopo@jmondi.org>
@ 2020-11-25 9:15 ` Sakari Ailus
2020-12-21 17:34 ` Jacopo Mondi
2020-12-21 20:23 ` Rob Herring
0 siblings, 2 replies; 8+ messages in thread
From: Sakari Ailus @ 2020-11-25 9:15 UTC (permalink / raw)
To: Jacopo Mondi
Cc: linux-media, mchehab, hverkuil, laurent.pinchart,
roman.kovalivskyi, dafna.hirschfeld, dave.stevenson, naush,
erosca, devicetree, Rob Herring
Cc Rob + DT list.
On Thu, Nov 19, 2020 at 05:19:27PM +0100, Jacopo Mondi wrote:
> Rename 'ov5647.yaml' as 'ovti,ov5647.yaml' and update the
> MAINTAINERS file entry accordingly.
>
> Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> ---
> .../bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} | 0
> MAINTAINERS | 2 +-
> 2 files changed, 1 insertion(+), 1 deletion(-)
> rename Documentation/devicetree/bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} (100%)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/ov5647.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> similarity index 100%
> rename from Documentation/devicetree/bindings/media/i2c/ov5647.yaml
> rename to Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 69d55ed67e1cf..3f2acc3a78687 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12928,7 +12928,7 @@ M: Jacopo Mondi <jacopo@jmondi.org>
> L: linux-media@vger.kernel.org
> S: Maintained
> T: git git://linuxtv.org/media_tree.git
> -F: Documentation/devicetree/bindings/media/i2c/ov5647.yaml
> +F: Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> F: drivers/media/i2c/ov5647.c
>
> OMNIVISION OV5670 SENSOR DRIVER
> --
> 2.29.1
>
--
Sakari Ailus
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2020-11-25 9:15 ` [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml Sakari Ailus
@ 2020-12-21 17:34 ` Jacopo Mondi
2020-12-22 9:43 ` Sakari Ailus
2020-12-21 20:23 ` Rob Herring
1 sibling, 1 reply; 8+ messages in thread
From: Jacopo Mondi @ 2020-12-21 17:34 UTC (permalink / raw)
To: Sakari Ailus
Cc: linux-media, mchehab, hverkuil, laurent.pinchart,
roman.kovalivskyi, dafna.hirschfeld, dave.stevenson, naush,
erosca, devicetree, Rob Herring
Hello,
On Wed, Nov 25, 2020 at 11:15:26AM +0200, Sakari Ailus wrote:
> Cc Rob + DT list.
>
Gentle ping.
Sakari, am I mistaken or last time we discussed this, the series is
ready for being collected (pending this ack I assume)
I wonder if it needs a rebase (pretty sure it does)
Thanks
j
> On Thu, Nov 19, 2020 at 05:19:27PM +0100, Jacopo Mondi wrote:
> > Rename 'ov5647.yaml' as 'ovti,ov5647.yaml' and update the
> > MAINTAINERS file entry accordingly.
> >
> > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> > ---
> > .../bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} | 0
> > MAINTAINERS | 2 +-
> > 2 files changed, 1 insertion(+), 1 deletion(-)
> > rename Documentation/devicetree/bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} (100%)
> >
> > diff --git a/Documentation/devicetree/bindings/media/i2c/ov5647.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> > similarity index 100%
> > rename from Documentation/devicetree/bindings/media/i2c/ov5647.yaml
> > rename to Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 69d55ed67e1cf..3f2acc3a78687 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -12928,7 +12928,7 @@ M: Jacopo Mondi <jacopo@jmondi.org>
> > L: linux-media@vger.kernel.org
> > S: Maintained
> > T: git git://linuxtv.org/media_tree.git
> > -F: Documentation/devicetree/bindings/media/i2c/ov5647.yaml
> > +F: Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> > F: drivers/media/i2c/ov5647.c
> >
> > OMNIVISION OV5670 SENSOR DRIVER
> > --
> > 2.29.1
> >
>
> --
> Sakari Ailus
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2020-11-25 9:15 ` [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml Sakari Ailus
2020-12-21 17:34 ` Jacopo Mondi
@ 2020-12-21 20:23 ` Rob Herring
2020-12-22 9:50 ` Sakari Ailus
1 sibling, 1 reply; 8+ messages in thread
From: Rob Herring @ 2020-12-21 20:23 UTC (permalink / raw)
To: Sakari Ailus
Cc: Jacopo Mondi, Linux Media Mailing List, Mauro Carvalho Chehab,
Hans Verkuil, Laurent Pinchart, roman.kovalivskyi,
Dafna Hirschfeld, Dave Stevenson, naush, Eugeniu Rosca,
devicetree
On Wed, Nov 25, 2020 at 2:15 AM Sakari Ailus
<sakari.ailus@linux.intel.com> wrote:
>
> Cc Rob + DT list.
You have to resend if you really want me to see things. In this case,
I don't think you need to wait on me for a rename.
Acked-by: Rob Herring <robh@kernel.org>
>
> On Thu, Nov 19, 2020 at 05:19:27PM +0100, Jacopo Mondi wrote:
> > Rename 'ov5647.yaml' as 'ovti,ov5647.yaml' and update the
> > MAINTAINERS file entry accordingly.
> >
> > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
> > ---
> > .../bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} | 0
> > MAINTAINERS | 2 +-
> > 2 files changed, 1 insertion(+), 1 deletion(-)
> > rename Documentation/devicetree/bindings/media/i2c/{ov5647.yaml => ovti,ov5647.yaml} (100%)
> >
> > diff --git a/Documentation/devicetree/bindings/media/i2c/ov5647.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> > similarity index 100%
> > rename from Documentation/devicetree/bindings/media/i2c/ov5647.yaml
> > rename to Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 69d55ed67e1cf..3f2acc3a78687 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -12928,7 +12928,7 @@ M: Jacopo Mondi <jacopo@jmondi.org>
> > L: linux-media@vger.kernel.org
> > S: Maintained
> > T: git git://linuxtv.org/media_tree.git
> > -F: Documentation/devicetree/bindings/media/i2c/ov5647.yaml
> > +F: Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml
> > F: drivers/media/i2c/ov5647.c
> >
> > OMNIVISION OV5670 SENSOR DRIVER
> > --
> > 2.29.1
> >
>
> --
> Sakari Ailus
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2020-12-21 17:34 ` Jacopo Mondi
@ 2020-12-22 9:43 ` Sakari Ailus
0 siblings, 0 replies; 8+ messages in thread
From: Sakari Ailus @ 2020-12-22 9:43 UTC (permalink / raw)
To: Jacopo Mondi
Cc: linux-media, mchehab, hverkuil, laurent.pinchart,
roman.kovalivskyi, dafna.hirschfeld, dave.stevenson, naush,
erosca, devicetree, Rob Herring
Hi Jacopo,
On Mon, Dec 21, 2020 at 06:34:46PM +0100, Jacopo Mondi wrote:
> Hello,
>
> On Wed, Nov 25, 2020 at 11:15:26AM +0200, Sakari Ailus wrote:
> > Cc Rob + DT list.
> >
>
> Gentle ping.
>
> Sakari, am I mistaken or last time we discussed this, the series is
> ready for being collected (pending this ack I assume)
>
> I wonder if it needs a rebase (pretty sure it does)
The set is in my tree already.
--
Sakari Ailus
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2020-12-21 20:23 ` Rob Herring
@ 2020-12-22 9:50 ` Sakari Ailus
2021-01-14 20:20 ` Rob Herring
0 siblings, 1 reply; 8+ messages in thread
From: Sakari Ailus @ 2020-12-22 9:50 UTC (permalink / raw)
To: Rob Herring
Cc: Jacopo Mondi, Linux Media Mailing List, Mauro Carvalho Chehab,
Hans Verkuil, Laurent Pinchart, roman.kovalivskyi,
Dafna Hirschfeld, Dave Stevenson, naush, Eugeniu Rosca,
devicetree
Hi Rob,
On Mon, Dec 21, 2020 at 01:23:29PM -0700, Rob Herring wrote:
> On Wed, Nov 25, 2020 at 2:15 AM Sakari Ailus
> <sakari.ailus@linux.intel.com> wrote:
> >
> > Cc Rob + DT list.
>
> You have to resend if you really want me to see things. In this case,
> I don't think you need to wait on me for a rename.
I merged the patch earlier based on the IRC discussion.
--
Regards,
Sakari Ailus
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2020-12-22 9:50 ` Sakari Ailus
@ 2021-01-14 20:20 ` Rob Herring
2021-01-15 9:03 ` Jacopo Mondi
0 siblings, 1 reply; 8+ messages in thread
From: Rob Herring @ 2021-01-14 20:20 UTC (permalink / raw)
To: Sakari Ailus
Cc: Jacopo Mondi, Linux Media Mailing List, Mauro Carvalho Chehab,
Hans Verkuil, Laurent Pinchart, roman.kovalivskyi,
Dafna Hirschfeld, Dave Stevenson, naush, Eugeniu Rosca,
devicetree
On Tue, Dec 22, 2020 at 3:50 AM Sakari Ailus
<sakari.ailus@linux.intel.com> wrote:
>
> Hi Rob,
>
> On Mon, Dec 21, 2020 at 01:23:29PM -0700, Rob Herring wrote:
> > On Wed, Nov 25, 2020 at 2:15 AM Sakari Ailus
> > <sakari.ailus@linux.intel.com> wrote:
> > >
> > > Cc Rob + DT list.
> >
> > You have to resend if you really want me to see things. In this case,
> > I don't think you need to wait on me for a rename.
>
> I merged the patch earlier based on the IRC discussion.
Seems this landed in -next:
./Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml: $id:
relative path/filename doesn't match actual path or filename
expected: http://devicetree.org/schemas/media/i2c/ovti,ov5647.yaml#
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2021-01-14 20:20 ` Rob Herring
@ 2021-01-15 9:03 ` Jacopo Mondi
2021-01-15 9:26 ` Sakari Ailus
0 siblings, 1 reply; 8+ messages in thread
From: Jacopo Mondi @ 2021-01-15 9:03 UTC (permalink / raw)
To: Rob Herring, Mauro Carvalho Chehab
Cc: Sakari Ailus, Linux Media Mailing List, Mauro Carvalho Chehab,
Hans Verkuil, Laurent Pinchart, roman.kovalivskyi,
Dafna Hirschfeld, Dave Stevenson, naush, Eugeniu Rosca,
devicetree
Ups,
I'll fix. A question for Mauro though
On Thu, Jan 14, 2021 at 02:20:34PM -0600, Rob Herring wrote:
> On Tue, Dec 22, 2020 at 3:50 AM Sakari Ailus
> <sakari.ailus@linux.intel.com> wrote:
> >
> > Hi Rob,
> >
> > On Mon, Dec 21, 2020 at 01:23:29PM -0700, Rob Herring wrote:
> > > On Wed, Nov 25, 2020 at 2:15 AM Sakari Ailus
> > > <sakari.ailus@linux.intel.com> wrote:
> > > >
> > > > Cc Rob + DT list.
> > >
> > > You have to resend if you really want me to see things. In this case,
> > > I don't think you need to wait on me for a rename.
> >
> > I merged the patch earlier based on the IRC discussion.
>
> Seems this landed in -next:
>
> ./Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml: $id:
> relative path/filename doesn't match actual path or filename
> expected: http://devicetree.org/schemas/media/i2c/ovti,ov5647.yaml#
>
The fix is worth a Fixes: tag, should I refer to the current commit
sha1 in linux-media/master or is there any risk for a rebase before
the tree is merged in the v5.12 merge window ?
Thanks
j
> Rob
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml
2021-01-15 9:03 ` Jacopo Mondi
@ 2021-01-15 9:26 ` Sakari Ailus
0 siblings, 0 replies; 8+ messages in thread
From: Sakari Ailus @ 2021-01-15 9:26 UTC (permalink / raw)
To: Jacopo Mondi
Cc: Rob Herring, Mauro Carvalho Chehab, Linux Media Mailing List,
Hans Verkuil, Laurent Pinchart, roman.kovalivskyi,
Dafna Hirschfeld, Dave Stevenson, naush, Eugeniu Rosca,
devicetree
Hi Jacopo,
On Fri, Jan 15, 2021 at 10:03:26AM +0100, Jacopo Mondi wrote:
> Ups,
>
> I'll fix. A question for Mauro though
>
> On Thu, Jan 14, 2021 at 02:20:34PM -0600, Rob Herring wrote:
> > On Tue, Dec 22, 2020 at 3:50 AM Sakari Ailus
> > <sakari.ailus@linux.intel.com> wrote:
> > >
> > > Hi Rob,
> > >
> > > On Mon, Dec 21, 2020 at 01:23:29PM -0700, Rob Herring wrote:
> > > > On Wed, Nov 25, 2020 at 2:15 AM Sakari Ailus
> > > > <sakari.ailus@linux.intel.com> wrote:
> > > > >
> > > > > Cc Rob + DT list.
> > > >
> > > > You have to resend if you really want me to see things. In this case,
> > > > I don't think you need to wait on me for a rename.
> > >
> > > I merged the patch earlier based on the IRC discussion.
> >
> > Seems this landed in -next:
> >
> > ./Documentation/devicetree/bindings/media/i2c/ovti,ov5647.yaml: $id:
> > relative path/filename doesn't match actual path or filename
> > expected: http://devicetree.org/schemas/media/i2c/ovti,ov5647.yaml#
> >
>
> The fix is worth a Fixes: tag, should I refer to the current commit
> sha1 in linux-media/master or is there any risk for a rebase before
> the tree is merged in the v5.12 merge window ?
I'd say that happens only in exceptional circumstances. I think you can do
that.
--
Regards,
Sakari Ailus
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-01-15 9:28 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20201119161956.756455-1-jacopo@jmondi.org>
[not found] ` <20201119161956.756455-2-jacopo@jmondi.org>
2020-11-25 9:15 ` [PATCH v4 01/30] dt-bindings: media: i2c: Rename ov5647.yaml Sakari Ailus
2020-12-21 17:34 ` Jacopo Mondi
2020-12-22 9:43 ` Sakari Ailus
2020-12-21 20:23 ` Rob Herring
2020-12-22 9:50 ` Sakari Ailus
2021-01-14 20:20 ` Rob Herring
2021-01-15 9:03 ` Jacopo Mondi
2021-01-15 9:26 ` Sakari Ailus
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).