linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: Christian Eggers <ceggers@gmx.de>
Cc: spi-devel-general@lists.sourceforge.net,
	linux-kernel@vger.kernel.org, trivial@kernel.org
Subject: Re: [resubmit2] [PATCH] spi: spi-davinci: Fix direction in dma_map_single()
Date: Mon, 29 Jul 2013 20:33:57 +0100	[thread overview]
Message-ID: <20130729193357.GF9858@sirena.org.uk> (raw)
In-Reply-To: <1388690.s58sU4PUYE@p2400>

[-- Attachment #1: Type: text/plain, Size: 1312 bytes --]

On Mon, Jul 29, 2013 at 08:54:09PM +0200, Christian Eggers wrote:

> The following patch I already sent on 2013-04-18 and 2013-06-06. 
> Unfortunately it hasn't been committed yet.

I'm not sure where you submitted this to but it doesn't appear to have
been to either me or to the mailing list.  If you want a patch applying
you need to send it to the maintainer as covered in SubmittingPatches.
Google and your current post suggest it went to lkml and the retired
sourceforge list...

As also covered in SubmittingPatches you should also not include things
like this in the body of the commit message, put them after the ---
where they will be stripped automatically by tools like git am.

> Commit 048177ce3b3962852fd34a7e04938959271c7e70 (spi: spi-davinci: 
> convert to DMA engine API) introduced a regression: dma_map_single() 
> is called with direction DMA_FROM_DEVICE for rx and for tx.

This doesn't apply automatically since:

> ---
> --- drivers/spi/spi-davinci.c.orig      2013-04-18 20:54:02.728719412 +0200
> +++ drivers/spi/spi-davinci.c   2013-04-18 20:54:51.900623956 +0200

you've generated the patch in -p0 format not -p1 format as covered in
SubmittingPatches.  I fixed this up manually but please do follow the
process in SubmittingPatches, it makes life easier.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

      reply	other threads:[~2013-07-29 19:33 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-29 18:54 [resubmit2] [PATCH] spi: spi-davinci: Fix direction in dma_map_single() Christian Eggers
2013-07-29 19:33 ` Mark Brown [this message]

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=20130729193357.GF9858@sirena.org.uk \
    --to=broonie@kernel.org \
    --cc=ceggers@gmx.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=spi-devel-general@lists.sourceforge.net \
    --cc=trivial@kernel.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).