[PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-10 Thread Jani Nikula
On Wed, 08 May 2013, Ville Syrj?l? wrote: > On Wed, May 08, 2013 at 05:03:31PM +0100, Damien Lespiau wrote: >> +static const char *connector_status_str(enum drm_connector_status status) >> +{ >> +switch (status) { >> +case connector_status_connected: >> +return "connected"; >>

Re: [PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-09 Thread Jani Nikula
On Wed, 08 May 2013, Ville Syrjälä wrote: > On Wed, May 08, 2013 at 05:03:31PM +0100, Damien Lespiau wrote: >> +static const char *connector_status_str(enum drm_connector_status status) >> +{ >> +switch (status) { >> +case connector_status_connected: >> +return "connected"; >>

[PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Ville Syrjälä
On Wed, May 08, 2013 at 05:03:31PM +0100, Damien Lespiau wrote: > Instead of just printing "status updated from 1 to 2", make those enum > numbers immediately readable. > > v2: Also patch output_poll_execute() (Daniel Vetter) > > Signed-off-by: Damien Lespiau > Reviewed-by: Jesse Barnes > --- >

[PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Chris Wilson
On Wed, May 08, 2013 at 05:03:31PM +0100, Damien Lespiau wrote: > > Instead of just printing "status updated from 1 to 2", make those enum > numbers immediately readable. > > v2: Also patch output_poll_execute() (Daniel Vetter) For bonus points, only emit the message when it is updated. -Chris

[PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Damien Lespiau
Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. v2: Also patch output_poll_execute() (Daniel Vetter) Signed-off-by: Damien Lespiau Reviewed-by: Jesse Barnes --- drivers/gpu/drm/drm_crtc_helper.c | 22 ++ 1 file changed, 1

Re: [PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Ville Syrjälä
On Wed, May 08, 2013 at 05:03:31PM +0100, Damien Lespiau wrote: > Instead of just printing "status updated from 1 to 2", make those enum > numbers immediately readable. > > v2: Also patch output_poll_execute() (Daniel Vetter) > > Signed-off-by: Damien Lespiau > Reviewed-by: Jesse Barnes > --- >

Re: [PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Chris Wilson
On Wed, May 08, 2013 at 05:03:31PM +0100, Damien Lespiau wrote: > > Instead of just printing "status updated from 1 to 2", make those enum > numbers immediately readable. > > v2: Also patch output_poll_execute() (Daniel Vetter) For bonus points, only emit the message when it is updated. -Chris

[PATCH 2/5] drm: Make the HPD status updates debug logs more readable

2013-05-08 Thread Damien Lespiau
Instead of just printing "status updated from 1 to 2", make those enum numbers immediately readable. v2: Also patch output_poll_execute() (Daniel Vetter) Signed-off-by: Damien Lespiau Reviewed-by: Jesse Barnes --- drivers/gpu/drm/drm_crtc_helper.c | 22 ++ 1 file changed, 1