RE: [PATCH 01/10] leds: lp55xx: add common data structure for program
Hi Bryan, > -Original Message- > From: Bryan Wu [mailto:coolo...@gmail.com] > Sent: Wednesday, August 14, 2013 3:56 AM > To: Milo Kim > Cc: Pali Rohár; Linux LED Subsystem; lkml; Kim, Milo > Subject: Re: [PATCH 01/10] leds: lp55xx: add common data structure for > program > > On Thu, Aug 8, 2013 at 12:59 AM, Milo Kim 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 for your help. Can I get more detailed information about this format problem? I need to check my configurations. Thanks, Milo- -- 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/
RE: [PATCH 01/10] leds: lp55xx: add common data structure for program
Hi Bryan, -Original Message- From: Bryan Wu [mailto:coolo...@gmail.com] Sent: Wednesday, August 14, 2013 3:56 AM To: Milo Kim Cc: Pali Rohár; Linux LED Subsystem; lkml; Kim, Milo Subject: Re: [PATCH 01/10] leds: lp55xx: add common data structure for program 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 for your help. Can I get more detailed information about this format problem? I need to check my configurations. Thanks, Milo- -- 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/
Re: [PATCH 01/10] leds: lp55xx: add common data structure for program
On Thu, Aug 8, 2013 at 12:59 AM, Milo Kim 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 > --- > 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/
Re: [PATCH 01/10] leds: lp55xx: add common data structure for program
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/