On Fri, Dec 14, 2012 at 5:57 AM, Inki Dae <inki.dae at samsung.com> wrote: > 2012/12/14 Daniel Vetter <daniel.vetter at ffwll.ch> >> >> On Thu, Dec 13, 2012 at 4:16 PM, Inki Dae <inki.dae at samsung.com> wrote: >> > How about rebasing this patch to top of >> > git://people.freedesktop.org/~airlied/linux.git drm-next? >> > Exynos's many patches have already been merged to drm-next. Or if you >> > are >> > ok, I'd like to rebase your patch to -next and test it. I don't care >> > either >> > way. :) >> >> I've pushed out a rebased version, but somehow I can't compile-test >> exynos any more :( Probably missing some other stuff from linux-next. >> Anyway, I don't mind which patch you pick or whether you just roll >> your own, the important thing is that drm_framebuffer_init is called >> _after_ everything is set up and initialized. Everything else doesn't >> really matter. > > > I've rebase your patch to -next and tested it. But your patch had null > pointer issue to gem object so I fixed it. > For this, you can refer to the below link, > > http://git.kernel.org/?p=linux/kernel/git/daeinki/drm-exynos.git;a=shortlog;h=refs/heads/exynos-drm-next > > And I gonna request git pull to -next including that patch today. If there > is any problem, please let me know. Oops, sorry for missing the gem obj assignment, your updated version looks good. Thanks for fixing it up. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch