[PATCH] drm/fb-helper: fixup set_config semantics

2013-02-12 Thread Daniel Vetter
While doing the modeset rework for drm/i915 I've noticed that the fb helper is very liberal with the semantics of the ->set_config interface: - It doesn't bother clearing stale modes (e.g. when unplugging a screen). - It unconditionally sets the fb, even if no mode will be set on a given crtc.

[PATCH] drm/fb-helper: fixup set_config semantics

2013-02-12 Thread Daniel Vetter
While doing the modeset rework for drm/i915 I've noticed that the fb helper is very liberal with the semantics of the ->set_config interface: - It doesn't bother clearing stale modes (e.g. when unplugging a screen). - It unconditionally sets the fb, even if no mode will be set on a given crtc.

[PATCH] drm/fb-helper: fixup set_config semantics

2013-02-11 Thread Rob Clark
On Mon, Feb 11, 2013 at 7:25 PM, Daniel Vetter wrote: > While doing the modeset rework for drm/i915 I've noticed that the fb > helper is very liberal with the semantics of the ->set_config > interface: > - It doesn't bother clearing stale modes (e.g. when unplugging a > screen). > - It uncondit

Re: [PATCH] drm/fb-helper: fixup set_config semantics

2013-02-11 Thread Rob Clark
On Mon, Feb 11, 2013 at 7:25 PM, Daniel Vetter wrote: > While doing the modeset rework for drm/i915 I've noticed that the fb > helper is very liberal with the semantics of the ->set_config > interface: > - It doesn't bother clearing stale modes (e.g. when unplugging a > screen). > - It unconditi

[PATCH] drm/fb-helper: fixup set_config semantics

2013-02-11 Thread Daniel Vetter
While doing the modeset rework for drm/i915 I've noticed that the fb helper is very liberal with the semantics of the ->set_config interface: - It doesn't bother clearing stale modes (e.g. when unplugging a screen). - It unconditionally sets the fb, even if no mode will be set on a given crtc.

[PATCH] drm/fb-helper: fixup set_config semantics

2013-02-11 Thread Daniel Vetter
While doing the modeset rework for drm/i915 I've noticed that the fb helper is very liberal with the semantics of the ->set_config interface: - It doesn't bother clearing stale modes (e.g. when unplugging a screen). - It unconditionally sets the fb, even if no mode will be set on a given crtc.