On Mon, Aug 5, 2013 at 9:09 AM, Kim, Milo wrote:
> In the meantime, I've reviewed the pin control subsystem,
> I think it is not the best way to implement LP3943 driver.
> The GPIO controller is OK, but I can't make flexible pin assignment for the
> PWM
> operation.
> For example, multiple outpu
> > > > Although, I think the 0 = 1, 1 = 2 ... stuff is really confusing. Is
> > > > there nothing we can do about that?
> > >
> > > OK, enum value of lp3943_pwm_output can be changed as below because
> > > LP3943_PWM_INVALID is not used anymore.
> > >
> > > enum lp3943_pwm_output {
> > > LP3943_
> > > Although, I think the 0 = 1, 1 = 2 ... stuff is really confusing. Is
> > > there nothing we can do about that?
> >
> > OK, enum value of lp3943_pwm_output can be changed as below because
> > LP3943_PWM_INVALID is not used anymore.
> >
> > enum lp3943_pwm_output {
> > LP3943_PWM_OUT0,
> >
On Wed, 31 Jul 2013, Kim, Milo wrote:
> Thanks for the review, please see my comments.
>
> > * looks good up to me up to here *
> >
> > Although, I think the 0 = 1, 1 = 2 ... stuff is really confusing. Is
> > there nothing we can do about that?
>
> OK, enum value of lp3943_pwm_output can be ch
Thanks for the review, please see my comments.
> * looks good up to me up to here *
>
> Although, I think the 0 = 1, 1 = 2 ... stuff is really confusing. Is
> there nothing we can do about that?
OK, enum value of lp3943_pwm_output can be changed as below
because LP3943_PWM_INVALID is not used a
> Cc: Lee Jones
> Cc: Linus Walleij
> Cc: Mark Brown
> Cc: Samuel Ortiz
> Cc: Thierry Reding
> Signed-off-by: Milo Kim
> ---
> drivers/mfd/Kconfig| 11 +++
> drivers/mfd/Makefile |1 +
> drivers/mfd/lp3943.c | 165
>
>
6 matches
Mail list logo