From: Antonio Ospite <ao2@ao2.it>
Date: Tue, 27 Feb 2018 17:41:44 +0000 (+0100)
Subject: am7xxx: return EINVAL instead of ENOTSUP in am7xxx_get_device_info()
X-Git-Tag: v0.1.7~16
X-Git-Url: https://git.ao2.it/libam7xxx.git/commitdiff_plain/627f4b2385f08222e7a38dec6066ac0c710a251b?ds=inline

am7xxx: return EINVAL instead of ENOTSUP in am7xxx_get_device_info()

All the other error paths about invalid results or unsupported
operations return EINVAL, so remove the only instance of ENOTSUP.

Since the API documentation makes no promises on the actual negative
value of the returned error, this little change of behavior should be
fine.
---

diff --git a/src/am7xxx.c b/src/am7xxx.c
index 4bf9abb..8573a59 100644
--- a/src/am7xxx.c
+++ b/src/am7xxx.c
@@ -1240,8 +1240,8 @@ AM7XXX_PUBLIC int am7xxx_get_device_info(am7xxx_device *dev,
 	if (h.packet_type != AM7XXX_PACKET_TYPE_DEVINFO) {
 		error(dev->ctx, "expected packet type: %d, got %d instead!\n",
 		      AM7XXX_PACKET_TYPE_DEVINFO, h.packet_type);
-		errno = ENOTSUP;
-		return -ENOTSUP;
+		errno = EINVAL;
+		return -EINVAL;
 	}
 
 	dev->device_info = malloc(sizeof(*dev->device_info));