<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/drivers/spi, branch v2017.01-rc2</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/drivers/spi?h=v2017.01-rc2</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/drivers/spi?h=v2017.01-rc2'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2016-12-15T15:57:28Z</updated>
<entry>
<title>spi: cadence_qspi: Move DT prop code to match layout</title>
<updated>2016-12-15T15:57:28Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:34Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6d72810c6640e13b4c1e25f6c26ac8c304d5b54e'/>
<id>urn:sha1:6d72810c6640e13b4c1e25f6c26ac8c304d5b54e</id>
<content type='text'>
Move the code to read the "sram-size" property into the other code
that reads properties from the node, rather than the SF subnode.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Fix CS timings</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:33Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=22e63ff3a23d189187d96dbcec50e94233027b3a'/>
<id>urn:sha1:22e63ff3a23d189187d96dbcec50e94233027b3a</id>
<content type='text'>
The Cadence QSPI controller has specified overheads for the various CS
times that are in addition to those programmed in to the Device Delay
register. The overheads are different for the delays.

In addition, the existing code does not handle the case when the delay
is less than a SCLK period.

This change accurately calculates the additional delays in Ref clocks.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Remove returns from end of void functions</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:32Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3c5695321929d3c3d1936cb8a7773566af0886b5'/>
<id>urn:sha1:3c5695321929d3c3d1936cb8a7773566af0886b5</id>
<content type='text'>
Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Acked-by: Marek Vasut &lt;marek.vasut@gmail.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Use spi mode at the point it is needed</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=7d403f284c814d6df9f1d116e691d6468c75282a'/>
<id>urn:sha1:7d403f284c814d6df9f1d116e691d6468c75282a</id>
<content type='text'>
Instead of extracting mode settings and passing them as separate
args to another function, just pass the SPI mode as an arg.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Clean up the #define names</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:30Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=7e76c4b08a30c0a90813ae56cd876555bef8d9f6'/>
<id>urn:sha1:7e76c4b08a30c0a90813ae56cd876555bef8d9f6</id>
<content type='text'>
A lot of the #defines are for single bits in a register, where the
name has _MASK on the end. Since this can be used for both a mask
and the value, remove _MASK from them.

Whilst doing so, also remove the unnecessary brackets around the
constants.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Acked-by: Marek Vasut &lt;marek.vasut@gmail.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Use #define for bits instead of bit shifts</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:29Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=db37cc9c3954d5e33b218e5635e7e08ff902e3f6'/>
<id>urn:sha1:db37cc9c3954d5e33b218e5635e7e08ff902e3f6</id>
<content type='text'>
Most of the code already uses #defines for the bit value, rather
than the shift required to get the value. This changes the remaining
code over.

Whislt at it, fix the names of the "Rd Data Capture" register defs.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Acked-by: Marek Vasut &lt;marek.vasut@gmail.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Better debug information on the SPI clock rate</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:28Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=0ceb4d9e9a64dcc662ea52150feebed37deda716'/>
<id>urn:sha1:0ceb4d9e9a64dcc662ea52150feebed37deda716</id>
<content type='text'>
Show what the output clock rate actually is.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Acked-by: Marek Vasut &lt;marek.vasut@gmail.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Fix baud rate calculation</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:27Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=32068c42a7230ad1ef756ed7a201cc3b3c580076'/>
<id>urn:sha1:32068c42a7230ad1ef756ed7a201cc3b3c580076</id>
<content type='text'>
With the existing code, when the requested SPI clock rate is near
to the lowest that can be achieved by the hardware (max divider
of the ref clock is 32), the generated clock rate is wrong.
For example, with a 50MHz ref clock, when asked for anything less
than a 1.5MHz SPI clock, the code sets up the divider to generate
25MHz.

This change fixes the calculation.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: cadence_qspi: Fix clearing of pol/pha bits</title>
<updated>2016-12-15T15:57:27Z</updated>
<author>
<name>Phil Edworthy</name>
<email>PHIL.EDWORTHY@renesas.com</email>
</author>
<published>2016-11-29T12:58:26Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=cc80a897e4fafbd9e9b6920eb866f0600a5cd5ee'/>
<id>urn:sha1:cc80a897e4fafbd9e9b6920eb866f0600a5cd5ee</id>
<content type='text'>
Or'ing together bit positions is clearly wrong.

Signed-off-by: Phil Edworthy &lt;phil.edworthy@renesas.com&gt;
Acked-by: Marek Vasut &lt;marek.vasut@gmail.com&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
<entry>
<title>spi: Add error checking for invalid bus widths</title>
<updated>2016-12-15T15:38:30Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2016-11-30T03:00:13Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1b7c28f5147144d7902d048ca90be58987899c25'/>
<id>urn:sha1:1b7c28f5147144d7902d048ca90be58987899c25</id>
<content type='text'>
At present an invalid bus width prints a message but does not return an
error. This is the opposite of the correct behaviour. Adjust it to avoid
code bloat in the common case, and avoid hard-to-debug failure in the
uncommon case.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Jagan Teki &lt;jagan@openedev.com&gt;
</content>
</entry>
</feed>
