Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753300Ab2HMHQg (ORCPT ); Mon, 13 Aug 2012 03:16:36 -0400 Received: from mail-lb0-f174.google.com ([209.85.217.174]:41113 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752216Ab2HMHQf (ORCPT ); Mon, 13 Aug 2012 03:16:35 -0400 MIME-Version: 1.0 In-Reply-To: <1344841259.4126.4.camel@mattotaupa> References: <1344823753-28780-1-git-send-email-chenhc@lemote.com> <1344841259.4126.4.camel@mattotaupa> Date: Mon, 13 Aug 2012 15:16:33 +0800 Message-ID: Subject: Re: [PATCH] drm/radeon: Include swiotlb.h if SWIOTLB configured. From: Huacai Chen To: Paul Menzel Cc: David Airlie , =?ISO-8859-1?Q?Michel_D=E4nzer?= , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Hongliang Tao , Fuxin Zhang , Huacai Chen , Hua Yan Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2534 Lines: 81 On Mon, Aug 13, 2012 at 3:00 PM, Paul Menzel wrote: > Dear Chen, > > > 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. > > 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. > >> Signed-off-by: Huacai Chen > > 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), but the mail is sent out by gmail, so I don't know how to make From line and Signed-off-by agree. > > From: Huacai Chen > > to the beginning of your message. > >> Signed-off-by: Hongliang Tao >> Signed-off-by: Hua Yan >> Cc: dri-devel@lists.freedesktop.org >> --- >> drivers/gpu/drm/radeon/radeon_ttm.c | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c >> index 5b71c71..fc3ac22 100644 >> --- a/drivers/gpu/drm/radeon/radeon_ttm.c >> +++ b/drivers/gpu/drm/radeon/radeon_ttm.c >> @@ -41,6 +41,10 @@ >> #include "radeon_reg.h" >> #include "radeon.h" >> >> +#ifdef CONFIG_SWIOTLB >> +#include >> +#endif >> + >> #define DRM_FILE_PAGE_OFFSET (0x100000000ULL >> PAGE_SHIFT) >> >> static int radeon_ttm_debugfs_init(struct radeon_device *rdev); > > With the clarification above > > Acked-by: Paul Menzel > > can be added to your patch. > > > Thanks, > > Paul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/