]> git.baikalelectronics.ru Git - kernel.git/commit
drm/vmwgfx: Convert macro to octal representation
authorØyvind A. Holm <sunny@sunbase.org>
Mon, 3 Apr 2017 20:06:24 +0000 (22:06 +0200)
committerSinclair Yeh <syeh@vmware.com>
Fri, 21 Apr 2017 16:25:47 +0000 (09:25 -0700)
commit96b834fbe18464f40551982b61b47a45a6882fde
tree4079bde33d78ab9055642cf8d1c97b4a9c7821d3
parent785c200bda19064226c85610906ea5b3f1b21057
drm/vmwgfx: Convert macro to octal representation

Change "S_IRUSR | S_IWUSR" to "0600", it's easier to parse mentally.

This change should be part of commit 455fe45901f7 ("drm/vmwgfx: Revert
"drm/vmwgfx: Replace numeric parameter like 0444 with macro""), but the
patch was truncated somewhere in the patch route, so add the missing
change. Extract from the original commit message:

  commit 455fe45901f7059c2f17d7a7c6aab8d5cac2e1d5
  Author: Øyvind A. Holm <sunny@sunbase.org>
  Date:   Thu Mar 23 14:54:48 2017 -0700

    drm/vmwgfx: Revert "drm/vmwgfx: Replace numeric parameter like 0444
    with macro"

    This reverts commit 01653d313d8e ("drm/vmwgfx: Replace numeric
    parameter like 0444 with macro")

    The commit belongs to the series of 1285 patches sent to LKML on
    2016-08-02, it changes the representation of file permissions from
    the octal value "0600" to "S_IRUSR | S_IWUSR".

    The general consensus was that the changes does not increase
    readability, quite the opposite; 0600 is easier to parse mentally
    than S_IRUSR | S_IWUSR.

Signed-off-by: Øyvind A. Holm <sunny@sunbase.org>
Reviewed-by: Sinclair Yeh <syeh@vmware.com>
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c