Re: [PATCH v2 3/5] drm/i915: Propagate "_release" function name suffix down

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Janusz Krzysztofik (2019-07-11 10:31:55)
> Replace mixed "_fini"/"_cleanup"/"_cleanup_hw" suffixes found in names
> of fucntions called from i915_driver_release() with "_release" suffix
> consistently.  This provides better code readability, especially
> helpful when trying to work out which phase the code is in.
> 
> Functions names starting with "i915_driver_", i.e., those defined in
> drivers/gpu/dri/i915/i915_drv.c, just have their "cleanup" or "fini"
> parts of their names replaced with the "_release" suffix, while names
> of functions coming from other source files have been suffixed with
> "_driver_release" to avoid ambiguity with other possible .release entry
> points.
> 
> v2: early_probe pairs better with late_release (Chris)
> 
> Suggested-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@xxxxxxxxxxxxxxx>

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>

Regards, Joonas
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux