[git pull] updated exynos-drm-fixes

[git pull] updated exynos-drm-fixes

[Inki Dae]

Hi Dave,

Please pull from
	git://git.infradead.org/users/kmpark/linux-samsung exynos-drm-fixes

this branch is based on git repository below:
	git://people.freedesktop.org/~airlied/linux.git drm-fixes
	commit-id: 62fb376e214d3c1bfdf6fbb77dac162f6da04d7e

this patch set includes page align and allocation bug fixes and
some code cleanup and also Joonyoung's patch set v2 like below.

v2: Split "[PATCH 2/4] drm/exynos: remove unused codes in hdmi and mixer
   driver" patch of patchset v1.

Please let me know if there is any problem.
Thanks.

Inki Dae (2):
  drm/exynos: fixed page align and code clean.
  drm/exynos: fixed duplicated page allocation bug.

Joonyoung Shim (6):
  drm/exynos: remove unnecessary type conversion of hdmi and mixer
  drm/exynos: remove unused codes in hdmi and mixer
  drm/exynos: rename s/HDMI_OVERLAY_NUMBER/MIXER_WIN_NR
  drm/exynos: use define instead of default_win member in struct
    mixer_context
  drm/exynos: fix struct for operation callback functions to driver
    name
  drm/exynos: fix to pointer manager member of struct exynos_drm_subdrv

Seung-Woo Kim (1):
  drm/exynos: add format list of plane

 drivers/gpu/drm/exynos/exynos_drm_buf.c   |   47 ++++---------
 drivers/gpu/drm/exynos/exynos_drm_core.c  |   14 ++--
 drivers/gpu/drm/exynos/exynos_drm_drv.h   |   10 ++--
 drivers/gpu/drm/exynos/exynos_drm_fimd.c  |   20 +++--
 drivers/gpu/drm/exynos/exynos_drm_gem.c   |   45 ++++++++++---
 drivers/gpu/drm/exynos/exynos_drm_gem.h   |    2 +
 drivers/gpu/drm/exynos/exynos_drm_hdmi.c  |  107 +++++++++++++----------------
 drivers/gpu/drm/exynos/exynos_drm_hdmi.h  |   23 +++----
 drivers/gpu/drm/exynos/exynos_drm_plane.c |    4 +
 drivers/gpu/drm/exynos/exynos_drm_vidi.c  |   20 +++--
 drivers/gpu/drm/exynos/exynos_hdmi.c      |   42 ++++++------
 drivers/gpu/drm/exynos/exynos_mixer.c     |   40 +++++------
 include/drm/exynos_drm.h                  |    3 +-
 13 files changed, 191 insertions(+), 186 deletions(-)

-- 
1.7.4.1

[GIT PULL] updated exynos-drm-fixes

[Inki Dae]

Hi Dave,

Please pull from
	git://git.infradead.org/users/kmpark/linux-samsung exynos-drm-fixes

this branch is based on git repository below:
	git://people.freedesktop.org/~airlied/linux.git drm-fixes
	commit-id: 47819ba234d41465b76f179ba674ff549255a5d2

this patch set had been already posted several weeks ago and includes
minor fixes.

Thanks.

Inki Dae (1):
  drm/exynos: fixed size type.

Laurent Pinchart (4):
  drm/exynos: DRIVER_BUS_PLATFORM is not a driver feature
  drm/exynos: Don't cast GEM object to Exynos GEM object when not
    needed
  drm/exynos: Keep a reference to frame buffer GEM objects
  drm/exynos: Remove dummy encoder get_crtc operation implementation

Seung-Woo Kim (1):
  drm/exynos: fixed blending for hdmi graphic layer

Ville Syrjälä (1):
  drm/exynos: Use DRM_FORMAT_{NV12, YUV420} instead of
    DRM_FORMAT_{NV12M, YUV420M}

 drivers/gpu/drm/exynos/exynos_drm_drv.c     |    4 ++--
 drivers/gpu/drm/exynos/exynos_drm_encoder.c |    7 -------
 drivers/gpu/drm/exynos/exynos_drm_fb.c      |   19 ++++++++++++++-----
 drivers/gpu/drm/exynos/exynos_drm_fb.h      |    4 ++--
 drivers/gpu/drm/exynos/exynos_drm_gem.c     |    9 +++------
 drivers/gpu/drm/exynos/exynos_mixer.c       |   12 +++++++-----
 include/drm/exynos_drm.h                    |    4 +++-
 7 files changed, 31 insertions(+), 28 deletions(-)

-- 
1.7.4.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[GIT PULL] updated exynos-drm-fixes

[Inki Dae]

Hello Dave,

Please pull from
	git://git.infradead.org/users/kmpark/linux-samsung exynos-drm-fixes

these patch sets are based on git repository below:
	git://people.freedesktop.org/~airlied/linux.git drm-fixes
	commit-id: a3928c6b5f43bcd561802b52049b6024b8da675f

this pull requst includes previous one and new two patches posted last week
for review. so please ignore previous request and pull this one.
for previous request, you can refer to below link:
	http://comments.gmane.org/gmane.comp.video.dri.devel/71136

this patch set includes the following:
- change exynos gem allocation interface.
  . this patch makes non-contiguous memory allocation request allocate pages
    from highmem through alloc_page() with GFP_HIGHUSER_MOVABLE because
    cpu may read garbage data from cache once gpu operation is completed with
    allocated pages.
    shmem_read_mapping_page_gfp() first tries to allocate pages from page
    cache so if pages are alloated from page cache then these pages could
    have valid line and we had faced with this issue.
- check if framebuffer and gem size are valid or not.
  . with addfb or addfb2 request by user, invalid framebuffer or gem size
    could be sent into kernel side so this could induce invalid memory
    access by dma of a device. so this patch checks if framebuffer and
    gem size are valid or not.
- add exception codes to vidi module.
  . edid data from user should be allocatd and copied into vidi context and
    also freed with disconnection.
- and code clean.

please let me know it there is any problem.
Thanks.

Cooper Yuan (1):
  drm/exynos: fix buffer pitch calculation

Inki Dae (14):
  drm/exynos: removed unnecessary declaration.
  drm/exynos: set edid fake data only for test.
  drm/exynos: check if raw edid data is fake or not for test
  drm/exynos: fixed edid data setting at vidi connection request
  drm/exynos: fixed build warning.
  drm/exynos: use alloc_page() to allocate pages.
  drm/exynos: set buffer type from exporter.
  drm/exynos: do not release memory region from exporter.
  drm/exynos: removed unnecessary variable
  drm/exynos: fixed a comment to gem size.
  drm/exynos: add packed_size not aligned in page unit.
  drm/exynos: check if framebuffer and gem size are valid or not.
  drm/exynos: use __free_page() to deallocate memory
  drm/exynos: fixed exception to page allocation failure

Sachin Kamat (4):
  drm/exynos: Add missing static storage class specifier
  drm/exynos: Use devm_* functions in exynos_drm_fimd.c
  drm/exynos: Use devm_* functions in exynos_hdmi.c
  drm/exynos: Use devm_* functions in exynos_mixer.c

Subash Patel (2):
  drm/exynos: return NULL if exynos_pages_to_sg fails
  drm/exynos: check for null in return value of
    dma_buf_map_attachment()

 drivers/gpu/drm/exynos/exynos_drm_connector.c |    3 +-
 drivers/gpu/drm/exynos/exynos_drm_core.c      |    5 --
 drivers/gpu/drm/exynos/exynos_drm_dmabuf.c    |   33 ++++++++++++----
 drivers/gpu/drm/exynos/exynos_drm_fb.c        |   47 +++++++++++++++++++++-
 drivers/gpu/drm/exynos/exynos_drm_fimd.c      |   40 ++++---------------
 drivers/gpu/drm/exynos/exynos_drm_gem.c       |   51 +++++++++++-------------
 drivers/gpu/drm/exynos/exynos_drm_gem.h       |    6 ++-
 drivers/gpu/drm/exynos/exynos_drm_vidi.c      |   53 +++++++++++++++++++------
 drivers/gpu/drm/exynos/exynos_hdmi.c          |   36 +++-------------
 drivers/gpu/drm/exynos/exynos_mixer.c         |   48 +++++++----------------
 10 files changed, 170 insertions(+), 152 deletions(-)

-- 
1.7.4.1

Re: [GIT PULL] updated exynos-drm-fixes

[Dave Airlie]

Hi Inki,

how urgent is this? Linus isn't really up for something this large
post -rc7, major regressions or oops fixes only at this point.

If its not urgent can we just stick it in -next and if needs be you
can get stable on required patches.

Dave.

> Please pull from
>         git://git.infradead.org/users/kmpark/linux-samsung exynos-drm-fixes
>
> these patch sets are based on git repository below:
>         git://people.freedesktop.org/~airlied/linux.git drm-fixes
>         commit-id: a3928c6b5f43bcd561802b52049b6024b8da675f
>
> this pull requst includes previous one and new two patches posted last week
> for review. so please ignore previous request and pull this one.
> for previous request, you can refer to below link:
>         http://comments.gmane.org/gmane.comp.video.dri.devel/71136
>
> this patch set includes the following:
> - change exynos gem allocation interface.
>   . this patch makes non-contiguous memory allocation request allocate pages
>     from highmem through alloc_page() with GFP_HIGHUSER_MOVABLE because
>     cpu may read garbage data from cache once gpu operation is completed with
>     allocated pages.
>     shmem_read_mapping_page_gfp() first tries to allocate pages from page
>     cache so if pages are alloated from page cache then these pages could
>     have valid line and we had faced with this issue.
> - check if framebuffer and gem size are valid or not.
>   . with addfb or addfb2 request by user, invalid framebuffer or gem size
>     could be sent into kernel side so this could induce invalid memory
>     access by dma of a device. so this patch checks if framebuffer and
>     gem size are valid or not.
> - add exception codes to vidi module.
>   . edid data from user should be allocatd and copied into vidi context and
>     also freed with disconnection.
> - and code clean.
>
> please let me know it there is any problem.
> Thanks.
>
> Cooper Yuan (1):
>   drm/exynos: fix buffer pitch calculation
>
> Inki Dae (14):
>   drm/exynos: removed unnecessary declaration.
>   drm/exynos: set edid fake data only for test.
>   drm/exynos: check if raw edid data is fake or not for test
>   drm/exynos: fixed edid data setting at vidi connection request
>   drm/exynos: fixed build warning.
>   drm/exynos: use alloc_page() to allocate pages.
>   drm/exynos: set buffer type from exporter.
>   drm/exynos: do not release memory region from exporter.
>   drm/exynos: removed unnecessary variable
>   drm/exynos: fixed a comment to gem size.
>   drm/exynos: add packed_size not aligned in page unit.
>   drm/exynos: check if framebuffer and gem size are valid or not.
>   drm/exynos: use __free_page() to deallocate memory
>   drm/exynos: fixed exception to page allocation failure
>
> Sachin Kamat (4):
>   drm/exynos: Add missing static storage class specifier
>   drm/exynos: Use devm_* functions in exynos_drm_fimd.c
>   drm/exynos: Use devm_* functions in exynos_hdmi.c
>   drm/exynos: Use devm_* functions in exynos_mixer.c
>
> Subash Patel (2):
>   drm/exynos: return NULL if exynos_pages_to_sg fails
>   drm/exynos: check for null in return value of
>     dma_buf_map_attachment()
>
>  drivers/gpu/drm/exynos/exynos_drm_connector.c |    3 +-
>  drivers/gpu/drm/exynos/exynos_drm_core.c      |    5 --
>  drivers/gpu/drm/exynos/exynos_drm_dmabuf.c    |   33 ++++++++++++----
>  drivers/gpu/drm/exynos/exynos_drm_fb.c        |   47 +++++++++++++++++++++-
>  drivers/gpu/drm/exynos/exynos_drm_fimd.c      |   40 ++++---------------
>  drivers/gpu/drm/exynos/exynos_drm_gem.c       |   51 +++++++++++-------------
>  drivers/gpu/drm/exynos/exynos_drm_gem.h       |    6 ++-
>  drivers/gpu/drm/exynos/exynos_drm_vidi.c      |   53 +++++++++++++++++++------
>  drivers/gpu/drm/exynos/exynos_hdmi.c          |   36 +++-------------
>  drivers/gpu/drm/exynos/exynos_mixer.c         |   48 +++++++----------------
>  10 files changed, 170 insertions(+), 152 deletions(-)
>
> --
> 1.7.4.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

RE: [GIT PULL] updated exynos-drm-fixes

[Inki Dae]

Hi Dave,

> -----Original Message-----
> From: Dave Airlie [mailto:airlied@gmail.com]
> Sent: Wednesday, July 18, 2012 5:14 PM
> To: Inki Dae
> Cc: airlied@linux.ie; dri-devel@lists.freedesktop.org;
> kyungmin.park@samsung.com; sw0312.kim@samsung.com
> Subject: Re: [GIT PULL] updated exynos-drm-fixes
> 
> Hi Inki,
> 
> how urgent is this? Linus isn't really up for something this large
> post -rc7, major regressions or oops fixes only at this point.
> 
> If its not urgent can we just stick it in -next and if needs be you
> can get stable on required patches.
>

Understood. we aren't urgent. so I will merge this patch set to
exynos-drm-next and then request pull again so that they can be merged to
drm-core-next.

Thanks,
Inki Dae.

> Dave.
> 
> > Please pull from
> >         git://git.infradead.org/users/kmpark/linux-samsung exynos-drm-
> fixes
> >
> > these patch sets are based on git repository below:
> >         git://people.freedesktop.org/~airlied/linux.git drm-fixes
> >         commit-id: a3928c6b5f43bcd561802b52049b6024b8da675f
> >
> > this pull requst includes previous one and new two patches posted last
> week
> > for review. so please ignore previous request and pull this one.
> > for previous request, you can refer to below link:
> >         http://comments.gmane.org/gmane.comp.video.dri.devel/71136
> >
> > this patch set includes the following:
> > - change exynos gem allocation interface.
> >   . this patch makes non-contiguous memory allocation request allocate
> pages
> >     from highmem through alloc_page() with GFP_HIGHUSER_MOVABLE because
> >     cpu may read garbage data from cache once gpu operation is completed
> with
> >     allocated pages.
> >     shmem_read_mapping_page_gfp() first tries to allocate pages from
page
> >     cache so if pages are alloated from page cache then these pages
could
> >     have valid line and we had faced with this issue.
> > - check if framebuffer and gem size are valid or not.
> >   . with addfb or addfb2 request by user, invalid framebuffer or gem
> size
> >     could be sent into kernel side so this could induce invalid memory
> >     access by dma of a device. so this patch checks if framebuffer and
> >     gem size are valid or not.
> > - add exception codes to vidi module.
> >   . edid data from user should be allocatd and copied into vidi context
> and
> >     also freed with disconnection.
> > - and code clean.
> >
> > please let me know it there is any problem.
> > Thanks.
> >
> > Cooper Yuan (1):
> >   drm/exynos: fix buffer pitch calculation
> >
> > Inki Dae (14):
> >   drm/exynos: removed unnecessary declaration.
> >   drm/exynos: set edid fake data only for test.
> >   drm/exynos: check if raw edid data is fake or not for test
> >   drm/exynos: fixed edid data setting at vidi connection request
> >   drm/exynos: fixed build warning.
> >   drm/exynos: use alloc_page() to allocate pages.
> >   drm/exynos: set buffer type from exporter.
> >   drm/exynos: do not release memory region from exporter.
> >   drm/exynos: removed unnecessary variable
> >   drm/exynos: fixed a comment to gem size.
> >   drm/exynos: add packed_size not aligned in page unit.
> >   drm/exynos: check if framebuffer and gem size are valid or not.
> >   drm/exynos: use __free_page() to deallocate memory
> >   drm/exynos: fixed exception to page allocation failure
> >
> > Sachin Kamat (4):
> >   drm/exynos: Add missing static storage class specifier
> >   drm/exynos: Use devm_* functions in exynos_drm_fimd.c
> >   drm/exynos: Use devm_* functions in exynos_hdmi.c
> >   drm/exynos: Use devm_* functions in exynos_mixer.c
> >
> > Subash Patel (2):
> >   drm/exynos: return NULL if exynos_pages_to_sg fails
> >   drm/exynos: check for null in return value of
> >     dma_buf_map_attachment()
> >
> >  drivers/gpu/drm/exynos/exynos_drm_connector.c |    3 +-
> >  drivers/gpu/drm/exynos/exynos_drm_core.c      |    5 --
> >  drivers/gpu/drm/exynos/exynos_drm_dmabuf.c    |   33 ++++++++++++----
> >  drivers/gpu/drm/exynos/exynos_drm_fb.c        |   47
> +++++++++++++++++++++-
> >  drivers/gpu/drm/exynos/exynos_drm_fimd.c      |   40
++++---------------
> >  drivers/gpu/drm/exynos/exynos_drm_gem.c       |   51
+++++++++++----------
> ---
> >  drivers/gpu/drm/exynos/exynos_drm_gem.h       |    6 ++-
> >  drivers/gpu/drm/exynos/exynos_drm_vidi.c      |   53
+++++++++++++++++++-
> -----
> >  drivers/gpu/drm/exynos/exynos_hdmi.c          |   36 +++-------------
> >  drivers/gpu/drm/exynos/exynos_mixer.c         |   48
+++++++--------------
> --
> >  10 files changed, 170 insertions(+), 152 deletions(-)
> >
> > --
> > 1.7.4.1
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel

[git pull] updated exynos-drm-fixes

[inki.dae]

Hi Dave,

As I posted before, we have added a new git repository for Exynos drm
to MAINTAINERS file so change it to new one like below,
   from git://git.infradead.org/users/kmpark/linux-samsung
   to git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos

And this pull request includes the following:
- fix display on issue when user requested dpms mode changing.
- add git repository for Exynos drm to MAINTAINERS file.
- add support for ARCH_MULTIPLATFORM.
- and code clean.

If there is any problem, please let me know.

Thanks,
Inki Dae

The following changes since commit 4936b172d699434547addbe452c2d600ea6a4baf:

  Merge branch 'drm-nouveau-fixes' of git://people.freedesktop.org/git/nouveau/linux-2.6 into drm-fixes (2012-10-31 13:46:09 +1000)

are available in the git repository at:


  git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos.git exynos-drm-fixes

for you to fetch changes up to 2cdc53b355873ab54b6a348ee5bf022967fb4159:

  drm: exynos: removed warning due to missing typecast for mixer driver data (2012-10-31 06:11:40 -0700)

----------------------------------------------------------------
Inki Dae (2):
      drm/exynos: fix display on issue
      MAINTAINERS: Add git repository for Exynos DRM

Rahul Sharma (1):
      drm: exynos: removed warning due to missing typecast for mixer driver data

Rob Clark (1):
      drm/exynos: add support for ARCH_MULTIPLATFORM

 MAINTAINERS                                   |  1 +
 drivers/gpu/drm/exynos/Kconfig                |  2 +-
 drivers/gpu/drm/exynos/exynos_drm_connector.c |  1 +
 drivers/gpu/drm/exynos/exynos_drm_encoder.c   | 33 ++++++++++++++-------------
 drivers/gpu/drm/exynos/exynos_mixer.c         |  2 +-
 5 files changed, 21 insertions(+), 18 deletions(-)

Re: [git pull] updated exynos-drm-fixes

[Inki Dae]

Hi Dave,

The below patch, "add support for ARCH_MULTIPLATFORM", makes exynos
drm driver to be complied for testing on different ARM DRM devices
without having to switch configs.
For this, you can refer to below link.
                    http://www.spinics.net/lists/dri-devel/msg29365.html

And this just flags ARCH_MULTIPLATFORM in Kconfig.

Thanks,
Inki Dae

2012/10/31  <inki.dae@samsung.com>:
> Hi Dave,
>
> As I posted before, we have added a new git repository for Exynos drm
> to MAINTAINERS file so change it to new one like below,
>    from git://git.infradead.org/users/kmpark/linux-samsung
>    to git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos
>
> And this pull request includes the following:
> - fix display on issue when user requested dpms mode changing.
> - add git repository for Exynos drm to MAINTAINERS file.
> - add support for ARCH_MULTIPLATFORM.
> - and code clean.
>
> If there is any problem, please let me know.
>
> Thanks,
> Inki Dae
>
> The following changes since commit 4936b172d699434547addbe452c2d600ea6a4baf:
>
>   Merge branch 'drm-nouveau-fixes' of git://people.freedesktop.org/git/nouveau/linux-2.6 into drm-fixes (2012-10-31 13:46:09 +1000)
>
> are available in the git repository at:
>
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos.git exynos-drm-fixes
>
> for you to fetch changes up to 2cdc53b355873ab54b6a348ee5bf022967fb4159:
>
>   drm: exynos: removed warning due to missing typecast for mixer driver data (2012-10-31 06:11:40 -0700)
>
> ----------------------------------------------------------------
> Inki Dae (2):
>       drm/exynos: fix display on issue
>       MAINTAINERS: Add git repository for Exynos DRM
>
> Rahul Sharma (1):
>       drm: exynos: removed warning due to missing typecast for mixer driver data
>
> Rob Clark (1):
>       drm/exynos: add support for ARCH_MULTIPLATFORM
>
>  MAINTAINERS                                   |  1 +
>  drivers/gpu/drm/exynos/Kconfig                |  2 +-
>  drivers/gpu/drm/exynos/exynos_drm_connector.c |  1 +
>  drivers/gpu/drm/exynos/exynos_drm_encoder.c   | 33 ++++++++++++++-------------
>  drivers/gpu/drm/exynos/exynos_mixer.c         |  2 +-
>  5 files changed, 21 insertions(+), 18 deletions(-)