Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Vitor Soares <Vitor.Soares@synopsys.com>
To:     Yangtao Li <tiny.windzz@gmail.com>,
        "bbrezillon@kernel.org" <bbrezillon@kernel.org>,
        "pgaj@cadence.com" <pgaj@cadence.com>
CC:     "linux-i3c@lists.infradead.org" <linux-i3c@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/2] i3c: dw: convert to devm_platform_ioremap_resource
Thread-Topic: [PATCH 1/2] i3c: dw: convert to devm_platform_ioremap_resource
Thread-Index: AQHVvbA5eAuj0awaTkuHsl9XvZifT6fSll7g
Date:   Mon, 30 Dec 2019 13:40:11 +0000
Message-ID: <MN2PR12MB4223E00D8862033751A7DEFFAE270@MN2PR12MB4223.namprd12.prod.outlook.com>
References: <20191228185406.26551-1-tiny.windzz@gmail.com>
In-Reply-To: <20191228185406.26551-1-tiny.windzz@gmail.com>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: synopsys.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 943c6744-c4d7-4179-869f-08d78d2dcb1a
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Dec 2019 13:40:11.4960
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: c33c9f88-1eb7-4099-9700-16013fd9e8aa
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 7b+Qb5Xsp27QJUFbxGEVp+UTK9zEItPDa7U1mx3BsvZ2QEK9VTtjTy3H+gfCQGwtYVX0sgG23HPnEKg/nAz7AA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB3773
X-OriginatorOrg: synopsys.com
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Yangtao,

Thanks for your patch.

From: Yangtao Li <tiny.windzz@gmail.com>
Date: Sat, Dec 28, 2019 at 18:54:05

> Use devm_platform_ioremap_resource() to simplify code.
>=20
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
>  drivers/i3c/master/dw-i3c-master.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>=20
> diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i=
3c-master.c
> index b0ff0e12d84c..7b941e93337f 100644
> --- a/drivers/i3c/master/dw-i3c-master.c
> +++ b/drivers/i3c/master/dw-i3c-master.c
> @@ -1100,15 +1100,13 @@ static const struct i3c_master_controller_ops dw_=
mipi_i3c_ops =3D {
>  static int dw_i3c_probe(struct platform_device *pdev)
>  {
>  	struct dw_i3c_master *master;
> -	struct resource *res;
>  	int ret, irq;
> =20
>  	master =3D devm_kzalloc(&pdev->dev, sizeof(*master), GFP_KERNEL);
>  	if (!master)
>  		return -ENOMEM;
> =20
> -	res =3D platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	master->regs =3D devm_ioremap_resource(&pdev->dev, res);
> +	master->regs =3D devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(master->regs))
>  		return PTR_ERR(master->regs);
> =20
> --=20
> 2.17.1

Acked-by: Vitor Soares <vitor.soares@synopsys.com>

Best regards,
Vitor=20
Soares