From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757408AbaHZHZC (ORCPT ); Tue, 26 Aug 2014 03:25:02 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:43191 "EHLO mx08-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755391AbaHZHZA (ORCPT ); Tue, 26 Aug 2014 03:25:00 -0400 Message-ID: <53FC363E.6050604@st.com> Date: Tue, 26 Aug 2014 09:24:46 +0200 From: Giuseppe CAVALLARO User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: Ley Foon Tan , , , "David S. Miller" Cc: , Vince Bridgers Subject: Re: [PATCH] net: stmmac: fix warning from Sparse for socfpga References: <1409037076-14775-1-git-send-email-lftan@altera.com> In-Reply-To: <1409037076-14775-1-git-send-email-lftan@altera.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.52.138.46] X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.12.52,1.0.27,0.0.0000 definitions=2014-08-26_03:2014-08-25,2014-08-26,1970-01-01 signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8/26/2014 9:11 AM, Ley Foon Tan wrote: > Warning: > drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c:122:41: > sparse: cast removes address space of expression > drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c:122:38: > sparse: incorrect type in assignment (different address spaces) > > Signed-off-by: Ley Foon Tan > --- > drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c > index cd613d7..c1addce 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c > +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-socfpga.c > @@ -119,7 +119,8 @@ static int socfpga_dwmac_parse_data(struct socfpga_dwmac *dwmac, struct device * > return -EINVAL; > } > > - dwmac->splitter_base = (void *)devm_ioremap_resource(dev, > + dwmac->splitter_base = > + (void __iomem *)devm_ioremap_resource(dev, I think, no casting should be done: dwmac->splitter_base = devm_ioremap_resource(dev, .... patch should be for net-next peppe > &res_splitter); > if (!dwmac->splitter_base) { > dev_info(dev, "Failed to mapping emac splitter\n"); >