On Wed, 4 Feb 2009, Anatolij Gustschin wrote:
> Hi Guennadi,
>
> > +struct s6e63d6 {
> > + unsigned int bus;
> > + unsigned int cs;
> > + struct spi_slave *slave;
> > +};
>
> could we use "bus" and "cs" from struct spi_slave? "struct spi_slave"
> declares them already. Maybe we should drop struct s6e63d6 entirely and use
> struct spi_slave instead?
No, cs and bus here are input parameters, set up by the caller to
s6e63d6_init() and used by the latter to setup the SPI-slave. slave is
driver-"private" field, the caller doesn't need to know what is under
s6e63d6.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
