From: Oliver Schinagl <oliver+list@schinagl.nl>
To: Lars Poeschel <poeschel@lemonage.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [RFC] pwm: add sysfs interface
Date: Thu, 23 May 2013 16:43:17 +0200 [thread overview]
Message-ID: <519E2B05.6060206@schinagl.nl> (raw)
In-Reply-To: <201305231412.15096.poeschel@lemonage.de>
On 05/23/13 14:12, Lars Poeschel wrote:
> Hi Oliver!
>
> As you are not the first one asking for the status of the pwm sysfs
> interface I post my answer to the linux-kernel list in hope others will
> benefit.
>
> On Thursday 23 May 2013 at 11:07:39, Oliver Schinagl wrote:
>> Hi lars,
>>
>> I have trouble seeing if your patch has been applied yet to the kernel
>> tree. I do see it in patchwork.
>> https://patchwork.kernel.org/patch/2163151/
>>
>> But don't see sysfs features in
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/dri
>> vers/pwm/core.c
>
> It is not merged and will never be.
>
>> If you haven't had time to take Greg's comments into account, would
>> there be a possibility for you to correct those and re-submit?
>
> I already did resubmit an updated version taking Greg's comments into
> account. You can find the thread here:
>
> http://marc.info/?l=linux-kernel&m=136499756101273&w=2
> https://patchwork.kernel.org/patch/2387391/
>
> Also this version will not get merged into mainline. I have to do a v2
> according to Thierry's comments. I have planed to do it, but I will not
> find the time during the next say 2 months.
Ok, keep me updated. If in the meantime I find time, I'll take the next
set of comments into account and re-submit it, with a CC to you. If you
hear nothing, don't let that hold you back ;)
>
>> If not, I'd be happy to pull your patch, fix and re-submit (giving you
>> full credit of course).
>
> You're welcome! This is a community project. And there is nothing special
> to pull or something. Just take the patch from patchwork and work on it.
>
>> Your sysfs driver feature is quite awesome and makes using PWM's quite
>> cool, so that's definitely a good feature to have in the kernel.
>
> Thanks!
> Thank you for your interest!
>
> Regards,
> Lars
>
prev parent reply other threads:[~2013-05-23 14:43 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <519DDC5B.4040403@schinagl.nl>
2013-05-23 12:12 ` Lars Poeschel
2013-05-23 14:43 ` Oliver Schinagl [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=519E2B05.6060206@schinagl.nl \
--to=oliver+list@schinagl.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=poeschel@lemonage.de \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®