mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier.adi@gmail.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-kernel@vger.kernel.org,
	uclinux-dist-devel@blackfin.uclinux.org,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Subject: Re: [PATCH 2/2 v2] Blackfin: fix dma-mapping build errors
Date: Tue, 23 Jun 2009 07:09:14 -0400	[thread overview]
Message-ID: <8bd0f97a0906230409u5ec65e45je59cc2bea4852cde@mail.gmail.com> (raw)
In-Reply-To: <200906231040.46767.arnd@arndb.de>

On Tue, Jun 23, 2009 at 04:40, Arnd Bergmann wrote:
> On Tuesday 23 June 2009, Mike Frysinger wrote:
>> +static inline void dma_sync_sg_for_device(struct device *dev,
>> +                                     struct scatterlist *sg,
>> +                                     int nents, enum dma_data_direction dir)
>> +{
>> +}
>
> If you flush the dcache in dma_map_sg, you probably also need to
> flush it in dma_sync_*_for_device:
>
> static inline void
> dma_sync_sg_for_device(struct device *dev, struct scatterlist *sglist,
>                      int nents, enum dma_data_direction direction)
> {
>       struct scatterlist *sg;
>       int i;
>
>       for_each_sg(sglist, sg, nents, i)
>               invalidate_dcache_range(sg_virt(sg), sg_virt(sg) + sg->length);
> }

i'll just integrate this into the patch you sent previously during the
next merge window.  these new functions werent actually being used
anywhere on Blackfin systems, so i'm not terribly worried about them
for now.
-mike

  reply	other threads:[~2009-06-23 11:09 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-23  1:54 [PATCH 1/2] Blackfin: hook up new perf_counter_open syscall Mike Frysinger
2009-06-23  1:54 ` [PATCH 2/2] Blackfin: fix dma-mapping build errors Mike Frysinger
2009-06-23  2:14   ` FUJITA Tomonori
2009-06-23  2:29     ` Mike Frysinger
2009-06-23  2:37       ` FUJITA Tomonori
2009-06-23  2:33     ` [PATCH 2/2 v2] " Mike Frysinger
2009-06-23  8:40       ` Arnd Bergmann
2009-06-23 11:09         ` Mike Frysinger [this message]
2009-06-23 13:07           ` Arnd Bergmann

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=8bd0f97a0906230409u5ec65e45je59cc2bea4852cde@mail.gmail.com \
    --to=vapier.adi@gmail.com \
    --cc=arnd@arndb.de \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=uclinux-dist-devel@blackfin.uclinux.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

all inboxes | Powered by JetHome®