Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c5c53314 authored by Junjie Wu's avatar Junjie Wu
Browse files

clock-rpm: Support deferred probe for enable_rpm_scaling 



enable_rpm_scaling() does not have a return value and thus it was
not possible to do deferred probing in case rpm-smd driver is not
probed yet.

Add a return value to enable_rpm_scaling() and propagate error from
underlying API calls.

Change-Id: Ifa2fe00aba9e460cf30fae1e0153be17ff55a2df
Signed-off-by: default avatarJunjie Wu <junjiew@codeaurora.org>
parent 2b979b24

arch/arm/mach-msm/clock-rpm-8974.c

+3 −1
Original line number Diff line number Diff line
@@ -304,7 +304,9 @@ static int msm_rpmcc_8974_probe(struct platform_device *pdev) 
	struct resource *res;

	int ret;



	enable_rpm_scaling();

	ret = enable_rpm_scaling();

	if (ret < 0)

		return ret;



	res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "cc_base");

	if (!res) {

drivers/clk/qcom/clock-rpm-8916.c

+3 −1
Original line number Diff line number Diff line
@@ -172,7 +172,9 @@ static int msm_rpmcc_8916_probe(struct platform_device *pdev) 
	struct resource *res;

	int ret;



	enable_rpm_scaling();

	ret = enable_rpm_scaling();

	if (ret)

		return ret;



	res =  platform_get_resource_byname(pdev, IORESOURCE_MEM, "cc_base");

	if (!res) {

drivers/clk/qcom/clock-rpm-plutonium.c

+3 −1
Original line number Diff line number Diff line
@@ -252,7 +252,9 @@ static int msm_rpmcc_plutonium_probe(struct platform_device *pdev) 
	struct resource *res;

	int ret;



	enable_rpm_scaling();

	ret = enable_rpm_scaling();

	if (ret < 0)

		return ret;



	res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "cc_base");

	if (!res) {

drivers/clk/qcom/clock-rpm.c

+13 −3
Original line number Diff line number Diff line
@@ -270,7 +270,7 @@ static enum handoff rpm_clk_handoff(struct clk *clk) 
#define RPM_MISC_CLK_TYPE	0x306b6c63

#define RPM_SCALING_ENABLE_ID	0x2



void enable_rpm_scaling(void)

int enable_rpm_scaling(void)

{

	int rc, value = 0x1;

	struct msm_rpm_kvp kvp = {
@@ -281,11 +281,21 @@ void enable_rpm_scaling(void) 


	rc = msm_rpm_send_message_noirq(MSM_RPM_CTX_SLEEP_SET,

			RPM_MISC_CLK_TYPE, RPM_SCALING_ENABLE_ID, &kvp, 1);

	WARN(rc < 0, "RPM clock scaling (sleep set) did not enable!\n");

	if (rc < 0) {

		if (rc != -EPROBE_DEFER)

			WARN(1, "RPM clock scaling (sleep set) did not enable!\n");

		return rc;

	}



	rc = msm_rpm_send_message_noirq(MSM_RPM_CTX_ACTIVE_SET,

			RPM_MISC_CLK_TYPE, RPM_SCALING_ENABLE_ID, &kvp, 1);

	WARN(rc < 0, "RPM clock scaling (active set) did not enable!\n");

	if (rc < 0) {

		if (rc != -EPROBE_DEFER)

			WARN(1, "RPM clock scaling (active set) did not enable!\n");

		return rc;

	}



	return 0;

}



struct clk_ops clk_ops_rpm = {

include/soc/qcom/clock-rpm.h

+1 −1
Original line number Diff line number Diff line
@@ -52,7 +52,7 @@ static inline struct rpm_clk *to_rpm_clk(struct clk *clk) 
 * RPM scaling enable function used for target that has an RPM resource for

 * rpm clock scaling enable.

 */

void enable_rpm_scaling(void);

int enable_rpm_scaling(void);



extern struct clk_rpmrs_data clk_rpmrs_data_smd;