Messages in this thread | | | Subject | Re: [PATCH] drm/radeon: Include swiotlb.h if SWIOTLB configured. | From | Paul Menzel <> | Date | Mon, 13 Aug 2012 09:50:19 +0200 |
| |
Dear Huacai,
Am Montag, den 13.08.2012, 15:16 +0800 schrieb Huacai Chen: > On Mon, Aug 13, 2012 at 3:00 PM, Paul Menzel wrote:
> > thanks for your patch. > > > > Firstly, is Chen your first or last name? If it is your first name, your > > From address should be switched. > Chen is may last name.
I am sorry then.
> > Am Montag, den 13.08.2012, 10:09 +0800 schrieb Huacai Chen: > >> When SWIOTLB is configured, if without this patch kernel compilation > >> fails. > > > > Secondly, could you please always paste part of the error message into > > the commit message so that people hitting this problem and searching for > > it on the WWW have a higher chance finding your patch. > > > OK, I'll paste error message. > > > Thirdly, do you know why this was not caught beforehand and compilation > > worked before? > Maybe radeon is mostly used on X86 and swiotlb.h is included > indirectly on X86 platform. This patch is split from my MIPS patchset, > on MIPS machines compilation always failed.
Ah, I thought your patches had already been submitted.
> >> Signed-off-by: Huacai Chen <chenhc@lemote.com> > > > > Lastly, the addresses in your From line and Signed-off-by line disagree. > > You can easily fix that, by adding > My patch is generated by git format-patch and there is surely a From > line in the beginning (from chenhc@lemote.com),
It should be `From: Huacai Chen <chenhc@lemote.com>`.
> but the mail is sent out by gmail, so I don't know how to make From > line and Signed-off-by agree.
Do you copy the content of the file created by `git format-patch` into the Gmail Web compositor?
If yes, just do not delete the `From` *and* `Date` lines from the beginning as for example Alex Deucher is doing [1].
[…]
Thanks,
Paul
[1] http://lists.freedesktop.org/archives/dri-devel/2012-July/025200.html [unhandled content-type:application/pgp-signature] | |