Hi, Kamil. Sorry. our patch has no problem and this is my mistake. so your patch has been merged to our git repository.
Thanks. > -----Original Message----- > From: Inki Dae [mailto:inki.dae at samsung.com] > Sent: Tuesday, February 14, 2012 5:58 PM > To: 'Kamil Debski'; 'dri-devel at lists.freedesktop.org' > Cc: 'kyungmin.park at samsung.com'; 'jy0922.shim at samsung.com'; > 'sw0312.kim at samsung.com'; 'airlied at linux.ie' > Subject: RE: [PATCH] drm/exynos: exynos_drm.h header file fixes > > Hi, Kamil. > Sorry for being late. > > Below is my comments. > > > -----Original Message----- > > From: Kamil Debski [mailto:k.debski at samsung.com] > > Sent: Saturday, February 11, 2012 2:32 AM > > To: dri-devel at lists.freedesktop.org > > Cc: kyungmin.park at samsung.com; inki.dae at samsung.com; > > jy0922.shim at samsung.com; sw0312.kim at samsung.com; airlied at linux.ie; > > Kamil > > Debski > > Subject: [PATCH] drm/exynos: exynos_drm.h header file fixes > > > > First of all #ifdef __KERNEL__ was added to exynos_drm.h to > > mark the part that should be left out of userspace. > > Secondly exynos_drm.h was added to include/drm/Kbuild, so it > > will be included when doing make headers_install. > > > > Signed-off-by: Kamil Debski <k.debski at samsung.com> > > Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com> > > --- > > include/drm/Kbuild | 1 + > > include/drm/exynos_drm.h | 5 +++-- > > 2 files changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/include/drm/Kbuild b/include/drm/Kbuild > > index a5c0e10..1e38a19 100644 > > --- a/include/drm/Kbuild > > +++ b/include/drm/Kbuild > > @@ -2,6 +2,7 @@ header-y += drm.h > > header-y += drm_fourcc.h > > header-y += drm_mode.h > > header-y += drm_sarea.h > > +header-y += exynos_drm.h > > header-y += i810_drm.h > > header-y += i915_drm.h > > header-y += mga_drm.h > > diff --git a/include/drm/exynos_drm.h b/include/drm/exynos_drm.h > > index 13b8660..5900e0b 100644 > > --- a/include/drm/exynos_drm.h > > +++ b/include/drm/exynos_drm.h > > @@ -99,6 +99,7 @@ struct drm_exynos_plane_set_zpos { > > #define DRM_IOCTL_EXYNOS_PLANE_SET_ZPOS DRM_IOWR(DRM_COMMAND_BASE + > > \ > > DRM_EXYNOS_PLANE_SET_ZPOS, struct drm_exynos_plane_set_zpos) > > > > +#ifdef __KERNEL__ > > could you please modify it for this macro to be place above of struct > exynos_drm_panel_info? struct exynos_drm_panel_info also isn't needed for > user side. > > Thanks. > > > /** > > * Platform Specific Structure for DRM based FIMD. > > * > > @@ -140,5 +141,5 @@ struct exynos_drm_hdmi_pdata { > > unsigned int default_win; > > unsigned int bpp; > > }; > > - > > -#endif > > +#endif /* __KERNEL__ */ > > +#endif /* _EXYNOS_DRM_H_ */ > > -- > > 1.7.0.4
