[PATCH 4/6] drm/crtc: Polish kerneldoc
- Make it clear that it's a negative errno (more in line with everything else). - Clean up the confusion around get_properties vs. getproperty ioctls: One reads per-obj property values, the other reads property metadata. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_crtc.c | 79 -- 1 file changed, 41 insertions(+), 38 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 5c878f172365..8c550302a9ef 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c @@ -1490,7 +1490,7 @@ EXPORT_SYMBOL(drm_mode_create_scaling_mode_property); * connectors. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_create_aspect_ratio_property(struct drm_device *dev) { @@ -1674,7 +1674,7 @@ static void drm_crtc_convert_to_umode(struct drm_mode_modeinfo *out, * the caller. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ static int drm_crtc_convert_umode(struct drm_display_mode *out, const struct drm_mode_modeinfo *in) @@ -1717,7 +1717,7 @@ static int drm_crtc_convert_umode(struct drm_display_mode *out, * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_getresources(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -1905,7 +1905,7 @@ out: * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_getcrtc(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -1966,7 +1966,7 @@ static bool drm_mode_expose_to_userspace(const struct drm_display_mode *mode, * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_getconnector(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2110,7 +2110,7 @@ out: * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_getencoder(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2151,7 +2151,7 @@ int drm_mode_getencoder(struct drm_device *dev, void *data, * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_getplane_res(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2212,7 +2212,7 @@ int drm_mode_getplane_res(struct drm_device *dev, void *data, * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_getplane(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2386,7 +2386,7 @@ static int setplane_internal(struct drm_plane *plane, * valid crtc). * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_setplane(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2461,7 +2461,7 @@ int drm_mode_setplane(struct drm_device *dev, void *data, * interface. The only thing it adds is correct refcounting dance. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_set_config_internal(struct drm_mode_set *set) { @@ -2554,7 +2554,7 @@ EXPORT_SYMBOL(drm_crtc_check_viewport); * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_setcrtc(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2717,7 +2717,7 @@ out: * userspace wants to make use of these capabilities. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ static int drm_mode_cursor_universal(struct drm_crtc *crtc, struct drm_mode_cursor2 *req, @@ -2865,7 +2865,7 @@ out: * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_cursor_ioctl(struct drm_device *dev, void *data, struct drm_file *file_priv) @@ -2892,7 +2892,7 @@ int drm_mode_cursor_ioctl(struct drm_device *dev, * Called by the user via ioctl. * * Returns: - * Zero on success, errno on failure. + * Zero on success, negative errno on failure. */ int drm_mode_cursor2_ioctl(struct drm_device *dev, void *data, str
[PATCH 4/6] drm/crtc: Polish kerneldoc
On Wed, Nov 19, 2014 at 12:38 PM, Daniel Vetter wrote: > - Make it clear that it's a negative errno (more in line with > everything else). > - Clean up the confusion around get_properties vs. getproperty ioctls: > One reads per-obj property values, the other reads property > metadata. > > Signed-off-by: Daniel Vetter Reviewed-by: Rob Clark > --- > drivers/gpu/drm/drm_crtc.c | 79 > -- > 1 file changed, 41 insertions(+), 38 deletions(-) > > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c > index 5c878f172365..8c550302a9ef 100644 > --- a/drivers/gpu/drm/drm_crtc.c > +++ b/drivers/gpu/drm/drm_crtc.c > @@ -1490,7 +1490,7 @@ EXPORT_SYMBOL(drm_mode_create_scaling_mode_property); > * connectors. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_create_aspect_ratio_property(struct drm_device *dev) > { > @@ -1674,7 +1674,7 @@ static void drm_crtc_convert_to_umode(struct > drm_mode_modeinfo *out, > * the caller. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > static int drm_crtc_convert_umode(struct drm_display_mode *out, > const struct drm_mode_modeinfo *in) > @@ -1717,7 +1717,7 @@ static int drm_crtc_convert_umode(struct > drm_display_mode *out, > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_getresources(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -1905,7 +1905,7 @@ out: > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_getcrtc(struct drm_device *dev, > void *data, struct drm_file *file_priv) > @@ -1966,7 +1966,7 @@ static bool drm_mode_expose_to_userspace(const struct > drm_display_mode *mode, > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_getconnector(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -2110,7 +2110,7 @@ out: > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_getencoder(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -2151,7 +2151,7 @@ int drm_mode_getencoder(struct drm_device *dev, void > *data, > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_getplane_res(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -2212,7 +2212,7 @@ int drm_mode_getplane_res(struct drm_device *dev, void > *data, > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_getplane(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -2386,7 +2386,7 @@ static int setplane_internal(struct drm_plane *plane, > * valid crtc). > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_setplane(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -2461,7 +2461,7 @@ int drm_mode_setplane(struct drm_device *dev, void > *data, > * interface. The only thing it adds is correct refcounting dance. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_set_config_internal(struct drm_mode_set *set) > { > @@ -2554,7 +2554,7 @@ EXPORT_SYMBOL(drm_crtc_check_viewport); > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_setcrtc(struct drm_device *dev, void *data, > struct drm_file *file_priv) > @@ -2717,7 +2717,7 @@ out: > * userspace wants to make use of these capabilities. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > static int drm_mode_cursor_universal(struct drm_crtc *crtc, > struct drm_mode_cursor2 *req, > @@ -2865,7 +2865,7 @@ out: > * Called by the user via ioctl. > * > * Returns: > - * Zero on success, errno on failure. > + * Zero on success, negative errno on failure. > */ > int drm_mode_cursor_ioctl(struct drm_device *dev, >