Re: [PATCH v3 3/3] xen/arm: Introduce support for Renesas R-Car Gen2 platform

[Ian Campbell <Ian.Campbell@citrix.com>]

On Wed, 2015-01-28 at 15:48 +0200, Oleksandr Tyshchenko wrote:
> Hi Ian
> 
> On Wed, Jan 28, 2015 at 3:26 PM, Ian Campbell <Ian.Campbell@citrix.com> wrote:
> > On Fri, 2015-01-23 at 17:33 +0200, Oleksandr Tyshchenko wrote:
> >> From: Iurii Konovalenko <iurii.konovalenko@globallogic.com>
> >>
> >> Signed-off-by: Iurii Konovalenko <iurii.konovalenko@globallogic.com>
> >> Reviewed-by: Julien Grall <julien.grall@linaro.org>
> >> ---
> >>  xen/arch/arm/platforms/Makefile   |  1 +
> >>  xen/arch/arm/platforms/shmobile.c | 71 +++++++++++++++++++++++++++++++++++++++
> >>  2 files changed, 72 insertions(+)
> >>  create mode 100644 xen/arch/arm/platforms/shmobile.c
> >>
> >> diff --git a/xen/arch/arm/platforms/Makefile b/xen/arch/arm/platforms/Makefile
> >> index 8f47c16..29e931a 100644
> >> --- a/xen/arch/arm/platforms/Makefile
> >> +++ b/xen/arch/arm/platforms/Makefile
> >> @@ -4,5 +4,6 @@ obj-$(CONFIG_ARM_32) += exynos5.o
> >>  obj-$(CONFIG_ARM_32) += midway.o
> >>  obj-$(CONFIG_ARM_32) += omap5.o
> >>  obj-$(CONFIG_ARM_32) += sunxi.o
> >> +obj-$(CONFIG_ARM_32) += shmobile.o
> >
> > Everything else calls the platform rcar. How does shmobile relate to
> > that?
> Unfortunately, the author of this patch in vacation now.
> 
> Perhaps Iurii took this name from kernel part (Linux/arch/arm/mach-shmobile/...)
> where the arch specific things for this platform is located,
> especially the SMP and timer support.
> 
> I would rename shmobile to rcar2.

I'd prefer us to be consistent one way or another.

Is RCAR2 a subset of shmobile or the other way around?

http://am.renesas.com/applications/automotive/cis/cis_highend/rcar_h2/index.jsp doesn't mention shmobile at all.

Having determined which is the "parent"/superset my next question would
be how similar are the various "children" -- similar enough to be a
single platform in Xen, or different enough that we will need multiple
platforms, since that would have an impact on which name I think we
should pick.

(To some extent we can fidge this since we can always git mv stuff, but
we should at least give it our best guess up front)


Ian.