diff options
author | Joel Stanley <joel@jms.id.au> | 2021-11-17 04:01:45 +0300 |
---|---|---|
committer | Maxime Ripard <maxime@cerno.tech> | 2021-11-19 14:00:14 +0300 |
commit | b4a6aaeaf4aa79f23775f6688a7e8db3ee1c1303 (patch) | |
tree | 9b21d9657e78c065b77c3bcaeabd2f425e724de3 /drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | |
parent | 96c5f82ef0a145d3e56e5b26f2bf6dcd2ffeae1c (diff) | |
download | linux-b4a6aaeaf4aa79f23775f6688a7e8db3ee1c1303.tar.xz |
drm/aspeed: Fix vga_pw sysfs output
Before the drm driver had support for this file there was a driver that
exposed the contents of the vga password register to userspace. It would
present the entire register instead of interpreting it.
The drm implementation chose to mask of the lower bit, without explaining
why. This breaks the existing userspace, which is looking for 0xa8 in
the lower byte.
Change our implementation to expose the entire register.
Fixes: 696029eb36c0 ("drm/aspeed: Add sysfs for output settings")
Reported-by: Oskar Senft <osk@google.com>
Signed-off-by: Joel Stanley <joel@jms.id.au>
Reviewed-by: Jeremy Kerr <jk@codeconstruct.com.au>
Tested-by: Oskar Senft <osk@google.com>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://patchwork.freedesktop.org/patch/msgid/20211117010145.297253-1-joel@jms.id.au
Diffstat (limited to 'drivers/gpu/drm/aspeed/aspeed_gfx_drv.c')
-rw-r--r-- | drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c index b53fee6f1c17..65f172807a0d 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c @@ -291,7 +291,7 @@ vga_pw_show(struct device *dev, struct device_attribute *attr, char *buf) if (rc) return rc; - return sprintf(buf, "%u\n", reg & 1); + return sprintf(buf, "%u\n", reg); } static DEVICE_ATTR_RO(vga_pw); |