Re: [PATCH v10 02/17] drm/exynos: use adjusted_mode of crtc_state instead of mode

[Gustavo Padovan <gustavo@padovan.org>]

Hi Inki,

2015-06-02 Inki Dae <inki.dae@samsung.com>:

> On 2015년 06월 02일 09:03, Joonyoung Shim wrote:
> > On 06/02/2015 12:09 AM, Tobias Jakobi wrote:
> >> Hello,
> >>
> >> as pointed out in [1] before, this gives me an kernel oops during boot.
> >>
> >> With best wishes,
> >> Tobias
> >>
> >> [1] http://www.spinics.net/lists/linux-samsung-soc/msg44736.html
> >>
> >
> > Yeah, this patch should go after switched by drm_helper_crtc_mode_set,
> > e.g. after the patch "drm/exynos: atomic phase 1: add .mode_set_nofb()
> > callback". Then crtc->base.state cannot be NULL.
> >
> > Gustavo, sorry for that, i meant rebase based on the patch "drm/exynos:
> > atomic phase 1: add .mode_set_nofb() callback"
> >
> > Inki, i think it's time to merge this patchset for switch to atomic
> > modeset functions if no any objection, just need to rebase of this patch
> > based on the patch "drm/exynos: atomic phase 1 : add .mode_set_nofb()
> > callback" and except a patch 17/17 from this merge, i think clean up
> > patches like it need to more review after merge atomic patchset.
> 
> Thanks Joonyoung. Got it. I will merge this patch series to my local
> repository for test and final review. If there is no any big problem,
> will merge it.

Thanks for that! As I will keep working on exynos I will be around to
send any follow up patch to send any follow up patch to fix potential
issues of the atomic modesetting patches.

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