question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-09 Thread Julia Lawall
On Mon, 9 Jul 2012, Patrik Jakobsson wrote: > On Sun, Jul 8, 2012 at 10:16 PM, Alan Cox wrote: >> On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) >> Julia Lawall wrote: >> >>> In the function oaktrail_lvds_mode_set, I don't think that the following >>> code makes any sense: >>> >>> /* Find

question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-09 Thread Patrik Jakobsson
On Sun, Jul 8, 2012 at 10:16 PM, Alan Cox wrote: > On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) > Julia Lawall wrote: > >> In the function oaktrail_lvds_mode_set, I don't think that the following >> code makes any sense: >> >> /* Find the connector we're trying to set up */ >>

Re: question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-09 Thread Julia Lawall
On Sun, 8 Jul 2012, Alan Cox wrote: On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) Julia Lawall julia.law...@lip6.fr wrote: In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense: /* Find the connector we're trying to set up */

Re: question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-09 Thread Julia Lawall
On Mon, 9 Jul 2012, Patrik Jakobsson wrote: On Sun, Jul 8, 2012 at 10:16 PM, Alan Cox a...@lxorguk.ukuu.org.uk wrote: On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) Julia Lawall julia.law...@lip6.fr wrote: In the function oaktrail_lvds_mode_set, I don't think that the following code makes any

question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-08 Thread Julia Lawall
On Sun, 8 Jul 2012, Alan Cox wrote: > On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) > Julia Lawall wrote: > >> In the function oaktrail_lvds_mode_set, I don't think that the following >> code makes any sense: >> >> /* Find the connector we're trying to set up */ >>

question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-08 Thread Alan Cox
On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) Julia Lawall wrote: > In the function oaktrail_lvds_mode_set, I don't think that the following > code makes any sense: > > /* Find the connector we're trying to set up */ > list_for_each_entry(connector, _config->connector_list, head)

question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-08 Thread Julia Lawall
In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense: /* Find the connector we're trying to set up */ list_for_each_entry(connector, _config->connector_list, head) { if (!connector->encoder || connector->encoder->crtc !=

question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-08 Thread Julia Lawall
In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense: /* Find the connector we're trying to set up */ list_for_each_entry(connector, mode_config-connector_list, head) { if (!connector-encoder || connector-encoder-crtc !=

Re: question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-08 Thread Alan Cox
On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) Julia Lawall julia.law...@lip6.fr wrote: In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense: /* Find the connector we're trying to set up */ list_for_each_entry(connector,

Re: question about drivers/gpu/drm/gma500/oaktrail_lvds.c

2012-07-08 Thread Patrik Jakobsson
On Sun, Jul 8, 2012 at 10:16 PM, Alan Cox a...@lxorguk.ukuu.org.uk wrote: On Sun, 8 Jul 2012 10:39:43 +0200 (CEST) Julia Lawall julia.law...@lip6.fr wrote: In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense: /* Find the connector we're