drm/drm_mm.h: Fix the name of the referenced function in comment

Message ID 20171101140445.2798-1-Liviu.Dudau@arm.com
State New
Headers show
Series
  • drm/drm_mm.h: Fix the name of the referenced function in comment
Related show

Commit Message

Liviu Dudau Nov. 1, 2017, 2:04 p.m.
drm_mm_insert_node_generic() is a simplified version of
drm_mm_insert_node_in_range(), update comment to reflect correct
function name.

Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>
---
 include/drm/drm_mm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alex Deucher Nov. 2, 2017, 4:02 a.m. | #1
On Wed, Nov 1, 2017 at 10:04 AM, Liviu Dudau <Liviu.Dudau@arm.com> wrote:
> drm_mm_insert_node_generic() is a simplified version of
> drm_mm_insert_node_in_range(), update comment to reflect correct
> function name.
>
> Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  include/drm/drm_mm.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/drm_mm.h b/include/drm/drm_mm.h
> index 8d10fc97801ce..101f566ae43d0 100644
> --- a/include/drm/drm_mm.h
> +++ b/include/drm/drm_mm.h
> @@ -386,7 +386,7 @@ int drm_mm_insert_node_in_range(struct drm_mm *mm,
>   * @color: opaque tag value to use for this node
>   * @mode: fine-tune the allocation search and placement
>   *
> - * This is a simplified version of drm_mm_insert_node_in_range_generic() with no
> + * This is a simplified version of drm_mm_insert_node_in_range() with no
>   * range restrictions applied.
>   *
>   * The preallocated node must be cleared to 0.
> --
> 2.14.3
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
=?UTF-8?q?Christian=20K=C3=B6nig?= Nov. 2, 2017, 9:46 a.m. | #2
Am 01.11.2017 um 15:04 schrieb Liviu Dudau:
> drm_mm_insert_node_generic() is a simplified version of
> drm_mm_insert_node_in_range(), update comment to reflect correct
> function name.
>
> Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>

Reviewed-by: Christian K├Ânig <christian.koenig@amd.com>

> ---
>   include/drm/drm_mm.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/drm_mm.h b/include/drm/drm_mm.h
> index 8d10fc97801ce..101f566ae43d0 100644
> --- a/include/drm/drm_mm.h
> +++ b/include/drm/drm_mm.h
> @@ -386,7 +386,7 @@ int drm_mm_insert_node_in_range(struct drm_mm *mm,
>    * @color: opaque tag value to use for this node
>    * @mode: fine-tune the allocation search and placement
>    *
> - * This is a simplified version of drm_mm_insert_node_in_range_generic() with no
> + * This is a simplified version of drm_mm_insert_node_in_range() with no
>    * range restrictions applied.
>    *
>    * The preallocated node must be cleared to 0.
Gustavo Padovan Nov. 2, 2017, 1:15 p.m. | #3
Hi Liviu,

2017-11-01 Liviu Dudau <Liviu.Dudau@arm.com>:

> drm_mm_insert_node_generic() is a simplified version of
> drm_mm_insert_node_in_range(), update comment to reflect correct
> function name.
> 
> Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>
> ---
>  include/drm/drm_mm.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

queued for 4.16. Thanks for your patch.

Gustavo

Patch

diff --git a/include/drm/drm_mm.h b/include/drm/drm_mm.h
index 8d10fc97801ce..101f566ae43d0 100644
--- a/include/drm/drm_mm.h
+++ b/include/drm/drm_mm.h
@@ -386,7 +386,7 @@  int drm_mm_insert_node_in_range(struct drm_mm *mm,
  * @color: opaque tag value to use for this node
  * @mode: fine-tune the allocation search and placement
  *
- * This is a simplified version of drm_mm_insert_node_in_range_generic() with no
+ * This is a simplified version of drm_mm_insert_node_in_range() with no
  * range restrictions applied.
  *
  * The preallocated node must be cleared to 0.