| Message ID | 1431959738-32220-1-git-send-email-ivan.ivanov@linaro.org (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Delegated to: | Andy Gross |
| Headers | show |
On Mon, 2015-05-18 at 17:35 +0300, Ivan T. Ivanov wrote: > Qualcomm Debug Subsystem clock is used by CoreSight components. > Add required definitions for it. > > Signed-off-by: Ivan T. Ivanov ivanov@linaro.org> > --- > drivers/mfd/qcom_rpm.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c > index eceb34f..313aeae 100644 > --- a/drivers/mfd/qcom_rpm.c > +++ b/drivers/mfd/qcom_rpm.c > @@ -149,6 +149,7 @@ static const struct qcom_rpm_resource apq8064_rpm_resource_table[] = { > [QCOM_RPM_USB_OTG_SWITCH] ={ 210, 125, 82, 1 }, > [QCOM_RPM_HDMI_SWITCH] ={ 211, 126, 83, 1 }, > [QCOM_RPM_DDR_DMM] ={ 212, 127, 84, 2 }, > + [QCOM_RPM_QDSS_CLK] ={ 214, 129, 7, 1 }, > [QCOM_RPM_VDDMIN_GPIO] ={ 215, 131, 89, 1 }, > }; Hi Bjorn, Could you take a look at this one? Thanks, Ivan -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon 18 May 07:35 PDT 2015, Ivan T. Ivanov wrote: > Qualcomm Debug Subsystem clock is used by CoreSight components. > Add required definitions for it. > > Signed-off-by: Ivan T. Ivanov <ivan.ivanov@linaro.org> Sorry for the slow response Ivan. > --- > drivers/mfd/qcom_rpm.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c > index eceb34f..313aeae 100644 > --- a/drivers/mfd/qcom_rpm.c > +++ b/drivers/mfd/qcom_rpm.c > @@ -149,6 +149,7 @@ static const struct qcom_rpm_resource apq8064_rpm_resource_table[] = { > [QCOM_RPM_USB_OTG_SWITCH] = { 210, 125, 82, 1 }, > [QCOM_RPM_HDMI_SWITCH] = { 211, 126, 83, 1 }, > [QCOM_RPM_DDR_DMM] = { 212, 127, 84, 2 }, > + [QCOM_RPM_QDSS_CLK] = { 214, 129, 7, 1 }, 214, 7 and 1 are correct. But as far as I can see in my tables the QDSS clock doesn't have any status (129 seems to be EBI1_CH0_RANGE). The status_id was originally included because I was reading it and I saw that Srinivas has some code that reads it, but unless it shows to be useful we should probably just drop that entire column... But for now please mark it as -1 or so and we can have a chat with Srinivas on dropping the column later. > [QCOM_RPM_VDDMIN_GPIO] = { 215, 131, 89, 1 }, > }; > Regards, Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 2015-07-06 at 13:33 -0700, Bjorn Andersson wrote: > On Mon 18 May 07:35 PDT 2015, Ivan T. Ivanov wrote: > > > Qualcomm Debug Subsystem clock is used by CoreSight components. > > Add required definitions for it. > > > > Signed-off-by: Ivan T. Ivanov ivanov@linaro.org> > > Sorry for the slow response Ivan. > > > --- > > drivers/mfd/qcom_rpm.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c > > index eceb34f..313aeae 100644 > > --- a/drivers/mfd/qcom_rpm.c > > +++ b/drivers/mfd/qcom_rpm.c > > @@ -149,6 +149,7 @@ static const struct qcom_rpm_resource apq8064_rpm_resource_table[] = { > > [QCOM_RPM_USB_OTG_SWITCH] ={ 210, 125, 82, 1 }, > > [QCOM_RPM_HDMI_SWITCH] ={ 211, 126, 83, 1 }, > > [QCOM_RPM_DDR_DMM] ={ 212, 127, 84, 2 }, > > + [QCOM_RPM_QDSS_CLK] ={ 214, 129, 7, 1 }, > > 214, 7 and 1 are correct. But as far as I can see in my tables the QDSS > clock doesn't have any status (129 seems to be EBI1_CH0_RANGE). > > The status_id was originally included because I was reading it and I saw > that Srinivas has some code that reads it, but unless it shows to be > useful we should probably just drop that entire column... > > But for now please mark it as -1 or so and we can have a chat with > Srinivas on dropping the column later. Thanks, I was unsure about this. This is just what I have found digging into codeaurora sources. Will change and resubmit. Ivan -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/mfd/qcom_rpm.c b/drivers/mfd/qcom_rpm.c index eceb34f..313aeae 100644 --- a/drivers/mfd/qcom_rpm.c +++ b/drivers/mfd/qcom_rpm.c @@ -149,6 +149,7 @@ static const struct qcom_rpm_resource apq8064_rpm_resource_table[] = { [QCOM_RPM_USB_OTG_SWITCH] = { 210, 125, 82, 1 }, [QCOM_RPM_HDMI_SWITCH] = { 211, 126, 83, 1 }, [QCOM_RPM_DDR_DMM] = { 212, 127, 84, 2 }, + [QCOM_RPM_QDSS_CLK] = { 214, 129, 7, 1 }, [QCOM_RPM_VDDMIN_GPIO] = { 215, 131, 89, 1 }, };
Qualcomm Debug Subsystem clock is used by CoreSight components. Add required definitions for it. Signed-off-by: Ivan T. Ivanov <ivan.ivanov@linaro.org> --- drivers/mfd/qcom_rpm.c | 1 + 1 file changed, 1 insertion(+) -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html