Make it compile on Linux 6.16 and improve performance

Makefile

@@ -2,18 +2,29 @@ ms912x-y := \
 	ms912x_registers.o \
 	ms912x_connector.o \
 	ms912x_transfer.o \
-	ms912x_drv.o
+	ms912x_drv.o 
 
 obj-m := ms912x.o
 
 KVER ?= $(shell uname -r)
 KSRC ?= /lib/modules/$(KVER)/build
+CURDIR := $(shell pwd)
+
+USE_SSE2 ?= 0
+
+ifeq ($(USE_SSE2),1)
+	CFLAGS_EXTRA += -msse2
+endif
+
+EXTRA_CFLAGS += $(CFLAGS_EXTRA)
 
 all:	modules
 
 modules:
-	make CHECK="/usr/bin/sparse" -C $(KSRC) M=$(PWD) modules
+	$(MAKE) CHECK="/usr/bin/sparse" -C $(KSRC) M=$(CURDIR) modules
 
 clean:
-	make -C $(KSRC) M=$(PWD) clean
-	rm -f $(PWD)/Module.symvers $(PWD)/*.ur-safe
+	$(MAKE) -C $(KSRC) M=$(CURDIR) clean
+	rm -f $(CURDIR)/Module.symvers $(CURDIR)/*.ur-safe
+
+.PHONY: all modules clean

README.md

@@ -6,7 +6,13 @@ There are two variants:
  - VID/PID is 534d:6021. Device is USB 2
  - VID/PID is 345f:9132. Device is USB 3
 
-For kernel 6.1 checkout branch kernel-6.1
+- Improved performance and Driver adaptation for Linux kernel 6.15. by Andrey Rodríguez Araya
+
+- For kernel 6.16 checkout branch kernel-6.16
+- For kernel 6.15 checkout branch kernel-6.15
+- For kernel 6.12 checkout branch kernel-6.12
+- For kernel 6.11 ?
+- For kernel 6.8 - 6.10 checkout branch kernel-6.8
 
 TODOs:
 
@@ -23,3 +29,12 @@ Driver is written by analyzing wireshark captures of the device.
 
 Run `sudo dkms install .`
 
+- make clean
+- make all -j
+- sudo rmmod ms912x # It will not work if the device is in use.
+- sudo modprobe drm_shmem_helper
+- sudo insmod ms912x.ko
+
+Forked From: https://github.com/rhgndf/ms912x
+
+

ms912x.h

@@ -23,6 +23,7 @@
 
 struct ms912x_usb_request {
 	void *transfer_buffer;
+	void *temp_buffer;
 	struct ms912x_device *ms912x;
 	size_t transfer_len;
 	size_t alloc_len;
@@ -40,14 +41,15 @@ struct ms912x_device {
 
 	struct drm_connector connector;
 	struct drm_simple_display_pipe display_pipe;
-	
+
 	struct drm_rect update_rect;
 
 	/* Double buffer to allow memcpy and transfer 
 	 * to happen in parallel
 	 */
 	int current_request;
 	struct ms912x_usb_request requests[2];
+	unsigned long last_send_jiffies;
 };
 
 struct ms912x_request {
@@ -116,4 +118,5 @@ int ms912x_fb_send_rect(struct drm_framebuffer *fb, const struct iosys_map *map,
 void ms912x_free_request(struct ms912x_usb_request *request);
 int ms912x_init_request(struct ms912x_device *ms912x,
 			struct ms912x_usb_request *request, size_t len);
-#endif
+void ms912x_init_yuv_lut(void);
+#endif

ms912x_connector.c

@@ -1,4 +1,3 @@
-
 #include <drm/drm_atomic_state_helper.h>
 #include <drm/drm_connector.h>
 #include <drm/drm_edid.h>
@@ -7,35 +6,77 @@
 
 #include "ms912x.h"
 
-static int ms912x_read_edid(void *data, u8 *buf, unsigned int block, size_t len)
+static int ms912x_read_edid_block(struct ms912x_device *ms912x, u8 *buf,
+				  unsigned int offset, size_t len)
 {
-	struct ms912x_device *ms912x = data;
-	int offset = block * EDID_LENGTH;
-	int i, ret;
-	for (i = 0; i < len; i++) {
-		u16 address = 0xc000 + offset + i;
-		ret = ms912x_read_byte(ms912x, address);
+	const u16 base = 0xc000 + offset;
+	for (size_t i = 0; i < len; i++) {
+		u16 address = base + i;
+		int ret = ms912x_read_byte(ms912x, address);
 		if (ret < 0)
 			return ret;
 		buf[i] = ret;
 	}
 	return 0;
 }
 
+static int ms912x_read_edid(void *data, u8 *buf, unsigned int block, size_t len)
+{
+	struct ms912x_device *ms912x = data;
+	const int offset = block * EDID_LENGTH;
+	int ret = ms912x_read_edid_block(ms912x, buf, offset, len);
+	if (ret < 0)
+		pr_err("ms912x: failed to read EDID block %u\n", block);
+	return ret;
+}
+
+static void ms912x_add_fallback_mode(struct drm_connector *connector)
+{
+	struct drm_display_mode *mode;
+
+	mode = drm_mode_create(connector->dev);
+	if (!mode)
+		return;
+
+	drm_mode_set_name(mode);
+	mode->type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED;
+
+	mode->clock = 65000;
+	mode->hdisplay = 1024;
+	mode->hsync_start = 1048;
+	mode->hsync_end = 1184;
+	mode->htotal = 1344;
+	mode->vdisplay = 768;
+	mode->vsync_start = 771;
+	mode->vsync_end = 777;
+	mode->vtotal = 806;
+	mode->flags = DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC;
+
+	drm_mode_probed_add(connector, mode);
+}
+
 static int ms912x_connector_get_modes(struct drm_connector *connector)
 {
-	int ret;
+	int ret = 0;
 	struct ms912x_device *ms912x = to_ms912x(connector->dev);
 	const struct drm_edid *edid;
+
 	edid = drm_edid_read_custom(connector, ms912x_read_edid, ms912x);
-	if (!edid)
-		return 0;
+	if (!edid) {
+		pr_warn("ms912x: EDID not found, falling back to default mode\n");
+		ms912x_add_fallback_mode(connector);
+		return 1;
+	}
+
 	ret = drm_edid_connector_update(connector, edid);
 	if (ret < 0) {
+		pr_err("ms912x: failed to update EDID connector\n");
 		ret = 0;
 		goto edid_free;
 	}
+
 	ret = drm_edid_connector_add_modes(connector);
+
 edid_free:
 	drm_edid_free(edid);
 	return ret;
@@ -47,12 +88,15 @@ static enum drm_connector_status ms912x_detect(struct drm_connector *connector,
 	struct ms912x_device *ms912x = to_ms912x(connector->dev);
 	int status = ms912x_read_byte(ms912x, 0x32);
 
-	if (status < 0)
+	if (status < 0) {
+		pr_err("ms912x: failed to detect HDMI status\n");
 		return connector_status_unknown;
+	}
 
-	return status == 1 ? connector_status_connected :
-			     connector_status_disconnected;
+	return (status == 1) ? connector_status_connected :
+			       connector_status_disconnected;
 }
+
 static const struct drm_connector_helper_funcs ms912x_connector_helper_funcs = {
 	.get_modes = ms912x_connector_get_modes,
 };
@@ -69,12 +113,20 @@ static const struct drm_connector_funcs ms912x_connector_funcs = {
 int ms912x_connector_init(struct ms912x_device *ms912x)
 {
 	int ret;
+
 	drm_connector_helper_add(&ms912x->connector,
 				 &ms912x_connector_helper_funcs);
+
 	ret = drm_connector_init(&ms912x->drm, &ms912x->connector,
 				 &ms912x_connector_funcs,
 				 DRM_MODE_CONNECTOR_HDMIA);
+	if (ret) {
+		pr_err("ms912x: failed to initialize connector\n");
+		return ret;
+	}
+
 	ms912x->connector.polled =
 		DRM_CONNECTOR_POLL_CONNECT | DRM_CONNECTOR_POLL_DISCONNECT;
-	return ret;
+
+	return 0;
 }