On 23.06.2009 00:09, Luc Verhaegen wrote:
On Mon, Jun 22, 2009 at 04:31:44PM +0200, Carl-Daniel Hailfinger wrote:
SB600 SPI: Kill unused variable.
Signed-off-by: Carl-Daniel Hailfinger c-d.hailfinger.devel.2006@gmx.net
If this builds, then what can be wrong with this :)
I guess you are keeping this struct because publically available documentation is perhaps not as clear as it should be?
Well, the documentation is pretty readable, but the struct is a handy compressed reference and useful if you want to avoid reading dozens of datasheet pages.
Why not comment it completely so that no-one else will complain about it in future?
Done.
Please also make reset_internal_fifo_pointer and execute_command static so thatthese symbols, when unused, will turn up in the build as well.
Done.
Apart from that, of course: Acked-by: Luc Verhaegen libv@skynet.be
Thanks, committed in r623.
Regards, Carl-Daniel