| Message ID | 1363679016-3761-1-git-send-email-wenyou.yang@atmel.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
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 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 ------------------------------------------------------------------------------ Own the Future-Intel(R) Level Up Game Demo Contest 2013 Rise to greatness in Intel's independent game demo contest. Compete for recognition, cash, and the chance to get your game on Steam. $5K grand prize plus 10 genre and skill prizes. Submit your demo by 6/6/13. http://altfarm.mediaplex.com/ad/ck/12124-176961-30367-2
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 ------------------------------------------------------------------------------ Own the Future-Intel(R) Level Up Game Demo Contest 2013 Rise to greatness in Intel's independent game demo contest. Compete for recognition, cash, and the chance to get your game on Steam. $5K grand prize plus 10 genre and skill prizes. Submit your demo by 6/6/13. http://altfarm.mediaplex.com/ad/ck/12124-176961-30367-2
diff --git a/drivers/spi/spi-atmel.c b/drivers/spi/spi-atmel.c index 64e2795..f3e1cd8 100644 --- a/drivers/spi/spi-atmel.c +++ 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; @@ -993,6 +994,7 @@ static int atmel_spi_probe(struct platform_device *pdev) as->regs = ioremap(regs->start, resource_size(regs)); if (!as->regs) goto out_free_buffer; + as->phybase = regs->start; as->irq = irq; as->clk = clk;