All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Herbert Xu <herbert@gondor.apana.org.au>
Cc: javier Martin <javier.martin@vista-silicon.com>,
	linux-crypto@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org,
	devicetree-discuss@lists.ozlabs.org, davem@davemloft.net,
	kernel@pengutronix.de, gcembed@gmail.com, shawn.guo@linaro.org,
	arnd@arndb.de, swarren@nvidia.com
Subject: Re: [PATCH v3 1/2] i.MX27: Add clock support for SAHARA2.
Date: Mon, 11 Mar 2013 08:42:06 +0100	[thread overview]
Message-ID: <20130311074206.GY1906@pengutronix.de> (raw)
In-Reply-To: <20130311011926.GA14930@gondor.apana.org.au>

On Mon, Mar 11, 2013 at 09:19:26AM +0800, Herbert Xu wrote:
> On Mon, Mar 11, 2013 at 12:08:56AM +0100, Sascha Hauer wrote:
> > On Sun, Mar 10, 2013 at 04:34:01PM +0800, Herbert Xu wrote:
> > > > >
> > > > > https://patchwork.kernel.org/patch/1817741/
> > > > >
> > > > > So the change above becomes unnecessary
> > > > 
> > > > Very good. Then this patch can be safely dropped.
> > > 
> > > So should I take this patch or not?
> > 
> > This clk patch, no. The sahara patch, yes, if it is fine for you.
> 
> But will the second patch work fine without the first?

It will work once a device is registered. The necessary clocks for it
will be provided by the devicetree then.

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 v3 1/2] i.MX27: Add clock support for SAHARA2.
Date: Mon, 11 Mar 2013 08:42:06 +0100	[thread overview]
Message-ID: <20130311074206.GY1906@pengutronix.de> (raw)
In-Reply-To: <20130311011926.GA14930@gondor.apana.org.au>

On Mon, Mar 11, 2013 at 09:19:26AM +0800, Herbert Xu wrote:
> On Mon, Mar 11, 2013 at 12:08:56AM +0100, Sascha Hauer wrote:
> > On Sun, Mar 10, 2013 at 04:34:01PM +0800, Herbert Xu wrote:
> > > > >
> > > > > https://patchwork.kernel.org/patch/1817741/
> > > > >
> > > > > So the change above becomes unnecessary
> > > > 
> > > > Very good. Then this patch can be safely dropped.
> > > 
> > > So should I take this patch or not?
> > 
> > This clk patch, no. The sahara patch, yes, if it is fine for you.
> 
> But will the second patch work fine without the first?

It will work once a device is registered. The necessary clocks for it
will be provided by the devicetree then.

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 |

  reply	other threads:[~2013-03-11  7:42 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-01 11:37 [PATCH v3 0/2] crypto: sahara: Add support for SAHARA in i.MX27 Javier Martin
2013-03-01 11:37 ` Javier Martin
2013-03-01 11:37 ` [PATCH v3 1/2] i.MX27: Add clock support for SAHARA2 Javier Martin
2013-03-01 11:37   ` Javier Martin
2013-03-04  7:53   ` Sascha Hauer
2013-03-04  7:53     ` Sascha Hauer
2013-03-05  7:48     ` javier Martin
2013-03-05  7:48       ` javier Martin
2013-03-10  8:34       ` Herbert Xu
2013-03-10  8:34         ` Herbert Xu
2013-03-10 23:08         ` Sascha Hauer
2013-03-10 23:08           ` Sascha Hauer
2013-03-11  1:19           ` Herbert Xu
2013-03-11  1:19             ` Herbert Xu
2013-03-11  7:42             ` Sascha Hauer [this message]
2013-03-11  7:42               ` Sascha Hauer
2013-03-11  8:04               ` javier Martin
2013-03-11  8:04                 ` javier Martin
2013-03-01 11:37 ` [PATCH v3 2/2] crypto: sahara: Add driver for SAHARA2 accelerator Javier Martin
2013-03-01 11:37   ` Javier Martin
2013-03-19  9:43   ` javier Martin
2013-03-19  9:43     ` javier Martin
2013-03-21  9:53   ` Herbert Xu
2013-03-21  9:53     ` Herbert Xu

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=20130311074206.GY1906@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=arnd@arndb.de \
    --cc=davem@davemloft.net \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --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=linux-doc@vger.kernel.org \
    --cc=shawn.guo@linaro.org \
    --cc=swarren@nvidia.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.