Re: [Uclinux-dist-devel] [PATCH] Clear control bits in drivers/spi/spi_bfin5xx.c as necessary

[Mike Frysinger <vapier.adi-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

On Tue, Jan 19, 2010 at 11:12, Grant Likely wrote:
> On Fri, Dec 18, 2009 at 10:34 AM, Mike Frysinger wrote:
>> On Fri, Dec 18, 2009 at 08:14, Mike Frysinger wrote:
>>> On Fri, Dec 18, 2009 at 02:29, James Getzendanner wrote:
>>>> From: Andy Getzendanner <james.getzendanner@students.olin.edu>
>>>> The patch also modifies bfin_spi_setup() to
>>>> set/clear the SIZE bit as appropriate.
>>>
>>> ok, but this isnt the place to do it.  the size can be changed on a
>>> per-transfer basis.  i have a patch locally to unify the transfer size
>>> logic and i'll include this fix in the process.
>>
>> i lied, this isnt necessary.  it's already handled in the transfer code.
>>    cr = (read_CTRL(drv_data) & (~BIT_CTL_TIMOD));
>>    cr |= (width << 8);
>>    write_CTRL(drv_data, cr);
>>
>> where width is CFG_SPI_WORDSIZE16 or CFG_SPI_WORDSIZE8 as dictated by
>> the transfer.
>
> Mike & James,
>
> It's not clear to me from the discussion.  What is the status of this
> patch, is it still needed?

he pointed out valid issues in the Blackfin driver that i have fixed,
just not with his versions
-mike

------------------------------------------------------------------------------
Throughout its 18-year history, RSA Conference consistently attracts the
world's best and brightest in the field, creating opportunities for Conference
attendees to learn about information security's most important issues through
interactions with peers, luminaries and emerging and established companies.
http://p.sf.net/sfu/rsaconf-dev2dev
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general