From: "Pali Rohár" <pali.rohar@gmail.com>
To: tony@atomide.com
Cc: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>,
robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com,
ijc+devicetree@hellion.org.uk, galak@codeaurora.org,
thierry.reding@gmail.com, bcousson@baylibre.com,
linux@arm.linux.org.uk, mchehab@osg.samsung.com,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pwm@vger.kernel.org, linux-omap@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-media@vger.kernel.org, sre@kernel.org
Subject: Re: [PATCH 0/7] ir-rx51 driver fixes
Date: Mon, 13 Jun 2016 20:17:38 +0200 [thread overview]
Message-ID: <201606132017.38629@pali> (raw)
In-Reply-To: <1462634508-24961-1-git-send-email-ivo.g.dimitrov.75@gmail.com>
[-- Attachment #1: Type: Text/Plain, Size: 2053 bytes --]
On Saturday 07 May 2016 17:21:41 Ivaylo Dimitrov wrote:
> ir-rx51 is a driver for Nokia N900 IR transmitter. The current series
> fixes the remaining problems in the driver:
>
> - replace GP timer 9 with PWM framework usage
> - replace pulse width timer dmtimer usage with hrtimer
> - add DT support to the driver
> - add driver to the board DTS
>
> Pathes 2 and 5 are needed so the driver to function correctly,
> without those PWM either refuses to set the needed carrier frequency
> (patch 2) or there are such a delays in the PWM framework, code that
> transmission duration raises to ~5s instead of half a second.
>
> Ivaylo Dimitrov (6):
> pwm: omap-dmtimer: Allow for setting dmtimer clock source
> [media] ir-rx51: use PWM framework instead of OMAP dmtimer
> [media] ir-rx51: add DT support to driver
> ARM: OMAP: dmtimer: Do not call PM runtime functions when not
> needed. [media] ir-rx51: use hrtimer instead of dmtimer
> ARM: dts: n900: enable lirc-rx51 driver
>
> Tony Lindgren (1):
> ir-rx51: Fix build after multiarch changes broke it
>
> .../devicetree/bindings/media/nokia,lirc-rx51 | 19 ++
> .../devicetree/bindings/pwm/pwm-omap-dmtimer.txt | 4 +
> arch/arm/boot/dts/omap3-n900.dts | 12 ++
> arch/arm/mach-omap2/board-rx51-peripherals.c | 5 -
> arch/arm/mach-omap2/pdata-quirks.c | 10 +-
> arch/arm/plat-omap/dmtimer.c | 9 +-
> arch/arm/plat-omap/include/plat/dmtimer.h | 1 +
> drivers/media/rc/Kconfig | 2 +-
> drivers/media/rc/ir-rx51.c | 229
> +++++++-------------- drivers/pwm/pwm-omap-dmtimer.c
> | 12 +- include/linux/platform_data/media/ir-rx51.h |
> 3 -
> 11 files changed, 131 insertions(+), 175 deletions(-)
> create mode 100644
> Documentation/devicetree/bindings/media/nokia,lirc-rx51
Patch series looks good, you can add my Acked-by.
--
Pali Rohár
pali.rohar@gmail.com
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2016-06-13 18:17 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-07 15:21 [PATCH 0/7] ir-rx51 driver fixes Ivaylo Dimitrov
2016-05-07 15:21 ` [PATCH 1/7] ir-rx51: Fix build after multiarch changes broke it Ivaylo Dimitrov
[not found] ` <1462634508-24961-1-git-send-email-ivo.g.dimitrov.75-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-07 15:21 ` [PATCH 2/7] pwm: omap-dmtimer: Allow for setting dmtimer clock source Ivaylo Dimitrov
2016-05-09 20:01 ` Rob Herring
2016-05-07 15:21 ` [PATCH 3/7] [media] ir-rx51: use PWM framework instead of OMAP dmtimer Ivaylo Dimitrov
2016-05-07 15:21 ` [PATCH 4/7] [media] ir-rx51: add DT support to driver Ivaylo Dimitrov
2016-05-09 20:06 ` Rob Herring
2016-05-09 20:53 ` Ivaylo Dimitrov
[not found] ` <5730F8BA.5000402-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-09 21:07 ` Rob Herring
2016-05-09 22:06 ` Ivaylo Dimitrov
2016-05-10 2:18 ` Sebastian Reichel
2016-05-11 14:14 ` Rob Herring
2016-05-13 6:15 ` Ivaylo Dimitrov
2016-05-13 14:01 ` Sebastian Reichel
2016-05-07 15:21 ` [PATCH 5/7] ARM: OMAP: dmtimer: Do not call PM runtime functions when not needed Ivaylo Dimitrov
[not found] ` <1462634508-24961-6-git-send-email-ivo.g.dimitrov.75-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-05-09 19:36 ` Tony Lindgren
2016-05-09 20:51 ` Ivaylo Dimitrov
2016-06-10 10:22 ` Tony Lindgren
2016-06-10 21:21 ` Ivaylo Dimitrov
2016-06-13 7:10 ` Tony Lindgren
2016-06-13 7:39 ` Tony Lindgren
2016-06-13 18:58 ` Ivaylo Dimitrov
[not found] ` <575F025F.7000101-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-06-14 7:53 ` Tony Lindgren
2016-05-07 15:21 ` [PATCH 6/7] [media] ir-rx51: use hrtimer instead of dmtimer Ivaylo Dimitrov
2016-05-07 15:21 ` [PATCH 7/7] ARM: dts: n900: enable lirc-rx51 driver Ivaylo Dimitrov
2016-06-10 10:20 ` Tony Lindgren
2016-06-13 18:17 ` Pali Rohár [this message]
2016-06-13 18:48 ` [PATCH 0/7] ir-rx51 driver fixes Ivaylo Dimitrov
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=201606132017.38629@pali \
--to=pali.rohar@gmail.com \
--cc=bcousson@baylibre.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=ivo.g.dimitrov.75@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=mchehab@osg.samsung.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
--cc=sre@kernel.org \
--cc=thierry.reding@gmail.com \
--cc=tony@atomide.com \
/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;
as well as URLs for NNTP newsgroup(s).