All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: Maxime Ripard
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>,
	Emilio Lopez <emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org>,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org,
	linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	sunny-0TFLnhJekD6UEPyfVivIlAC/G2K4zDHf@public.gmane.org,
	shuge-0TFLnhJekD6UEPyfVivIlAC/G2K4zDHf@public.gmane.org,
	kevin-0TFLnhJekD6UEPyfVivIlAC/G2K4zDHf@public.gmane.org
Subject: Re: [PATCHv3 0/6] Add I2C support for Allwinner SoCs
Date: Mon, 27 May 2013 21:17:27 +0200	[thread overview]
Message-ID: <2987322.J4qAl07e8R@flatron> (raw)
In-Reply-To: <1369648843-1640-1-git-send-email-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>

Hi Maxime,

On Monday 27 of May 2013 12:00:37 Maxime Ripard wrote:
> Hi,
> 
> This patchset adds support for the I2C controller found on most of the
> Allwinner SoCs, especially the already supported A10 and A13, and the
> yet to come A31.
> 
> It has been tested on a A13-Olinuxino.
> 
> Thanks,
> Maxime
> 
> Changes from v2:
>   * Slightly modified the switch comments again
>   * Removed the of_* calls in favor of platform_get_* functions
> 
> Changes from v1:
>   * Added comments to the switch statements to clarify when the fall
> through to the next case is made on purpose
>   * Use devm_ioremap_resource instead of of_iomap
>   * Moved the reset after enabling the clocks
>   * Added Emilio Lopez' patch to add the available i2c controllers to
> the cubieboard
> 
> Emilio López (1):
>   ARM: sun4i: cubieboard: Enable the i2c controllers
> 
> Maxime Ripard (5):
>   i2c: sunxi: Add Allwinner A1X i2c driver
>   ARM: sunxi: dt: Add i2c controller nodes to the DTSI
>   ARM: sun4i: dt: Add i2c muxing options
>   ARM: sun5i: dt: Add i2c muxing options
>   ARM: sun5i: olinuxino: Enable the i2c controllers
> 
>  .../devicetree/bindings/i2c/i2c-sunxi.txt          |  19 +
>  arch/arm/boot/dts/sun4i-a10-cubieboard.dts         |  12 +
>  arch/arm/boot/dts/sun4i-a10.dtsi                   |  48 +++
>  arch/arm/boot/dts/sun5i-a13-olinuxino.dts          |  18 +
>  arch/arm/boot/dts/sun5i-a13.dtsi                   |  48 +++
>  drivers/i2c/busses/Kconfig                         |  10 +
>  drivers/i2c/busses/Makefile                        |   1 +
>  drivers/i2c/busses/i2c-sunxi.c                     | 444
> +++++++++++++++++++++ 8 files changed, 600 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/i2c/i2c-sunxi.txt
>  create mode 100644 drivers/i2c/busses/i2c-sunxi.c

All the patches look good to me.

Reviewed-by: Tomasz Figa <tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

Best regards,
Tomasz

-- 
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/groups/opt_out.

WARNING: multiple messages have this Message-ID (diff)
From: tomasz.figa@gmail.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv3 0/6] Add I2C support for Allwinner SoCs
Date: Mon, 27 May 2013 21:17:27 +0200	[thread overview]
Message-ID: <2987322.J4qAl07e8R@flatron> (raw)
In-Reply-To: <1369648843-1640-1-git-send-email-maxime.ripard@free-electrons.com>

Hi Maxime,

On Monday 27 of May 2013 12:00:37 Maxime Ripard wrote:
> Hi,
> 
> This patchset adds support for the I2C controller found on most of the
> Allwinner SoCs, especially the already supported A10 and A13, and the
> yet to come A31.
> 
> It has been tested on a A13-Olinuxino.
> 
> Thanks,
> Maxime
> 
> Changes from v2:
>   * Slightly modified the switch comments again
>   * Removed the of_* calls in favor of platform_get_* functions
> 
> Changes from v1:
>   * Added comments to the switch statements to clarify when the fall
> through to the next case is made on purpose
>   * Use devm_ioremap_resource instead of of_iomap
>   * Moved the reset after enabling the clocks
>   * Added Emilio Lopez' patch to add the available i2c controllers to
> the cubieboard
> 
> Emilio L?pez (1):
>   ARM: sun4i: cubieboard: Enable the i2c controllers
> 
> Maxime Ripard (5):
>   i2c: sunxi: Add Allwinner A1X i2c driver
>   ARM: sunxi: dt: Add i2c controller nodes to the DTSI
>   ARM: sun4i: dt: Add i2c muxing options
>   ARM: sun5i: dt: Add i2c muxing options
>   ARM: sun5i: olinuxino: Enable the i2c controllers
> 
>  .../devicetree/bindings/i2c/i2c-sunxi.txt          |  19 +
>  arch/arm/boot/dts/sun4i-a10-cubieboard.dts         |  12 +
>  arch/arm/boot/dts/sun4i-a10.dtsi                   |  48 +++
>  arch/arm/boot/dts/sun5i-a13-olinuxino.dts          |  18 +
>  arch/arm/boot/dts/sun5i-a13.dtsi                   |  48 +++
>  drivers/i2c/busses/Kconfig                         |  10 +
>  drivers/i2c/busses/Makefile                        |   1 +
>  drivers/i2c/busses/i2c-sunxi.c                     | 444
> +++++++++++++++++++++ 8 files changed, 600 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/i2c/i2c-sunxi.txt
>  create mode 100644 drivers/i2c/busses/i2c-sunxi.c

All the patches look good to me.

Reviewed-by: Tomasz Figa <tomasz.figa@gmail.com>

Best regards,
Tomasz

  parent reply	other threads:[~2013-05-27 19:17 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-27 10:00 [PATCHv3 0/6] Add I2C support for Allwinner SoCs Maxime Ripard
2013-05-27 10:00 ` Maxime Ripard
     [not found] ` <1369648843-1640-1-git-send-email-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-05-27 10:00   ` [PATCHv3 1/6] i2c: sunxi: Add Allwinner A1X i2c driver Maxime Ripard
2013-05-27 10:00     ` Maxime Ripard
     [not found]     ` <1369648843-1640-2-git-send-email-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-06-05 13:39       ` Wolfram Sang
2013-06-05 13:39         ` Wolfram Sang
2013-06-05 21:31         ` Maxime Ripard
2013-06-05 21:31           ` Maxime Ripard
2013-06-10 10:08           ` Maxime Ripard
2013-06-10 10:08             ` Maxime Ripard
2013-06-11 18:22             ` Wolfram Sang
2013-06-11 18:22               ` Wolfram Sang
2013-06-11 18:21           ` Wolfram Sang
2013-06-11 18:21             ` Wolfram Sang
2013-06-12  8:11             ` Maxime Ripard
2013-06-12  8:11               ` Maxime Ripard
2013-05-27 10:00   ` [PATCHv3 2/6] ARM: sunxi: dt: Add i2c controller nodes to the DTSI Maxime Ripard
2013-05-27 10:00     ` Maxime Ripard
2013-05-27 10:00   ` [PATCHv3 3/6] ARM: sun4i: dt: Add i2c muxing options Maxime Ripard
2013-05-27 10:00     ` Maxime Ripard
2013-05-27 10:00   ` [PATCHv3 4/6] ARM: sun5i: " Maxime Ripard
2013-05-27 10:00     ` Maxime Ripard
2013-05-27 10:00   ` [PATCHv3 5/6] ARM: sun5i: olinuxino: Enable the i2c controllers Maxime Ripard
2013-05-27 10:00     ` Maxime Ripard
2013-05-27 10:00   ` [PATCHv3 6/6] ARM: sun4i: cubieboard: " Maxime Ripard
2013-05-27 10:00     ` Maxime Ripard
2013-05-27 19:17   ` Tomasz Figa [this message]
2013-05-27 19:17     ` [PATCHv3 0/6] Add I2C support for Allwinner SoCs Tomasz Figa
2013-05-29  7:34     ` Maxime Ripard
2013-05-29  7:34       ` Maxime Ripard

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=2987322.J4qAl07e8R@flatron \
    --to=tomasz.figa-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=emilio-0Z03zUJReD5OxF6Tv1QG9Q@public.gmane.org \
    --cc=kevin-0TFLnhJekD6UEPyfVivIlAC/G2K4zDHf@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
    --cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
    --cc=shuge-0TFLnhJekD6UEPyfVivIlAC/G2K4zDHf@public.gmane.org \
    --cc=sunny-0TFLnhJekD6UEPyfVivIlAC/G2K4zDHf@public.gmane.org \
    --cc=wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.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.