From 5b426d3e0f1f73319c5d423598a7b386611dd9a1 Mon Sep 17 00:00:00 2001 From: Matt Madison Date: Sat, 15 Jul 2023 06:48:22 -0700 Subject: [PATCH] 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 --- .../tegra-binaries/tegra-nvpmodel/nvpmodel.service | 8 ++++---- recipes-bsp/tegra-binaries/tegra-nvpmodel_35.3.1.bb | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/recipes-bsp/tegra-binaries/tegra-nvpmodel/nvpmodel.service b/recipes-bsp/tegra-binaries/tegra-nvpmodel/nvpmodel.service index df975945a6a7..9ecda3bda408 100644 --- 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 index b6fa70700155..bbd76acf247e 100644 --- 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"