Commit d5318d30 authored by Johan Hovold's avatar Johan Hovold Committed by Lee Jones

backlight: as3711_bl: Fix Device Tree node leaks

Two framebuffer device-node names were looked up during probe, but were
only used as flags to indicate the presence of two framebuffer device.

Drop the unused framebuffer name along with a likewise unused device
pointer from the driver data, and update the platform data to pass in
booleans instead of the framebuffer strings. This allows us do drop the
node references acquired during probe, which would otherwise leak.

Note that there are no other in-kernel users of the modified
platform-data fields.

Fixes: 59eb2b5e ("drivers/video/backlight/as3711_bl.c: add OF support")
Signed-off-by: default avatarJohan Hovold <johan@kernel.org>
Acked-by: default avatarDaniel Thompson <daniel.thompson@linaro.org>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
parent 2b12dfa1
...@@ -28,8 +28,6 @@ enum as3711_bl_type { ...@@ -28,8 +28,6 @@ enum as3711_bl_type {
struct as3711_bl_data { struct as3711_bl_data {
bool powered; bool powered;
const char *fb_name;
struct device *fb_dev;
enum as3711_bl_type type; enum as3711_bl_type type;
int brightness; int brightness;
struct backlight_device *bl; struct backlight_device *bl;
...@@ -273,7 +271,9 @@ static int as3711_backlight_parse_dt(struct device *dev) ...@@ -273,7 +271,9 @@ static int as3711_backlight_parse_dt(struct device *dev)
fb = of_parse_phandle(bl, "su1-dev", 0); fb = of_parse_phandle(bl, "su1-dev", 0);
if (fb) { if (fb) {
pdata->su1_fb = fb->full_name; of_node_put(fb);
pdata->su1_fb = true;
ret = of_property_read_u32(bl, "su1-max-uA", &pdata->su1_max_uA); ret = of_property_read_u32(bl, "su1-max-uA", &pdata->su1_max_uA);
if (pdata->su1_max_uA <= 0) if (pdata->su1_max_uA <= 0)
...@@ -286,7 +286,9 @@ static int as3711_backlight_parse_dt(struct device *dev) ...@@ -286,7 +286,9 @@ static int as3711_backlight_parse_dt(struct device *dev)
if (fb) { if (fb) {
int count = 0; int count = 0;
pdata->su2_fb = fb->full_name; of_node_put(fb);
pdata->su2_fb = true;
ret = of_property_read_u32(bl, "su2-max-uA", &pdata->su2_max_uA); ret = of_property_read_u32(bl, "su2-max-uA", &pdata->su2_max_uA);
if (pdata->su2_max_uA <= 0) if (pdata->su2_max_uA <= 0)
...@@ -425,7 +427,6 @@ static int as3711_backlight_probe(struct platform_device *pdev) ...@@ -425,7 +427,6 @@ static int as3711_backlight_probe(struct platform_device *pdev)
if (pdata->su1_fb) { if (pdata->su1_fb) {
su = &supply->su1; su = &supply->su1;
su->fb_name = pdata->su1_fb;
su->type = AS3711_BL_SU1; su->type = AS3711_BL_SU1;
max_brightness = min(pdata->su1_max_uA, 31); max_brightness = min(pdata->su1_max_uA, 31);
...@@ -436,7 +437,6 @@ static int as3711_backlight_probe(struct platform_device *pdev) ...@@ -436,7 +437,6 @@ static int as3711_backlight_probe(struct platform_device *pdev)
if (pdata->su2_fb) { if (pdata->su2_fb) {
su = &supply->su2; su = &supply->su2;
su->fb_name = pdata->su2_fb;
su->type = AS3711_BL_SU2; su->type = AS3711_BL_SU2;
switch (pdata->su2_fbprot) { switch (pdata->su2_fbprot) {
......
...@@ -108,9 +108,9 @@ struct as3711_regulator_pdata { ...@@ -108,9 +108,9 @@ struct as3711_regulator_pdata {
}; };
struct as3711_bl_pdata { struct as3711_bl_pdata {
const char *su1_fb; bool su1_fb;
int su1_max_uA; int su1_max_uA;
const char *su2_fb; bool su2_fb;
int su2_max_uA; int su2_max_uA;
enum as3711_su2_feedback su2_feedback; enum as3711_su2_feedback su2_feedback;
enum as3711_su2_fbprot su2_fbprot; enum as3711_su2_fbprot su2_fbprot;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment