[PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-02-12 Thread Daniel Vetter
On Tue, Feb 12, 2013 at 1:24 AM, Rob Clark wrote: > On Thu, Jan 24, 2013 at 10:20 AM, Daniel Vetter > wrote: >> No need to check whether we've allocated a new fb since we're not >> always doing that. Also, we always need to register the fbdev and add >> it to the panic notifier. > > hmm, how is

[PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-02-11 Thread Rob Clark
On Mon, Feb 11, 2013 at 7:33 PM, Daniel Vetter wrote: > On Tue, Feb 12, 2013 at 1:24 AM, Rob Clark wrote: >> On Thu, Jan 24, 2013 at 10:20 AM, Daniel Vetter >> wrote: >>> No need to check whether we've allocated a new fb since we're not >>> always doing that. Also, we always need to register t

[PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-02-11 Thread Rob Clark
On Thu, Jan 24, 2013 at 10:20 AM, Daniel Vetter wrote: > No need to check whether we've allocated a new fb since we're not > always doing that. Also, we always need to register the fbdev and add > it to the panic notifier. hmm, how is this not leading to a register_framebuffer() on every hotplug

Re: [PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-02-11 Thread Rob Clark
On Mon, Feb 11, 2013 at 7:33 PM, Daniel Vetter wrote: > On Tue, Feb 12, 2013 at 1:24 AM, Rob Clark wrote: >> On Thu, Jan 24, 2013 at 10:20 AM, Daniel Vetter >> wrote: >>> No need to check whether we've allocated a new fb since we're not >>> always doing that. Also, we always need to register th

Re: [PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-02-11 Thread Daniel Vetter
On Tue, Feb 12, 2013 at 1:24 AM, Rob Clark wrote: > On Thu, Jan 24, 2013 at 10:20 AM, Daniel Vetter > wrote: >> No need to check whether we've allocated a new fb since we're not >> always doing that. Also, we always need to register the fbdev and add >> it to the panic notifier. > > hmm, how is

Re: [PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-02-11 Thread Rob Clark
On Thu, Jan 24, 2013 at 10:20 AM, Daniel Vetter wrote: > No need to check whether we've allocated a new fb since we're not > always doing that. Also, we always need to register the fbdev and add > it to the panic notifier. hmm, how is this not leading to a register_framebuffer() on every hotplug

[PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-01-24 Thread Daniel Vetter
No need to check whether we've allocated a new fb since we're not always doing that. Also, we always need to register the fbdev and add it to the panic notifier. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_fb_helper.c | 29 +++-- 1 file changed, 15 insertions(+

[PATCH 13/16] drm/fb-helper: streamline drm_fb_helper_single_fb_probe

2013-01-24 Thread Daniel Vetter
No need to check whether we've allocated a new fb since we're not always doing that. Also, we always need to register the fbdev and add it to the panic notifier. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_fb_helper.c | 29 +++-- 1 file changed, 15 insertions(+