diff mbox

ARM: shmobile: r8a7790: DU and IPMMU-MX prototype

Message ID 20151217064535.30114.91840.sendpatchset@little-apple (mailing list archive)
State RFC
Headers show

Commit Message

Magnus Damm Dec. 17, 2015, 6:45 a.m. UTC
From: Magnus Damm <damm+renesas@opensource.se>

Enable the IPMMU-MX device and hook up the DU on r8a7790,
and also include enable debug messages.

On boards such as r8a7790 Lager the DU will with this change
included use IOMMU framework and IPMMU-MX device  when driving
on-board video output via VGA, HDMI and LVDS.

Not-Yet-Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 Written against renesas-devel-20151217v2-v4.4-rc5

 arch/arm/boot/dts/r8a7790.dtsi |    3 ++-
 drivers/iommu/iommu.c          |    2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

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

Comments

Sergei Shtylyov Dec. 17, 2015, 11:38 a.m. UTC | #1
Hello.

On 12/17/2015 9:45 AM, Magnus Damm wrote:

> From: Magnus Damm <damm+renesas@opensource.se>
>
> Enable the IPMMU-MX device and hook up the DU on r8a7790,
> and also include enable debug messages.
>
> On boards such as r8a7790 Lager the DU will with this change
> included use IOMMU framework and IPMMU-MX device  when driving
> on-board video output via VGA, HDMI and LVDS.
>
> Not-Yet-Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
>
>   Written against renesas-devel-20151217v2-v4.4-rc5
>
>   arch/arm/boot/dts/r8a7790.dtsi |    3 ++-
>   drivers/iommu/iommu.c          |    2 +-
>   2 files changed, 3 insertions(+), 2 deletions(-)
>
> --- 0001/arch/arm/boot/dts/r8a7790.dtsi
> +++ work/arch/arm/boot/dts/r8a7790.dtsi	2015-12-17 15:38:20.890513000 +0900
> @@ -886,6 +886,7 @@
>   			 <&mstp7_clks R8A7790_CLK_LVDS0>,
>   			 <&mstp7_clks R8A7790_CLK_LVDS1>;
>   		clock-names = "du.0", "du.1", "du.2", "lvds.0", "lvds.1";
> +		iommus = <&ipmmu_mx 15>, <&ipmmu_mx 16>;
>   		status = "disabled";
>
>   		ports {
> @@ -1809,7 +1810,7 @@
>   		interrupts = <0 222 IRQ_TYPE_LEVEL_HIGH>,
>   			     <0 221 IRQ_TYPE_LEVEL_HIGH>;
>   		#iommu-cells = <1>;
> -		status = "disabled";
> +		status = "okay";

    It's enough to just omit the "status" prop, AFAIK.

[...]
> --- 0001/drivers/iommu/iommu.c
> +++ work/drivers/iommu/iommu.c	2015-12-17 15:38:34.880513000 +0900
> @@ -15,7 +15,7 @@
>    * along with this program; if not, write to the Free Software
>    * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
>    */
> -
> +#define DEBUG

    Unrelated change?

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sergei Shtylyov Dec. 17, 2015, 11:42 a.m. UTC | #2
Hello.

On 12/17/2015 9:45 AM, Magnus Damm wrote:

> From: Magnus Damm <damm+renesas@opensource.se>
>
> Enable the IPMMU-MX device and hook up the DU on r8a7790,
> and also include enable debug messages.
>
> On boards such as r8a7790 Lager the DU will with this change
> included use IOMMU framework and IPMMU-MX device  when driving
> on-board video output via VGA, HDMI and LVDS.
>
> Not-Yet-Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
>
>   Written against renesas-devel-20151217v2-v4.4-rc5
>
>   arch/arm/boot/dts/r8a7790.dtsi |    3 ++-
>   drivers/iommu/iommu.c          |    2 +-
>   2 files changed, 3 insertions(+), 2 deletions(-)
>
> --- 0001/arch/arm/boot/dts/r8a7790.dtsi
> +++ work/arch/arm/boot/dts/r8a7790.dtsi	2015-12-17 15:38:20.890513000 +0900
> @@ -886,6 +886,7 @@
>   			 <&mstp7_clks R8A7790_CLK_LVDS0>,
>   			 <&mstp7_clks R8A7790_CLK_LVDS1>;
>   		clock-names = "du.0", "du.1", "du.2", "lvds.0", "lvds.1";
> +		iommus = <&ipmmu_mx 15>, <&ipmmu_mx 16>;
>   		status = "disabled";
>
>   		ports {
> @@ -1809,7 +1810,7 @@
>   		interrupts = <0 222 IRQ_TYPE_LEVEL_HIGH>,
>   			     <0 221 IRQ_TYPE_LEVEL_HIGH>;
>   		#iommu-cells = <1>;
> -		status = "disabled";
> +		status = "okay";

    It's enough to just omit the "status" prop, AFAIK.

[...]
> --- 0001/drivers/iommu/iommu.c
> +++ work/drivers/iommu/iommu.c	2015-12-17 15:38:34.880513000 +0900
> @@ -15,7 +15,7 @@
>    * along with this program; if not, write to the Free Software
>    * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
>    */
> -
> +#define DEBUG

    Unrelated change?

MBR, Sergei

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

Patch

--- 0001/arch/arm/boot/dts/r8a7790.dtsi
+++ work/arch/arm/boot/dts/r8a7790.dtsi	2015-12-17 15:38:20.890513000 +0900
@@ -886,6 +886,7 @@ 
 			 <&mstp7_clks R8A7790_CLK_LVDS0>,
 			 <&mstp7_clks R8A7790_CLK_LVDS1>;
 		clock-names = "du.0", "du.1", "du.2", "lvds.0", "lvds.1";
+		iommus = <&ipmmu_mx 15>, <&ipmmu_mx 16>;
 		status = "disabled";
 
 		ports {
@@ -1809,7 +1810,7 @@ 
 		interrupts = <0 222 IRQ_TYPE_LEVEL_HIGH>,
 			     <0 221 IRQ_TYPE_LEVEL_HIGH>;
 		#iommu-cells = <1>;
-		status = "disabled";
+		status = "okay";
 	};
 
 	ipmmu_rt: mmu@ffc80000 {
--- 0001/drivers/iommu/iommu.c
+++ work/drivers/iommu/iommu.c	2015-12-17 15:38:34.880513000 +0900
@@ -15,7 +15,7 @@ 
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
  */
-
+#define DEBUG
 #define pr_fmt(fmt)    "iommu: " fmt
 
 #include <linux/device.h>