Change the Korad IDN to work for more devices

Author: Martin.JM

src/hardware/korad-kaxxxxp/api.c

@@ -281,14 +281,20 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 	sr_dbg("Want max %zu bytes.", len);
 
 	/*
-	 * TODO: check if adding the newline breaks some other devices - I cannot do so
-	 * This is required for the KA3005PS
+	 * There is a quirk rolled into here:
+	 *  - The KORAD_QUIRK_NEWLINE needs a `\n`
+	 * If there's more at the end they still function perfectly fine.
+	 * But the normal ones need an `*IDN?` without anything at the end.
+	 * So this sends a combination that satisfies these constraints.
 	 */
-	ret = korad_kaxxxxp_send_cmd(serial, "*IDN?", true);
+	ret = korad_kaxxxxp_send_cmd(serial, "*IDN?\n*IDN?", false);
 	if (ret < 0)
 		return NULL;
 
-	/* Newline stripping enabled by default - doesn't matter for devices that don't add one */
+	/*
+	 * Newline stripping enabled by default - doesn't matter for devices that
+	 * don't add one
+	 */
 	ret = korad_kaxxxxp_read_chars(serial, len, reply, true);
 	if (ret < 0)
 		return NULL;