On Mon, Aug 18, 2014 at 4:19 PM, Maxime Ripard <[email protected]> wrote: > On Mon, Aug 18, 2014 at 03:53:08PM -0400, [email protected] wrote: >> On Mon, Aug 18, 2014 at 3:23 PM, Maxime Ripard >> <[email protected]> wrote: >> > Hi, >> > >> > On Mon, Aug 18, 2014 at 02:03:16PM -0400, Jon Smirl wrote: >> >> found on Allwinner SoCs.x >> >> >> >> Signed-off-by: Jon Smirl <[email protected]> >> > >> > Is there any notable difference with Alexandre's driver? >> > https://lkml.org/lkml/2014/5/19/568 >> > >> > If so, why didn't you posted incremental patches on top of his driver? >> >> Better A10 support. > > Better how? > >> And I wrote this one without know about his. > > So you're just require us to trash an already extensively reviewed > driver and start from scratch again?
Forget mine then. Problems in the other one.... 1) The math is not right in either patch. Selsinork has been checking mine with a scope and just found another math error in mine. The internal math needs to be done in picoseconds or round off errors mess up the high frequency outputs (24Mhz, 12Mhz, 8Mhz, ...). 3) By-pass mode on the A20 is not implemented. 2) A10 is not correctly handled. Specifically the prescaler is different on the A10 vs A20. > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com -- Jon Smirl [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
