[PATCH] drm_crtc.c: fix some typo mistake and some annotations mistake

John Hunter zhjwpku at gmail.com
Thu Mar 12 19:53:00 PDT 2015


Got it, maybe I should work on the drm-next or latest linux master.

I am working on the drm git repo below for now as Daniel Vetter told me
git://people.freedesktop.org/~airlied/linux
I am quite confused now 'cause there are a bounch of git repo, I really
do not know which one should I git clone.


On Fri, Mar 13, 2015 at 10:33 AM, Rob Clark <robdclark at gmail.com> wrote:

> On Thu, Mar 12, 2015 at 4:02 AM, John Hunter <zhaojunwang at pku.edu.cn>
> wrote:
> > From: John Hunter <zhjwpku at gmail.com>
> >
> > IMHO, some annotations were copypaste from somewhere else, it is
> > obviously not rightly modified. Hope I am right with that.
> > ---
> >  drivers/gpu/drm/drm_crtc.c | 115
> ++++++++++++++++++++++-----------------------
> >  1 file changed, 56 insertions(+), 59 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> > index f2d667b..7f771e9 100644
> > --- a/drivers/gpu/drm/drm_crtc.c
> > +++ b/drivers/gpu/drm/drm_crtc.c
> > @@ -205,12 +205,12 @@ char *drm_get_connector_status_name(enum
> drm_connector_status status)
> >  /**
> >   * drm_mode_object_get - allocate a new identifier
> >   * @dev: DRM device
> > - * @ptr: object pointer, used to generate unique ID
> > - * @type: object type
> > + * @obj: object pointer, used to generate unique ID
> > + * @obj_type: object type
> >   *
> >   * LOCKING:
> >   *
> > - * Create a unique identifier based on @ptr in @dev's identifier
> space.  Used
> > + * Create a unique identifier based on @obj in @dev's identifier
> space.  Used
> >   * for tracking modes, CRTCs and connectors.
>
> so, I don't suppose you can rebase against a newer upstream kernel?
>
> From a quick look, it seems like the arg name errors where corrected
> in 065a50ed3ef75cb265e12e3e1b615db0835150bc while the description
> bodies fixes from your patch are still applicable.  In the current
> form, the patch does not apply against drm-next (or latest linux
> master), but most of it is still applicable and would be a very
> welcome improvement.
>
> Thanks :-)
>
> BR,
> -R
>
>
> >   *
> >   * RETURNS:
> > @@ -245,12 +245,12 @@ again:
> >  /**
> >   * drm_mode_object_put - free an identifer
> >   * @dev: DRM device
> > - * @id: ID to free
> > + * @object: object to free
> >   *
> >   * LOCKING:
> >   * Caller must hold DRM mode_config lock.
> >   *
> > - * Free @id from @dev's unique identifier pool.
> > + * Free @object from @dev's unique identifier pool.
> >   */
> >  static void drm_mode_object_put(struct drm_device *dev,
> >                                 struct drm_mode_object *object)
> > @@ -353,10 +353,11 @@ EXPORT_SYMBOL(drm_framebuffer_reference);
> >  void drm_framebuffer_cleanup(struct drm_framebuffer *fb)
> >  {
> >         struct drm_device *dev = fb->dev;
> > +
> >         /*
> >          * This could be moved to drm_framebuffer_remove(), but for
> >          * debugging is nice to keep around the list of fb's that are
> > -        * no longer associated w/ a drm_file but are not unreferenced
> > +        * no longer associated with a drm_file but are not unreferenced
> >          * yet.  (i915 and omapdrm have debugfs files which will show
> >          * this.)
> >          */
> > @@ -519,7 +520,7 @@ EXPORT_SYMBOL(drm_mode_remove);
> >   * @dev: DRM device
> >   * @connector: the connector to init
> >   * @funcs: callbacks for this connector
> > - * @name: user visible name of the connector
> > + * @connector_type: type of the connector
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -683,7 +684,8 @@ int drm_plane_init(struct drm_device *dev, struct
> drm_plane *plane,
> >         plane->format_count = format_count;
> >         plane->possible_crtcs = possible_crtcs;
> >
> > -       /* private planes are not exposed to userspace, but depending on
> > +       /*
> > +        * private planes are not exposed to userspace, but depending on
> >          * display hardware, might be convenient to allow sharing
> programming
> >          * for the scanout engine with the crtc implementation.
> >          */
> > @@ -1188,10 +1190,9 @@ static int drm_crtc_convert_umode(struct
> drm_display_mode *out,
> >
> >  /**
> >   * drm_mode_getresources - get graphics configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -1249,7 +1250,6 @@ int drm_mode_getresources(struct drm_device *dev,
> void *data,
> >                 list_for_each(lh, &dev->mode_config.encoder_list)
> >                         encoder_count++;
> >         } else {
> > -
> >                 crtc_count = mode_group->num_crtcs;
> >                 connector_count = mode_group->num_connectors;
> >                 encoder_count = mode_group->num_encoders;
> > @@ -1328,7 +1328,6 @@ int drm_mode_getresources(struct drm_device *dev,
> void *data,
> >                                 }
> >                                 copied++;
> >                         }
> > -
> >                 }
> >         }
> >         card_res->count_encoders = encoder_count;
> > @@ -1376,10 +1375,9 @@ out:
> >
> >  /**
> >   * drm_mode_getcrtc - get CRTC configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -1421,10 +1419,8 @@ int drm_mode_getcrtc(struct drm_device *dev,
> >                 crtc_resp->fb_id = 0;
> >
> >         if (crtc->enabled) {
> > -
> >                 drm_crtc_convert_to_umode(&crtc_resp->mode, &crtc->mode);
> >                 crtc_resp->mode_valid = 1;
> > -
> >         } else {
> >                 crtc_resp->mode_valid = 0;
> >         }
> > @@ -1436,10 +1432,9 @@ out:
> >
> >  /**
> >   * drm_mode_getconnector - get connector configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -1732,8 +1727,8 @@ out:
> >  /**
> >   * drm_mode_setplane - set up or tear down an plane
> >   * @dev: DRM device
> > - * @data: ioctl data*
> > - * @file_prive: DRM file info
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -1862,10 +1857,9 @@ out:
> >
> >  /**
> >   * drm_mode_setcrtc - set CRTC configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -2077,6 +2071,7 @@ int drm_mode_cursor_ioctl(struct drm_device *dev,
> >                         goto out;
> >                 }
> >         }
> > +
> >  out:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > @@ -2120,10 +2115,9 @@ EXPORT_SYMBOL(drm_mode_legacy_fb_format);
> >
> >  /**
> >   * drm_mode_addfb - add an FB to the graphics configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -2304,10 +2298,9 @@ static int framebuffer_check(const struct
> drm_mode_fb_cmd2 *r)
> >
> >  /**
> >   * drm_mode_addfb2 - add an FB to the graphics configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -2370,10 +2363,9 @@ out:
> >
> >  /**
> >   * drm_mode_rmfb - remove an FB from the configuration
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -2425,10 +2417,9 @@ out:
> >
> >  /**
> >   * drm_mode_getfb - get FB info
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> > @@ -2515,6 +2506,7 @@ int drm_mode_dirtyfb_ioctl(struct drm_device *dev,
> >                         ret = -EINVAL;
> >                         goto out_err1;
> >                 }
> > +
> >                 clips = kzalloc(num_clips * sizeof(*clips), GFP_KERNEL);
> >                 if (!clips) {
> >                         ret = -ENOMEM;
> > @@ -2544,15 +2536,14 @@ out_err1:
> >         return ret;
> >  }
> >
> > -
> >  /**
> >   * drm_fb_release - remove and free the FBs on this file
> > - * @filp: file * from the ioctl
> > + * @priv: file * from the ioctl
> >   *
> >   * LOCKING:
> >   * Takes mode config lock.
> >   *
> > - * Destroy all the FBs associated with @filp.
> > + * Destroy all the FBs associated with @priv.
> >   *
> >   * Called by the user via ioctl.
> >   *
> > @@ -2616,7 +2607,7 @@ int drm_mode_attachmode_crtc(struct drm_device
> *dev, struct drm_crtc *crtc,
> >
> >         WARN_ON(!list_empty(&list));
> >
> > - out:
> > +out:
> >         list_for_each_entry_safe(dup_mode, next, &list, head)
> >                 drm_mode_destroy(dev, dup_mode);
> >
> > @@ -2660,10 +2651,9 @@ EXPORT_SYMBOL(drm_mode_detachmode_crtc);
> >
> >  /**
> >   * drm_fb_attachmode - Attach a user mode to an connector
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * This attaches a user specified mode to an connector.
> >   * Called by the user via ioctl.
> > @@ -2707,6 +2697,7 @@ int drm_mode_attachmode_ioctl(struct drm_device
> *dev,
> >         }
> >
> >         drm_mode_attachmode(dev, connector, mode);
> > +
> >  out:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > @@ -2715,10 +2706,9 @@ out:
> >
> >  /**
> >   * drm_fb_detachmode - Detach a user specified mode from an connector
> > - * @inode: inode from the ioctl
> > - * @filp: file * from the ioctl
> > - * @cmd: cmd from ioctl
> > - * @arg: arg from ioctl
> > + * @dev: DRM device
> > + * @data: ioctl data
> > + * @file_priv: DRM file info
> >   *
> >   * Called by the user via ioctl.
> >   *
> > @@ -2754,6 +2744,7 @@ int drm_mode_detachmode_ioctl(struct drm_device
> *dev,
> >         }
> >
> >         ret = drm_mode_detachmode(dev, connector, &mode);
> > +
> >  out:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > @@ -2790,6 +2781,7 @@ struct drm_property *drm_property_create(struct
> drm_device *dev, int flags,
> >
> >         list_add_tail(&property->head, &dev->mode_config.property_list);
> >         return property;
> > +
> >  fail:
> >         kfree(property->values);
> >         kfree(property);
> > @@ -3078,6 +3070,7 @@ int drm_mode_getproperty_ioctl(struct drm_device
> *dev,
> >                 }
> >                 out_resp->count_enum_blobs = blob_count;
> >         }
> > +
> >  done:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > @@ -3187,6 +3180,7 @@ static bool drm_property_change_is_valid(struct
> drm_property *property,
> >  {
> >         if (property->flags & DRM_MODE_PROP_IMMUTABLE)
> >                 return false;
> > +
> >         if (property->flags & DRM_MODE_PROP_RANGE) {
> >                 if (value < property->values[0] || value >
> property->values[1])
> >                         return false;
> > @@ -3194,6 +3188,7 @@ static bool drm_property_change_is_valid(struct
> drm_property *property,
> >         } else if (property->flags & DRM_MODE_PROP_BITMASK) {
> >                 int i;
> >                 uint64_t valid_mask = 0;
> > +
> >                 for (i = 0; i < property->num_values; i++)
> >                         valid_mask |= (1ULL << property->values[i]);
> >                 return !(value & ~valid_mask);
> > @@ -3326,6 +3321,7 @@ int drm_mode_obj_get_properties_ioctl(struct
> drm_device *dev, void *data,
> >                 }
> >         }
> >         arg->count_props = props_count;
> > +
> >  out:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > @@ -3487,7 +3483,6 @@ int drm_mode_gamma_set_ioctl(struct drm_device
> *dev,
> >  out:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > -
> >  }
> >
> >  int drm_mode_gamma_get_ioctl(struct drm_device *dev,
> > @@ -3535,6 +3530,7 @@ int drm_mode_gamma_get_ioctl(struct drm_device
> *dev,
> >                 ret = -EFAULT;
> >                 goto out;
> >         }
> > +
> >  out:
> >         mutex_unlock(&dev->mode_config.mutex);
> >         return ret;
> > @@ -3668,6 +3664,7 @@ int drm_mode_create_dumb_ioctl(struct drm_device
> *dev,
> >
> >         if (!dev->driver->dumb_create)
> >                 return -ENOSYS;
> > +
> >         return dev->driver->dumb_create(file_priv, dev, args);
> >  }
> >
> > --
> > 1.9.1
> >
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>



-- 
Best regards
Junwang Zhao
Microprocessor Research and Develop Center
Department of Computer Science &Technology
Peking University
Beijing, 100871, PRC
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150313/f80e2494/attachment-0001.html>


More information about the dri-devel mailing list