Hi Daniel, ... > Subject: Re: [PATCH v1 1/2] backlight: mp3309c: Make use of device > properties > > +++ b/drivers/video/backlight/mp3309c.c > > @@ -15,6 +15,8 @@ > > #include <linux/delay.h> > > #include <linux/gpio/consumer.h> > > #include <linux/i2c.h> > > +#include <linux/mod_devicetable.h> > > +#include <linux/property.h> > > #include <linux/pwm.h> > > #include <linux/regmap.h> > > > > @@ -202,15 +204,12 @@ static const struct backlight_ops mp3309c_bl_ops > = { > > static int pm3309c_parse_dt_node(struct mp3309c_chip *chip, > > Pretty minor... but I wonder if it should be renamed: > mp3309c_parse_fwnode(). Right! It was my oversight! Thanks! > > > Daniel. Flavio