RE: [PATCH v5 0/5] Add new driver for generic scaler

2012-08-01 Thread Sungchun Kang
On 07/31/2012 10:44 PM, Shaik Ameer Basha wrote:
 -Original Message-
 From: Shaik Ameer Basha [mailto:shaik.am...@samsung.com]
 Sent: Tuesday, July 31, 2012 10:44 PM
 To: linux-media@vger.kernel.org
 Cc: sungchun.k...@samsung.com; khw0178@samsung.com; mche...@infradead.org;
 laurent.pinch...@ideasonboard.com; sy0816.k...@samsung.com; 
 s.nawro...@samsung.com;
 posc...@google.com; hverk...@xs4all.nl; alim.akh...@gmail.com; 
 prashant...@samsung.com;
 jo...@samsung.com; shaik.sams...@gmail.com; shaik.am...@samsung.com
 Subject: [PATCH v5 0/5] Add new driver for generic scaler
 
 This patch adds support for the gscaler device which is a new device
 for scaling and color space conversion on EXYNOS5 SoCs.
 
 This device supports the followings as key feature.
  1) Input image format
- RGB888/565, YUV422 1P/2P, YUV420 2P/3P, TILE
  2) Output image format
- RGB888/565, YUV422 1P/2P, YUV420 2P/3P, YUV444
  3) Input rotation
- 0/90/180/270 degree, X/Y Flip
  4) Scale ratio
- 1/16 scale down to 8 scale up
  5) CSC
- RGB to YUV / YUV to RGB
  6) Size
- 2048 x 2048 for tile or rotation
- 4800 x 3344 other case
 
 changes since v4:
 - Rebased on latest media-tree git, branch staging/for_v3.6.
 http://linuxtv.org/git/media_tree.git
 - Addressed review comments from Hans Verkuil
   http://www.mail-archive.com/linux-media@vger.kernel.org/msg49754.html
 - Removed the v4l: Add fourcc definitions for new formats patch dependency.
 
 Shaik Ameer Basha (2):
   v4l: Add new YVU420 multi planar fourcc definition
   media: gscaler: Add Makefile for G-Scaler Driver
 
 Sungchun Kang (3):
   media: gscaler: Add new driver for generic scaler
   media: gscaler: Add core functionality for the G-Scaler driver
   media: gscaler: Add m2m functionality for the G-Scaler driver
 
  Documentation/DocBook/media/v4l/pixfmt-yvu420m.xml |  154 +++
  Documentation/DocBook/media/v4l/pixfmt.xml |1 +
  drivers/media/video/Kconfig|8 +
  drivers/media/video/Makefile   |2 +
  drivers/media/video/exynos-gsc/Makefile|3 +
  drivers/media/video/exynos-gsc/gsc-core.c  | 1254 
 
  drivers/media/video/exynos-gsc/gsc-core.h  |  532 +
  drivers/media/video/exynos-gsc/gsc-m2m.c   |  771 
  drivers/media/video/exynos-gsc/gsc-regs.c  |  425 +++
  drivers/media/video/exynos-gsc/gsc-regs.h  |  172 +++
  include/linux/videodev2.h  |1 +
  11 files changed, 3323 insertions(+), 0 deletions(-)
  create mode 100644 Documentation/DocBook/media/v4l/pixfmt-yvu420m.xml
  create mode 100644 drivers/media/video/exynos-gsc/Makefile
  create mode 100644 drivers/media/video/exynos-gsc/gsc-core.c
  create mode 100644 drivers/media/video/exynos-gsc/gsc-core.h
  create mode 100644 drivers/media/video/exynos-gsc/gsc-m2m.c
  create mode 100644 drivers/media/video/exynos-gsc/gsc-regs.c
  create mode 100644 drivers/media/video/exynos-gsc/gsc-regs.h

I'm sorry to be so late.
Basically, I wonder important one thing.
What would you implement a device driver connected with gscaler.
For example, fimc-lite, mipi-csis.
As you know Exynos5 has local-path with gscaler
MIPI-CSIS = Fimc-lite = Gscaler
And, you should use media control framework.
So, We made exynos folder, and implement drivers with mc.
We use mdev that is virtual device driver for connecting gscaler, fimc-lite, 
mipi-csis with MC.
This is camera path. 
There are not only camera path but also rendering path.
Gscaler = FIMD or TV
Rendering path use mdev-0,
Camera path use mdev-1.
In conclusion, because we use to connect each other devices with MC, we made 
exynos folder.

And how you make to implement devices with MC?


--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/5] Add new driver for generic scaler

2012-08-01 Thread Sylwester Nawrocki
On 08/01/2012 09:35 AM, Sungchun Kang wrote:
 I'm sorry to be so late.
 Basically, I wonder important one thing.
 What would you implement a device driver connected with gscaler.
 For example, fimc-lite, mipi-csis.
 As you know Exynos5 has local-path with gscaler
 MIPI-CSIS = Fimc-lite = Gscaler
 And, you should use media control framework.
 So, We made exynos folder, and implement drivers with mc.
 We use mdev that is virtual device driver for connecting gscaler, fimc-lite, 
 mipi-csis with MC.
 This is camera path. 
 There are not only camera path but also rendering path.
 Gscaler = FIMD or TV
 Rendering path use mdev-0,
 Camera path use mdev-1.
 In conclusion, because we use to connect each other devices with MC, we made 
 exynos folder.
 
 And how you make to implement devices with MC?

As you may know, these patches only add mem-to-mem functionality,
which can be used together with the Exynos multi-format video codec.

Remaining features, as you listed, are planned to be added later,
in subsequent steps, after discussing it here on the mailing list.

I think it's much better approach, than coming up with a complete huge
driver with many API compliance issues. Especially that some drivers,
like MIPI-CSIS or FIMC-LITE are already in the mainline kernel.

As for the driver directory name, IMHO drivers/media/exynos is too
generic, s5p-fimc, s5p-jpeg, s5p-tv also cover some Exynos SoCs.

I don't think having drivers/media/exynos directory would be helpful
in anything.

Regards,
Sylwester
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v5 0/5] Add new driver for generic scaler

2012-07-31 Thread Shaik Ameer Basha
This patch adds support for the gscaler device which is a new device
for scaling and color space conversion on EXYNOS5 SoCs.

This device supports the followings as key feature.
 1) Input image format
   - RGB888/565, YUV422 1P/2P, YUV420 2P/3P, TILE
 2) Output image format
   - RGB888/565, YUV422 1P/2P, YUV420 2P/3P, YUV444
 3) Input rotation
   - 0/90/180/270 degree, X/Y Flip
 4) Scale ratio
   - 1/16 scale down to 8 scale up
 5) CSC
   - RGB to YUV / YUV to RGB
 6) Size
   - 2048 x 2048 for tile or rotation
   - 4800 x 3344 other case

changes since v4:
- Rebased on latest media-tree git, branch staging/for_v3.6.
http://linuxtv.org/git/media_tree.git
- Addressed review comments from Hans Verkuil
http://www.mail-archive.com/linux-media@vger.kernel.org/msg49754.html
- Removed the v4l: Add fourcc definitions for new formats patch dependency.

Shaik Ameer Basha (2):
  v4l: Add new YVU420 multi planar fourcc definition
  media: gscaler: Add Makefile for G-Scaler Driver

Sungchun Kang (3):
  media: gscaler: Add new driver for generic scaler
  media: gscaler: Add core functionality for the G-Scaler driver
  media: gscaler: Add m2m functionality for the G-Scaler driver

 Documentation/DocBook/media/v4l/pixfmt-yvu420m.xml |  154 +++
 Documentation/DocBook/media/v4l/pixfmt.xml |1 +
 drivers/media/video/Kconfig|8 +
 drivers/media/video/Makefile   |2 +
 drivers/media/video/exynos-gsc/Makefile|3 +
 drivers/media/video/exynos-gsc/gsc-core.c  | 1254 
 drivers/media/video/exynos-gsc/gsc-core.h  |  532 +
 drivers/media/video/exynos-gsc/gsc-m2m.c   |  771 
 drivers/media/video/exynos-gsc/gsc-regs.c  |  425 +++
 drivers/media/video/exynos-gsc/gsc-regs.h  |  172 +++
 include/linux/videodev2.h  |1 +
 11 files changed, 3323 insertions(+), 0 deletions(-)
 create mode 100644 Documentation/DocBook/media/v4l/pixfmt-yvu420m.xml
 create mode 100644 drivers/media/video/exynos-gsc/Makefile
 create mode 100644 drivers/media/video/exynos-gsc/gsc-core.c
 create mode 100644 drivers/media/video/exynos-gsc/gsc-core.h
 create mode 100644 drivers/media/video/exynos-gsc/gsc-m2m.c
 create mode 100644 drivers/media/video/exynos-gsc/gsc-regs.c
 create mode 100644 drivers/media/video/exynos-gsc/gsc-regs.h

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html