Date: Tue, 18 Oct 2011 12:15:40 +0300 From: Aleksandr Rybalko <ray@dlink.ua> To: Adrian Chadd <adrian@freebsd.org> Cc: Aleksandr Rybalko <ray@ddteam.net>, current@freebsd.org Subject: Re: SPI rework Message-ID: <20111018121540.a402040e.ray@dlink.ua> In-Reply-To: <CAJ-VmomDTtnUROGbhDN9DDev0PWdt5HdbZifRvLC6ua8AC7%2BFw@mail.gmail.com> References: <20111017194940.ed720115.ray@ddteam.net> <CAJ-VmomDTtnUROGbhDN9DDev0PWdt5HdbZifRvLC6ua8AC7%2BFw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 18 Oct 2011 06:17:46 +0800 Adrian Chadd <adrian@freebsd.org> wrote: >> Hi, >> >> That sounds logical to me. Maybe getting it done before 9.0-RELEASE >> is a bit rushed? >> >> I can help you test out the flash side of things on my atheros SoC >> boards. >> >> >> Adrian More thinking give me a better way to fix that. We leave same structure, but remove KASSERT(cmd->tx_cmd_sz == cmd->rx_cmd_sz,""); and fix SPI controllers drivers to care about possible different sizes. And of course will fix consumers to set exact what they need. (dev/flash: cmd_tx_sz=1, data_rx_sz=3 for "device ID" call) That way better because we will have ability to duplex SPI transfers on controllers that able to do that, RT305x SPI will return error in case when sizes specified for both directions. Comments? WBW -- Alexandr Rybalko <ray@dlink.ua> aka Alex RAY <ray@ddteam.net>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20111018121540.a402040e.ray>