Commit c566d299 authored by Daniel Drake's avatar Daniel Drake Committed by Anton Vorontsov

olpc_battery: Ambient temperature is not available on XO-1.5

The XO-1.5 does not support the ambient temperature property.
Create a separate list of properties for that configuration where
ambient temperature is not included, and apply the correct property
list at runtime.
Signed-off-by: default avatarDaniel Drake <dsd@laptop.org>
Signed-off-by: default avatarAnton Vorontsov <cbouatmailru@gmail.com>
parent f3dcab70
...@@ -331,7 +331,7 @@ static int olpc_bat_get_property(struct power_supply *psy, ...@@ -331,7 +331,7 @@ static int olpc_bat_get_property(struct power_supply *psy,
return ret; return ret;
} }
static enum power_supply_property olpc_bat_props[] = { static enum power_supply_property olpc_xo1_bat_props[] = {
POWER_SUPPLY_PROP_STATUS, POWER_SUPPLY_PROP_STATUS,
POWER_SUPPLY_PROP_CHARGE_TYPE, POWER_SUPPLY_PROP_CHARGE_TYPE,
POWER_SUPPLY_PROP_PRESENT, POWER_SUPPLY_PROP_PRESENT,
...@@ -348,6 +348,23 @@ static enum power_supply_property olpc_bat_props[] = { ...@@ -348,6 +348,23 @@ static enum power_supply_property olpc_bat_props[] = {
POWER_SUPPLY_PROP_CHARGE_COUNTER, POWER_SUPPLY_PROP_CHARGE_COUNTER,
}; };
/* XO-1.5 does not have ambient temperature property */
static enum power_supply_property olpc_xo15_bat_props[] = {
POWER_SUPPLY_PROP_STATUS,
POWER_SUPPLY_PROP_CHARGE_TYPE,
POWER_SUPPLY_PROP_PRESENT,
POWER_SUPPLY_PROP_HEALTH,
POWER_SUPPLY_PROP_TECHNOLOGY,
POWER_SUPPLY_PROP_VOLTAGE_AVG,
POWER_SUPPLY_PROP_CURRENT_AVG,
POWER_SUPPLY_PROP_CAPACITY,
POWER_SUPPLY_PROP_CAPACITY_LEVEL,
POWER_SUPPLY_PROP_TEMP,
POWER_SUPPLY_PROP_MANUFACTURER,
POWER_SUPPLY_PROP_SERIAL_NUMBER,
POWER_SUPPLY_PROP_CHARGE_COUNTER,
};
/* EEPROM reading goes completely around the power_supply API, sadly */ /* EEPROM reading goes completely around the power_supply API, sadly */
#define EEPROM_START 0x20 #define EEPROM_START 0x20
...@@ -419,8 +436,6 @@ static struct device_attribute olpc_bat_error = { ...@@ -419,8 +436,6 @@ static struct device_attribute olpc_bat_error = {
static struct platform_device *bat_pdev; static struct platform_device *bat_pdev;
static struct power_supply olpc_bat = { static struct power_supply olpc_bat = {
.properties = olpc_bat_props,
.num_properties = ARRAY_SIZE(olpc_bat_props),
.get_property = olpc_bat_get_property, .get_property = olpc_bat_get_property,
.use_for_apm = 1, .use_for_apm = 1,
}; };
...@@ -466,6 +481,13 @@ static int __init olpc_bat_init(void) ...@@ -466,6 +481,13 @@ static int __init olpc_bat_init(void)
goto ac_failed; goto ac_failed;
olpc_bat.name = bat_pdev->name; olpc_bat.name = bat_pdev->name;
if (olpc_board_at_least(olpc_board_pre(0xd0))) { /* XO-1.5 */
olpc_bat.properties = olpc_xo15_bat_props;
olpc_bat.num_properties = ARRAY_SIZE(olpc_xo15_bat_props);
} else { /* XO-1 */
olpc_bat.properties = olpc_xo1_bat_props;
olpc_bat.num_properties = ARRAY_SIZE(olpc_xo1_bat_props);
}
ret = power_supply_register(&bat_pdev->dev, &olpc_bat); ret = power_supply_register(&bat_pdev->dev, &olpc_bat);
if (ret) if (ret)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment