Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754730Ab3DBGRR (ORCPT ); Tue, 2 Apr 2013 02:17:17 -0400 Received: from nasmtp02.atmel.com ([204.2.163.16]:45126 "EHLO nasmtp02.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752343Ab3DBGRQ (ORCPT ); Tue, 2 Apr 2013 02:17:16 -0400 From: "Yang, Wenyou" To: "Yang, Wenyou" , Mark Brown CC: "linux-arm-kernel@lists.infradead.org" , "grant.likely@secretlab.ca" , "richard.genoud@gmail.com" , "plagnioj@jcrosoft.com" , "Ferre, Nicolas" , "Lin, JM" , "spi-devel-general@lists.sourceforge.net" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH v7 03/14] spi/spi-atmel: add physical base address Thread-Topic: [PATCH v7 03/14] spi/spi-atmel: add physical base address Thread-Index: AQHOJHXx8mQi9R5DskWA68i2eZE0h5jA7keAgAGT0mCAAAdeYA== Date: Tue, 2 Apr 2013 06:17:06 +0000 Message-ID: References: <1363678866-3567-1-git-send-email-wenyou.yang@atmel.com> <1363679016-3761-1-git-send-email-wenyou.yang@atmel.com> <20130401134147.GV18636@opensource.wolfsonmicro.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.168.5.13] Content-Type: text/plain; charset="gb2312" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id r326HUvA015515 Content-Length: 1720 Lines: 55 Hi, Mark, > -----Original Message----- > From: Yang, Wenyou > Sent: 2013??4??2?? 13:50 > To: 'Mark Brown' > Cc: linux-arm-kernel@lists.infradead.org; grant.likely@secretlab.ca; > richard.genoud@gmail.com; plagnioj@jcrosoft.com; Ferre, Nicolas; Lin, JM; > spi-devel-general@lists.sourceforge.net; linux-kernel@vger.kernel.org > Subject: RE: [PATCH v7 03/14] spi/spi-atmel: add physical base address > > Hi, Mark, > > > -----Original Message----- > > From: Mark Brown [mailto:broonie@opensource.wolfsonmicro.com] > > Sent: 2013??4??1?? 21:42 > > To: Yang, Wenyou > > Cc: linux-arm-kernel@lists.infradead.org; grant.likely@secretlab.ca; > > richard.genoud@gmail.com; plagnioj@jcrosoft.com; Ferre, Nicolas; Lin, JM; > > spi-devel-general@lists.sourceforge.net; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH v7 03/14] spi/spi-atmel: add physical base address > > > > On Tue, Mar 19, 2013 at 03:43:36PM +0800, Wenyou Yang wrote: > > > > > +++ b/drivers/spi/spi-atmel.c > > > @@ -196,6 +196,7 @@ struct atmel_spi_caps { struct atmel_spi { > > > spinlock_t lock; > > > > > > + resource_size_t phybase; > > > void __iomem *regs; > > > int irq; > > > struct clk *clk; > > > > This looks like the wrong type? > I got it, you are right, you mean It should be used " phys_addr_t " type. phys_addr_t phybase; Thanks. Best Regards, Wenyou Yang. > I double checked. > > The type ' resource_size_t ' is defined in include/linux/ types.h as below, > > typedef phys_addr_t resource_size_t; > > I think it is OK. > > Best Regards, > Wenyou Yang ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?