diff mbox series

[7/8] soc: ti: omap-prm: add dra7 PRM data

Message ID 1565164139-21886-8-git-send-email-t-kristo@ti.com
State New
Headers show
Series soc: ti: Add OMAP PRM driver | expand

Commit Message

Tero Kristo Aug. 7, 2019, 7:48 a.m. UTC
Add PRM data for dra7 family of SoCs.

Signed-off-by: Tero Kristo <t-kristo@ti.com>

---
 drivers/soc/ti/omap_prm.c | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

-- 
1.9.1

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Comments

Suman Anna Aug. 19, 2019, 11:12 p.m. UTC | #1
On 8/7/19 2:48 AM, Tero Kristo wrote:
> Add PRM data for dra7 family of SoCs.

> 

> Signed-off-by: Tero Kristo <t-kristo@ti.com>

> ---

>  drivers/soc/ti/omap_prm.c | 26 ++++++++++++++++++++++++++

>  1 file changed, 26 insertions(+)

> 

> diff --git a/drivers/soc/ti/omap_prm.c b/drivers/soc/ti/omap_prm.c

> index fadfc7f..05b7749 100644

> --- a/drivers/soc/ti/omap_prm.c

> +++ b/drivers/soc/ti/omap_prm.c

> @@ -73,6 +73,31 @@ struct omap_prm_data omap4_prm_data[] = {

>  	{ },

>  };

>  

> +static struct omap_prm_data dra7_prm_data[] = {


Same comment as on other data, static const.

regards
Suman

> +	{ .name = "mpu", .base = 0x4ae06300, .pwstst = 0x4 },

> +	{ .name = "dsp1", .base = 0x4ae06400, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

> +	{ .name = "ipu", .base = 0x4ae06500, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14, .clkdm_name = "ipu1" },

> +	{ .name = "coreaon", .base = 0x4ae06628, .pwstst = 0x4 },

> +	{ .name = "core", .base = 0x4ae06700, .pwstst = 0x4, .rstctl = 0x210, .rstst = 0x214, .clkdm_name = "ipu2" },

> +	{ .name = "iva", .base = 0x4ae06f00, .pwstst = 0x4 },

> +	{ .name = "cam", .base = 0x4ae07000, .pwstst = 0x4 },

> +	{ .name = "dss", .base = 0x4ae07100, .pwstst = 0x4 },

> +	{ .name = "gpu", .base = 0x4ae07200, .pwstst = 0x4 },

> +	{ .name = "l3init", .base = 0x4ae07300, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

> +	{ .name = "l4per", .base = 0x4ae07400, .pwstst = 0x4 },

> +	{ .name = "custefuse", .base = 0x4ae07600, .pwstst = 0x4 },

> +	{ .name = "wkupaon", .base = 0x4ae07724, .pwstst = 0x4 },

> +	{ .name = "emu", .base = 0x4ae07900, .pwstst = 0x4 },

> +	{ .name = "dsp2", .base = 0x4ae07b00, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

> +	{ .name = "eve1", .base = 0x4ae07b40, .pwstst = 0x4 },

> +	{ .name = "eve2", .base = 0x4ae07b80, .pwstst = 0x4 },

> +	{ .name = "eve3", .base = 0x4ae07bc0, .pwstst = 0x4 },

> +	{ .name = "eve4", .base = 0x4ae07c00, .pwstst = 0x4 },

> +	{ .name = "rtc", .base = 0x4ae07c60, .pwstst = 0x4 },

> +	{ .name = "vpe", .base = 0x4ae07c80, .pwstst = 0x4 },

> +	{ },

> +};

> +

>  struct omap_rst_map am3_wkup_rst_map[] = {

>  	{ .rst = 3, .st = 5 },

>  	{ .rst = -1 },

> @@ -91,6 +116,7 @@ struct omap_prm_data am3_prm_data[] = {

>  

>  static const struct of_device_id omap_prm_id_table[] = {

>  	{ .compatible = "ti,omap4-prm-inst", .data = omap4_prm_data },

> +	{ .compatible = "ti,dra7-prm-inst", .data = dra7_prm_data },

>  	{ .compatible = "ti,am3-prm-inst", .data = am3_prm_data },

>  	{ },

>  };

>
Suman Anna Aug. 20, 2019, 7:03 p.m. UTC | #2
Hi Tero,

On 8/7/19 2:48 AM, Tero Kristo wrote:
> Add PRM data for dra7 family of SoCs.

> 

> Signed-off-by: Tero Kristo <t-kristo@ti.com>

> ---

>  drivers/soc/ti/omap_prm.c | 26 ++++++++++++++++++++++++++

>  1 file changed, 26 insertions(+)

> 

> diff --git a/drivers/soc/ti/omap_prm.c b/drivers/soc/ti/omap_prm.c

> index fadfc7f..05b7749 100644

> --- a/drivers/soc/ti/omap_prm.c

> +++ b/drivers/soc/ti/omap_prm.c

> @@ -73,6 +73,31 @@ struct omap_prm_data omap4_prm_data[] = {

>  	{ },

>  };

>  

> +static struct omap_prm_data dra7_prm_data[] = {

> +	{ .name = "mpu", .base = 0x4ae06300, .pwstst = 0x4 },

> +	{ .name = "dsp1", .base = 0x4ae06400, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

> +	{ .name = "ipu", .base = 0x4ae06500, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14, .clkdm_name = "ipu1" },

> +	{ .name = "coreaon", .base = 0x4ae06628, .pwstst = 0x4 },


Public TRM marks this region Reserved. Do you need it for anything?

> +	{ .name = "core", .base = 0x4ae06700, .pwstst = 0x4, .rstctl = 0x210, .rstst = 0x214, .clkdm_name = "ipu2" },

> +	{ .name = "iva", .base = 0x4ae06f00, .pwstst = 0x4 },


Missing rstctrl and rstst offsets.

> +	{ .name = "cam", .base = 0x4ae07000, .pwstst = 0x4 },

> +	{ .name = "dss", .base = 0x4ae07100, .pwstst = 0x4 },

> +	{ .name = "gpu", .base = 0x4ae07200, .pwstst = 0x4 },

> +	{ .name = "l3init", .base = 0x4ae07300, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

> +	{ .name = "l4per", .base = 0x4ae07400, .pwstst = 0x4 },

> +	{ .name = "custefuse", .base = 0x4ae07600, .pwstst = 0x4 },

> +	{ .name = "wkupaon", .base = 0x4ae07724, .pwstst = 0x4 },


No pwstctrl and pwstst bits documented in TRM or are marked reserved.

> +	{ .name = "emu", .base = 0x4ae07900, .pwstst = 0x4 },

> +	{ .name = "dsp2", .base = 0x4ae07b00, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

> +	{ .name = "eve1", .base = 0x4ae07b40, .pwstst = 0x4 },

> +	{ .name = "eve2", .base = 0x4ae07b80, .pwstst = 0x4 },

> +	{ .name = "eve3", .base = 0x4ae07bc0, .pwstst = 0x4 },

> +	{ .name = "eve4", .base = 0x4ae07c00, .pwstst = 0x4 },


All EVEs are missing rstctrl and rstst fields.

> +	{ .name = "rtc", .base = 0x4ae07c60, .pwstst = 0x4 },


Undocumented pwstctrl and pwstst registers.

> +	{ .name = "vpe", .base = 0x4ae07c80, .pwstst = 0x4 },


Missing "device" and "instr" PRM. The latter doesn't have any pwrstctl
and pwrstst though.

regards
Suman

> +	{ },

> +};

> +

>  struct omap_rst_map am3_wkup_rst_map[] = {

>  	{ .rst = 3, .st = 5 },

>  	{ .rst = -1 },

> @@ -91,6 +116,7 @@ struct omap_prm_data am3_prm_data[] = {

>  

>  static const struct of_device_id omap_prm_id_table[] = {

>  	{ .compatible = "ti,omap4-prm-inst", .data = omap4_prm_data },

> +	{ .compatible = "ti,dra7-prm-inst", .data = dra7_prm_data },

>  	{ .compatible = "ti,am3-prm-inst", .data = am3_prm_data },

>  	{ },

>  };

>
Tero Kristo Aug. 21, 2019, 7:36 a.m. UTC | #3
On 20.8.2019 22.03, Suman Anna wrote:
> Hi Tero,

> 

> On 8/7/19 2:48 AM, Tero Kristo wrote:

>> Add PRM data for dra7 family of SoCs.

>>

>> Signed-off-by: Tero Kristo <t-kristo@ti.com>

>> ---

>>   drivers/soc/ti/omap_prm.c | 26 ++++++++++++++++++++++++++

>>   1 file changed, 26 insertions(+)

>>

>> diff --git a/drivers/soc/ti/omap_prm.c b/drivers/soc/ti/omap_prm.c

>> index fadfc7f..05b7749 100644

>> --- a/drivers/soc/ti/omap_prm.c

>> +++ b/drivers/soc/ti/omap_prm.c

>> @@ -73,6 +73,31 @@ struct omap_prm_data omap4_prm_data[] = {

>>   	{ },

>>   };

>>   

>> +static struct omap_prm_data dra7_prm_data[] = {

>> +	{ .name = "mpu", .base = 0x4ae06300, .pwstst = 0x4 },

>> +	{ .name = "dsp1", .base = 0x4ae06400, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

>> +	{ .name = "ipu", .base = 0x4ae06500, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14, .clkdm_name = "ipu1" },

>> +	{ .name = "coreaon", .base = 0x4ae06628, .pwstst = 0x4 },

> 

> Public TRM marks this region Reserved. Do you need it for anything?

This is copied from existing PRM data from kernel. However, I'll ditch 
these for now and only retain the reset enabled domains.

> 

>> +	{ .name = "core", .base = 0x4ae06700, .pwstst = 0x4, .rstctl = 0x210, .rstst = 0x214, .clkdm_name = "ipu2" },

>> +	{ .name = "iva", .base = 0x4ae06f00, .pwstst = 0x4 },

> 

> Missing rstctrl and rstst offsets.


Will add.

> 

>> +	{ .name = "cam", .base = 0x4ae07000, .pwstst = 0x4 },

>> +	{ .name = "dss", .base = 0x4ae07100, .pwstst = 0x4 },

>> +	{ .name = "gpu", .base = 0x4ae07200, .pwstst = 0x4 },

>> +	{ .name = "l3init", .base = 0x4ae07300, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

>> +	{ .name = "l4per", .base = 0x4ae07400, .pwstst = 0x4 },

>> +	{ .name = "custefuse", .base = 0x4ae07600, .pwstst = 0x4 },

>> +	{ .name = "wkupaon", .base = 0x4ae07724, .pwstst = 0x4 },

> 

> No pwstctrl and pwstst bits documented in TRM or are marked reserved.


Same as coreaon.

> 

>> +	{ .name = "emu", .base = 0x4ae07900, .pwstst = 0x4 },

>> +	{ .name = "dsp2", .base = 0x4ae07b00, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },

>> +	{ .name = "eve1", .base = 0x4ae07b40, .pwstst = 0x4 },

>> +	{ .name = "eve2", .base = 0x4ae07b80, .pwstst = 0x4 },

>> +	{ .name = "eve3", .base = 0x4ae07bc0, .pwstst = 0x4 },

>> +	{ .name = "eve4", .base = 0x4ae07c00, .pwstst = 0x4 },

> 

> All EVEs are missing rstctrl and rstst fields.


Will add.

> 

>> +	{ .name = "rtc", .base = 0x4ae07c60, .pwstst = 0x4 },

> 

> Undocumented pwstctrl and pwstst registers.

> 

>> +	{ .name = "vpe", .base = 0x4ae07c80, .pwstst = 0x4 },

> 

> Missing "device" and "instr" PRM. The latter doesn't have any pwrstctl

> and pwrstst though.


Will ditch those.

-Tero

> 

> regards

> Suman

> 

>> +	{ },

>> +};

>> +

>>   struct omap_rst_map am3_wkup_rst_map[] = {

>>   	{ .rst = 3, .st = 5 },

>>   	{ .rst = -1 },

>> @@ -91,6 +116,7 @@ struct omap_prm_data am3_prm_data[] = {

>>   

>>   static const struct of_device_id omap_prm_id_table[] = {

>>   	{ .compatible = "ti,omap4-prm-inst", .data = omap4_prm_data },

>> +	{ .compatible = "ti,dra7-prm-inst", .data = dra7_prm_data },

>>   	{ .compatible = "ti,am3-prm-inst", .data = am3_prm_data },

>>   	{ },

>>   };

>>

> 


--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
diff mbox series

Patch

diff --git a/drivers/soc/ti/omap_prm.c b/drivers/soc/ti/omap_prm.c
index fadfc7f..05b7749 100644
--- a/drivers/soc/ti/omap_prm.c
+++ b/drivers/soc/ti/omap_prm.c
@@ -73,6 +73,31 @@  struct omap_prm_data omap4_prm_data[] = {
 	{ },
 };
 
+static struct omap_prm_data dra7_prm_data[] = {
+	{ .name = "mpu", .base = 0x4ae06300, .pwstst = 0x4 },
+	{ .name = "dsp1", .base = 0x4ae06400, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },
+	{ .name = "ipu", .base = 0x4ae06500, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14, .clkdm_name = "ipu1" },
+	{ .name = "coreaon", .base = 0x4ae06628, .pwstst = 0x4 },
+	{ .name = "core", .base = 0x4ae06700, .pwstst = 0x4, .rstctl = 0x210, .rstst = 0x214, .clkdm_name = "ipu2" },
+	{ .name = "iva", .base = 0x4ae06f00, .pwstst = 0x4 },
+	{ .name = "cam", .base = 0x4ae07000, .pwstst = 0x4 },
+	{ .name = "dss", .base = 0x4ae07100, .pwstst = 0x4 },
+	{ .name = "gpu", .base = 0x4ae07200, .pwstst = 0x4 },
+	{ .name = "l3init", .base = 0x4ae07300, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },
+	{ .name = "l4per", .base = 0x4ae07400, .pwstst = 0x4 },
+	{ .name = "custefuse", .base = 0x4ae07600, .pwstst = 0x4 },
+	{ .name = "wkupaon", .base = 0x4ae07724, .pwstst = 0x4 },
+	{ .name = "emu", .base = 0x4ae07900, .pwstst = 0x4 },
+	{ .name = "dsp2", .base = 0x4ae07b00, .pwstst = 0x4, .rstctl = 0x10, .rstst = 0x14 },
+	{ .name = "eve1", .base = 0x4ae07b40, .pwstst = 0x4 },
+	{ .name = "eve2", .base = 0x4ae07b80, .pwstst = 0x4 },
+	{ .name = "eve3", .base = 0x4ae07bc0, .pwstst = 0x4 },
+	{ .name = "eve4", .base = 0x4ae07c00, .pwstst = 0x4 },
+	{ .name = "rtc", .base = 0x4ae07c60, .pwstst = 0x4 },
+	{ .name = "vpe", .base = 0x4ae07c80, .pwstst = 0x4 },
+	{ },
+};
+
 struct omap_rst_map am3_wkup_rst_map[] = {
 	{ .rst = 3, .st = 5 },
 	{ .rst = -1 },
@@ -91,6 +116,7 @@  struct omap_prm_data am3_prm_data[] = {
 
 static const struct of_device_id omap_prm_id_table[] = {
 	{ .compatible = "ti,omap4-prm-inst", .data = omap4_prm_data },
+	{ .compatible = "ti,dra7-prm-inst", .data = dra7_prm_data },
 	{ .compatible = "ti,am3-prm-inst", .data = am3_prm_data },
 	{ },
 };