linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Simon Horman <horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
To: Geert Uytterhoeven <geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org>
Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 0/8] SoC and board integration for RSPI on RZ/A1H
Date: Tue, 7 Jan 2014 10:26:28 +0900	[thread overview]
Message-ID: <20140107012628.GD2196@verge.net.au> (raw)
In-Reply-To: <1388175362-17132-1-git-send-email-geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org>

On Fri, Dec 27, 2013 at 09:15:54PM +0100, Geert Uytterhoeven wrote:
> This patch series integrates Renesas RSPI on the RZ/A1H aka R7S72100 SoC,
> and the Genmai development board.
> 
> The series is based on renesas-devel-v3.13-rc5-20131226, with Magnus'
> pinctrl, Wolfram's riic, and Simon's sh_eth work applied on top.
> 
> It was tested on the Genmai development board (genmai-reference), using
> an in-kernel version of Documentation/spi/spidev_test.c, exercising
> loopback mode on rspi1, with the same outcome as the SDK reference code.
> 
> Notes:
>   1. The proposed DT bindings also talk about QSPI on the R8A779x, as those
>      SoCs use the same driver.
>   2. As the actual spi-rspi driver doesn't use DT yet, platform devices and
>      spi board info are still used from C to instantiate SPI buses and their
>      children, while pinctrl is performed by DT.
>   3. It will not work with the non-reference version of genmai, due to
>      missing pinmux configuration in setup-r7s72100.c.
>   4. Patches [4/8] and [5/8] are new.
> 
>     [1/8] pinctrl: sh-pfc: r7s72100: Add RSPI support

I believe that Laurent should handle this.

>     [2/8] ARM: shmobile: r7s72100: Add RSPI clocks

This seems fine and independent of the other patches.
But unfortunately it no longer applies.

Feel free to rebase it if you would like me to queue it up sooner
than later.

>     [3/8] ARM: shmobile: r7s72100: Add RSPI platform devices

I assume this depends on 1/3.

Please add a patch to update the koelsch defconfig.

>     [4/8] ARM: shmobile: genmai: Add RSPI children (NEW)

This seems reasonable but I assume it depends on earlier patches.

Please add a patch to update the koelsch defconfig or Kconfig.

>     [5/8] [RFC] ARM: shmobile: genmai: Add preliminary RSPI pinmux (NEW)
>     [6/8] [RFC] Documentation: dt: Add Renesas RSPI/QSPI bindings
>     [7/8] [RFC] ARM: shmobile: r7s72100 dtsi: Add RSPI nodes
>     [8/8] [RFC] ARM: shmobile: genmai reference: Add RSPI nodes
> 
> Gr{oetje,eeting}s,
> 
> 						Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> 							    -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2014-01-07  1:26 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-27 20:15 [PATCH v2 0/8] SoC and board integration for RSPI on RZ/A1H Geert Uytterhoeven
2013-12-27 20:15 ` [PATCH v2 1/8] pinctrl: sh-pfc: r7s72100: Add RSPI support Geert Uytterhoeven
2013-12-27 20:16 ` [PATCH v2 6/8] [RFC] Documentation: dt: Add Renesas RSPI/QSPI bindings Geert Uytterhoeven
     [not found]   ` <1388175362-17132-7-git-send-email-geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org>
2013-12-30 12:58     ` Mark Brown
2013-12-30 13:43       ` Geert Uytterhoeven
2013-12-30 13:50         ` Mark Brown
2013-12-27 20:16 ` [PATCH v2 7/8] [RFC] ARM: shmobile: r7s72100 dtsi: Add RSPI nodes Geert Uytterhoeven
2013-12-27 20:16 ` [PATCH v2 8/8] [RFC] ARM: shmobile: genmai reference: " Geert Uytterhoeven
     [not found] ` <1388175362-17132-1-git-send-email-geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org>
2013-12-27 20:15   ` [PATCH v2 2/8] ARM: shmobile: r7s72100: Add RSPI clocks Geert Uytterhoeven
2013-12-27 20:15   ` [PATCH v2 3/8] ARM: shmobile: r7s72100: Add RSPI platform devices Geert Uytterhoeven
2013-12-27 20:15   ` [PATCH v2 4/8] ARM: shmobile: genmai: Add RSPI children Geert Uytterhoeven
2013-12-27 20:15   ` [PATCH v2 5/8] [RFC] ARM: shmobile: genmai: Add preliminary RSPI pinmux setup Geert Uytterhoeven
2014-01-07  1:26   ` Simon Horman [this message]
     [not found]     ` <20140107012628.GD2196-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org>
2014-01-07  8:31       ` [PATCH v2 0/8] SoC and board integration for RSPI on RZ/A1H Geert Uytterhoeven
     [not found]         ` <CAMuHMdW5e74keNhRkSG6q6crGWZ0Gqtb5-hntbCUNm7Md78Bcw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-01-07  8:52           ` Simon Horman
2014-01-07 15:01       ` Laurent Pinchart
2014-01-08  0:00         ` Simon Horman

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=20140107012628.GD2196@verge.net.au \
    --to=horms-/r6kz+ddxgppr4jqbcensq@public.gmane.org \
    --cc=geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-spi-u79uwXL29TY76Z2rM5mHXA@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 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).