On Mon, 07 Nov 2011 20:26:59 +0000
Chris Wilson <ch...@chris-wilson.co.uk> wrote:

> On Mon,  7 Nov 2011 12:03:15 -0800, Jesse Barnes <jbar...@virtuousgeek.org> 
> wrote:
> > Also fix up the wrapping to 80 columns.
> > 
> > Signed-off-by: Jesse Barnes <jbar...@virtuousgeek.org>
> > ---
> >  include/drm/drm_crtc.h |   20 +++++++-------------
> >  1 files changed, 7 insertions(+), 13 deletions(-)
> > 
> > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > index ad2a605..6ab20f8 100644
> > --- a/include/drm/drm_crtc.h
> > +++ b/include/drm/drm_crtc.h
> > @@ -386,26 +386,19 @@ struct drm_crtc {
> >  /**
> >   * drm_connector_funcs - control connectors on a given device
> >   * @dpms: set power state (see drm_crtc_funcs above)
> > - * @save: save connector state
> > - * @restore: restore connector state
> >   * @reset: reset connector after state has been invalidate (e.g. resume)
> > - * @mode_valid: is this mode valid on the given connector?
> > - * @mode_fixup: try to fixup proposed mode for this connector
> > - * @mode_set: set this mode
> >   * @detect: is this connector active?
> > - * @get_modes: get mode list for this connector
> > + * @fill_modes: build a mode list for this connector
> >   * @set_property: property for this connector may need update
> >   * @destroy: make object go away
> >   * @force: notify the driver the connector is forced on
> >   *
> >   * Each CRTC may have one or more connectors attached to it.  The functions
> > - * below allow the core DRM code to control connectors, enumerate 
> > available modes,
> > - * etc.
> > + * below allow the core DRM code to control connectors, enumerate available
> > + * modes, etc.
> >   */
> >  struct drm_connector_funcs {
> >     void (*dpms)(struct drm_connector *connector, int mode);
> > -   void (*save)(struct drm_connector *connector);
> > -   void (*restore)(struct drm_connector *connector);
> >     void (*reset)(struct drm_connector *connector);
> >  
> >     /* Check to see if anything is attached to the connector.
> > @@ -416,9 +409,10 @@ struct drm_connector_funcs {
> >      */
> >     enum drm_connector_status (*detect)(struct drm_connector *connector,
> >                                         bool force);
> > -   int (*fill_modes)(struct drm_connector *connector, uint32_t max_width, 
> > uint32_t max_height);
> > -   int (*set_property)(struct drm_connector *connector, struct 
> > drm_property *property,
> > -                        uint64_t val);
> > +   int (*fill_modes)(struct drm_connector *connector, uint32_t max_width,
> > +                     uint32_t max_height);
> 
> Attack of the random 80-column limit!
> 
> int (*fill_modes)(struct drm_connector *connector,
>                 uint32_t max_width,  uint32_t max_height);
> 
> keeps the logically more connected width&height together.
> 
> What colour do you want your bikeshed today?

/me burns down the bikeshed.

-- 
Jesse Barnes, Intel Open Source Technology Center

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to