Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
To:     ohad@wizery.com, bjorn.andersson@linaro.org,
        mathieu.poirier@linaro.org, o.rempel@pengutronix.de
Cc:     shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de,
        festevam@gmail.com, linux-imx@nxp.com,
        linux-remoteproc@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        aisheng.dong@nxp.com, Peng Fan <peng.fan@nxp.com>
Subject: [PATCH V4 6/6] remoteproc: imx_rproc: change to ioremap_wc for dram
Date:   Fri, 10 Sep 2021 17:06:21 +0800
Message-Id: <20210910090621.3073540-7-peng.fan@oss.nxp.com>
In-Reply-To: <20210910090621.3073540-1-peng.fan@oss.nxp.com>
References: <20210910090621.3073540-1-peng.fan@oss.nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from localhost.localdomain (119.31.174.66) by SG3P274CA0012.SGPP274.PROD.OUTLOOK.COM (2603:1096:4:be::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.14 via Frontend Transport; Fri, 10 Sep 2021 09:07:44 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 00b902b4-0c93-47b2-d470-08d9743a75ec
X-MS-TrafficTypeDiagnostic: DB7PR04MB4763:
X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <DB7PR04MB47630AFFFB621809EF5569FCC9D69@DB7PR04MB4763.eurprd04.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:1284;
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: yhJCQsheu9mCLh+Rmz1JHfGllTF+3klKkdcq+MDmlzvFBaIUs1YGTnxB5xDNE9hbxyfI9D6GSyASwDxN+3kE9WG5vNey0XdXyfrw0bdJCpYjAU+u1f9Xmyik4a7dMVs+fqDRydK0twAI4Hx/JxqivOxKmDCB7ayQh+THCVlDlXkRcsupLB38/xZE3pf9MdcQc7kqR0sN5UT37EO6TXd/RT5+fHlXxCY917PBxR1pYM3s04HFRLHH+NOdF+aqtIQvJT8dB9E2WGteiytGVGozzuJo3uEOTChYMy+t59BEsmVN/TNFWxZGz37WzibsTPG2ZqmOKCDJ92q8RF9eeTCztqmMavimSFdgmv21RHRHHElhmil+XJlP8afPyAvQ4DSH3P3q9iUg2FwUzUGRTmc5f8KY3uR27buWW3IxVODqwR4L+eExAmZZP1JFtu3FrJ5NsKCsrmmsgtOkbYAwF2lUmIdsbMyQhxqxksVK+6fqDwgPuMAE98Agc92nVmS8du9VPsfSNnuXldJghpEG7CwqNxYwo4RaOXQbh1cQ6mlZCk8FrRGBCktCEXj89X5iQd9ed0W9BD47KBpf8Cu630O4K6YkKQueabR2/9WJqOJvlWqfZbEtc7UmjeQV+WCg7R8sAMApAOKgE7g6K2ZO3Zua65/vRb27buj4+gEuGw+IDzRFpurHNmWhQld/32nI9/5FTSAIaAjKyqiBb5jjpaM22A==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB6PR0402MB2760.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(39860400002)(346002)(366004)(136003)(376002)(396003)(6512007)(8936002)(66946007)(66556008)(66476007)(83380400001)(7416002)(6506007)(1076003)(6666004)(38350700002)(38100700002)(6486002)(8676002)(316002)(186003)(52116002)(86362001)(26005)(478600001)(2906002)(2616005)(5660300002)(956004)(4326008);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?YATnlWXZEyM6V2CXuxswoJjV48nrruj0Y+ZKBukC6e/Z2f2vM83GxxT2m3jv?=
 =?us-ascii?Q?Oo8kBB7youEi8uMT8+MnFUrSGJKkaDtMf316CtVntIozuUby94tqpj2O3phf?=
 =?us-ascii?Q?YIUmwrFvZFdI7X9/A1/rkNLMBZBz16WyohlVKsx15jBzosF4KrEzin/fyEUT?=
 =?us-ascii?Q?6gXiG1mDgIRb9+BxTI8B+OHkGbyfiPOu4zG2gy3kWhnPcA3OYVww4IRUzlpY?=
 =?us-ascii?Q?kEIr+P9gIQhCqAspB6YG+H3GvLyXf1BrZp8IzXXpt4KVqwDqxcoJxXrj1Pk/?=
 =?us-ascii?Q?gwptqIaOnC/LVKAuA8Vqt4eYveXlzs0PQOAETnop3npQLQgS5iWyz90oXe1A?=
 =?us-ascii?Q?xbPPYZBKn+Gpemqe6ObNNI2rqpxYlvJQuQ5wDt69rf/dz+75PZV00S1o8ChR?=
 =?us-ascii?Q?/IGxPWT4mTY5q2ELDAJgIN3bJuX7vBgUAqhXPQ/yynm629kAn8L/1k7FoEpf?=
 =?us-ascii?Q?R9DW2B9I/tKgxqnHKyeNnsx8F6NUS4AY4PaN7KYRMItWsgGFEg22RbsIHldp?=
 =?us-ascii?Q?Whckq8SFrlx7M3W9vZpBcAPXkO69+qLqq5htvpijVZ62tUNK3qWidjWU/t30?=
 =?us-ascii?Q?zAdLHRt3jZjzu/bBV5HqwPIHBKiaG+REGhHO1Nz/ijhYVQ0qy8Tnb4OUGO6H?=
 =?us-ascii?Q?+U8oUn99WsfyY5VzThdDiSLxgupn36nFHQaA42b1sdbtlRyRFIpz5eI4M77S?=
 =?us-ascii?Q?0NKSdgnrQWDI3gR3qLmEJNMEaTSbG8daLsBjBYSiwLGu4UALbVI5F0H6tKhw?=
 =?us-ascii?Q?97uATx1K8HV8AmIGdp3brJIR/D51oYE7RKVCo5C7vue4Bf/OL7Hs7fl1Z/Xs?=
 =?us-ascii?Q?b2QkZsHkUyK/e6Mka2W7QdcXJtI7JLFBhjJv3EvFofP1Z3DSVTBtGjc96Y1R?=
 =?us-ascii?Q?xDtNSzPei4XHJlihQnqWeEGG8v1AWAi3oIyRbT/VxtdRCZFyBHEeQeuDmGct?=
 =?us-ascii?Q?OoaMSP14PMyxcqU27mUSXyhGnTrIXYEAxP5RiV698vT470YpC5v7Rdbl36kD?=
 =?us-ascii?Q?ZfKwnD9ZUrcr090M3niFTmiRUQLCGX+69dlXN88OOmX3fK30FPoVZ4crA2tw?=
 =?us-ascii?Q?DKhx6/iL/rzSshd0KJfbsniKR3bywtau5l0RPUc8tqQZ4DQGbvFzr6kwelzk?=
 =?us-ascii?Q?mBGu1Pdi5UlZksu3b16s9UX7N9bZPvrYirHEJXnn58NYxOz9iaGORnF/4Zkf?=
 =?us-ascii?Q?Bg/nSWbUKudjA3Ufyr4uUr8Iut2zJ8ZjjbOnjOmIZKiU4idzHt3PD2+++XbF?=
 =?us-ascii?Q?/GcH6qra6+QhkrmkHSRgcVgTMrBRfktTWmzvQHuFVJ+0108NwI7nYpxluj/J?=
 =?us-ascii?Q?7r+rK3L94vopvneuqM/xR0xz?=
X-OriginatorOrg: oss.nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 00b902b4-0c93-47b2-d470-08d9743a75ec
X-MS-Exchange-CrossTenant-AuthSource: DB6PR0402MB2760.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Sep 2021 09:07:48.9274
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: jK7PBDKDRGRpeTq16i7jWFhO0g3qpW6Q1nV5//MS9lBjsB9aYFT/ayICi8ZaHLin2/RcBfxfQ6adknsGvY2BVw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB7PR04MB4763
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Dong Aisheng <aisheng.dong@nxp.com>

DRAM is not io memory, so changed to ioremap_wc. This is also
aligned with core io accessories. e.g. memcpy/memset and cpu direct
access.

Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Reviewed-and-tested-by: Peng Fan <peng.fan@nxp.com>
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/remoteproc/imx_rproc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/remoteproc/imx_rproc.c b/drivers/remoteproc/imx_rproc.c
index ff620688fad94..4ae416ba50807 100644
--- a/drivers/remoteproc/imx_rproc.c
+++ b/drivers/remoteproc/imx_rproc.c
@@ -597,7 +597,7 @@ static int imx_rproc_addr_init(struct imx_rproc *priv,
 			break;
 
 		/* Not use resource version, because we might share region */
-		priv->mem[b].cpu_addr = devm_ioremap(&pdev->dev, res.start, resource_size(&res));
+		priv->mem[b].cpu_addr = devm_ioremap_wc(&pdev->dev, res.start, resource_size(&res));
 		if (!priv->mem[b].cpu_addr) {
 			dev_err(dev, "failed to remap %pr\n", &res);
 			return -ENOMEM;
-- 
2.25.1