From: sshtylyov@mvista.com (Sergei Shtylyov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM
Date: Fri, 15 Apr 2011 16:25:49 +0400 [thread overview]
Message-ID: <4DA8394D.80405@ru.mvista.com> (raw)
In-Reply-To: <201104132316.01922.jkrzyszt@tis.icnet.pl>
Hello.
On 14-04-2011 1:16, Janusz Krzysztofik wrote:
>>> Janusz Krzysztofik wrote:
>>>>>> After switching from mem->dma_handle to
>>>>>> virt_to_phys(mem->vaddr) used for obtaining page frame number
>>>>>> passed to remap_pfn_range() (commit
>>>>>> 35d9f510b67b10338161aba6229d4f55b4000f5b),
>>>>>> videobuf-dma-contig
>>>>> Please specify the commit summary -- for the human readers.
>>>> Hi,
>>>> OK, I'll try to reword the summary using a more human friendly
>>>> language as soon as I have signs that Mauro (who seemed to
>>>> understand the message well enough) is willing to accept the
>>>> code.
>>> I wasn't asking you to rework your summary but to specify the
>>> summary of the commit you've mentioned (in parens).
>> Ah, I see. How about just reordered wording:
>> After commit 35d9f510b67b10338161aba6229d4f55b4000f5b (switching from
>> mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page
>> frame number passed to remap_pfn_range()), ....
>> Do you think this would be clear enough?
> Oh no, I probably missed your point again.
> You meant just quoting the commit original summary line, didn't you?
Yes.
> Thanks,
> Janusz
WBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: Sergei Shtylyov <sshtylyov@mvista.com>
To: Janusz Krzysztofik <jkrzyszt@tis.icnet.pl>
Cc: Sergei Shtylyov <sshtylyov@mvista.com>,
Linux Media Mailing List <linux-media@vger.kernel.org>,
Jiri Slaby <jslaby@suse.cz>,
linux-arm-kernel@lists.infradead.org,
Mauro Carvalho Chehab <mchehab@infradead.org>
Subject: Re: [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM
Date: Fri, 15 Apr 2011 16:25:49 +0400 [thread overview]
Message-ID: <4DA8394D.80405@ru.mvista.com> (raw)
In-Reply-To: <201104132316.01922.jkrzyszt@tis.icnet.pl>
Hello.
On 14-04-2011 1:16, Janusz Krzysztofik wrote:
>>> Janusz Krzysztofik wrote:
>>>>>> After switching from mem->dma_handle to
>>>>>> virt_to_phys(mem->vaddr) used for obtaining page frame number
>>>>>> passed to remap_pfn_range() (commit
>>>>>> 35d9f510b67b10338161aba6229d4f55b4000f5b),
>>>>>> videobuf-dma-contig
>>>>> Please specify the commit summary -- for the human readers.
>>>> Hi,
>>>> OK, I'll try to reword the summary using a more human friendly
>>>> language as soon as I have signs that Mauro (who seemed to
>>>> understand the message well enough) is willing to accept the
>>>> code.
>>> I wasn't asking you to rework your summary but to specify the
>>> summary of the commit you've mentioned (in parens).
>> Ah, I see. How about just reordered wording:
>> After commit 35d9f510b67b10338161aba6229d4f55b4000f5b (switching from
>> mem->dma_handle to virt_to_phys(mem->vaddr) used for obtaining page
>> frame number passed to remap_pfn_range()), ....
>> Do you think this would be clear enough?
> Oh no, I probably missed your point again.
> You meant just quoting the commit original summary line, didn't you?
Yes.
> Thanks,
> Janusz
WBR, Sergei
next prev parent reply other threads:[~2011-04-15 12:25 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-12 21:06 [PATCH 2.6.39 v2] V4L: videobuf-dma-contig: fix mmap_mapper broken on ARM Janusz Krzysztofik
2011-04-12 21:06 ` Janusz Krzysztofik
2011-04-12 21:40 ` Russell King - ARM Linux
2011-04-12 21:40 ` Russell King - ARM Linux
2011-04-12 22:12 ` Theodore Kilgore
2011-04-12 22:12 ` Theodore Kilgore
2011-04-13 10:52 ` Janusz Krzysztofik
2011-04-13 10:52 ` Janusz Krzysztofik
2011-04-13 18:32 ` Russell King - ARM Linux
2011-04-13 18:32 ` Russell King - ARM Linux
2011-04-13 20:56 ` Janusz Krzysztofik
2011-04-13 20:56 ` Janusz Krzysztofik
2011-04-13 22:00 ` Russell King - ARM Linux
2011-04-13 22:00 ` Russell King - ARM Linux
2011-04-13 22:43 ` Janusz Krzysztofik
2011-04-13 22:43 ` Janusz Krzysztofik
2011-04-19 6:08 ` [REVERT] " Jiri Slaby
2011-04-19 6:08 ` Jiri Slaby
2011-04-19 12:37 ` Mauro Carvalho Chehab
2011-04-19 12:37 ` Mauro Carvalho Chehab
2011-04-13 12:03 ` [PATCH 2.6.39 v2] " Sergei Shtylyov
2011-04-13 12:03 ` Sergei Shtylyov
2011-04-13 13:11 ` Janusz Krzysztofik
2011-04-13 13:11 ` Janusz Krzysztofik
2011-04-13 17:36 ` Sergei Shtylyov
2011-04-13 17:36 ` Sergei Shtylyov
2011-04-13 21:01 ` Janusz Krzysztofik
2011-04-13 21:01 ` Janusz Krzysztofik
2011-04-13 21:16 ` Janusz Krzysztofik
2011-04-13 21:16 ` Janusz Krzysztofik
2011-04-15 12:25 ` Sergei Shtylyov [this message]
2011-04-15 12:25 ` Sergei Shtylyov
2011-04-14 11:17 ` Sergei Shtylyov
2011-04-14 11:17 ` Sergei Shtylyov
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=4DA8394D.80405@ru.mvista.com \
--to=sshtylyov@mvista.com \
--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.