All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Arnd Bergmann <arnd@arndb.de>
Cc: javier Martin <javier.martin@vista-silicon.com>,
	kernel@pengutronix.de, herbert@gondor.apana.org.au,
	davem@davemloft.net, tony@atomide.com, swarren@nvidia.com,
	shawn.guo@linaro.org, gcembed@gmail.com,
	linux-arm-kernel@lists.infradead.org,
	linux-crypto@vger.kernel.org
Subject: Re: [PATCH 0/3] crypto: sahara: Add support for SAHARA in i.MX27.
Date: Thu, 21 Feb 2013 16:23:10 +0100	[thread overview]
Message-ID: <20130221152310.GA1906@pengutronix.de> (raw)
In-Reply-To: <201302211440.59577.arnd@arndb.de>

On Thu, Feb 21, 2013 at 02:40:59PM +0000, Arnd Bergmann wrote:
> On Thursday 21 February 2013, javier Martin wrote:
> > We know about the existence of DT and the constant migration process
> > that is taking place towards it.
> > 
> > Moreover we are strongly interested in converting the Visstrim SM10
> > platform to DT. Unfortunately, there are several issues that need to
> > be solved so that Visstrim M10 can be fully converted to DT:
> > - full SoC camera DT support
> > - DT support for ov7670
> > - pinctrl support for i.MX27
> 
> Ok, I see. Full device tree support is certainly asking too much then.
> 
> > Furthermore, we have limited resources at the moment and we have to
> > decide priorities. But having an hybrid platform-DT support is our
> > next task for Visstrim M10.
> 
> Ok, cool.
> 
> > However, I understand your concern about new drivers having DT
> > support. Maybe a good approach could be that I added compile-tested
> > only DT support for this driver and remove platform support for
> > mainline submission (although we maintain it internally in our
> > repositories).
> > 
> > What do you think?
> 
> Sounds good. Since it sounds like the hardware is available in all
> imx27 device, maybe someone else can test your driver on an imx27
> board that already has partial DT support when you add it to
> arch/arm/boot/dts/imx27.dtsi?
> 
> Is there any reason why an imx27 based board would not support this
> driver? Are there any other chips that could use it, e.g. imx25 or
> imx6?

Neither i.MX25, i.MX31 nor i.MX21 have this unit. I haven't checked the
others, but I assume this is i.MX27 only.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

WARNING: multiple messages have this Message-ID (diff)
From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] crypto: sahara: Add support for SAHARA in i.MX27.
Date: Thu, 21 Feb 2013 16:23:10 +0100	[thread overview]
Message-ID: <20130221152310.GA1906@pengutronix.de> (raw)
In-Reply-To: <201302211440.59577.arnd@arndb.de>

On Thu, Feb 21, 2013 at 02:40:59PM +0000, Arnd Bergmann wrote:
> On Thursday 21 February 2013, javier Martin wrote:
> > We know about the existence of DT and the constant migration process
> > that is taking place towards it.
> > 
> > Moreover we are strongly interested in converting the Visstrim SM10
> > platform to DT. Unfortunately, there are several issues that need to
> > be solved so that Visstrim M10 can be fully converted to DT:
> > - full SoC camera DT support
> > - DT support for ov7670
> > - pinctrl support for i.MX27
> 
> Ok, I see. Full device tree support is certainly asking too much then.
> 
> > Furthermore, we have limited resources at the moment and we have to
> > decide priorities. But having an hybrid platform-DT support is our
> > next task for Visstrim M10.
> 
> Ok, cool.
> 
> > However, I understand your concern about new drivers having DT
> > support. Maybe a good approach could be that I added compile-tested
> > only DT support for this driver and remove platform support for
> > mainline submission (although we maintain it internally in our
> > repositories).
> > 
> > What do you think?
> 
> Sounds good. Since it sounds like the hardware is available in all
> imx27 device, maybe someone else can test your driver on an imx27
> board that already has partial DT support when you add it to
> arch/arm/boot/dts/imx27.dtsi?
> 
> Is there any reason why an imx27 based board would not support this
> driver? Are there any other chips that could use it, e.g. imx25 or
> imx6?

Neither i.MX25, i.MX31 nor i.MX21 have this unit. I haven't checked the
others, but I assume this is i.MX27 only.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

  parent reply	other threads:[~2013-02-21 15:23 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-21 12:19 [PATCH 0/3] crypto: sahara: Add support for SAHARA in i.MX27 Javier Martin
2013-02-21 12:19 ` Javier Martin
2013-02-21 12:19 ` [PATCH 1/3] i.MX27: Add platform support for SAHARA2 Javier Martin
2013-02-21 12:19   ` Javier Martin
2013-02-21 12:19 ` [PATCH 2/3] crypto: sahara: Add driver for SAHARA2 accelerator Javier Martin
2013-02-21 12:19   ` Javier Martin
2013-02-21 13:13   ` Arnd Bergmann
2013-02-21 13:13     ` Arnd Bergmann
2013-02-21 14:35     ` javier Martin
2013-02-21 14:35       ` javier Martin
2013-02-23 20:16   ` Arnaud Patard
2013-02-23 20:16     ` Arnaud Patard (Rtp)
2013-02-26 10:34     ` javier Martin
2013-02-26 10:34       ` javier Martin
2013-02-21 12:19 ` [PATCH 3/3] Visstrim M10: Add support for SAHARA2 module Javier Martin
2013-02-21 12:19   ` Javier Martin
2013-02-21 12:59 ` [PATCH 0/3] crypto: sahara: Add support for SAHARA in i.MX27 Arnd Bergmann
2013-02-21 12:59   ` Arnd Bergmann
2013-02-21 14:18   ` javier Martin
2013-02-21 14:18     ` javier Martin
2013-02-21 14:40     ` Arnd Bergmann
2013-02-21 14:40       ` Arnd Bergmann
2013-02-21 14:57       ` javier Martin
2013-02-21 14:57         ` javier Martin
2013-02-21 15:23       ` Sascha Hauer [this message]
2013-02-21 15:23         ` Sascha Hauer
2013-02-21 15:40         ` Fabio Estevam
2013-02-21 15:40           ` Fabio Estevam
2013-02-21 16:40           ` Arnaud Patard
2013-02-21 16:40             ` Arnaud Patard (Rtp)
2013-02-21 16:47             ` Fabio Estevam
2013-02-21 16:47               ` Fabio Estevam
2013-02-21 20:59               ` Arnaud Patard
2013-02-21 20:59                 ` Arnaud Patard (Rtp)
2013-02-21 15:18     ` Sascha Hauer
2013-02-21 15:18       ` Sascha Hauer
2013-02-21 15:23       ` javier Martin
2013-02-21 15:23         ` javier Martin

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=20130221152310.GA1906@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=arnd@arndb.de \
    --cc=davem@davemloft.net \
    --cc=gcembed@gmail.com \
    --cc=herbert@gondor.apana.org.au \
    --cc=javier.martin@vista-silicon.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-crypto@vger.kernel.org \
    --cc=shawn.guo@linaro.org \
    --cc=swarren@nvidia.com \
    --cc=tony@atomide.com \
    /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.