tegra-nvpmodel: update nvpmodel.service unit file

* Fix the service type to be oneshot, with RemainAfterExit set to true. Trying to execute nvpmodel multiple times leads to spurious failures. * Add a dependency on nvpower.service, following how Jetson Linux does. Evidently the nvpower script now performs some setup the nvpmodel needs, which wasn't true in older L4T versions. Signed-off-by: Matt Madison <[email protected]>
OE4T · Jul 15, 2023 · 1e589d0 · 1e589d0
1 parent 3c459ad
commit 1e589d0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/recipes-bsp/tegra-binaries/tegra-nvpmodel/nvpmodel.service b/recipes-bsp/tegra-binaries/tegra-nvpmodel/nvpmodel.service
@@ -1,13 +1,13 @@
 [Unit]
 Description=NVIDIA power model daemon
-Requires=nvstartup.service
-After=nvstartup.service
+Requires=nvstartup.service nvpower.service
+After=nvstartup.service nvpower.service
 Before=graphical.target display-manager.service
 
 [Service]
-Type=simple
+Type=oneshot
+RemainAfterExit=yes
 ExecStart=/usr/sbin/nvpmodel -f /etc/nvpmodel.conf
-Restart=on-failure
 
 [Install]
 WantedBy=multi-user.target
diff --git a/recipes-bsp/tegra-binaries/tegra-nvpmodel_35.3.1.bb b/recipes-bsp/tegra-binaries/tegra-nvpmodel_35.3.1.bb
@@ -24,4 +24,4 @@ inherit systemd update-rc.d
 INITSCRIPT_NAME = "nvpmodel"
 INITSCRIPT_PARAMS = "defaults"
 SYSTEMD_SERVICE:${PN} = "nvpmodel.service"
-RDEPENDS:${PN} = "tegra-nvpmodel-base"
+RDEPENDS:${PN} = "tegra-nvpmodel-base tegra-nvpower"