linux-kernel - Re: [PATCH V3] pwm: lpc32xx - Add a driver for the motor PWM

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 9 Sep 2014 18:05:30 +0200
From:	Alban Bedel <alban.bedel@...onic-design.de>
To:	Arnd Bergmann <arnd@...db.de>
Cc:	Thierry Reding <thierry.reding@...il.com>,
	linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
	linux-pwm@...r.kernel.org, Grant Likely <grant.likely@...aro.org>,
	Kumar Gala <galak@...eaurora.org>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Mark Rutland <mark.rutland@....com>,
	Pawel Moll <pawel.moll@....com>,
	Rob Herring <robh+dt@...nel.org>,
	Roland Stigge <stigge@...com.de>
Subject: Re: [PATCH V3] pwm: lpc32xx - Add a driver for the motor PWM

On Tue, 09 Sep 2014 17:47:53 +0200
Arnd Bergmann <arnd@...db.de> wrote:

> On Tuesday 09 September 2014 17:42:41 Alban Bedel wrote:
> > +config PWM_LPC32XX_MOTOR
> > +	tristate "LPC32xx Motor PWM support"
> > +	depends on ARCH_LPC32XX
> > +	help
> > +	  Generic PWM framework driver for LPC32xx motor PWM. The LPC32xx SOC
> > +	  has one motor PWM controllers.
> > +
> > +	  To compile this driver as a module, choose M here: the module
> > +	  will be called pwm-lpc32xx-motor.
> > +
> 
> Can you change the dependency to ARCH_LPC32XX || COMPILE_TEST and
> add explicit dependencies for the subsystems the driver depends
> on (pwm and clk, I guess)? That would give us better build-time
> coverage with allmodconfig.

I'll add COMPILE_TEST, PWM is already there because this is in an if
PWM block. However I'm not sure for the clk, the LPC32xx use its own
implementation of the clk functions, what should I add in this case?

> > +	/* Write to limit register -> period */
> > +	__raw_writel(period, lpc32xx->base + MCLIM_REG_OFFSET(pwm));
> > +
> > +	/* Write to match register -> duty */
> > +	__raw_writel(period - duty, lpc32xx->base + MCMAT_REG_OFFSET(pwm));
> 
> Please don't use __raw_{writel,readl} in driver, and change that to use
> readl_relaxed()/writel_relaxed().

Will do.

Alban

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)