From:   Bernard Zhao <bernard@vivo.com>
To:     Alex Deucher <alexander.deucher@amd.com>,
        =?UTF-8?q?Christian=20K=C3=B6nig?= <christian.koenig@amd.com>,
        "Pan, Xinhui" <Xinhui.Pan@amd.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        Jingwen Chen <Jingwen.Chen2@amd.com>,
        Candice Li <candice.li@amd.com>,
        John Clements <john.clements@amd.com>,
        Monk liu <monk.liu@amd.com>,
        Peng Ju Zhou <PengJu.Zhou@amd.com>,
        Bernard Zhao <bernard@vivo.com>, Jiawei Gu <Jiawei.Gu@amd.com>,
        Bokun Zhang <bokun.zhang@amd.com>,
        Zhigang Luo <zhigang.luo@amd.com>,
        Lee Jones <lee.jones@linaro.org>,
        amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
        linux-kernel@vger.kernel.org
Subject: [PATCH] drm/amd/amdgpu: cleanup the code style a bit
Date:   Sun, 14 Nov 2021 23:07:08 -0800
Message-Id: <20211115070714.7007-1-bernard@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Ul1/6tZrEO7ovvDl74ImVhZzb0+pNVka9DXrrM16PT8Es1AzHPiUJCeP3dY5?=
 =?us-ascii?Q?/GTybKpUiA2CH5eiKvnYm0oQnZ9F8OV5I5jJ27KaotI/nnZ6CqZQCxZST5Tz?=
 =?us-ascii?Q?IooJuIxud49+sfsLwz+VaSHlDs7IMK4Mm+wXG9W80Sfs5fuXZ1Qv9mh3gdQU?=
 =?us-ascii?Q?owGviA/peUM4dgNsWw6MzPZFW6bJFdc8zhYkzdzPT6mdwO2lwp8DWEdgd8xC?=
 =?us-ascii?Q?QpXh0hfJcDK1BDFUCF2n3Rq1VtY5g1iqCmGfeIyPKJjYXPH2Y7+PfU/2tM/x?=
 =?us-ascii?Q?2gxDyBtQ0ur+U5hBuVqUSrz80KaGQX33Gj1e1/Dz6hKdOUY+WWEgCPJcARj3?=
 =?us-ascii?Q?OvKD8fegVqVmBJ04ob8KBnrjO5TmdghXgy4ZDp/YDsxUFD+hob+Sj8Jxfail?=
 =?us-ascii?Q?JibuTHPgl0cXhAgJQUyfMzQzwmbi1/KwnYAjCutwdYN9gk8dSrmq2NIZYknN?=
 =?us-ascii?Q?6CgI62JM/jcz6+fXg5PZohOJU6bEszGaa52hNr5ApzugizhLEKqapM1rWzpY?=
 =?us-ascii?Q?nqtgUayb72lfYobvrM9QJWquDjI5HLCiJ4gPTkqVxLKiQflYtCLZYekx4zB0?=
 =?us-ascii?Q?XPR3N1p4eO5p6KZcbp9vt0UP9jLcxw3+LJDM2cRyTYdRhK+eKb4bAXSF6O27?=
 =?us-ascii?Q?D4JhTeGQltTRfnO+oJcGEYwb7FomiHOtWt7T9FPT8bx6u4XqISnwMWb/DW+I?=
 =?us-ascii?Q?f4yA4K/oHg1v4QqbCRPNvzJtKYsJ1IUTP+sRAFIwuxtx4/Sg4cOqFeXHIxzC?=
 =?us-ascii?Q?TlUBGnt852Vc/mVgGfkDt56roj8HS22Sqfb8yeqVi7SnscIGZ9GGzbQMxMZW?=
 =?us-ascii?Q?3eekL13lC1t9GzAvDzjKCW7w7bRDrKpbeMnojguAKPur+LQTCAoiWrATYNNK?=
 =?us-ascii?Q?DhS4raPvdnLxnkxTg1wG0wRaeNFNHrQv5eH58y8n10iSwQModO+sL+bhN4WN?=
 =?us-ascii?Q?400ArRf/VPf5U9K+p+DJOSMRZ9MncBYqgCdH31sPfiZfMWMj41t7gX4qPW2h?=
 =?us-ascii?Q?fEtROaI+szZp2uv/3ckVsm1m4EJ7UTOAVWByjV8mTgSgpTN8qtZQmwHZLXjc?=
 =?us-ascii?Q?vlsDzHFIJIIKBeJLGGxy/+Fg/WXTzxuIu53+xqYtzXCfxyNLmzWsd5DZUpS5?=
 =?us-ascii?Q?ZLZZiw/HeoKSEgs4OBQE4t4gmXEZIaP5PtVYjc+Qx1K2XXvrn153m8zW6M9n?=
 =?us-ascii?Q?85U2RZTuFAuWu9LyuDdqHwm+7hu4zDGiYDXZUL3xFPs6mnxL3+JSJz1w1x7I?=
 =?us-ascii?Q?jTZPJdz3oc2Tuqf3UaYKZjL5xICwHIVzuwvs5ybls7LY0PkVnoqt029aoko7?=
 =?us-ascii?Q?W03fU8M5ZlxtA8+WcF2iusmH4tfmO7kC8JMpJGVbbAAVzJZ14TNRC3zvsm/4?=
 =?us-ascii?Q?wmsPhkO8AGEI11N7R38StPkwDFv85PByAC/aJUNhEIK0qsCdX7aeyecTK8B9?=
 =?us-ascii?Q?O+ZGSTWWiaP6X0dLHqRrgewf2/ytKOVB0GWIm6ADkwBaaWTDRs+M+DPdELBy?=
 =?us-ascii?Q?K/BQrpYoPsdFsps9saZOOeKBo3TsTRxgqGju+rw12xn79GRYHyNYYHclsTaB?=
 =?us-ascii?Q?LaaEd+aJrJAi6ZAy9Qu6qOmBuKHaJruXvrMVd9uLy0S/Pt6C/cU8U1FJAZdI?=
 =?us-ascii?Q?ROv5p3H6ts6/8y9UHBBRITE=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 22fd5543-cac2-4f24-ab57-08d9a8069453
X-MS-Exchange-CrossTenant-AuthSource: PSAPR06MB4021.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Nov 2021 07:07:26.4921
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: DBC7jJBHu4yF5OvNp0Cr1utNx3sVRz6Xlj3jnZ4BLP/IrZrW8m0DrgCU758gZy3fm22HXkKyjOL9omW+kppK5w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PU1PR06MB2360
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

This change is to cleanup the code style a bit.

Signed-off-by: Bernard Zhao <bernard@vivo.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
index 04cf9b207e62..90070b41136a 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
@@ -286,12 +286,14 @@ static int amdgpu_virt_init_ras_err_handler_data(struct amdgpu_device *adev)
 		return -ENOMEM;
 
 	bps = kmalloc_array(align_space, sizeof((*data)->bps), GFP_KERNEL);
+	if (!bps) {
+		kfree(*data);
+		return -ENOMEM;
+	}
 	bps_bo = kmalloc_array(align_space, sizeof((*data)->bps_bo), GFP_KERNEL);
-
-	if (!bps || !bps_bo) {
-		kfree(bps);
-		kfree(bps_bo);
+	if (!bps_bo) {
 		kfree(*data);
+		kfree(bps);
 		return -ENOMEM;
 	}
 
-- 
2.33.1