devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Alexander Bersenev <bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
Cc: linux-sunxi <linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org>,
	"david-0WFrdVpeebksCylrc8G9yg@public.gmane.org"
	<david-0WFrdVpeebksCylrc8G9yg@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org"
	<galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	"grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org"
	<grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org"
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	"james.hogan-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org"
	<james.hogan-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	"linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org"
	<linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	"m.chehab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
	<m.chehab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
	"mark.rutland-5wv7dgnIgG8@public.gmane.org"
	<mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	"maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org"
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	"pawel.moll-5wv7dgnIgG8@public.gmane.org"
	<pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	"rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org"
	<rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
	"robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org"
	<robh+dt@kernel.>
Subject: Re: [PATCH v5 0/3] ARM: sunxi: Add support for consumer infrared devices
Date: Fri, 09 May 2014 15:26:52 +0200	[thread overview]
Message-ID: <536CD79C.8080800@redhat.com> (raw)
In-Reply-To: <E54FF549-B53A-406B-9871-E2C3B1ED8157-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>

Hi,

On 05/09/2014 03:21 PM, Alexander Bersenev wrote:
> Hello, I am in Austria/Germany in 1-11 may. I plan to make v6 after returning back to Russia.

Ok, when you do please also pick up this fix:
https://github.com/jwrdegoede/linux-sunxi/commit/180a3e59361ceddeed26543529ca757cc8112fb0

Which fixes an oops at startup.

Other then needing that fix + coding style fixes it works well for me on
the cubietruck. I'll try to also test it on sun4i and sun5i if I can find some
time to do so.

Regards,

Hans


> 
> Best,
> Alexander Bersenev
> 
>> 08 мая 2014 г., в 15:55, Hans de Goede <hdegoede@redhat.com> написал(а):
>>
>> Hi,
>>
>>> On 04/30/2014 05:16 PM, Alexander Bersenev wrote:
>>> This patch introduces Consumer IR(CIR) support for sunxi boards.
>>>
>>> This is based on Alexsey Shestacov's work based on the original driver 
>>> supplied by Allwinner.
>>>
>>> Signed-off-by: Alexander Bersenev <bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
>>> Signed-off-by: Alexsey Shestacov <wingrime-3kdeTeqwOZ9EV1b7eY7vFQ@public.gmane.org>
>>
>> Alexander, v5 still has various issues which need fixing,
>> do you plan to do a v6 soon ?
>>
>> Regards,
>>
>> Hans
>>
>>>
>>> ---
>>> Changes since version 1: 
>>> - Fix timer memory leaks 
>>> - Fix race condition when driver unloads while interrupt handler is active
>>> - Support Cubieboard 2(need testing)
>>>
>>> Changes since version 2:
>>> - More reliable keydown events
>>> - Documentation fixes
>>> - Rename registers accurding to A20 user manual
>>> - Remove some includes, order includes alphabetically
>>> - Use BIT macro
>>> - Typo fixes
>>>
>>> Changes since version 3:
>>> - Split the patch on smaller parts
>>> - More documentation fixes
>>> - Add clock-names in DT
>>> - Use devm_clk_get function to get the clocks
>>> - Removed gpios property from ir's DT
>>> - Changed compatible from allwinner,sunxi-ir to allwinner,sun7i-a20-ir in DT
>>> - Use spin_lock_irq instead spin_lock_irqsave in interrupt handler
>>> - Add myself in the copyright ;)
>>> - Coding style and indentation fixes
>>>
>>> Changes since version 4:
>>> - Try to fix indentation errors by sending patches with git send-mail
>>>
>>> Alexander Bersenev (3):
>>>  ARM: sunxi: Add documentation for sunxi consumer infrared devices
>>>  ARM: sunxi: Add driver for sunxi IR controller
>>>  ARM: sunxi: Add IR controller support in DT on A20
>>>
>>> .../devicetree/bindings/media/sunxi-ir.txt         |  23 ++
>>> arch/arm/boot/dts/sun7i-a20-cubieboard2.dts        |   6 +
>>> arch/arm/boot/dts/sun7i-a20-cubietruck.dts         |   6 +
>>> arch/arm/boot/dts/sun7i-a20.dtsi                   |  31 +++
>>> drivers/media/rc/Kconfig                           |  10 +
>>> drivers/media/rc/Makefile                          |   1 +
>>> drivers/media/rc/sunxi-ir.c                        | 303 +++++++++++++++++++++
>>> 7 files changed, 380 insertions(+)
>>> create mode 100644 Documentation/devicetree/bindings/media/sunxi-ir.txt
>>> create mode 100644 drivers/media/rc/sunxi-ir.c
>>>

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

      parent reply	other threads:[~2014-05-09 13:26 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-30 15:16 [PATCH v5 0/3] ARM: sunxi: Add support for consumer infrared devices Alexander Bersenev
     [not found] ` <1398871010-30681-1-git-send-email-bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
2014-04-30 15:16   ` [PATCH v5 1/3] ARM: sunxi: Add documentation for sunxi " Alexander Bersenev
     [not found]     ` <1398871010-30681-2-git-send-email-bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
2014-05-03 17:45       ` Maxime Ripard
2014-04-30 15:16   ` [PATCH v5 2/3] ARM: sunxi: Add driver for sunxi IR controller Alexander Bersenev
     [not found]     ` <1398871010-30681-3-git-send-email-bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
2014-05-03 17:53       ` Maxime Ripard
2014-04-30 15:16   ` [PATCH v5 3/3] ARM: sunxi: Add IR controller support in DT on A20 Alexander Bersenev
     [not found]     ` <1398871010-30681-4-git-send-email-bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
2014-05-03 18:00       ` Maxime Ripard
2014-05-01  6:00   ` [PATCH v5 0/3] ARM: sunxi: Add support for consumer infrared devices Priit Laes
2014-05-08 13:55   ` Hans de Goede
     [not found]     ` <536B8CDD.7040600-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-05-09 13:21       ` Alexander Bersenev
     [not found]         ` <E54FF549-B53A-406B-9871-E2C3B1ED8157-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org>
2014-05-09 13:26           ` Hans de Goede [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=536CD79C.8080800@redhat.com \
    --to=hdegoede-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=bay-d8LqKIyZzER6k2ZkT/Rh8A@public.gmane.org \
    --cc=david-0WFrdVpeebksCylrc8G9yg@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=james.hogan-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
    --cc=m.chehab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
    --cc=robh+dt@kernel. \
    /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).