diff mbox series

[v4,01/30] dt-bindings: media: i2c: Rename ov5647.yaml

Message ID 20201119161956.756455-2-jacopo@jmondi.org (mailing list archive)
State New, archived
Headers show
Series media: ov5647: Support RaspberryPi Camera Module v1 | expand

Commit Message

Jacopo Mondi Nov. 19, 2020, 4:19 p.m. UTC
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%)

Comments

Sakari Ailus Nov. 25, 2020, 9:15 a.m. UTC | #1
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
>
Jacopo Mondi Dec. 21, 2020, 5:34 p.m. UTC | #2
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
Rob Herring Dec. 21, 2020, 8:23 p.m. UTC | #3
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
Sakari Ailus Dec. 22, 2020, 9:43 a.m. UTC | #4
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 Dec. 22, 2020, 9:50 a.m. UTC | #5
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.
Rob Herring Jan. 14, 2021, 8:20 p.m. UTC | #6
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
Jacopo Mondi Jan. 15, 2021, 9:03 a.m. UTC | #7
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
Sakari Ailus Jan. 15, 2021, 9:26 a.m. UTC | #8
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.
diff mbox series

Patch

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