All of lore.kernel.org
 help / color / mirror / Atom feed
From: hdegoede@redhat.com (Hans de Goede)
To: linux-arm-kernel@lists.infradead.org
Subject: [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@hackerdom.ru>

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@hackerdom.ru>
>>> Signed-off-by: Alexsey Shestacov <wingrime@linux-sunxi.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
>>>

WARNING: multiple messages have this Message-ID (diff)
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.

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

Thread overview: 31+ 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
2014-04-30 15:16 ` Alexander Bersenev
2014-04-30 15:16 ` Alexander Bersenev
2014-04-30 15:16 ` [PATCH v5 1/3] ARM: sunxi: Add documentation for sunxi " Alexander Bersenev
2014-04-30 15:16   ` Alexander Bersenev
2014-04-30 15:16   ` Alexander Bersenev
2014-05-03 17:45   ` Maxime Ripard
2014-05-03 17:45     ` Maxime Ripard
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
2014-04-30 15:16   ` Alexander Bersenev
2014-04-30 15:16   ` Alexander Bersenev
2014-05-03 17:53   ` Maxime Ripard
2014-05-03 17:53     ` Maxime Ripard
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
2014-04-30 15:16   ` Alexander Bersenev
2014-04-30 15:16   ` Alexander Bersenev
2014-05-03 18:00   ` Maxime Ripard
2014-05-03 18:00     ` Maxime Ripard
2014-05-03 18:00     ` Maxime Ripard
2014-05-01  6:00 ` [linux-sunxi] [PATCH v5 0/3] ARM: sunxi: Add support for consumer infrared devices Priit Laes
2014-05-01  6:00   ` Priit Laes
2014-05-01  6:00   ` Priit Laes
2014-05-08 13:55 ` Hans de Goede
2014-05-08 13:55   ` Hans de Goede
2014-05-08 13:55   ` Hans de Goede
2014-05-09 13:21   ` Alexander Bersenev
2014-05-09 13:21     ` Alexander Bersenev
2014-05-09 13:26     ` Hans de Goede [this message]
2014-05-09 13:26       ` Hans de Goede

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@redhat.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.