On Thu, Aug 8, 2013 at 12:59 AM, Milo Kim <woogyom....@gmail.com> wrote: > LP55xx family devices have internal three program engines which are used for > loading LED patterns. > To maintain legacy device attributes, specific data structure is used, 'mode' > and 'led_mux'. > The mode is used for showing/storing current engine mode such like disabled, > load and run. > Then led_mux is used for showing/storing current output LED selection. > This is only for LP5523/55231. >
This patch looks good to me, but the commit message format is little bit odd to me. I will fix that and merge into my tree. Thanks, -Bryan > Signed-off-by: Milo Kim <milo....@ti.com> > --- > drivers/leds/leds-lp55xx-common.h | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/drivers/leds/leds-lp55xx-common.h > b/drivers/leds/leds-lp55xx-common.h > index dbbf86d..04c1d4f 100644 > --- a/drivers/leds/leds-lp55xx-common.h > +++ b/drivers/leds/leds-lp55xx-common.h > @@ -20,6 +20,13 @@ enum lp55xx_engine_index { > LP55XX_ENGINE_1, > LP55XX_ENGINE_2, > LP55XX_ENGINE_3, > + LP55XX_ENGINE_MAX = LP55XX_ENGINE_3, > +}; > + > +enum lp55xx_engine_mode { > + LP55XX_ENGINE_DISABLED, > + LP55XX_ENGINE_LOAD, > + LP55XX_ENGINE_RUN, > }; > > struct lp55xx_led; > @@ -72,6 +79,16 @@ struct lp55xx_device_config { > }; > > /* > + * struct lp55xx_engine > + * @mode : Engine mode > + * @led_mux : Mux bits for LED selection. Only used in LP5523 > + */ > +struct lp55xx_engine { > + enum lp55xx_engine_mode mode; > + u16 led_mux; > +}; > + > +/* > * struct lp55xx_chip > * @cl : I2C communication for access registers > * @pdata : Platform specific data > @@ -79,6 +96,7 @@ struct lp55xx_device_config { > * @num_leds : Number of registered LEDs > * @cfg : Device specific configuration data > * @engine_idx : Selected engine number > + * @engines : Engine structure for the device attribute R/W interface > * @fw : Firmware data for running a LED pattern > */ > struct lp55xx_chip { > @@ -89,6 +107,7 @@ struct lp55xx_chip { > int num_leds; > struct lp55xx_device_config *cfg; > enum lp55xx_engine_index engine_idx; > + struct lp55xx_engine engines[LP55XX_ENGINE_MAX]; > const struct firmware *fw; > }; > > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/