Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1765356AbYJMKKr (ORCPT ); Mon, 13 Oct 2008 06:10:47 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1761027AbYJMJvK (ORCPT ); Mon, 13 Oct 2008 05:51:10 -0400 Received: from yx-out-2324.google.com ([74.125.44.30]:63231 "EHLO yx-out-2324.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932198AbYJMJvI (ORCPT ); Mon, 13 Oct 2008 05:51:08 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references:x-google-sender-auth; b=dxJLBtKk6r+BnSbNAdl1P2nq5P4guJ9NyZAmhTL1Yz9EAH7sVcAJTvjt2BQii/d0EV warMuIW+mP4gV4dOUihveDNHPxii2VNKjgdMbNiNxr5gg74PyXBB9wS/fSdNqZGBrE1D H2WTqqis66lIl5r4LpfmHu4Ds8d40YQnC4CkE= Message-ID: <386072610810130251k3699a8f0lcbcfc67c6c1364b4@mail.gmail.com> Date: Mon, 13 Oct 2008 17:51:07 +0800 From: "Bryan Wu" To: "Ben Dooks" Subject: Re: [PATCH 1/1] netdev: DM9000: Added typecasting to supress some warnings on Blackfin Cc: ben@simtec.co.uk, s.hauer@pengutronix.de, jeff@garzik.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "Javier Herrero" In-Reply-To: <20081013091725.GG31627@trinity.fluff.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <1223889009-31336-1-git-send-email-cooloney@kernel.org> <20081013091725.GG31627@trinity.fluff.org> X-Google-Sender-Auth: 8d923d31b766ecb6 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2379 Lines: 82 On Mon, Oct 13, 2008 at 5:17 PM, Ben Dooks wrote: > On Mon, Oct 13, 2008 at 05:10:09PM +0800, Bryan Wu wrote: >> From: Javier Herrero > > No, your arch definitions of writesb and co are wrong if they > are not taking 'void __iomem *' arguments. > Right, we should check the I/O functions and make them support "void __iomem *" arguments. Thanks -Bryan >> Signed-off-by: Javier Herrero >> Signed-off-by: Bryan Wu >> --- >> drivers/net/dm9000.c | 12 ++++++------ >> 1 files changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/net/dm9000.c b/drivers/net/dm9000.c >> index f42c23f..c4737ca 100644 >> --- a/drivers/net/dm9000.c >> +++ b/drivers/net/dm9000.c >> @@ -188,35 +188,35 @@ iow(board_info_t * db, int reg, int value) >> >> static void dm9000_outblk_8bit(void __iomem *reg, void *data, int count) >> { >> - writesb(reg, data, count); >> + writesb((int)reg, data, count); >> } >> >> static void dm9000_outblk_16bit(void __iomem *reg, void *data, int count) >> { >> - writesw(reg, data, (count+1) >> 1); >> + writesw((int)reg, data, (count+1) >> 1); >> } >> >> static void dm9000_outblk_32bit(void __iomem *reg, void *data, int count) >> { >> - writesl(reg, data, (count+3) >> 2); >> + writesl((int)reg, data, (count+3) >> 2); >> } >> >> /* input block from chip to memory */ >> >> static void dm9000_inblk_8bit(void __iomem *reg, void *data, int count) >> { >> - readsb(reg, data, count); >> + readsb((int)reg, data, count); >> } >> >> >> static void dm9000_inblk_16bit(void __iomem *reg, void *data, int count) >> { >> - readsw(reg, data, (count+1) >> 1); >> + readsw((int)reg, data, (count+1) >> 1); >> } >> >> static void dm9000_inblk_32bit(void __iomem *reg, void *data, int count) >> { >> - readsl(reg, data, (count+3) >> 2); >> + readsl((int)reg, data, (count+3) >> 2); >> } >> >> /* dump block from chip to null */ >> -- >> 1.5.6 > > -- > -- > Ben > > Q: What's a light-year? > A: One-third less calories than a regular year. > > -- 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/