[PATCH] ep93xx: add errata info about pwm outputs
Ryan Mallon
ryan at bluewatersys.com
Tue Oct 20 20:55:47 EDT 2009
H Hartley Sweeten wrote:
> The EP93xx User's guide has an error about the number of pwm
> channels available in the various chips. Add comments about
> this in the driver since the guide will probably never get
> updated.
>
> EP9301/02 only have pwm.1, pin PMWOUT is not on the chip
> EP9307 only has pwm.0, pin PWMOUT1 (EGPIO14) is not on the chip
> EP9312/15 have both pwm0 and pwm.1
>
> Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
> Cc: Ryan Mallon <ryan at bluewatersys.com>
>
> ---
>
> diff --git a/arch/arm/mach-ep93xx/core.c b/arch/arm/mach-ep93xx/core.c
> index f95dc16..f30be28 100644
> --- a/arch/arm/mach-ep93xx/core.c
> +++ b/arch/arm/mach-ep93xx/core.c
> @@ -648,6 +648,7 @@ static struct platform_device ep93xx_pwm1_device = {
>
> void __init ep93xx_register_pwm(int pwm0, int pwm1)
> {
> + /* NOTE: EP9301/02 do not have PWMOUT */
> if (pwm0)
> platform_device_register(&ep93xx_pwm0_device);
>
> diff --git a/drivers/misc/ep93xx_pwm.c b/drivers/misc/ep93xx_pwm.c
> index ba46941..de9f405 100644
> --- a/drivers/misc/ep93xx_pwm.c
> +++ b/drivers/misc/ep93xx_pwm.c
> @@ -9,12 +9,18 @@
> * as published by the Free Software Foundation; either version
> * 2 of the License, or (at your option) any later version.
> *
> - * EP9307 has only one channel:
> - * - PWMOUT
> + * The EP93xx User's Guide has an error about the number of pwm channels
> + * available in the various chips.
Is there an official errata for this that we can link to?
~Ryan
--
Bluewater Systems Ltd - ARM Technology Solution Centre
Ryan Mallon 5 Amuri Park, 404 Barbadoes St
ryan at bluewatersys.com PO Box 13 889, Christchurch 8013
http://www.bluewatersys.com New Zealand
Phone: +64 3 3779127 Freecall: Australia 1800 148 751
Fax: +64 3 3779135 USA 1800 261 2934
More information about the linux-arm-kernel
mailing list