Re: [RFC 5/6] [PWM] Install new Atmel PWMC driver in Kconfig, expunge old one

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hans-Christian Egtvedt wrote:
> On Thu, 9 Oct 2008 14:16:01 +0200
> Haavard Skinnemoen <haavard.skinnemoen@xxxxxxxxx> wrote:
> 
>> Hans-Christian Egtvedt <hans-christian.egtvedt@xxxxxxxxx> wrote:
>>> You know that there are other users of ATMEL_PWM ?
>>>
>>> There is a atmel-pwm-bl driver in drivers/video/backlight which
>>> could probably be converted to pwm-bl using the new pwm framework.
>> There's already a pwm_bl driver there which I'm assuming is using the
>> generic pwm API. Hopefully, we can simply delete the atmel-pwm-bl
>> driver. Maybe some board code needs updating before that can happen
>> though.

Yes.  I think atmel-pwm-bl goes away completely, and video/backlight/pwm_bl.c
gets modified (slightly) to use the new PWM API and becomes the general
backlight-via-PWM-peripheral solution.  After that, anyone who wants to write a
backend pwm_device driver suddenly can use their peripheral for backlights (and
other stuff) out-of-the-box.

I just glanced over drivers/video/backlight, it looks like there are a couple
other implementations that could move over to Generic PWM as well.

> That would be the Favr-32 board, since the atmel-pwm-bl driver was
> developed for that board.
> 
> Weird how this stuff almost was inserted almost in parallel.

Indeed.  But also a bit tragic--- some wasted effort.  :(

Not entirely weird, however, since I try to follow AVR32 as closely as I do ARM.
 I have an NGW100 and an STK1000, in addition to about five different ARM
machines including ones based on the AT91SAM9263.  These common frameworks help
me slide into insanity a little more slowly.  :)

(I have some PPC and MIPS boards, too.  Occupational hazard).



b.g.
-- 
Bill Gatliff
bgat@xxxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-embedded" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Gstreamer Embedded]     [Linux MMC Devel]     [U-Boot V2]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux ARM Kernel]     [Linux OMAP]     [Linux SCSI]

  Powered by Linux