[alsa-devel] [PATCH 1/2 v2] asoc: add sport driver for bf6xx soc

Mark Brown broonie at opensource.wolfsonmicro.com
Wed Jun 20 12:25:55 CEST 2012

On Wed, Jun 20, 2012 at 05:00:30PM -0400, Scott Jiang wrote:

Applied, thanks.  There is one minor fixup below - please send an
incremental patch for that.

> The SPORT(Serial Port) module on bf6xx soc has a totally different ip comparing to bf5xx
> soc. An individual SPORT module consists of two independently configurable SPORT halves
> with identical functionality. Each SPORT half can be configured for either transmitter
> or receiver.

Please take more care with the formatting of your changelogs, they
should be wrapped to within 80 columns and the subject line should
correspond to the subsystem style.  I reflowed.

> +static irqreturn_t sport_err_irq(int irq, void *dev_id)
> +{
> +	struct sport_device *sport = dev_id;
> +	struct device *dev = &sport->pdev->dev;
> +
> +	if (sport->tx_regs->spctl & SPORT_CTL_DERRPRI)
> +		dev_dbg(dev, "sport error: TUVF\n");
> +	if (sport->rx_regs->spctl & SPORT_CTL_DERRPRI)
> +		dev_dbg(dev, "sport error: ROVF\n");

These should be dev_err() not dev_dbg() - they're errors.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20120620/d4a7dde7/attachment.sig 

More information about the Alsa-devel mailing list