Re: [GIT PULL FOR v4.11] MediaTek JPEG encoder

2017-02-16 Thread Matthias Brugger 



On 13/02/17 13:00, Mauro Carvalho Chehab wrote:

Em Mon, 13 Feb 2017 11:52:07 +0100
Hans Verkuil  escreveu:


Hi Mauro,

This adds the MediaTek JPEG encoder to the media subsystem.

This patch https://patchwork.linuxtv.org/patch/38645/ needs to go through
Matthias Brugger,


Why? It seems easier to just merge it via media tree, as it doesn't
seem to be dependent of anything else.

IMHO, the better would be if Matthias would ack to merge it via
the media tree. On the other hand, it is just a documentation path.
It wouldn't hurt if merged for 4.11, even if the remaining patches
go for 4.12.



Sorry for the late answer:
I'm fine with merging it through your tree. Any patches on dts/dtsi 
files should go through mine, but for the bindings it's fine (and make 
sense) to take them through the driver subsystem maintainers tree.


Regards,
Matthias


so you need to coordinate with him for which kernel this
driver will be merged. This pull request is for 4.11, but since it is so
late in the cycle I can understand if this slips to 4.12.

In any case, this should be coordinated.


It is too late for 4.11. We close our merge window by -rc6, in
order to give janitors some time to check if everything is ok.

Also, unfortunately I won't have any time this week to review
this patchset.

So, let's not rush it and merge it after 4.11-rc1, for 4.12.

Regards,
Mauro



Regards,

Hans

The following changes since commit 9eeb0ed0f30938f31a3d9135a88b9502192c18dd:

  [media] mtk-vcodec: fix build warnings without DEBUG (2017-02-08 12:08:20 
-0200)

are available in the git repository at:

  git://linuxtv.org/hverkuil/media_tree.git for-v4.11e

for you to fetch changes up to 0f492f30d15aec43248fbdd4d6ceea8f495f4457:

  vcodec: mediatek: Add Maintainers entry for Mediatek JPEG driver (2017-02-13 
11:35:29 +0100)


Rick Chang (3):
  dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder
  vcodec: mediatek: Add Mediatek JPEG Decoder Driver
  vcodec: mediatek: Add Maintainers entry for Mediatek JPEG driver

 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt |   37 +
 MAINTAINERS   |7 +
 drivers/media/platform/Kconfig|   15 +
 drivers/media/platform/Makefile   |2 +
 drivers/media/platform/mtk-jpeg/Makefile  |2 +
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c   | 1306 
+
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h   |  139 
 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.c |  417 
+++
 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.h |   91 +++
 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.c  |  160 
 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.h  |   25 +
 drivers/media/platform/mtk-jpeg/mtk_jpeg_reg.h|   58 ++
 12 files changed, 2259 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt
 create mode 100644 drivers/media/platform/mtk-jpeg/Makefile
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.c
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.h
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.c
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.h
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_reg.h




Thanks,
Mauro

Re: [GIT PULL FOR v4.11] MediaTek JPEG encoder

2017-02-13 Thread Mauro Carvalho Chehab 
Em Mon, 13 Feb 2017 11:52:07 +0100
Hans Verkuil  escreveu:

> Hi Mauro,
> 
> This adds the MediaTek JPEG encoder to the media subsystem.
> 
> This patch https://patchwork.linuxtv.org/patch/38645/ needs to go through
> Matthias Brugger,

Why? It seems easier to just merge it via media tree, as it doesn't
seem to be dependent of anything else.

IMHO, the better would be if Matthias would ack to merge it via
the media tree. On the other hand, it is just a documentation path.
It wouldn't hurt if merged for 4.11, even if the remaining patches
go for 4.12.

> so you need to coordinate with him for which kernel this
> driver will be merged. This pull request is for 4.11, but since it is so
> late in the cycle I can understand if this slips to 4.12.
> 
> In any case, this should be coordinated.

It is too late for 4.11. We close our merge window by -rc6, in
order to give janitors some time to check if everything is ok.

Also, unfortunately I won't have any time this week to review
this patchset.

So, let's not rush it and merge it after 4.11-rc1, for 4.12.

Regards,
Mauro

> 
> Regards,
> 
>   Hans
> 
> The following changes since commit 9eeb0ed0f30938f31a3d9135a88b9502192c18dd:
> 
>   [media] mtk-vcodec: fix build warnings without DEBUG (2017-02-08 12:08:20 
> -0200)
> 
> are available in the git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git for-v4.11e
> 
> for you to fetch changes up to 0f492f30d15aec43248fbdd4d6ceea8f495f4457:
> 
>   vcodec: mediatek: Add Maintainers entry for Mediatek JPEG driver 
> (2017-02-13 11:35:29 +0100)
> 
> 
> Rick Chang (3):
>   dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder
>   vcodec: mediatek: Add Mediatek JPEG Decoder Driver
>   vcodec: mediatek: Add Maintainers entry for Mediatek JPEG driver
> 
>  Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt |   37 +
>  MAINTAINERS   |7 +
>  drivers/media/platform/Kconfig|   15 +
>  drivers/media/platform/Makefile   |2 +
>  drivers/media/platform/mtk-jpeg/Makefile  |2 +
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c   | 1306 
> +
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h   |  139 
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.c |  417 
> +++
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.h |   91 +++
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.c  |  160 
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.h  |   25 +
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_reg.h|   58 ++
>  12 files changed, 2259 insertions(+)
>  create mode 100644 
> Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt
>  create mode 100644 drivers/media/platform/mtk-jpeg/Makefile
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.c
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.h
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.c
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.h
>  create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_reg.h



Thanks,
Mauro

[GIT PULL FOR v4.11] MediaTek JPEG encoder

2017-02-13 Thread Hans Verkuil 
Hi Mauro,

This adds the MediaTek JPEG encoder to the media subsystem.

This patch https://patchwork.linuxtv.org/patch/38645/ needs to go through
Matthias Brugger, so you need to coordinate with him for which kernel this
driver will be merged. This pull request is for 4.11, but since it is so
late in the cycle I can understand if this slips to 4.12.

In any case, this should be coordinated.

Regards,

Hans

The following changes since commit 9eeb0ed0f30938f31a3d9135a88b9502192c18dd:

  [media] mtk-vcodec: fix build warnings without DEBUG (2017-02-08 12:08:20 
-0200)

are available in the git repository at:

  git://linuxtv.org/hverkuil/media_tree.git for-v4.11e

for you to fetch changes up to 0f492f30d15aec43248fbdd4d6ceea8f495f4457:

  vcodec: mediatek: Add Maintainers entry for Mediatek JPEG driver (2017-02-13 
11:35:29 +0100)


Rick Chang (3):
  dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder
  vcodec: mediatek: Add Mediatek JPEG Decoder Driver
  vcodec: mediatek: Add Maintainers entry for Mediatek JPEG driver

 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt |   37 +
 MAINTAINERS   |7 +
 drivers/media/platform/Kconfig|   15 +
 drivers/media/platform/Makefile   |2 +
 drivers/media/platform/mtk-jpeg/Makefile  |2 +
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c   | 1306 
+
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h   |  139 
 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.c |  417 
+++
 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.h |   91 +++
 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.c  |  160 
 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.h  |   25 +
 drivers/media/platform/mtk-jpeg/mtk_jpeg_reg.h|   58 ++
 12 files changed, 2259 insertions(+)
 create mode 100644 
Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt
 create mode 100644 drivers/media/platform/mtk-jpeg/Makefile
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.c
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_hw.h
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.c
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_parse.h
 create mode 100644 drivers/media/platform/mtk-jpeg/mtk_jpeg_reg.h