linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: imx6q-sabrelite.dts: Add SPI NOR support
Date: Fri, 27 Apr 2012 10:38:51 +0200	[thread overview]
Message-ID: <20120427083851.GZ17184@pengutronix.de> (raw)
In-Reply-To: <20120427073546.GR2234@S2101-09.ap.freescale.net>

On Fri, Apr 27, 2012 at 03:35:48PM +0800, Shawn Guo wrote:
> On Fri, Apr 27, 2012 at 08:27:32AM +0200, Uwe Kleine-K?nig wrote:
> > This seems to me to be a userspace problem? My guess is applying the
> > patch changed the kernel version (appends '+') and you didn't run make
> > modules_install and/or depmod?!
> > 
> Hmm, what the modules are we missing here?  That's actually my question.
> Fabio, Uwe, Sascha, you do not see this error message with your testing?

Every imx spi user (in fact most spi users no matter which driver)
will see this message.

See:

commit ffbbdd21329f3e15eeca6df2d4bc11c04d9d91c0
Author: Linus Walleij <linus.walleij@linaro.org>
Date:   Wed Feb 22 10:05:38 2012 +0100

    spi: create a message queueing infrastructure
    
    This rips the message queue in the PL022 driver out and pushes
    it into (optional) common infrastructure. Drivers that want to
    use the message pumping thread will need to define the new
    per-messags transfer methods and leave the deprecated transfer()
    method as NULL.
    
    Most of the design is described in the documentation changes that
    are included in this patch.
    
    Since there is a queue that need to be stopped when the system
    is suspending/resuming, two new calls are implemented for the
    device drivers to call in their suspend()/resume() functions:
    spi_master_suspend() and spi_master_resume().
    
    ChangeLog v1->v2:
    - Remove Kconfig entry and do not make the queue support optional
      at all, instead be more agressive and have it as part of the
      compulsory infrastructure.
    - If the .transfer() method is implemented, delete print a small
      deprecation notice and do not start the transfer pump.
    - Fix a bitrotted comment.
    ChangeLog v2->v3:
    - Fix up a problematic sequence courtesy of Chris Blair.
    - Stop rather than destroy the queue on suspend() courtesy of
      Chris Blair.
    
    Signed-off-by: Chris Blair <chris.blair@stericsson.com>
    Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
    Tested-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
    Reviewed-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
    Signed-off-by: Grant Likely <grant.likely@secretlab.ca>


-- 
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:[~2012-04-27  8:38 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-24 23:35 [PATCH 1/2] ARM: imx6q-sabrelite.dts: Add SPI NOR support Fabio Estevam
2012-04-24 23:35 ` [PATCH 2/2] ARM: imx_v6_v7_defconfig: " Fabio Estevam
2012-04-27  3:17   ` Shawn Guo
2012-04-26  8:16 ` [PATCH 1/2] ARM: imx6q-sabrelite.dts: " Sascha Hauer
2012-04-27  3:51   ` Shawn Guo
2012-04-27  3:44 ` Shawn Guo
2012-04-27  3:51   ` Fabio Estevam
2012-04-27  5:05     ` Shawn Guo
2012-04-27  6:27       ` Uwe Kleine-König
2012-04-27  7:35         ` Shawn Guo
2012-04-27  8:38           ` Sascha Hauer [this message]
2012-04-27  7:19       ` Sascha Hauer
2012-04-27  7:31         ` Shawn Guo
2012-04-27 12:37     ` Linus Walleij
2012-04-27  4:05 ` [PATCH v2 2/2] ARM: imx_v6_v7_defconfig: " Fabio Estevam
2012-05-01 14:14 ` [PATCH 1/2] ARM: imx6q-sabrelite.dts: " Shawn Guo

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=20120427083851.GZ17184@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.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).