From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ajay Bhargav <ajay.bhargav@einfochips.com>
Date: Wed, 20 Jul 2011 17:50:25 +0530 (IST)
Subject: [U-Boot] [PATCH 1/4] gpio: Adds GPIO driver support for
 Armada100
In-Reply-To: <CALZhoSSvPgMR0K7Mp2bQCBDn3R72ejE7y2KVmLqFe4fEEV4yXQ@mail.gmail.com>
Message-ID: <1281824279.36304.1311164425547.JavaMail.root@ahm.einfochips.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de


----- "Lei Wen" <adrian.wenl@gmail.com> wrote:

> Hi Ajay,
> 
> On Wed, Jul 20, 2011 at 6:43 PM, Ajay Bhargav
> <ajay.bhargav@einfochips.com> wrote:
> > Hi Lei,
> >
> >> I think we make thing complicated here. For GPIO driver, the only
> >> structure we need to
> >> define is the GPIO register itself, like GPIO_PLR, GPIO_PDR,
> etc...
> >>
> >
> > I got your point, but lemme show you where the problem is..
> > GPIO_PLR0 ? ?0x0000
> > GPIO_PLR1 ? ?0x0004
> > GPIO_PLR2 ? ?0x0008
> > GPIO_PLR3 ? ?0x0100
> >
> > till 3 registers its fine... fourth register is way out of league.
> >
> 
> I know this is a bit tricky, but couldn't be included in a logic
> hidden by MACRO or funcion?
> 
> Best regards,
> Lei
> 

Hi Lei,

I am rewriting the whole logic. Will submit the patch soon :)
I will use function to get base address.

Regards,
Ajay Bhargav