From: Thierry Reding <email@example.com> To: Arun MURTHY <firstname.lastname@example.org> Cc: Linus Walleij <email@example.com>, Srinidhi KASAGAR <firstname.lastname@example.org>, Arnd Bergmann <email@example.com>, Greg Kroah-Hartman <firstname.lastname@example.org>, Lee Jones <email@example.com>, "firstname.lastname@example.org" <email@example.com> Subject: Re: [PATCH] pwm: Move AB8500 PWM driver to PWM framework Date: Thu, 20 Sep 2012 12:54:09 +0200 [thread overview] Message-ID: <20120920105409.GA4033@avionic-0098.mockup.avionic-design.de> (raw) In-Reply-To: <F45880696056844FA6A73F415B568C695B44A603C3@EXDCVYMBSTM006.EQ1STM.local> [-- Attachment #1: Type: text/plain, Size: 1835 bytes --] On Mon, Sep 17, 2012 at 07:01:16AM +0200, Arun MURTHY wrote: > > On Fri, Sep 07, 2012 at 03:22:09PM +0200, Thierry Reding wrote: > > > On Mon, Sep 03, 2012 at 01:28:44PM +0200, Arun MURTHY wrote: > > > > > On Sun, Sep 2, 2012 at 12:30 PM, Thierry Reding > > > > > <thierry.reding@avionic- design.de> wrote: > > > > > > > > > > > This commit moves the driver to drivers/pwm and converts it to > > > > > > the new PWM framework. > > > > > > > > > > > > Signed-off-by: Thierry Reding <firstname.lastname@example.org> > > > > > > --- > > > > > > Note: I'll take this through the PWM tree, but I'd like to have > > > > > > it acknowledged by a few people who know the hardware and can > > > > > > actually test whether this still works. > > > > > > > > > > Pls include Arun on these patches, he'll know best if it'll work or not. > > > > > > > > > > I have one generic question: will this alter the layout of sysfs > > > > > so we need to change userspace? I had such issues before when > > > > > moving drivers to new busses... > > > > > > > > > > > > > Since this pwm framework is still a set of exported functions in a > > > > header file, the file architecture doesn't really matter. > > > > Using this driver, either leds-pwm and backlight-pwm is used to > > > > register and this is where actual registration happens(sysfs entry is > > created). > > > > > > Hi Arun, > > > > > > Have you been able to verify, then, that the driver still works after > > > this patch? Can I add your Acked-by as well? > > > > Hi Arun, > > > > any news on this? > > > > Thierry > > Hi Thierry, > Works with certain changes in platform data which I can manage > to push it later. > > Acked-by: Arun Murthy<email@example.com> Thanks. I've pushed this patch to my for-next branch. Thierry [-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
prev parent reply other threads:[~2012-09-20 10:54 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-09-02 10:30 Thierry Reding 2012-09-02 18:55 ` Arnd Bergmann 2012-09-02 19:09 ` Thierry Reding 2012-09-03 11:10 ` Linus Walleij 2012-09-03 11:28 ` Arun MURTHY 2012-09-03 11:51 ` Linus Walleij 2012-09-07 13:22 ` Thierry Reding 2012-09-13 15:25 ` Thierry Reding 2012-09-17 5:01 ` Arun MURTHY 2012-09-20 10:54 ` Thierry Reding [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20120920105409.GA4033@avionic-0098.mockup.avionic-design.de \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH] pwm: Move AB8500 PWM driver to PWM framework' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).