On Fri, 2014-02-21 at 14:01 +0900, Mark Brown wrote:
On Thu, Feb 20, 2014 at 09:48:42PM +0000, Liam Girdwood wrote:
Add support for low level differentiation functions for Haswell and Broadwell SST DSPs. This includes suppoprt for DSP boot and reset, DSP firmware module parsing and DSP memory block map initialisation.
Applied, thanks. Some terribly minor things below:
Sorry, been away a few days. Will get around to fixes over the next few days.
+u32 sst_shim32_read(void __iomem *addr, u32 offset) +{
- return readl(addr + offset);
+} +EXPORT_SYMBOL_GPL(sst_shim32_read);
This lot all look like a regmap to me, though some extensions may be needed for the 64 bit variants!
The SHIM is the register space that is used to configure the DSP and also for sending IPC messages. It's really just PCI register space that has some registers shared with the DSP. IIrc, there may be 2 shared registers out of about 8 in total, but the different cores may access them differently. Iirc, Vinod has a case where they needed to write then read a value for one of his cores.
+struct sst_ops haswell_ops = {
- .reset = hsw_reset,
.boot = hsw_boot,- .write = sst_shim32_write,
Something appears funky with the indentation for .boot - tab/space confusion?
oops, will fix.
Thanks !
Liam