add locks and missing comments

Author: ym

Makefile

@@ -10,7 +10,7 @@ KVM_PKG_NAME := github.com/jetkvm/kvm
 
 BUILDKIT_FLAVOR := arm-rockchip830-linux-uclibcgnueabihf
 BUILDKIT_PATH ?= /opt/jetkvm-native-buildkit
-SKIP_NATIVE_IF_EXISTS ?= 1
+SKIP_NATIVE_IF_EXISTS ?= 0
 
 
 GO_BUILD_ARGS := -tags netgo -tags timetzdata
@@ -111,7 +111,7 @@ dev_release: frontend build_dev
 	rclone copyto bin/jetkvm_app r2://jetkvm-update/app/$(VERSION_DEV)/jetkvm_app
 	rclone copyto bin/jetkvm_app.sha256 r2://jetkvm-update/app/$(VERSION_DEV)/jetkvm_app.sha256
 
-build_release: frontend
+build_release: frontend build_native
 	@echo "Building release..."
 	$(GO_CMD) build \
 		-ldflags="$(GO_LDFLAGS) -X $(KVM_PKG_NAME).builtAppVersion=$(VERSION)" \

dev_deploy.sh

@@ -104,7 +104,8 @@ if [ -z "$REMOTE_HOST" ]; then
 fi
 
 # Build the development version on the host
-if [ "$SKIP_UI_BUILD" = false ]; then
+# When using `make build_release`, the frontend will be built regardless of the `SKIP_UI_BUILD` flag
+if [[ "$SKIP_UI_BUILD" = false && "$INSTALL_APP" = false ]]; then 
     msg_info "▶ Building frontend"
     make frontend
 fi

internal/native/cgo/edid.c

@@ -173,7 +173,7 @@ const char *videoc_log_status()
     }
     else
     {
-        log_error("Failed to read kernel log\n");
+        log_error("Failed to read kernel log");
         return NULL;
     }
 

internal/native/cgo/log_handler.h

@@ -2,8 +2,25 @@
 #define LOG_HANDLER_H
 
 typedef void (jetkvm_log_handler_t)(int level, const char *filename, const char *funcname, const int line, const char *message);
+
+/**
+ * @brief Log a message
+ *
+ * @param level The level of the message
+ * @param filename The filename of the message
+ * @param funcname The function name of the message
+ * @param line The line number of the message
+ * @param message The message to log
+ * @return void
+ */
 void log_message(int level, const char *filename, const char *funcname, const int line, const char *message);
 
+/**
+ * @brief Set the log handler
+ *
+ * @param handler The handler to set
+ * @return void
+ */
 void log_set_handler(jetkvm_log_handler_t *handler);
 
 #endif

internal/native/cgo/screen.c

@@ -188,7 +188,7 @@ lv_img_dsc_t *ui_get_image(const char *name) {
 void ui_set_text(const char *name, const char *text) {
     lv_obj_t *obj = ui_get_obj(name);
     if(obj == NULL) {
-        log_error("ui_set_text %s %s, obj not found\n", name, text);
+        log_error("ui_set_text %s %s, obj not found", name, text);
         return;
     }
     lv_label_set_text(obj, text);

internal/native/cgo/screen.h

@@ -12,10 +12,37 @@ void lvgl_tick(void);
 
 void lvgl_set_rotation(lv_display_t *disp, u_int16_t rotation);
 
+/**
+ * @brief Set the text of an object
+ *
+ * @param name The name of the object
+ * @param text The text to set
+ * @return void
+ */
 void ui_set_text(const char *name, const char *text);
 
+/**
+ * @brief Get the object with the given name
+ *
+ * @param name The name of the object
+ * @return lv_obj_t* The object with the given name
+ */
 lv_obj_t *ui_get_obj(const char *name);
+
+/**
+ * @brief Get the style with the given name
+ *
+ * @param name The name of the style
+ * @return lv_style_t* The style with the given name
+ */
 lv_style_t *ui_get_style(const char *name);
+
+/**
+ * @brief Get the image with the given name
+ *
+ * @param name The name of the image
+ * @return lv_img_dsc_t* The image with the given name
+ */
 lv_img_dsc_t *ui_get_image(const char *name);
 
 #endif // SCREEN_H