From 8eab233dbab1289b5b7c2f4fd12a322e7440cd6d Mon Sep 17 00:00:00 2001 From: Mahesh Kallelil Date: Tue, 21 Mar 2023 15:55:48 -0700 Subject: [PATCH] Assign owner/group for cp_temp after the kernel module is loaded. The chown operation on the cp_temp sysfs attribute needs to happen after the cp_thermal_zone driver has been loaded by the insmod_sh service. If not, the owner/group won't be updated and after the driver gets loaded, cp_temp will default to root/root. Hence, we are moving this operation from init to the post-fs-data stage. Test: Verified owner/group for the sysfs attribute Bug: 267485434 Change-Id: Ib463dd5f2de332164714a77bdb0302afeaad5217 Signed-off-by: Mahesh Kallelil --- conf/init.zuma.rc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/conf/init.zuma.rc b/conf/init.zuma.rc index acd42e0f..bf3eddff 100644 --- a/conf/init.zuma.rc +++ b/conf/init.zuma.rc @@ -223,9 +223,6 @@ on init chown system system /dev/battery_history chmod 0644 /dev/battery_history - # Modem temperature driver - chown radio system /sys/devices/platform/cp-tm1/cp_temp - # Loading common kernel modules in background start init_display @@ -347,6 +344,9 @@ on late-fs mount_all --late on post-fs-data + # Modem temperature driver + chown radio system /sys/devices/platform/cp-tm1/cp_temp + # Log data folder mkdir /data/vendor 0771 radio system mkdir /data/vendor/log 0771 radio system