All of lore.kernel.org
 help / color / mirror / Atom feed
From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] dma: imx-sdma: clarify firmare not found warning
Date: Tue, 21 Jan 2014 08:06:49 +0100	[thread overview]
Message-ID: <20140121070649.GT16215@pengutronix.de> (raw)
In-Reply-To: <20140120123627.1147f252@ipc1.ka-ro>

Hi Lothar,

On Mon, Jan 20, 2014 at 12:36:27PM +0100, Lothar Wa?mann wrote:
> Hi,
> 
> Lothar Wa?mann wrote:
> > Hi,
> > 
> erroneously pressed the 'Send' button...
> 
> > Sascha Hauer wrote:
> > > When a firmware cannot be found for the SDMA engine then we can
> > > continue with the inernal ROM firmware.
> > > 
> > > The meaning of this message is frequently asked for, so make clear
> > > that the driver still works with the internal ROM firmware and reduce
> > > the loglevel from err to info.
> > > 
> In case user space firmware loading support (CONFIG_FW_LOADER) is
> enabled, this message may still be inadequate, since the firmware may
> very well be loaded lateron.

Then at least until the RAM firmware is loaded the message is still adequate,
no? We could add an addional message when the firmware is loaded if you
like.

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:[~2014-01-21  7:06 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-20 10:07 [PATCH v2] dma: imx-sdma: clarify firmare not found warning Sascha Hauer
2014-01-20 11:28 ` Lothar Waßmann
2014-01-20 11:36   ` Lothar Waßmann
2014-01-20 12:38     ` Russell King - ARM Linux
2014-01-20 11:59       ` Vinod Koul
2014-01-20 13:09         ` Sascha Hauer
2014-01-20 14:09       ` Lothar Waßmann
2014-01-20 15:11         ` Russell King - ARM Linux
2014-01-21  6:59           ` Lothar Waßmann
2014-01-21 11:01             ` Russell King - ARM Linux
2014-01-21 11:52               ` Lothar Waßmann
2014-01-21 11:57                 ` Russell King - ARM Linux
2014-01-21 13:36                   ` Lothar Waßmann
2014-01-21 14:10                     ` Russell King - ARM Linux
2014-01-21  7:06     ` Sascha Hauer [this message]
2014-01-21  8:31       ` Lothar Waßmann
2014-10-25 12:28 ` Fabio Estevam
2014-11-12 11:13   ` Vinod Koul
2014-11-12 22:05     ` Fabio Estevam

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=20140121070649.GT16215@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 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.