From: Seung-Woo Kim <sw0312.kim@samsung.com>
Date: Mon, 15 May 2023 03:39:38 +0000 (+0900)
Subject: maru: Remove useless null checks for array address
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=91c37dc9a2a38bc426af5e8ec18bafdfa5bcadb3;p=sdk%2Femulator%2Femulator-kernel.git

maru: Remove useless null checks for array address

The array address is always not null, so no need to check. Remove
useless null checks to fix -Waddress build warnings.

Change-Id: I3bef4644e37d8ed4a2a88fb21408ac1b4ff7a64b
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
---

diff --git a/drivers/maru/maru_jack.c b/drivers/maru/maru_jack.c
index abfecf0ecf4c..7cc8c194eb04 100644
--- a/drivers/maru/maru_jack.c
+++ b/drivers/maru/maru_jack.c
@@ -144,10 +144,6 @@ static void jack_vq_done(struct virtqueue *vq) {
 		DLOG(KERN_DEBUG, "receive queue- not an answer message: %d", msg->req);
 		return;
 	}
-	if (msg->buf == NULL) {
-		DLOG(KERN_ERR, "receive queue- message from host is NULL.");
-		return;
-	}
 
 	DLOG(KERN_DEBUG, "msg buf: %s, req: %d, type: %d", msg->buf, msg->req, msg->type);
 
diff --git a/drivers/maru/maru_power_supply.c b/drivers/maru/maru_power_supply.c
index b9535136edc9..0fb94652c9af 100644
--- a/drivers/maru/maru_power_supply.c
+++ b/drivers/maru/maru_power_supply.c
@@ -120,10 +120,6 @@ static void power_vq_done(struct virtqueue *vq) {
 		DLOG(KERN_DEBUG, "receive queue- not an answer message: %d", msg->req);
 		return;
 	}
-	if (msg->buf == NULL) {
-		DLOG(KERN_ERR, "receive queue- message from host is NULL.");
-		return;
-	}
 
 	DLOG(KERN_DEBUG, "msg buf: %s, req: %d, type: %d", msg->buf, msg->req, msg->type);
 
diff --git a/drivers/maru/sensors/maru_virtio_sensor.c b/drivers/maru/sensors/maru_virtio_sensor.c
index 33d6157f5192..60117c3dd692 100644
--- a/drivers/maru/sensors/maru_virtio_sensor.c
+++ b/drivers/maru/sensors/maru_virtio_sensor.c
@@ -160,11 +160,6 @@ static void sensor_vq_done(struct virtqueue *rvq) {
 		return;
 	}
 
-	if (msg->buf == NULL) {
-		ERR("receive queue- message from host is NULL.");
-		return;
-	}
-
 	mutex_lock(&vs->lock);
 	LOG(1, "msg buf: %s, req: %d, type: %d, vs->flags: %d", msg->buf, msg->req, msg->type, vs->flags);