| Message ID | 20201119161956.756455-2-jacopo@jmondi.org |
|---|---|
| State | Accepted |
| Commit | 1b5071af824039cb71592d5b2a60773ee1ee94be |
| Headers | show |
| Series | media: ov5647: Support RaspberryPi Camera Module v1 | expand |
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 >
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
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
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
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
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
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 --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
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%)