Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-07-30 Thread Tomi Valkeinen
On 23/07/14 00:14, Paul Walmsley wrote:
 On Tue, 17 Jun 2014, Tomi Valkeinen wrote:
 
 From: Sathya Prakash M R sath...@ti.com

 Add DSS hwmod data for AM43xx.

 Signed-off-by: Sathya Prakash M R sath...@ti.com
 [tomi.valkei...@ti.com: added missing dispc flags]
 Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
 Acked-by: Rajendra Nayak rna...@ti.com
 
 This one didn't compile on an AM43xx-only build:
 
 
 arch/arm/mach-omap2/built-in.o:(.data+0x3f2c): undefined reference to 
 `omap2_dss_hwmod_class'
 arch/arm/mach-omap2/built-in.o:(.data+0x405c): undefined reference to 
 `omap2_rfbi_hwmod_class'
 make: *** [vmlinux] Error 1
 test_build: Tue Jul 22 13:48:50 MDT 2014: FAILED on 
 omap2plus_defconfig_am43xx_only hwmod-a-v3.17
 
 
 Have queued the following patch instead.

Thanks. Tested on AM437x GP EVM with today's linux-next. Works fine for me.

 Tomi




signature.asc
Description: OpenPGP digital signature


Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-07-29 Thread Tony Lindgren
* Felipe Balbi ba...@ti.com [140725 09:55]:
 On Tue, Jul 22, 2014 at 09:14:20PM +, Paul Walmsley wrote:
  On Tue, 17 Jun 2014, Tomi Valkeinen wrote:
  
   From: Sathya Prakash M R sath...@ti.com
   
   Add DSS hwmod data for AM43xx.
   
   Signed-off-by: Sathya Prakash M R sath...@ti.com
   [tomi.valkei...@ti.com: added missing dispc flags]
   Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
   Acked-by: Rajendra Nayak rna...@ti.com
  
  This one didn't compile on an AM43xx-only build:
  
  
  arch/arm/mach-omap2/built-in.o:(.data+0x3f2c): undefined reference to 
  `omap2_dss_hwmod_class'
  arch/arm/mach-omap2/built-in.o:(.data+0x405c): undefined reference to 
  `omap2_rfbi_hwmod_class'
  make: *** [vmlinux] Error 1
  test_build: Tue Jul 22 13:48:50 MDT 2014: FAILED on 
  omap2plus_defconfig_am43xx_only hwmod-a-v3.17
  
  
  Have queued the following patch instead.
 
 queued for v3.17, I assume. Still not available from linux-next though.

Should be in linux-next now.

Tony
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-07-25 Thread Felipe Balbi
On Tue, Jul 22, 2014 at 09:14:20PM +, Paul Walmsley wrote:
 On Tue, 17 Jun 2014, Tomi Valkeinen wrote:
 
  From: Sathya Prakash M R sath...@ti.com
  
  Add DSS hwmod data for AM43xx.
  
  Signed-off-by: Sathya Prakash M R sath...@ti.com
  [tomi.valkei...@ti.com: added missing dispc flags]
  Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
  Acked-by: Rajendra Nayak rna...@ti.com
 
 This one didn't compile on an AM43xx-only build:
 
 
 arch/arm/mach-omap2/built-in.o:(.data+0x3f2c): undefined reference to 
 `omap2_dss_hwmod_class'
 arch/arm/mach-omap2/built-in.o:(.data+0x405c): undefined reference to 
 `omap2_rfbi_hwmod_class'
 make: *** [vmlinux] Error 1
 test_build: Tue Jul 22 13:48:50 MDT 2014: FAILED on 
 omap2plus_defconfig_am43xx_only hwmod-a-v3.17
 
 
 Have queued the following patch instead.

queued for v3.17, I assume. Still not available from linux-next though.

-- 
balbi


signature.asc
Description: Digital signature


Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-07-22 Thread Paul Walmsley
On Tue, 17 Jun 2014, Tomi Valkeinen wrote:

 From: Sathya Prakash M R sath...@ti.com
 
 Add DSS hwmod data for AM43xx.
 
 Signed-off-by: Sathya Prakash M R sath...@ti.com
 [tomi.valkei...@ti.com: added missing dispc flags]
 Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
 Acked-by: Rajendra Nayak rna...@ti.com

This one didn't compile on an AM43xx-only build:


arch/arm/mach-omap2/built-in.o:(.data+0x3f2c): undefined reference to 
`omap2_dss_hwmod_class'
arch/arm/mach-omap2/built-in.o:(.data+0x405c): undefined reference to 
`omap2_rfbi_hwmod_class'
make: *** [vmlinux] Error 1
test_build: Tue Jul 22 13:48:50 MDT 2014: FAILED on 
omap2plus_defconfig_am43xx_only hwmod-a-v3.17


Have queued the following patch instead.


- Paul

From: Sathya Prakash M R sath...@ti.com
Date: Sat, 5 Jul 2014 17:44:57 -0600
Subject: [PATCH] ARM: AM43xx: hwmod: add DSS hwmod data

Add DSS hwmod data for AM43xx.

Signed-off-by: Sathya Prakash M R sath...@ti.com
[tomi.valkei...@ti.com: added missing dispc flags]
Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
Acked-by: Rajendra Nayak rna...@ti.com
Acked-by: Tony Lindgren t...@atomide.com
Tested-by: Felipe Balbi ba...@ti.com # on linux-next 5f295cdf5c5d
[p...@pwsan.com: fixed build break on AM43xx-only config]
Signed-off-by: Paul Walmsley p...@pwsan.com
---
 arch/arm/mach-omap2/Makefile   |   1 +
 .../mach-omap2/omap_hwmod_2xxx_3xxx_ipblock_data.c |  40 -
 arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 100 +
 .../mach-omap2/omap_hwmod_common_ipblock_data.c|  55 
 arch/arm/mach-omap2/prcm43xx.h |   1 +
 5 files changed, 157 insertions(+), 40 deletions(-)
 create mode 100644 arch/arm/mach-omap2/omap_hwmod_common_ipblock_data.c

diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
index 8421f38cf445..75c73f253604 100644
--- a/arch/arm/mach-omap2/Makefile
+++ b/arch/arm/mach-omap2/Makefile
@@ -200,6 +200,7 @@ obj-$(CONFIG_SOC_OMAP2420)  += opp2420_data.o
 obj-$(CONFIG_SOC_OMAP2430) += opp2430_data.o
 
 # hwmod data
+obj-y  += omap_hwmod_common_ipblock_data.o
 obj-$(CONFIG_SOC_OMAP2420) += omap_hwmod_2xxx_ipblock_data.o
 obj-$(CONFIG_SOC_OMAP2420) += omap_hwmod_2xxx_3xxx_ipblock_data.o
 obj-$(CONFIG_SOC_OMAP2420) += omap_hwmod_2xxx_interconnect_data.o
diff --git a/arch/arm/mach-omap2/omap_hwmod_2xxx_3xxx_ipblock_data.c 
b/arch/arm/mach-omap2/omap_hwmod_2xxx_3xxx_ipblock_data.c
index 5da7a42a6d90..c6c6384de867 100644
--- a/arch/arm/mach-omap2/omap_hwmod_2xxx_3xxx_ipblock_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_2xxx_3xxx_ipblock_data.c
@@ -37,46 +37,6 @@ struct omap_hwmod_class omap2_uart_class = {
 };
 
 /*
- * 'dss' class
- * display sub-system
- */
-
-static struct omap_hwmod_class_sysconfig omap2_dss_sysc = {
-   .rev_offs   = 0x,
-   .sysc_offs  = 0x0010,
-   .syss_offs  = 0x0014,
-   .sysc_flags = (SYSC_HAS_SOFTRESET | SYSC_HAS_AUTOIDLE |
-  SYSS_HAS_RESET_STATUS),
-   .sysc_fields= omap_hwmod_sysc_type1,
-};
-
-struct omap_hwmod_class omap2_dss_hwmod_class = {
-   .name   = dss,
-   .sysc   = omap2_dss_sysc,
-   .reset  = omap_dss_reset,
-};
-
-/*
- * 'rfbi' class
- * remote frame buffer interface
- */
-
-static struct omap_hwmod_class_sysconfig omap2_rfbi_sysc = {
-   .rev_offs   = 0x,
-   .sysc_offs  = 0x0010,
-   .syss_offs  = 0x0014,
-   .sysc_flags = (SYSC_HAS_SIDLEMODE | SYSC_HAS_SOFTRESET |
-  SYSC_HAS_AUTOIDLE),
-   .idlemodes  = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART),
-   .sysc_fields= omap_hwmod_sysc_type1,
-};
-
-struct omap_hwmod_class omap2_rfbi_hwmod_class = {
-   .name   = rfbi,
-   .sysc   = omap2_rfbi_sysc,
-};
-
-/*
  * 'venc' class
  * video encoder
  */
diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c 
b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
index 5c2cc8083fdd..fea01aa3ef42 100644
--- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
@@ -19,6 +19,8 @@
 #include omap_hwmod.h
 #include omap_hwmod_33xx_43xx_common_data.h
 #include prcm43xx.h
+#include omap_hwmod_common_data.h
+
 
 /* IP blocks */
 static struct omap_hwmod am43xx_l4_hs_hwmod = {
@@ -415,6 +417,72 @@ static struct omap_hwmod am43xx_qspi_hwmod = {
},
 };
 
+/* dss */
+
+static struct omap_hwmod am43xx_dss_core_hwmod = {
+   .name   = dss_core,
+   .class  = omap2_dss_hwmod_class,
+   .clkdm_name = dss_clkdm,
+   .main_clk   = disp_clk,
+   .prcm = {
+   .omap4 = {
+   .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
+   .modulemode   = MODULEMODE_SWCTRL,
+   },
+   },
+};
+
+/* dispc */
+
+struct omap_dss_dispc_dev_attr am43xx_dss_dispc_dev_attr = {
+

Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-06-22 Thread Paul Walmsley
On Tue, 17 Jun 2014, Tomi Valkeinen wrote:

 From: Sathya Prakash M R sath...@ti.com
 
 Add DSS hwmod data for AM43xx.
 
 Signed-off-by: Sathya Prakash M R sath...@ti.com
 [tomi.valkei...@ti.com: added missing dispc flags]
 Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
 Acked-by: Rajendra Nayak rna...@ti.com

Thanks, queued with Tony's Acked-by: and Felipe's Tested-by: for 3.17.


- Paul
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-06-17 Thread Tomi Valkeinen
From: Sathya Prakash M R sath...@ti.com

Add DSS hwmod data for AM43xx.

Signed-off-by: Sathya Prakash M R sath...@ti.com
[tomi.valkei...@ti.com: added missing dispc flags]
Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
Acked-by: Rajendra Nayak rna...@ti.com
---
Changes to v1:

* added missing dispc .sysc_flags and .idlemodes
* reordered entries, as the dss and dispc were a bit mixed together

 arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 100 +
 arch/arm/mach-omap2/prcm43xx.h |   1 +
 2 files changed, 101 insertions(+)

diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c 
b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
index 5c2cc8083fdd..fea01aa3ef42 100644
--- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
@@ -19,6 +19,8 @@
 #include omap_hwmod.h
 #include omap_hwmod_33xx_43xx_common_data.h
 #include prcm43xx.h
+#include omap_hwmod_common_data.h
+
 
 /* IP blocks */
 static struct omap_hwmod am43xx_l4_hs_hwmod = {
@@ -415,6 +417,72 @@ static struct omap_hwmod am43xx_qspi_hwmod = {
},
 };
 
+/* dss */
+
+static struct omap_hwmod am43xx_dss_core_hwmod = {
+   .name   = dss_core,
+   .class  = omap2_dss_hwmod_class,
+   .clkdm_name = dss_clkdm,
+   .main_clk   = disp_clk,
+   .prcm = {
+   .omap4 = {
+   .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
+   .modulemode   = MODULEMODE_SWCTRL,
+   },
+   },
+};
+
+/* dispc */
+
+struct omap_dss_dispc_dev_attr am43xx_dss_dispc_dev_attr = {
+   .manager_count  = 1,
+   .has_framedonetv_irq= 0
+};
+
+static struct omap_hwmod_class_sysconfig am43xx_dispc_sysc = {
+   .rev_offs   = 0x,
+   .sysc_offs  = 0x0010,
+   .syss_offs  = 0x0014,
+   .sysc_flags = (SYSC_HAS_AUTOIDLE | SYSC_HAS_SOFTRESET |
+  SYSC_HAS_ENAWAKEUP | SYSC_HAS_SIDLEMODE |
+  SYSC_HAS_CLOCKACTIVITY | SYSC_HAS_MIDLEMODE),
+   .idlemodes  = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART |
+  MSTANDBY_FORCE | MSTANDBY_NO | MSTANDBY_SMART),
+   .sysc_fields= omap_hwmod_sysc_type1,
+};
+
+static struct omap_hwmod_class am43xx_dispc_hwmod_class = {
+   .name   = dispc,
+   .sysc   = am43xx_dispc_sysc,
+};
+
+static struct omap_hwmod am43xx_dss_dispc_hwmod = {
+   .name   = dss_dispc,
+   .class  = am43xx_dispc_hwmod_class,
+   .clkdm_name = dss_clkdm,
+   .main_clk   = disp_clk,
+   .prcm = {
+   .omap4 = {
+   .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
+   },
+   },
+   .dev_attr   = am43xx_dss_dispc_dev_attr,
+};
+
+/* rfbi */
+
+static struct omap_hwmod am43xx_dss_rfbi_hwmod = {
+   .name   = dss_rfbi,
+   .class  = omap2_rfbi_hwmod_class,
+   .clkdm_name = dss_clkdm,
+   .main_clk   = disp_clk,
+   .prcm = {
+   .omap4 = {
+   .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
+   },
+   },
+};
+
 /* Interfaces */
 static struct omap_hwmod_ocp_if am43xx_l3_main__l4_hs = {
.master = am33xx_l3_main_hwmod,
@@ -654,6 +722,34 @@ static struct omap_hwmod_ocp_if am43xx_l3_s__qspi = {
.user   = OCP_USER_MPU | OCP_USER_SDMA,
 };
 
+static struct omap_hwmod_ocp_if am43xx_dss__l3_main = {
+   .master = am43xx_dss_core_hwmod,
+   .slave  = am33xx_l3_main_hwmod,
+   .clk= l3_gclk,
+   .user   = OCP_USER_MPU | OCP_USER_SDMA,
+};
+
+static struct omap_hwmod_ocp_if am43xx_l4_ls__dss = {
+   .master = am33xx_l4_ls_hwmod,
+   .slave  = am43xx_dss_core_hwmod,
+   .clk= l4ls_gclk,
+   .user   = OCP_USER_MPU | OCP_USER_SDMA,
+};
+
+static struct omap_hwmod_ocp_if am43xx_l4_ls__dss_dispc = {
+   .master = am33xx_l4_ls_hwmod,
+   .slave  = am43xx_dss_dispc_hwmod,
+   .clk= l4ls_gclk,
+   .user   = OCP_USER_MPU | OCP_USER_SDMA,
+};
+
+static struct omap_hwmod_ocp_if am43xx_l4_ls__dss_rfbi = {
+   .master = am33xx_l4_ls_hwmod,
+   .slave  = am43xx_dss_rfbi_hwmod,
+   .clk= l4ls_gclk,
+   .user   = OCP_USER_MPU | OCP_USER_SDMA,
+};
+
 static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = {
am33xx_l4_wkup__synctimer,
am43xx_l4_ls__timer8,
@@ -748,6 +844,10 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] 
__initdata = {
am43xx_l4_ls__ocp2scp1,
am43xx_l3_s__usbotgss0,
am43xx_l3_s__usbotgss1,
+   am43xx_dss__l3_main,
+   am43xx_l4_ls__dss,
+   am43xx_l4_ls__dss_dispc,
+   am43xx_l4_ls__dss_rfbi,
NULL,
 };
 
diff --git a/arch/arm/mach-omap2/prcm43xx.h 

Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-06-17 Thread Tony Lindgren
* Tomi Valkeinen tomi.valkei...@ti.com [140617 01:39]:
 From: Sathya Prakash M R sath...@ti.com
 
 Add DSS hwmod data for AM43xx.
 
 Signed-off-by: Sathya Prakash M R sath...@ti.com
 [tomi.valkei...@ti.com: added missing dispc flags]
 Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
 Acked-by: Rajendra Nayak rna...@ti.com
 ---
 Changes to v1:
 
 * added missing dispc .sysc_flags and .idlemodes
 * reordered entries, as the dss and dispc were a bit mixed together
 
  arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 100 
 +
  arch/arm/mach-omap2/prcm43xx.h |   1 +
  2 files changed, 101 insertions(+)
 
 diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c 
 b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
 index 5c2cc8083fdd..fea01aa3ef42 100644
 --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
 +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
 @@ -19,6 +19,8 @@
  #include omap_hwmod.h
  #include omap_hwmod_33xx_43xx_common_data.h
  #include prcm43xx.h
 +#include omap_hwmod_common_data.h
 +
  
  /* IP blocks */
  static struct omap_hwmod am43xx_l4_hs_hwmod = {
 @@ -415,6 +417,72 @@ static struct omap_hwmod am43xx_qspi_hwmod = {
   },
  };
  
 +/* dss */
 +
 +static struct omap_hwmod am43xx_dss_core_hwmod = {
 + .name   = dss_core,
 + .class  = omap2_dss_hwmod_class,
 + .clkdm_name = dss_clkdm,
 + .main_clk   = disp_clk,
 + .prcm = {
 + .omap4 = {
 + .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
 + .modulemode   = MODULEMODE_SWCTRL,
 + },
 + },
 +};
 +
 +/* dispc */
 +
 +struct omap_dss_dispc_dev_attr am43xx_dss_dispc_dev_attr = {
 + .manager_count  = 1,
 + .has_framedonetv_irq= 0
 +};
 +
 +static struct omap_hwmod_class_sysconfig am43xx_dispc_sysc = {
 + .rev_offs   = 0x,
 + .sysc_offs  = 0x0010,
 + .syss_offs  = 0x0014,
 + .sysc_flags = (SYSC_HAS_AUTOIDLE | SYSC_HAS_SOFTRESET |
 +SYSC_HAS_ENAWAKEUP | SYSC_HAS_SIDLEMODE |
 +SYSC_HAS_CLOCKACTIVITY | SYSC_HAS_MIDLEMODE),
 + .idlemodes  = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART |
 +MSTANDBY_FORCE | MSTANDBY_NO | MSTANDBY_SMART),
 + .sysc_fields= omap_hwmod_sysc_type1,
 +};
 +
 +static struct omap_hwmod_class am43xx_dispc_hwmod_class = {
 + .name   = dispc,
 + .sysc   = am43xx_dispc_sysc,
 +};
 +
 +static struct omap_hwmod am43xx_dss_dispc_hwmod = {
 + .name   = dss_dispc,
 + .class  = am43xx_dispc_hwmod_class,
 + .clkdm_name = dss_clkdm,
 + .main_clk   = disp_clk,
 + .prcm = {
 + .omap4 = {
 + .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
 + },
 + },
 + .dev_attr   = am43xx_dss_dispc_dev_attr,
 +};
 +
 +/* rfbi */

I think we're still missing am43x_rfbi_sysc entry though?

 +static struct omap_hwmod am43xx_dss_rfbi_hwmod = {
 + .name   = dss_rfbi,
 + .class  = omap2_rfbi_hwmod_class,
 + .clkdm_name = dss_clkdm,
 + .main_clk   = disp_clk,
 + .prcm = {
 + .omap4 = {
 + .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
 + },
 + },
 +};
 +
  /* Interfaces */
  static struct omap_hwmod_ocp_if am43xx_l3_main__l4_hs = {
   .master = am33xx_l3_main_hwmod,
 @@ -654,6 +722,34 @@ static struct omap_hwmod_ocp_if am43xx_l3_s__qspi = {
   .user   = OCP_USER_MPU | OCP_USER_SDMA,
  };
  
 +static struct omap_hwmod_ocp_if am43xx_dss__l3_main = {
 + .master = am43xx_dss_core_hwmod,
 + .slave  = am33xx_l3_main_hwmod,
 + .clk= l3_gclk,
 + .user   = OCP_USER_MPU | OCP_USER_SDMA,
 +};
 +
 +static struct omap_hwmod_ocp_if am43xx_l4_ls__dss = {
 + .master = am33xx_l4_ls_hwmod,
 + .slave  = am43xx_dss_core_hwmod,
 + .clk= l4ls_gclk,
 + .user   = OCP_USER_MPU | OCP_USER_SDMA,
 +};
 +
 +static struct omap_hwmod_ocp_if am43xx_l4_ls__dss_dispc = {
 + .master = am33xx_l4_ls_hwmod,
 + .slave  = am43xx_dss_dispc_hwmod,
 + .clk= l4ls_gclk,
 + .user   = OCP_USER_MPU | OCP_USER_SDMA,
 +};
 +
 +static struct omap_hwmod_ocp_if am43xx_l4_ls__dss_rfbi = {
 + .master = am33xx_l4_ls_hwmod,
 + .slave  = am43xx_dss_rfbi_hwmod,
 + .clk= l4ls_gclk,
 + .user   = OCP_USER_MPU | OCP_USER_SDMA,
 +};
 +
  static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = {
   am33xx_l4_wkup__synctimer,
   am43xx_l4_ls__timer8,
 @@ -748,6 +844,10 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] 
 __initdata = {
   am43xx_l4_ls__ocp2scp1,
   am43xx_l3_s__usbotgss0,
   am43xx_l3_s__usbotgss1,
 + am43xx_dss__l3_main,
 + am43xx_l4_ls__dss,
 + 

Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-06-17 Thread Tomi Valkeinen
On 17/06/14 11:58, Tony Lindgren wrote:

 +/* rfbi */
 
 I think we're still missing am43x_rfbi_sysc entry though?
 
 +static struct omap_hwmod am43xx_dss_rfbi_hwmod = {
 +.name   = dss_rfbi,
 +.class  = omap2_rfbi_hwmod_class,
 +.clkdm_name = dss_clkdm,
 +.main_clk   = disp_clk,
 +.prcm = {
 +.omap4 = {
 +.clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
 +},
 +},
 +};

That should come from omap2/3 hwmod data, as the above references
omap2_rfbi_hwmod_class. The bits there looked correct for AM43xx to me.

 Tomi




signature.asc
Description: OpenPGP digital signature


Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-06-17 Thread Tony Lindgren
* Tomi Valkeinen tomi.valkei...@ti.com [140617 02:26]:
 On 17/06/14 11:58, Tony Lindgren wrote:
 
  +/* rfbi */
  
  I think we're still missing am43x_rfbi_sysc entry though?
  
  +static struct omap_hwmod am43xx_dss_rfbi_hwmod = {
  +  .name   = dss_rfbi,
  +  .class  = omap2_rfbi_hwmod_class,
  +  .clkdm_name = dss_clkdm,
  +  .main_clk   = disp_clk,
  +  .prcm = {
  +  .omap4 = {
  +  .clkctrl_offs = AM43XX_CM_PER_DSS_CLKCTRL_OFFSET,
  +  },
  +  },
  +};
 
 That should come from omap2/3 hwmod data, as the above references
 omap2_rfbi_hwmod_class. The bits there looked correct for AM43xx to me.

OK looks good to me then:

Acked-by: Tony Lindgren t...@atomide.com
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2] ARM: AM43xx: hwmod: add DSS hwmod data

2014-06-17 Thread Felipe Balbi
On Tue, Jun 17, 2014 at 11:37:13AM +0300, Tomi Valkeinen wrote:
 From: Sathya Prakash M R sath...@ti.com
 
 Add DSS hwmod data for AM43xx.
 
 Signed-off-by: Sathya Prakash M R sath...@ti.com
 [tomi.valkei...@ti.com: added missing dispc flags]
 Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
 Acked-by: Rajendra Nayak rna...@ti.com

tested on top of today's linux-next
(5f295cdf5c5dbbb0c40f10f2ddae02ff46bbf773) with am437x-sk. Works fine:

Tested-by: Felipe Balbi ba...@ti.com

-- 
balbi


minicom.cap
Description: application/vnd.tcpdump.pcap


signature.asc
Description: Digital signature