power: supply: test-power: implement charge_behaviour property
authorThomas Weißschuh <linux@weissschuh.net>
Wed, 6 Mar 2024 19:37:04 +0000 (20:37 +0100)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Mon, 25 Mar 2024 22:29:06 +0000 (23:29 +0100)
To validate the special formatting of the "charge_behaviour" sysfs
property add it to the example driver.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Link: https://lore.kernel.org/r/20240306-power_supply-charge_behaviour_prop-v3-1-d04cf1f5f0af@weissschuh.net
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/test_power.c

index 0d0a77584c5db60dc9b5998f56ed37e290224b6a..442ceb7795e1d84e34da2801d228d53fb67e08d9 100644 (file)
@@ -35,6 +35,8 @@ static int battery_capacity           = 50;
 static int battery_voltage             = 3300;
 static int battery_charge_counter      = -1000;
 static int battery_current             = -1600;
+static enum power_supply_charge_behaviour battery_charge_behaviour =
+       POWER_SUPPLY_CHARGE_BEHAVIOUR_AUTO;
 
 static bool module_initialized;
 
@@ -123,6 +125,9 @@ static int test_power_get_battery_property(struct power_supply *psy,
        case POWER_SUPPLY_PROP_CURRENT_NOW:
                val->intval = battery_current;
                break;
+       case POWER_SUPPLY_PROP_CHARGE_BEHAVIOUR:
+               val->intval = battery_charge_behaviour;
+               break;
        default:
                pr_info("%s: some properties deliberately report errors.\n",
                        __func__);
@@ -131,6 +136,31 @@ static int test_power_get_battery_property(struct power_supply *psy,
        return 0;
 }
 
+static int test_power_battery_property_is_writeable(struct power_supply *psy,
+                                                   enum power_supply_property psp)
+{
+       return psp == POWER_SUPPLY_PROP_CHARGE_BEHAVIOUR;
+}
+
+static int test_power_set_battery_property(struct power_supply *psy,
+                                          enum power_supply_property psp,
+                                          const union power_supply_propval *val)
+{
+       switch (psp) {
+       case POWER_SUPPLY_PROP_CHARGE_BEHAVIOUR:
+               if (val->intval < 0 ||
+                   val->intval >= BITS_PER_TYPE(typeof(psy->desc->charge_behaviours)) ||
+                   !(BIT(val->intval) & psy->desc->charge_behaviours)) {
+                       return -EINVAL;
+               }
+               battery_charge_behaviour = val->intval;
+               break;
+       default:
+               return -EINVAL;
+       }
+       return 0;
+}
+
 static enum power_supply_property test_power_ac_props[] = {
        POWER_SUPPLY_PROP_ONLINE,
 };
@@ -156,6 +186,7 @@ static enum power_supply_property test_power_battery_props[] = {
        POWER_SUPPLY_PROP_VOLTAGE_NOW,
        POWER_SUPPLY_PROP_CURRENT_AVG,
        POWER_SUPPLY_PROP_CURRENT_NOW,
+       POWER_SUPPLY_PROP_CHARGE_BEHAVIOUR,
 };
 
 static char *test_power_ac_supplied_to[] = {
@@ -178,6 +209,11 @@ static const struct power_supply_desc test_power_desc[] = {
                .properties = test_power_battery_props,
                .num_properties = ARRAY_SIZE(test_power_battery_props),
                .get_property = test_power_get_battery_property,
+               .set_property = test_power_set_battery_property,
+               .property_is_writeable = test_power_battery_property_is_writeable,
+               .charge_behaviours = BIT(POWER_SUPPLY_CHARGE_BEHAVIOUR_AUTO)
+                                  | BIT(POWER_SUPPLY_CHARGE_BEHAVIOUR_INHIBIT_CHARGE)
+                                  | BIT(POWER_SUPPLY_CHARGE_BEHAVIOUR_FORCE_DISCHARGE),
        },
        [TEST_USB] = {
                .name = "test_usb",