[PATCH 2/4] nios2/altera_tse.c: switch to ressources

Franck JULLIEN franck.jullien at gmail.com
Tue Jul 26 07:41:06 EDT 2011


Hi Sascha,

2011/7/26 Sascha Hauer <s.hauer at pengutronix.de>:
> Hi Franck,
>
> On Mon, Jul 25, 2011 at 09:50:18PM +0200, franck.jullien at gmail.com wrote:
>> From: Franck Jullien <franck.jullien at gmail.com>
>>
>> Signed-off-by: Franck Jullien <franck.jullien at gmail.com>
>> ---
>>  drivers/net/altera_tse.c |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/net/altera_tse.c b/drivers/net/altera_tse.c
>> index ac8cd2d..4298739 100644
>> --- a/drivers/net/altera_tse.c
>> +++ b/drivers/net/altera_tse.c
>> @@ -510,7 +510,7 @@ static int tse_probe(struct device_d *dev)
>>       dev->type_data = edev;
>>       edev->priv = (struct altera_tse_priv *)(edev + 1);
>>
>> -     edev->iobase = dev->map_base;
>> +     edev->iobase = (int)dev_request_mem_region(dev, 0);
>
> Damned. I wasn't aware that struct eth_device has an int iobase.
> Can you please move your iobase to edev->priv (as a void __iomem *)?
>

No problem. I'll also use resources for sgdma_tx and sgdma_rx. I'll resend the
patch tonight.


> Then we can remove this otherwise unused field from eth_device.
>
> Sascha
>
> --
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
>



More information about the barebox mailing list