Hi Bryan, > -----Original Message----- > From: Bryan Wu [mailto:cooloney@xxxxxxxxx] > 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.kim@xxxxxxxxx> 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-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html