From mboxrd@z Thu Jan  1 00:00:00 1970
From: Anatolij Gustschin <agust@denx.de>
Date: Wed, 04 Feb 2009 23:20:26 +0100
Subject: [U-Boot] [PATCH 4/9] A driver for the S6E63D6 SPI display
 controller from Samsung
In-Reply-To: <Pine.LNX.4.64.0902041540350.6279@axis700.grange>
References: <Pine.LNX.4.64.0902041532100.6279@axis700.grange>
	<Pine.LNX.4.64.0902041540350.6279@axis700.grange>
Message-ID: <498A14AA.4080300@denx.de>
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

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?

Best regards,
Anatolij