> -----Original Message-----
> From: Premi, Sanjeev
> Sent: Thursday, June 23, 2011 4:48 PM
> To: Premi, Sanjeev; Igor Grinberg
> Cc: Govindarajan, Sriramakrishnan; u-boot@lists.denx.de
> Subject: RE: [U-Boot] [PATCH 2/3] omap3evm: Update ethernet
> reset sequence for Rev.G board
>
> > -----Original Message-----
> > From: u-boot-boun...@lists.denx.de
> > [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Premi, Sanjeev
> > Sent: Thursday, June 23, 2011 4:43 PM
> > To: Igor Grinberg
> > Cc: Govindarajan, Sriramakrishnan; u-boot@lists.denx.de
> > Subject: Re: [U-Boot] [PATCH 2/3] omap3evm: Update ethernet
> > reset sequence for Rev.G board
> >
> > > -----Original Message-----
> > > From: Igor Grinberg [mailto:grinb...@compulab.co.il]
> > > Sent: Thursday, June 23, 2011 2:38 PM
> > > To: Premi, Sanjeev
> > > Cc: u-boot@lists.denx.de; Govindarajan, Sriramakrishnan
> > > Subject: Re: [U-Boot] [PATCH 2/3] omap3evm: Update ethernet
> > > reset sequence for Rev.G board
> > >
> > > Hi Sanjeev,
> > >
> > > On 06/22/11 22:24, Sanjeev Premi wrote:
> > > > From: Sriramakrishnan <s...@ti.com>
> > > >
> > > > The GPIO pin used for resetting the external LAN chip has
> > > > changed for Rev.G board.
> > > >
> > > > Signed-off-by: Sriramakrishnan <s...@ti.com>
> > > > Signed-off-by: Sanjeev Premi <pr...@ti.com>
> > > > ---
> > > > board/ti/evm/evm.c | 27 ++++++++++++++++++---------
> > > > 1 files changed, 18 insertions(+), 9 deletions(-)
> > > >
[snip]...[snip]
> > > > + /* Send a pulse on the GPIO pin */
> > > > + writel(pin, &gpio_base->setdataout);
> > > > udelay(1);
> > > > - writel(GPIO0, &gpio3_base->cleardataout);
> > > > + writel(pin, &gpio_base->cleardataout);
> > > > udelay(1);
> > > > - writel(GPIO0, &gpio3_base->setdataout);
> > > > + writel(pin, &gpio_base->setdataout);
> > >
> > > Why keep messing with the gpio registers?
> > > Why not use gpio framework?
> > > Though it is omap specific, but it will be much cleaner then
> > > the above.
> >
> > [sp] I guess the intent was to keep code similar. But yes,
> > gpio framework can be used.
> >
>
> [sp] Sorry, mail went earlier than I wanted :(
>
> The only issue is that I couln't see gpio framework for omap.
> Let me dig further...
>
[sp] Implementing GPIO for OMAP would be a long task. It should be
done for long term; but is it necessary pre-condition for the
patch?
~sanjeev
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
