From: Vipul Pandya <vipul@chelsio•com>
To: Fengguang Wu <fengguang.wu@intel•com>
Cc: "kernel-janitors@vger•kernel.org"
<kernel-janitors@vger•kernel.org>,
Jay Hernandez <jay@chelsio•com>,
"netdev@vger•kernel.org" <netdev@vger•kernel.org>
Subject: Re: drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:363:64: sparse: incorrect type in argument 3 (different base types)
Date: Mon, 08 Oct 2012 18:38:40 +0530 [thread overview]
Message-ID: <5072D058.4080201@chelsio.com> (raw)
In-Reply-To: <20120928163653.GC5171@localhost>
On 28-09-2012 22:06, Fengguang Wu wrote:
> Hi Vipul,
>
> FYI, there are new sparse warnings show up in
>
> commit: 5afc8b84eb7b29e4646d6e8ca7e6d7196031d6f7 cxgb4: Add functions to read memory via PCIE memory window
>
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:361:33: sparse: incorrect type in assignment (different base types)
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:361:33: expected restricted __be32 [usertype] <noident>
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:361:33: got unsigned int
> + drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:363:64: sparse: incorrect type in argument 3 (different base types)
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:363:64: expected unsigned int [unsigned] [usertype] val
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:363:64: got restricted __be32 [usertype] <noident>
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:694:31: sparse: incorrect type in assignment (different base types)
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:694:31: expected unsigned int [unsigned] [usertype] <noident>
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:694:31: got restricted __be32 [usertype] <noident>
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:898:25: sparse: cast to restricted __be32
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:898:25: sparse: cast to restricted __be32
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:898:25: sparse: cast to restricted __be32
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:898:25: sparse: cast to restricted __be32
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:898:25: sparse: cast to restricted __be32
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:898:25: sparse: cast to restricted __be32
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:2177:25: sparse: incorrect type in assignment (different base types)
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:2177:25: expected restricted __be32 [usertype] <noident>
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:2177:25: got unsigned int
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c: In function 't4_memory_rw.constprop.6':
> drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:462:1: warning: the frame size of 2056 bytes is larger than 1024 bytes [-Wframe-larger-than=]
>
> vim +363 drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
>
> 5afc8b84 (Vipul Pandya 2012-09-26 347)
> 5afc8b84 (Vipul Pandya 2012-09-26 348) /*
> 5afc8b84 (Vipul Pandya 2012-09-26 349) * Setup offset into PCIE memory window. Address must be a
> 5afc8b84 (Vipul Pandya 2012-09-26 350) * MEMWIN0_APERTURE-byte-aligned address. (Read back MA register to
> 5afc8b84 (Vipul Pandya 2012-09-26 351) * ensure that changes propagate before we attempt to use the new
> 5afc8b84 (Vipul Pandya 2012-09-26 352) * values.)
> 5afc8b84 (Vipul Pandya 2012-09-26 353) */
> 5afc8b84 (Vipul Pandya 2012-09-26 354) t4_write_reg(adap, PCIE_MEM_ACCESS_OFFSET,
> 5afc8b84 (Vipul Pandya 2012-09-26 355) addr & ~(MEMWIN0_APERTURE - 1));
> 5afc8b84 (Vipul Pandya 2012-09-26 356) t4_read_reg(adap, PCIE_MEM_ACCESS_OFFSET);
> 5afc8b84 (Vipul Pandya 2012-09-26 357)
> 5afc8b84 (Vipul Pandya 2012-09-26 358) /* Collecting data 4 bytes at a time upto MEMWIN0_APERTURE */
> 5afc8b84 (Vipul Pandya 2012-09-26 359) for (i = 0; i < MEMWIN0_APERTURE; i = i+0x4) {
> 5afc8b84 (Vipul Pandya 2012-09-26 360) if (dir)
> 5afc8b84 (Vipul Pandya 2012-09-26 361) *data++ = t4_read_reg(adap, (MEMWIN0_BASE + i));
> 5afc8b84 (Vipul Pandya 2012-09-26 362) else
> 5afc8b84 (Vipul Pandya 2012-09-26 @363) t4_write_reg(adap, (MEMWIN0_BASE + i), *data++);
> 5afc8b84 (Vipul Pandya 2012-09-26 364) }
> 5afc8b84 (Vipul Pandya 2012-09-26 365)
> 5afc8b84 (Vipul Pandya 2012-09-26 366) return 0;
> 5afc8b84 (Vipul Pandya 2012-09-26 367) }
> 5afc8b84 (Vipul Pandya 2012-09-26 368)
> 5afc8b84 (Vipul Pandya 2012-09-26 369) /**
> 5afc8b84 (Vipul Pandya 2012-09-26 370) * t4_memory_rw - read/write EDC 0, EDC 1 or MC via PCIE memory window
> 5afc8b84 (Vipul Pandya 2012-09-26 371) * @adap: the adapter
>
> ---
> 0-DAY kernel build testing backend Open Source Technology Centre
> Fengguang Wu, Yuanhan Liu Intel Corporation
>
Hi Fengguang Wu,
I have submitted the patch to fix this sparse warnings to David Miller's
net-next tree. The subject of the patch is 'cxgb4: Address various
sparse warnings'.
Thanks,
Vipul Pandya
prev parent reply other threads:[~2012-10-08 13:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-28 16:36 drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:363:64: sparse: incorrect type in argument 3 (different base types) Fengguang Wu
2012-10-02 0:48 ` Jay Hernandez
2012-10-02 8:46 ` Dan Carpenter
2012-10-08 13:08 ` Vipul Pandya [this message]
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=5072D058.4080201@chelsio.com \
--to=vipul@chelsio$(echo .)com \
--cc=fengguang.wu@intel$(echo .)com \
--cc=jay@chelsio$(echo .)com \
--cc=kernel-janitors@vger$(echo .)kernel.org \
--cc=netdev@vger$(echo .)kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox