]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
mfd: max8925: fix dt code for backlight
authorOlof Johansson <olof@lixom.net>
Wed, 24 Jul 2013 15:42:27 +0000 (08:42 -0700)
committerOlof Johansson <olof@lixom.net>
Thu, 25 Jul 2013 23:57:00 +0000 (16:57 -0700)
commit515c0967205f2e6d0ca1602ce0de65f9aec1d215
treee0f1280c5d8b5c702940e9b030e0b65560f57b75
parent51378066fcb49c2cdee368ee1272fa8d963c7917
mfd: max8925: fix dt code for backlight

The device-tree enablement for max8925 has several problems, but besides
the bindings being wrong (and not having seen review) there's also some
bad coding practices on how to fill in the platform_data from device tree.

I came across this since it causes a warning when compiling
mmp2_defconfig, and instead of doing the minimal fix to silence the
warning, I restructured the code a bit.

This silences the warning:
drivers/video/backlight/max8925_bl.c: In function 'max8925_backlight_probe':
drivers/video/backlight/max8925_bl.c:177:3: warning: statement with no effect [-Wunused-value]

Note that the bindings themselves need to be revisited too, but that will
affect more than just the backlight driver and is best done separately;
this just fixes the bad code for the backlight driver.

Acked-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Olof Johansson <olof@lixom.net>
drivers/video/backlight/max8925_bl.c