linux-kernel - Re: [PWM PATCH 3/5] Expunge old Atmel PWMC driver, replacing it with one that conforms to the PWM API

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, 02 Feb 2010 22:10:51 -0600
From:	Bill Gatliff <bgat@...lgatliff.com>
To:	H Hartley Sweeten <hartleys@...ionengravers.com>
CC:	linux-embedded@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PWM PATCH 3/5] Expunge old Atmel PWMC driver, replacing it with
 one that conforms to the PWM API

H Hartley Sweeten wrote:
>
> You have a couple name changes here, were they intentional?
>
>     Filename: atmel_pwm.c        -> atmel-pwm.c
>  Driver name: atmel_pwm          -> atmel_pwmc
> Module alias: platform:atmel_pwm -> platform:atmel_pwmc
>
>   

Yes.  In the datasheet, the peripheral is called the "PWMC" and so
that's why the driver name changed.  For consistency, I guess I should
have named the file atmel-pwmc.c as well.  Finally, I find it easier to
type file names with dashes rather than underscores.  :)

> Also, will this new driver still work for all the existing users of
> the old one?  Are there any platform changes that need to be done?
>   

There are platform changes necessary, to register and use the devices.  
The driver API is different from the previous code, so there are changes
there as well.  Functionally, however, the new stuff is a superset of
what the previous driver provided.

b.g.

-- 
Bill Gatliff
bgat@...lgatliff.com

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/