<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/drivers/mtd/spi, branch v2016.03</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/drivers/mtd/spi?h=v2016.03</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/drivers/mtd/spi?h=v2016.03'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2016-03-12T14:25:42Z</updated>
<entry>
<title>sf: Correct data types in stm_is_locked_sr()</title>
<updated>2016-03-12T14:25:42Z</updated>
<author>
<name>Marek Vasut</name>
<email>marex@denx.de</email>
</author>
<published>2016-03-11T02:20:16Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ea9619aed6908e83b0679bd9c9aa4ae97714ef97'/>
<id>urn:sha1:ea9619aed6908e83b0679bd9c9aa4ae97714ef97</id>
<content type='text'>
The stm_is_locked_sr() function is picked from Linux kernel. For reason
unknown, the 64bit data types used by the function and present in Linux
were replaced with 32bit unsigned ones, which causes trouble.

The testcase performed was done using ST M25P80 chip.
The command used was:
 =&gt; sf protect unlock 0 0x10000

The call chain starts in stm_unlock(), which calls stm_is_locked_sr()
with negative ofs argument. This works fine in Linux, where the "ofs"
is loff_t, which is signed long long, while this fails in U-Boot, where
"ofs" is u32 (unsigned int). Because of this signedness problem, the
expression past the return statement to be incorrectly evaluated to 1,
which in turn propagates back to stm_unlock() and results in -EINVAL.

The correction is very simple, just use the correctly sized data types
with correct signedness in the function to make it work as intended.

Signed-off-by: Marek Vasut &lt;marex@denx.de&gt;
Cc: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: Correct two error return values</title>
<updated>2016-02-26T15:53:10Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-02-24T16:14:56Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=0badb23d110c886cf25550325f36ce1eba1eb5e2'/>
<id>urn:sha1:0badb23d110c886cf25550325f36ce1eba1eb5e2</id>
<content type='text'>
When an error number is provided we should use it, not change it. This fixes
the SPI and SPI flash tests.

One of these is long-standing. The other seems to have been introduced by
commit 1e90d9fd (sf: Move read_id code to sf_ops).

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Fixes: 1e90d9fd (sf: Move read_id code to sf_ops)
Reviewed-by: Jagan Teki &lt;jteki@openedev.com&gt;
Tested-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
<entry>
<title>sandbox: spi: Remove an incorrect free()</title>
<updated>2016-02-26T15:53:10Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-02-24T16:14:55Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ffe276d27a538ca410d697126c562a1ce0ac8bbc'/>
<id>urn:sha1:ffe276d27a538ca410d697126c562a1ce0ac8bbc</id>
<content type='text'>
We must not free data that is managed by driver mode. Remove this line,
which is a hangover from the pre-driver-model code.

This fixes a problem where 'sf probe' crashes U-Boot if the backing file
for the SPI flash cannot be found.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Jagan Teki &lt;jteki@openedev.com&gt;
Tested-by: Jagan Teki &lt;jteki@openedev.com&gt;
Reviewed-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>sandbox: spi: Add more debugging to SPI emulation</title>
<updated>2016-02-26T15:53:10Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-02-24T16:14:54Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=20f655da11cdddfeb0814197f9b3c71d4d297061'/>
<id>urn:sha1:20f655da11cdddfeb0814197f9b3c71d4d297061</id>
<content type='text'>
Add a little more debugging to help when things go wrong.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Jagan Teki &lt;jteki@openedev.com&gt;
Tested-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
<entry>
<title>sf: spi_flash: use dma to copy data from mmap region if platform supports</title>
<updated>2016-02-23T10:44:46Z</updated>
<author>
<name>Mugunthan V N</name>
<email>mugunthanvnm@ti.com</email>
</author>
<published>2016-02-15T10:01:39Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=7bd1c59bdb5ccf5a0fbd8522fd35b9db1a450fb3'/>
<id>urn:sha1:7bd1c59bdb5ccf5a0fbd8522fd35b9db1a450fb3</id>
<content type='text'>
Add dma memcpy api to the default spi_flash_copy_mmap(), so that
dma will be used to copy data when CONFIG_DMA is defined for the
platform.

Signed-off-by: Mugunthan V N &lt;mugunthanvnm@ti.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: Correct device tree usage in spi_flash_decode_fdt()</title>
<updated>2016-01-22T03:42:35Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-01-22T02:43:54Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d178a1c5b2b3d47a6380b7990a8a3f44b1cafe24'/>
<id>urn:sha1:d178a1c5b2b3d47a6380b7990a8a3f44b1cafe24</id>
<content type='text'>
This function currently searches the entire device tree for a node that
it thinks is relevant. But the node is known and is passed in. Correct the
code and enable it only with driver model, since only driver-model boards
will use it.

This avoids bringing in a large number of strings from fdtdec.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>dm: spi_flash: Allow the uclass to work without printf()</title>
<updated>2016-01-15T12:39:27Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2015-12-29T12:22:47Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3c8fb12b77849400ffc04737c113a7acff7d7f18'/>
<id>urn:sha1:3c8fb12b77849400ffc04737c113a7acff7d7f18</id>
<content type='text'>
For SPL we don't really need sprintf() and with tiny-printf this is not
available. Allow this to be dropped in SPL when using tiny-printf.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>sf: Make IO modes at last in read modes</title>
<updated>2016-01-13T13:17:27Z</updated>
<author>
<name>Jagan Teki</name>
<email>jteki@openedev.com</email>
</author>
<published>2015-12-28T16:38:40Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1c17f5ec571cac6e0867f64607b052d6ffa0b1c1'/>
<id>urn:sha1:1c17f5ec571cac6e0867f64607b052d6ffa0b1c1</id>
<content type='text'>
SLOW, FAST, DUAL, DUAL_IO, QUAD, QUAD_IO changed order to
SLOW, FAST, DUAL, QUAD, DUAL_IO, QUAD_IO

Cc: Simon Glass &lt;sjg@chromium.org&gt;
Cc: Michal Simek &lt;michal.simek@xilinx.com&gt;
Cc: Siva Durga Prasad Paladugu &lt;sivadur@xilinx.com&gt;
Tested-by: Jagan Teki &lt;jteki@openedev.com&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Signed-off-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: Rename op_mode_rx to mode_rx</title>
<updated>2016-01-13T13:17:26Z</updated>
<author>
<name>Jagan Teki</name>
<email>jteki@openedev.com</email>
</author>
<published>2015-12-16T09:54:24Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=91292e0bee9e59f029f7d1039fe5f2914b31ea85'/>
<id>urn:sha1:91292e0bee9e59f029f7d1039fe5f2914b31ea85</id>
<content type='text'>
Since spi rx mode macro's are renamed to simple and
meaninfull, this patch will rename the respective
structure members.

Cc: Simon Glass &lt;sjg@chromium.org&gt;
Cc: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Cc: Michal Simek &lt;michal.simek@xilinx.com&gt;
Cc: Siva Durga Prasad Paladugu &lt;sivadur@xilinx.com&gt;
Tested-by: Mugunthan V N &lt;mugunthanvnm@ti.com&gt;
Tested-by: Jagan Teki &lt;jteki@openedev.com&gt;
Signed-off-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
<entry>
<title>sf: Write quad bit along with read status</title>
<updated>2016-01-13T13:17:26Z</updated>
<author>
<name>Jagan Teki</name>
<email>jteki@openedev.com</email>
</author>
<published>2015-12-16T08:18:08Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d9a0ab6c0dc318750ce5d389bf9268e31fd8af60'/>
<id>urn:sha1:d9a0ab6c0dc318750ce5d389bf9268e31fd8af60</id>
<content type='text'>
While setting quad bit on spansion, macronix code
is writing only particular quad bit this may give
wrong functionality with other register bits,
So this patch fix the issue where it with write
previous read reg status along  particular quad bit.

Cc: Vignesh R &lt;vigneshr@ti.com&gt;
Cc: Mugunthan V N &lt;mugunthanvnm@ti.com&gt;
Cc: Simon Glass &lt;sjg@chromium.org&gt;
Cc: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Cc: Michal Simek &lt;michal.simek@xilinx.com&gt;
Cc: Siva Durga Prasad Paladugu &lt;sivadur@xilinx.com&gt;
Signed-off-by: Jagan Teki &lt;jteki@openedev.com&gt;
</content>
</entry>
</feed>
