[Outreachy kernel] [PATCH v2 0/3] drm: Code cleanup

Sean Paul seanpaul at chromium.org
Wed Sep 13 21:49:05 UTC 2017


On Wed, Sep 13, 2017 at 12:12 PM, Meghana Madhyastha
<meghana.madhyastha at gmail.com> wrote:
> These patches fix the checkpath warnings and errors
> in drm_agpsupport.c
>
> changes in v2:
>  -Changed the commit subject prefix to follow the style used in drm.
>  -Combine multiple redundant if conditions to a single condition in
>   patch 3.
>
> Meghana Madhyastha (3):
>   drm: Move EXPORT_SYMBOL so that it immediately follows its function
>   drm: Replace "foo * bar" with "foo *bar"
>   drm: Remove assignment in if condition

Hi Meghana,
Thank you for your submissions!

Would you mind prefixing the subjects with drm/agpsupport instead of
just drm? I know the git log shows mostly drm:, but I think that's
more because the only time this file is touched is during subsystem
wide refactors/cleanups. While this may seem like a small nit,
narrowing down the affected file(s) in the subject really helps when
glancing at the log.

Sean



>
>  drivers/gpu/drm/drm_agpsupport.c | 45 ++++++++++++++++++++--------------------
>  1 file changed, 23 insertions(+), 22 deletions(-)
>
> --
> 2.7.4
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe at googlegroups.com.
> To post to this group, send email to outreachy-kernel at googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20170913161235.GA3155%40meghana-HP-Pavilion-Notebook.
> For more options, visit https://groups.google.com/d/optout.


More information about the dri-devel mailing list