<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/drivers/soc/qcom, branch v4.4.123</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.4.123</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.4.123'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2016-09-24T08:07:42Z</updated>
<entry>
<title>soc: qcom/spm: shut up uninitialized variable warning</title>
<updated>2016-09-24T08:07:42Z</updated>
<author>
<name>Arnd Bergmann</name>
<email>arnd@arndb.de</email>
</author>
<published>2016-01-17T00:02:56Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=e50d38bdfc5c58b07e3b15b6fd790ed0c0d97f02'/>
<id>urn:sha1:e50d38bdfc5c58b07e3b15b6fd790ed0c0d97f02</id>
<content type='text'>
commit 00affcac69c7aae6c2cfcbc71f724e1c16d0b445 upstream.

gcc warns about the 'found' variable possibly being used uninitialized:

drivers/soc/qcom/spm.c: In function 'spm_dev_probe':
drivers/soc/qcom/spm.c:305:5: error: 'found' may be used uninitialized in this function [-Werror=maybe-uninitialized]

However, the code is correct because we know that there is
always at least one online CPU. This initializes the 'found'
variable to zero before the loop so the compiler knows
it does not have to warn about it.

Signed-off-by: Arnd Bergmann &lt;arnd@arndb.de&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;

</content>
</entry>
<entry>
<title>soc: qcom: smd-rpm: Correct size of outgoing message</title>
<updated>2015-10-31T01:51:20Z</updated>
<author>
<name>Bjorn Andersson</name>
<email>bjorn.andersson@sonymobile.com</email>
</author>
<published>2015-10-29T22:09:54Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=d13a5c8c4c3dbe299659bcff805f79a2c83e2bbc'/>
<id>urn:sha1:d13a5c8c4c3dbe299659bcff805f79a2c83e2bbc</id>
<content type='text'>
With the removal of VLAIS the size was incorrectly changed to only cover
the headers of the packet, resulting in "empty" requests being sent to
the RPM. Correct this so the entire message is transfered.

Fixes: 50e1b29b4438 ("soc: qcom: smd: Remove use of VLAIS")
Signed-off-by: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Reviewed-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
Signed-off-by: Olof Johansson &lt;olof@lixom.net&gt;
</content>
</entry>
<entry>
<title>soc: qcom/smem: add HWSPINLOCK dependency</title>
<updated>2015-10-15T21:04:11Z</updated>
<author>
<name>Arnd Bergmann</name>
<email>arnd@arndb.de</email>
</author>
<published>2015-10-13T15:05:39Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=73ebb85444b0472d90bb70a1a9e6b5df3f92c14c'/>
<id>urn:sha1:73ebb85444b0472d90bb70a1a9e6b5df3f92c14c</id>
<content type='text'>
This fixes a build error when smem is enabled without hwspinlock:

drivers/built-in.o: In function `qcom_smem_alloc':
rockchip-efuse.c:(.text+0x7a3e4): undefined reference to `__hwspin_lock_timeout'
rockchip-efuse.c:(.text+0x7a568): undefined reference to `__hwspin_unlock'
drivers/built-in.o: In function `qcom_smem_remove':
rockchip-efuse.c:(.text+0x7a5cc): undefined reference to `hwspin_lock_free'
drivers/built-in.o: In function `qcom_smem_probe':
rockchip-efuse.c:(.text+0x7a960): undefined reference to `hwspin_lock_request_specific'
rockchip-efuse.c:(.text+0x7a988): undefined reference to `of_hwspin_lock_get_id'
drivers/built-in.o: In function `qcom_smem_get':
rockchip-efuse.c:(.text+0x7aa24): undefined reference to `__hwspin_lock_timeout'
rockchip-efuse.c:(.text+0x7aafc): undefined reference to `__hwspin_unlock'

Signed-off-by: Arnd Bergmann &lt;arnd@arndb.de&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smem: Move RPM message ram out of smem DT node</title>
<updated>2015-10-14T19:51:40Z</updated>
<author>
<name>Stephen Boyd</name>
<email>sboyd@codeaurora.org</email>
</author>
<published>2015-10-08T18:34:09Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=d0bfd7c9b162612de55ca2d204403b90dc278db6'/>
<id>urn:sha1:d0bfd7c9b162612de55ca2d204403b90dc278db6</id>
<content type='text'>
SMEM is a software construct built on top of a DDR reserved region
and sometimes a device memory region called RPM message ram. Having
the RPM message ram in the smem DT node's reg property leads to the
smem node being located in different places depending on if the
message ram is being used or not. Let's add a qcom specific
property, qcom,rpm-msg-ram, and point to the device memory from
the SMEM node via a phandle.  As SMEM is a software construct, it
really needs to reside at the root of the DT regardless of whether
it's using the message ram or not.

Cc: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smd-rpm: Correct the active vs sleep state flagging</title>
<updated>2015-10-14T19:51:22Z</updated>
<author>
<name>Bjorn Andersson</name>
<email>bjorn.andersson@sonymobile.com</email>
</author>
<published>2015-10-13T20:57:43Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a8ddd1b998bdad9489876a3068cc45917bee5441'/>
<id>urn:sha1:a8ddd1b998bdad9489876a3068cc45917bee5441</id>
<content type='text'>
The BIT() was incorrectly inherited from family A and should not be used
on family B where the state is denoted by an enum.

Reported-by: Georgi Djakov &lt;georgi.djakov@linaro.org&gt;
Signed-off-by: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Tested-by: Georgi Djakov &lt;georgi.djakov@linaro.org&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smd: delete unneeded of_node_put</title>
<updated>2015-10-14T19:51:22Z</updated>
<author>
<name>Julia Lawall</name>
<email>Julia.Lawall@lip6.fr</email>
</author>
<published>2015-10-12T20:43:15Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=6083096d961f3cf754c4da282179053f7b85ac16'/>
<id>urn:sha1:6083096d961f3cf754c4da282179053f7b85ac16</id>
<content type='text'>
Device node iterators perform an of_node_put on each iteration, so putting
an of_node_put before a continue results in a double put.

A simplified version of the semantic match that finds this problem is as
follows (http://coccinelle.lip6.fr):

// &lt;smpl&gt;
@@
expression root,e;
local idexpression child;
iterator i;
@@

 i(..., child, ...) {
   ... when != of_node_get(child)
*  of_node_put(child);
   ...
*  continue;
}
// &lt;/smpl&gt;

Signed-off-by: Julia Lawall &lt;Julia.Lawall@lip6.fr&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smd: Correct SMEM items for upper channels</title>
<updated>2015-10-14T19:51:22Z</updated>
<author>
<name>Bjorn Andersson</name>
<email>bjorn.andersson@sonymobile.com</email>
</author>
<published>2015-10-10T05:06:44Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=ea4683e0b6c38ed54e5a66160c99d65f5022ecb2'/>
<id>urn:sha1:ea4683e0b6c38ed54e5a66160c99d65f5022ecb2</id>
<content type='text'>
Update the SMEM items for the second set of SMD channels, as these where
incorrect.

Signed-off-by: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smd: Reject send of too big packets</title>
<updated>2015-10-14T19:51:21Z</updated>
<author>
<name>Bjorn Andersson</name>
<email>bjorn.andersson@sonymobile.com</email>
</author>
<published>2015-09-25T01:37:18Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a208ca98ff63e5c94d03810bc80fee9b02fe9972'/>
<id>urn:sha1:a208ca98ff63e5c94d03810bc80fee9b02fe9972</id>
<content type='text'>
Attempting to find room for a packet that's bigger than the fifo will
never succeed and the calling process will be sleeping forever in the
loop, waiting for enough room. So fail early instead.

Reported-by: Courtney Cavin &lt;courtney.cavin@sonymobile.com&gt;
Signed-off-by: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Reviewed-by: Andy Gross &lt;agross@codeaurora.org&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smd: Handle big endian CPUs</title>
<updated>2015-10-14T19:51:21Z</updated>
<author>
<name>Stephen Boyd</name>
<email>sboyd@codeaurora.org</email>
</author>
<published>2015-09-17T21:50:53Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=24f60e3776e340136f8d822aff75fe06b78a315f'/>
<id>urn:sha1:24f60e3776e340136f8d822aff75fe06b78a315f</id>
<content type='text'>
The smd structures are always in little endian, but the smd
driver is not capable of being used on big endian CPUs. Annotate
the little endian data members and update the code to do the
proper byte swapping.

Cc: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
<entry>
<title>soc: qcom: smd_rpm: Handle big endian CPUs</title>
<updated>2015-10-14T19:51:21Z</updated>
<author>
<name>Stephen Boyd</name>
<email>sboyd@codeaurora.org</email>
</author>
<published>2015-09-02T22:46:50Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=30b7ea5eda0afb13164b2d7049a56645cfc1d47b'/>
<id>urn:sha1:30b7ea5eda0afb13164b2d7049a56645cfc1d47b</id>
<content type='text'>
The smd rpm structures are always in little endian, but this
driver is not capable of being used on big endian CPUs. Annotate
the little endian data members and update the code to do the
proper byte swapping.

Cc: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Signed-off-by: Stephen Boyd &lt;sboyd@codeaurora.org&gt;
Reviewed-by: Bjorn Andersson &lt;bjorn.andersson@sonymobile.com&gt;
Signed-off-by: Andy Gross &lt;agross@codeaurora.org&gt;
</content>
</entry>
</feed>
