]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
ACPI: TAD: Clear unused RT data in acpi_tad_set_real_time()
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 4 Mar 2026 18:15:21 +0000 (19:15 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Sat, 4 Apr 2026 17:18:57 +0000 (19:18 +0200)
Move the clearing of the fields in struct acpi_tad_rt that are not used
on the real time setting side to acpi_tad_set_real_time().

No intentional functional impact.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Link: https://patch.msgid.link/8660506.T7Z3S40VBb@rafael.j.wysocki
drivers/acpi/acpi_tad.c

index 240e5d53c4a5a754c6b01b7ef83941c53efba7ae..b570e1bda23615c6184de4749a1d20e06ea1ca17 100644 (file)
@@ -96,6 +96,10 @@ static int acpi_tad_set_real_time(struct device *dev, struct acpi_tad_rt *rt)
        if (acpi_tad_rt_is_invalid(rt))
                return -EINVAL;
 
+       rt->valid = 0;
+       rt->msec = 0;
+       memset(rt->padding, 0, 3);
+
        args[0].buffer.pointer = (u8 *)rt;
        args[0].buffer.length = sizeof(*rt);
 
@@ -233,10 +237,6 @@ static ssize_t time_store(struct device *dev, struct device_attribute *attr,
 
        rt.daylight = val;
 
-       rt.valid = 0;
-       rt.msec = 0;
-       memset(rt.padding, 0, 3);
-
        ret = acpi_tad_set_real_time(dev, &rt);
        if (ret)
                return ret;