<feed xmlns='http://www.w3.org/2005/Atom'>
<title>pm24.git/drivers/devfreq, branch v4.10-rc8</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<id>https://git.kobert.dev/pm24.git/atom?h=v4.10-rc8</id>
<link rel='self' href='https://git.kobert.dev/pm24.git/atom?h=v4.10-rc8'/>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/'/>
<updated>2017-01-02T23:21:45Z</updated>
<entry>
<title>PM / devfreq: exynos-bus: Fix the wrong return value</title>
<updated>2017-01-02T23:21:45Z</updated>
<author>
<name>Chanwoo Choi</name>
<email>cw00.choi@samsung.com</email>
</author>
<published>2016-12-28T11:52:36Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=32dd7731699765f21dbe6df9020e613d4ed73fc3'/>
<id>urn:sha1:32dd7731699765f21dbe6df9020e613d4ed73fc3</id>
<content type='text'>
This patch fixes the wrong return value. If devfreq driver requires the wrong
and non-available governor, it is fail. So, this patch returns the error
insead of -EPROBE_DEFER.

Fixes: 403e0689d2a9 (PM / devfreq: exynos: Add support of bus frequency of sub-blocks using passive governor)
Signed-off-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</content>
</entry>
<entry>
<title>PM / devfreq: Fix the bug of devfreq_add_device when governor is NULL</title>
<updated>2017-01-02T23:21:45Z</updated>
<author>
<name>Chanwoo Choi</name>
<email>cw00.choi@samsung.com</email>
</author>
<published>2016-12-28T11:52:35Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=73613b16cb5c5d5a659fc8832eff99eead3f9afb'/>
<id>urn:sha1:73613b16cb5c5d5a659fc8832eff99eead3f9afb</id>
<content type='text'>
This patch fixes the bug of devfreq_add_device(). The devfreq device must
have the default governor. If find_devfreq_governor() returns error,
devfreq_add_device() fail to add the devfreq instance.

Fixes: 1b5c1be2c88e (PM / devfreq: map devfreq drivers to governor using name)
Signed-off-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</content>
</entry>
<entry>
<title>devfreq: rk3399_dmc: Don't use OPP structures outside of RCU locks</title>
<updated>2016-12-08T00:46:07Z</updated>
<author>
<name>Viresh Kumar</name>
<email>viresh.kumar@linaro.org</email>
</author>
<published>2016-12-05T03:23:52Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=e37d35082e75982ef714f9e25bfa43a061d0c5e6'/>
<id>urn:sha1:e37d35082e75982ef714f9e25bfa43a061d0c5e6</id>
<content type='text'>
The OPP structures are abused to the best here, without understanding
how the OPP core and RCU locks work.

In short, the OPP pointer saved in 'rk3399_dmcfreq' can become invalid
under your nose, as the OPP core may free it.

Fix various abuses around OPP structures and calls.

Reviewed-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: Viresh Kumar &lt;viresh.kumar@linaro.org&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</content>
</entry>
<entry>
<title>devfreq: rk3399_dmc: Remove dangling rcu_read_unlock()</title>
<updated>2016-12-08T00:46:06Z</updated>
<author>
<name>Viresh Kumar</name>
<email>viresh.kumar@linaro.org</email>
</author>
<published>2016-12-01T10:42:14Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=d8323de3d4062e1b5a5aa7e9c0f935138405a582'/>
<id>urn:sha1:d8323de3d4062e1b5a5aa7e9c0f935138405a582</id>
<content type='text'>
This call never had the rcu_read_lock() counterpart. Remove the unlock
part as well.

Signed-off-by: Viresh Kumar &lt;viresh.kumar@linaro.org&gt;
Reviewed-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</content>
</entry>
<entry>
<title>devfreq: exynos: Don't use OPP structures outside of RCU locks</title>
<updated>2016-12-08T00:46:06Z</updated>
<author>
<name>Viresh Kumar</name>
<email>viresh.kumar@linaro.org</email>
</author>
<published>2016-12-01T10:25:40Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=c8ce82b9b9c40d66709cce588f6281dd47cc3922'/>
<id>urn:sha1:c8ce82b9b9c40d66709cce588f6281dd47cc3922</id>
<content type='text'>
The OPP structures are abused to the best here, without understanding
how the OPP core and RCU locks work.

In short, the OPP pointer saved 'struct exynos_bus' can become invalid
under your nose, as the OPP core may free it.

Fix various abuses around OPP structures and calls.

Signed-off-by: Viresh Kumar &lt;viresh.kumar@linaro.org&gt;
Acked-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Tested-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</content>
</entry>
<entry>
<title>PM / devfreq: rk3399_dmc: Use the resource-managed function to add devfreq dev</title>
<updated>2016-11-17T02:31:29Z</updated>
<author>
<name>Chanwoo Choi</name>
<email>cw00.choi@samsung.com</email>
</author>
<published>2016-11-08T09:13:27Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=927b75a628b1d80fef171420fbd694fc28d9b8e9'/>
<id>urn:sha1:927b75a628b1d80fef171420fbd694fc28d9b8e9</id>
<content type='text'>
This patch uses the resource-managed to add the devfreq device.
This function will make it easy to handle the devfreq device.

- struct devfreq *devm_devfreq_add_device(struct device *dev,
				  struct devfreq_dev_profile *profile,
				  const char *governor_name,
				  void *data);

Signed-off-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: MyungJoo Ham &lt;myungjoo.ham@samsung.com&gt;
</content>
</entry>
<entry>
<title>PM / devfreq: correct comment typo.</title>
<updated>2016-11-17T02:31:28Z</updated>
<author>
<name>MyungJoo Ham</name>
<email>myungjoo.ham@samsung.com</email>
</author>
<published>2016-11-09T01:29:14Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=bafeb42bd80fb269d5cc396d28165818aae5994c'/>
<id>urn:sha1:bafeb42bd80fb269d5cc396d28165818aae5994c</id>
<content type='text'>
The function name in the comment was incorrect.

Signed-off-by: MyungJoo Ham &lt;myungjoo.ham@samsung.com&gt;
Reviewed-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
</content>
</entry>
<entry>
<title>PM / devfreq: exynos-ppmu: Remove unused mutex from struct exynos_ppmu</title>
<updated>2016-11-17T02:31:28Z</updated>
<author>
<name>Axel Lin</name>
<email>axel.lin@ingics.com</email>
</author>
<published>2016-10-21T01:09:09Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=6bbda2d4f83f7bfeb20fcf67670c47193d6ee6b0'/>
<id>urn:sha1:6bbda2d4f83f7bfeb20fcf67670c47193d6ee6b0</id>
<content type='text'>
The mutex is not used at all, remove it.

Signed-off-by: Axel Lin &lt;axel.lin@ingics.com&gt;
Acked-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: MyungJoo Ham &lt;myungjoo.ham@samsung.com&gt;
</content>
</entry>
<entry>
<title>PM / devfreq: exynos-ppmu: ppmu_events array should not be NULL terminated</title>
<updated>2016-11-17T02:31:28Z</updated>
<author>
<name>Axel Lin</name>
<email>axel.lin@ingics.com</email>
</author>
<published>2016-10-21T01:09:08Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=f8dbe363ba01de9bebb023f499e7bf9c31f028ab'/>
<id>urn:sha1:f8dbe363ba01de9bebb023f499e7bf9c31f028ab</id>
<content type='text'>
The rest of the code uses ARRAY_SIZE to count the number of entries in
ppmu_events array. The NULL terminated entry makes ARRAY_SIZE return
off-by-one value.

Signed-off-by: Axel Lin &lt;axel.lin@ingics.com&gt;
Acked-by: Chanwoo Choi &lt;cw00.choi@samsung.com&gt;
Signed-off-by: MyungJoo Ham &lt;myungjoo.ham@samsung.com&gt;
</content>
</entry>
<entry>
<title>PM / devfreq: exynos-ppmu: Fix module autoload</title>
<updated>2016-11-17T02:31:28Z</updated>
<author>
<name>Javier Martinez Canillas</name>
<email>javier@osg.samsung.com</email>
</author>
<published>2016-10-19T21:06:27Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=29e477f23598b850b08bae22d142bd3abf491248'/>
<id>urn:sha1:29e477f23598b850b08bae22d142bd3abf491248</id>
<content type='text'>
If the driver is built as a module, autoload won't work because the module
alias information is not filled. So user-space can't match the registered
device with the corresponding module.

Export the module alias information using the MODULE_DEVICE_TABLE() macro.

Before this patch:

$ modinfo drivers/devfreq/event/exynos-ppmu.ko | grep alias
$

After this patch:

$ modinfo drivers/devfreq/event/exynos-ppmu.ko | grep alias
alias:          of:N*T*Csamsung,exynos-ppmu-v2C*
alias:          of:N*T*Csamsung,exynos-ppmu-v2
alias:          of:N*T*Csamsung,exynos-ppmuC*
alias:          of:N*T*Csamsung,exynos-ppmu

Signed-off-by: Javier Martinez Canillas &lt;javier@osg.samsung.com&gt;
Signed-off-by: MyungJoo Ham &lt;myungjoo.ham@samsung.com&gt;
</content>
</entry>
</feed>
