On Mon, Jun 17, 2013 at 11:53:53PM -0700, Prafulla Wadaskar wrote:
> 
> 
> > -----Original Message-----
> > From: Simon Guinot [mailto:simon.gui...@sequanux.org]
> > Sent: 18 June 2013 02:28
> > To: Prafulla Wadaskar
> > Cc: Albert ARIBAUD; u-boot@lists.denx.de
> > Subject: [PATCH 2/3] LaCie/common: add support for the
> > GPIO extension bus
> > 
> > This patch adds support for the GPIO extension bus
> > found on some LaCie
> > boards (as the 2Big/5Big Network v2 and the 2Big NAS).
> > This bus allows
> > to configure the devices (mostly the LEDs) connected to
> > a CPLD via two
> > GPIO-based registers (address and data).
> 
> Hi Simon

Hi Prafulla,

Thanks for your quick answer.

> The name given here for the patch as well as files "gpio-ext" is confusing 
> for me.
> 
> As you stated here those are the IOs generated by CPLD on your board which 
> can be programmed using bit-banging on two GPIOs.

Not really bit-banging. Here we have 3 GPIOs used as a kind of address
register and 3 other GPIOs used as a kind of data register. An another
GPIO is used to notify the CPLD than the registers have been updated.
I will update the patch description to make clear that a register may
include several GPIOs.

At LaCie, this kind of bus is called a "GPIO extension bus". It is like
a cheap GPIO expander.

> 
> So to me it those are "cpld_ios".

I am not sure that behind this kind of bus we always have a CPLD. I will
check this. If it is the case, what do you think about cpld-gpio-bus ?

Moreover as this "bus" is really a lacie specific stuff, should I
prepend a "lacie-" prefix ?

Regards,

Simon

> 
> May you please rename these files accordingly and resend the patch series for 
> better understanding?
> 
> To me "gpio-ext" is generic term application for "General purpose IOs"
> 
> Regards...
> Prafulla . . .

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to