* [RESEND PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats
@ 2015-01-06 11:43 Boris Brezillon
2015-02-02 14:57 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Boris Brezillon @ 2015-01-06 11:43 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Hans Verkuil, Laurent Pinchart, linux-media
Cc: Nicolas Ferre, Jean-Christophe Plagniol-Villard,
Alexandre Belloni, linux-kernel, Boris Brezillon
Add RGB444_1X12 and RGB565_1X16 format definitions and update the
documentation.
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Acked-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
Hi Mauro, Sakari,
This patch has been rejected as 'Not Applicable'.
Is there anyting wrong in it ?
Best Regards,
Boris
Documentation/DocBook/media/v4l/subdev-formats.xml | 40 ++++++++++++++++++++++
include/uapi/linux/media-bus-format.h | 4 ++-
2 files changed, 43 insertions(+), 1 deletion(-)
diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
index c5ea868..be57efa 100644
--- a/Documentation/DocBook/media/v4l/subdev-formats.xml
+++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
@@ -192,6 +192,24 @@ see <xref linkend="colorspaces" />.</entry>
</row>
</thead>
<tbody valign="top">
+ <row id="MEDIA-BUS-FMT-RGB444-1X12">
+ <entry>MEDIA_BUS_FMT_RGB444_1X12</entry>
+ <entry>0x100d</entry>
+ <entry></entry>
+ &dash-ent-20;
+ <entry>r<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ <entry>g<subscript>0</subscript></entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>b<subscript>2</subscript></entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>b<subscript>0</subscript></entry>
+ </row>
<row id="MEDIA-BUS-FMT-RGB444-2X8-PADHI-BE">
<entry>MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE</entry>
<entry>0x1001</entry>
@@ -304,6 +322,28 @@ see <xref linkend="colorspaces" />.</entry>
<entry>g<subscript>4</subscript></entry>
<entry>g<subscript>3</subscript></entry>
</row>
+ <row id="MEDIA-BUS-FMT-RGB565-1X16">
+ <entry>MEDIA_BUS_FMT_RGB565_1X16</entry>
+ <entry>0x100d</entry>
+ <entry></entry>
+ &dash-ent-16;
+ <entry>r<subscript>4</subscript></entry>
+ <entry>r<subscript>3</subscript></entry>
+ <entry>r<subscript>2</subscript></entry>
+ <entry>r<subscript>1</subscript></entry>
+ <entry>r<subscript>0</subscript></entry>
+ <entry>g<subscript>5</subscript></entry>
+ <entry>g<subscript>4</subscript></entry>
+ <entry>g<subscript>3</subscript></entry>
+ <entry>g<subscript>2</subscript></entry>
+ <entry>g<subscript>1</subscript></entry>
+ <entry>g<subscript>0</subscript></entry>
+ <entry>b<subscript>4</subscript></entry>
+ <entry>b<subscript>3</subscript></entry>
+ <entry>b<subscript>2</subscript></entry>
+ <entry>b<subscript>1</subscript></entry>
+ <entry>b<subscript>0</subscript></entry>
+ </row>
<row id="MEDIA-BUS-FMT-BGR565-2X8-BE">
<entry>MEDIA_BUS_FMT_BGR565_2X8_BE</entry>
<entry>0x1005</entry>
diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
index 23b4090..37091c6 100644
--- a/include/uapi/linux/media-bus-format.h
+++ b/include/uapi/linux/media-bus-format.h
@@ -33,11 +33,13 @@
#define MEDIA_BUS_FMT_FIXED 0x0001
-/* RGB - next is 0x100e */
+/* RGB - next is 0x1010 */
+#define MEDIA_BUS_FMT_RGB444_1X12 0x100e
#define MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE 0x1001
#define MEDIA_BUS_FMT_RGB444_2X8_PADHI_LE 0x1002
#define MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE 0x1003
#define MEDIA_BUS_FMT_RGB555_2X8_PADHI_LE 0x1004
+#define MEDIA_BUS_FMT_RGB565_1X16 0x100f
#define MEDIA_BUS_FMT_BGR565_2X8_BE 0x1005
#define MEDIA_BUS_FMT_BGR565_2X8_LE 0x1006
#define MEDIA_BUS_FMT_RGB565_2X8_BE 0x1007
--
1.9.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RESEND PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats
2015-01-06 11:43 [RESEND PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats Boris Brezillon
@ 2015-02-02 14:57 ` Mauro Carvalho Chehab
2015-02-02 15:32 ` Boris Brezillon
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2015-02-02 14:57 UTC (permalink / raw)
To: Boris Brezillon
Cc: Hans Verkuil, Laurent Pinchart, linux-media, Nicolas Ferre,
Jean-Christophe Plagniol-Villard, Alexandre Belloni,
linux-kernel
Em Tue, 6 Jan 2015 12:43:35 +0100
Boris Brezillon <boris.brezillon@free-electrons.com> escreveu:
> Add RGB444_1X12 and RGB565_1X16 format definitions and update the
> documentation.
>
> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> Acked-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
> Hi Mauro, Sakari,
>
> This patch has been rejected as 'Not Applicable'.
> Is there anyting wrong in it ?
I was expecting that this patch would be merged together with the
remaining series, via the DRM tree. That's basically why I gave
my ack:
https://lkml.org/lkml/2014/11/3/661
HINT: when a subsystem maintainer gives an ack, that likely means that
he expects that the patch will be applied via some other tree.
Regards,
Mauro
>
> Best Regards,
>
> Boris
>
> Documentation/DocBook/media/v4l/subdev-formats.xml | 40 ++++++++++++++++++++++
> include/uapi/linux/media-bus-format.h | 4 ++-
> 2 files changed, 43 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml
> index c5ea868..be57efa 100644
> --- a/Documentation/DocBook/media/v4l/subdev-formats.xml
> +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml
> @@ -192,6 +192,24 @@ see <xref linkend="colorspaces" />.</entry>
> </row>
> </thead>
> <tbody valign="top">
> + <row id="MEDIA-BUS-FMT-RGB444-1X12">
> + <entry>MEDIA_BUS_FMT_RGB444_1X12</entry>
> + <entry>0x100d</entry>
> + <entry></entry>
> + &dash-ent-20;
> + <entry>r<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + <entry>g<subscript>0</subscript></entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>b<subscript>2</subscript></entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>b<subscript>0</subscript></entry>
> + </row>
> <row id="MEDIA-BUS-FMT-RGB444-2X8-PADHI-BE">
> <entry>MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE</entry>
> <entry>0x1001</entry>
> @@ -304,6 +322,28 @@ see <xref linkend="colorspaces" />.</entry>
> <entry>g<subscript>4</subscript></entry>
> <entry>g<subscript>3</subscript></entry>
> </row>
> + <row id="MEDIA-BUS-FMT-RGB565-1X16">
> + <entry>MEDIA_BUS_FMT_RGB565_1X16</entry>
> + <entry>0x100d</entry>
> + <entry></entry>
> + &dash-ent-16;
> + <entry>r<subscript>4</subscript></entry>
> + <entry>r<subscript>3</subscript></entry>
> + <entry>r<subscript>2</subscript></entry>
> + <entry>r<subscript>1</subscript></entry>
> + <entry>r<subscript>0</subscript></entry>
> + <entry>g<subscript>5</subscript></entry>
> + <entry>g<subscript>4</subscript></entry>
> + <entry>g<subscript>3</subscript></entry>
> + <entry>g<subscript>2</subscript></entry>
> + <entry>g<subscript>1</subscript></entry>
> + <entry>g<subscript>0</subscript></entry>
> + <entry>b<subscript>4</subscript></entry>
> + <entry>b<subscript>3</subscript></entry>
> + <entry>b<subscript>2</subscript></entry>
> + <entry>b<subscript>1</subscript></entry>
> + <entry>b<subscript>0</subscript></entry>
> + </row>
> <row id="MEDIA-BUS-FMT-BGR565-2X8-BE">
> <entry>MEDIA_BUS_FMT_BGR565_2X8_BE</entry>
> <entry>0x1005</entry>
> diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h
> index 23b4090..37091c6 100644
> --- a/include/uapi/linux/media-bus-format.h
> +++ b/include/uapi/linux/media-bus-format.h
> @@ -33,11 +33,13 @@
>
> #define MEDIA_BUS_FMT_FIXED 0x0001
>
> -/* RGB - next is 0x100e */
> +/* RGB - next is 0x1010 */
> +#define MEDIA_BUS_FMT_RGB444_1X12 0x100e
> #define MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE 0x1001
> #define MEDIA_BUS_FMT_RGB444_2X8_PADHI_LE 0x1002
> #define MEDIA_BUS_FMT_RGB555_2X8_PADHI_BE 0x1003
> #define MEDIA_BUS_FMT_RGB555_2X8_PADHI_LE 0x1004
> +#define MEDIA_BUS_FMT_RGB565_1X16 0x100f
> #define MEDIA_BUS_FMT_BGR565_2X8_BE 0x1005
> #define MEDIA_BUS_FMT_BGR565_2X8_LE 0x1006
> #define MEDIA_BUS_FMT_RGB565_2X8_BE 0x1007
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RESEND PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats
2015-02-02 14:57 ` Mauro Carvalho Chehab
@ 2015-02-02 15:32 ` Boris Brezillon
2015-02-02 15:40 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Boris Brezillon @ 2015-02-02 15:32 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Hans Verkuil, Laurent Pinchart, linux-media, Nicolas Ferre,
Jean-Christophe Plagniol-Villard, Alexandre Belloni,
linux-kernel, dri-devel, Dave Airlie
Hi Mauro,
On Mon, 02 Feb 2015 12:57:55 -0200
Mauro Carvalho Chehab <m.chehab@samsung.com> wrote:
> Em Tue, 6 Jan 2015 12:43:35 +0100
> Boris Brezillon <boris.brezillon@free-electrons.com> escreveu:
>
> > Add RGB444_1X12 and RGB565_1X16 format definitions and update the
> > documentation.
> >
> > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > Acked-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > ---
> > Hi Mauro, Sakari,
> >
> > This patch has been rejected as 'Not Applicable'.
> > Is there anyting wrong in it ?
>
> I was expecting that this patch would be merged together with the
> remaining series, via the DRM tree. That's basically why I gave
> my ack:
> https://lkml.org/lkml/2014/11/3/661
>
> HINT: when a subsystem maintainer gives an ack, that likely means that
> he expects that the patch will be applied via some other tree.
My bad, I thought this would go into the media tree since this single
patch is not exactly related to a DRM feature (except the fact that I
was planning to use it in my DRM driver).
Actually, I didn't send it to the DRM maintainer or dri-devel ML in the
first place :-(.
Can you reconsider taking it in the media tree ?
I you can't, I'll ask Dave (just added him in Cc) to take it into the
DRM tree.
Thanks.
Best Regards,
Boris
--
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RESEND PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats
2015-02-02 15:32 ` Boris Brezillon
@ 2015-02-02 15:40 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2015-02-02 15:40 UTC (permalink / raw)
To: Boris Brezillon
Cc: Hans Verkuil, Laurent Pinchart, linux-media, Nicolas Ferre,
Jean-Christophe Plagniol-Villard, Alexandre Belloni,
linux-kernel, dri-devel, Dave Airlie
Em Mon, 02 Feb 2015 16:32:07 +0100
Boris Brezillon <boris.brezillon@free-electrons.com> escreveu:
> Hi Mauro,
>
> On Mon, 02 Feb 2015 12:57:55 -0200
> Mauro Carvalho Chehab <m.chehab@samsung.com> wrote:
>
> > Em Tue, 6 Jan 2015 12:43:35 +0100
> > Boris Brezillon <boris.brezillon@free-electrons.com> escreveu:
> >
> > > Add RGB444_1X12 and RGB565_1X16 format definitions and update the
> > > documentation.
> > >
> > > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > > Acked-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> > > Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > > ---
> > > Hi Mauro, Sakari,
> > >
> > > This patch has been rejected as 'Not Applicable'.
> > > Is there anyting wrong in it ?
> >
> > I was expecting that this patch would be merged together with the
> > remaining series, via the DRM tree. That's basically why I gave
> > my ack:
> > https://lkml.org/lkml/2014/11/3/661
> >
> > HINT: when a subsystem maintainer gives an ack, that likely means that
> > he expects that the patch will be applied via some other tree.
>
> My bad, I thought this would go into the media tree since this single
> patch is not exactly related to a DRM feature (except the fact that I
> was planning to use it in my DRM driver).
> Actually, I didn't send it to the DRM maintainer or dri-devel ML in the
> first place :-(.
> Can you reconsider taking it in the media tree ?
> I you can't, I'll ask Dave (just added him in Cc) to take it into the
> DRM tree.
I really prefer if you submit this together with the DRM series.
We don't apply API changes at media, except if the API change is
needed by some driver that it is also submitted in the same series.
I don't mind applying it via media, but in this case, I'll apply
together with the remaining DRM drivers, and will require DRM
maintainer's ack. So, it is probably easier to just apply this
change via the DRM subtree than the reverse.
Regards,
Mauro
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-02-02 15:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-06 11:43 [RESEND PATCH v2] [media] Add RGB444_1X12 and RGB565_1X16 media bus formats Boris Brezillon
2015-02-02 14:57 ` Mauro Carvalho Chehab
2015-02-02 15:32 ` Boris Brezillon
2015-02-02 15:40 ` Mauro Carvalho Chehab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®