Merge 63500f8dd0 into cbbe4b41ea

.github/workflows/compile.yml

@@ -75,6 +75,7 @@ jobs:
                 gcc \
                 gcc-aarch64-linux-gnu \
                 gcc-arm-linux-gnueabihf \
+                gcc-mips-linux-gnu \
                 libgps-dev \
                 make
 
@@ -138,6 +139,13 @@ jobs:
         podman exec -i stable cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_TOOLCHAIN_FILE=cmake/${toolchain}.cmake -DENABLE_WERROR=ON -DENABLE_GPS=${gps} -B build-${toolchain}
         podman exec -i stable cmake --build build-${toolchain}
 
+    - name: Configure & Build with mips-linux-gnu-gcc
+      env:
+        toolchain: mips-linux-gnu-gcc
+      run: |
+        podman exec -i stable cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_TOOLCHAIN_FILE=cmake/${toolchain}.cmake -DENABLE_WERROR=ON -B build-${toolchain}
+        podman exec -i stable cmake --build build-${toolchain}
+
     - name: Configure & Build with gcc (Makefile)
       env:
         cc: gcc

canerrsim.c

@@ -25,7 +25,6 @@
 #include <linux/can/error.h>
 #include <linux/can/raw.h>
 #include <net/if.h>
-#include <stdarg.h>
 #include <stdbool.h>
 #include <stdint.h>
 #include <stdio.h>
@@ -123,25 +122,27 @@ void show_help_and_exit()
 	exit(EXIT_SUCCESS);
 }
 
-void __attribute__((format (printf, 1, 2))) err_exit(const char *format, ...)
+void err_exit(const char *msg)
 {
-	va_list ap;
+	printf("%s", msg);
-
-	va_start(ap, format);
-	vfprintf(stdout, format, ap);
-	va_end(ap);
-
 	exit(EXIT_FAILURE);
 }
 
+void show_custom_format_and_exit(const char *param, const char *format)
+{
+	char str_buf[80];
+	sprintf(str_buf, format, param);
+	err_exit(str_buf);
+}
+
 void show_invalid_option(const char *option)
 {
-	err_exit("Error: Invalid option %s\n", option);
+	show_custom_format_and_exit(option, "Error: Invalid option %s\n");
 }
 
 void show_err_and_exit(const char *err_type)
 {
-	err_exit("Error: You can only have one %s parameter!\n", err_type);
+	show_custom_format_and_exit(err_type, "Error: You can only have one %s parameter!\n");
 }
 
 void show_loc_err_and_exit()
@@ -175,6 +176,7 @@ int main(int argc, char *argv[])
 	struct ifreq ifr;
 	struct can_frame frame;
 	bool show_bits = false, location_processed = false, transceiver_processed = false, arbitration_processed = false;
+	char tmp_str[256];
 
 	printf("CAN Sockets Error Messages Simulator\n");
 	if (argc < 3)
@@ -535,25 +537,24 @@ int main(int argc, char *argv[])
 
 	// create socket
 	if ((sock = socket(PF_CAN, SOCK_RAW, CAN_RAW)) < 0)
-		err_exit("Error while opening socket\n");
+		err_exit("Error while opening socket");
 
 	// set interface name
-	if (strlen(argv[1]) >= IFNAMSIZ)
-		err_exit("Name of CAN device '%s' is too long!\n\n", argv[1]);
-
 	strcpy(ifr.ifr_name, argv[1]); // can0, vcan0...
-	if (ioctl(sock, SIOCGIFINDEX, &ifr) < 0)
+	if (ioctl(sock, SIOCGIFINDEX, &ifr) < 0) {
-		err_exit("Error setting CAN interface name %s\n", argv[1]);
+		sprintf(tmp_str, "Error setting CAN interface name %s", argv[1]);
+		err_exit(tmp_str);
+	}
 
 	// bind socket to the CAN interface
 	addr.can_family = AF_CAN;
 	addr.can_ifindex = ifr.ifr_ifindex;
 	if (bind(sock, (struct sockaddr *)&addr, sizeof(addr)) < 0)
-		err_exit("Error in socket bind\n");
+		err_exit("Error in socket bind");
 
 	// Send CAN error frame
 	if (write(sock, &frame, sizeof(frame)) < 0)
-		err_exit("Error writing to socket\n");
+		err_exit("Error writing to socket");
 	else
 		printf("CAN error frame sent\n");
 

mcp251xfd/mcp251xfd-regmap.c

@@ -72,7 +72,8 @@ int mcp251xfd_regmap_read(struct mcp251xfd_priv *priv,
 		return 0;
 
 	/* maybe it's something like "spi0.0" */
-	if (strchr(file_path, '/'))
+	tmp = strchr(file_path, '/');
+	if (tmp)
 		return -ENOENT;
 
 	/* first try literally */