Skip to content

add shutdown support to cirque (to save lots of power in deep sleep) #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
geeksville wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add shutdown support to cirque (to save lots of power in deep sleep) #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 49 additions & 2 deletions drivers/input/input_pinnacle.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -421,6 +421,51 @@ int pinnacle_set_sleep(const struct device *dev, bool enabled) {
return ret;
}


int pinnacle_set_shutdown(const struct device *dev, bool enabled) {
uint8_t sys_cfg;
int ret = pinnacle_seq_read(dev, PINNACLE_SYS_CFG, &sys_cfg, 1);
if (ret < 0) {
LOG_ERR("can't read sys config %d", ret);
return ret;
}

if (((sys_cfg & PINNACLE_SYS_CFG_SHUTDOWN) != 0) == enabled) {
LOG_WRN("Shutdown already set to %s", (enabled ? "on" : "off"));
}

LOG_DBG("Setting shutdown: %s", (enabled ? "on" : "off"));

if(enabled) {
// interrupt pin might have bogus asserts in shtdown so disable ints here
// FIXME if this is in things are fucked later
set_int(dev, false);
}

WRITE_BIT(sys_cfg, PINNACLE_SYS_CFG_SHUTDOWN_BIT, enabled ? 1 : 0);

ret = pinnacle_write(dev, PINNACLE_SYS_CFG, sys_cfg);
if (ret < 0) {
LOG_ERR("can't write shutdown config %d", ret);
return ret;
}

if (!enabled) {
// pinnacle_clear_status(dev); // clear any spurious ints on wake
ret = set_int(dev, true);
}
else {
ret = pinnacle_seq_read(dev, PINNACLE_SYS_CFG, &sys_cfg, 1);
if (ret < 0) {
LOG_ERR("can't read sys config %d", ret);
return ret;
}
LOG_DBG("Shutdown readback: %s", (sys_cfg & PINNACLE_SYS_CFG_SHUTDOWN) ? "on" : "off");
}

return ret;
}

static int pinnacle_init(const struct device *dev) {
struct pinnacle_data *data = dev->data;
const struct pinnacle_config *config = dev->config;
Expand Down Expand Up @@ -549,9 +594,11 @@ static int pinnacle_init(const struct device *dev) {
static int pinnacle_pm_action(const struct device *dev, enum pm_device_action action) {
switch (action) {
case PM_DEVICE_ACTION_SUSPEND:
return set_int(dev, false);
pinnacle_set_shutdown(dev, true);
return 0;
case PM_DEVICE_ACTION_RESUME:
return set_int(dev, true);
pinnacle_set_shutdown(dev, false);
return 0;
default:
return -ENOTSUP;
}
Expand Down
2 changes: 2 additions & 0 deletions drivers/input/input_pinnacle.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
#define PINNACLE_SYS_CFG 0x03 // Contains system operation and configuration bits.
#define PINNACLE_SYS_CFG_EN_SLEEP_BIT 2
#define PINNACLE_SYS_CFG_EN_SLEEP BIT(2)
#define PINNACLE_SYS_CFG_SHUTDOWN_BIT 1
#define PINNACLE_SYS_CFG_SHUTDOWN BIT(1)
#define PINNACLE_SYS_CFG_RESET BIT(0)

Expand Down Expand Up @@ -107,3 +108,4 @@ struct pinnacle_config {
};

int pinnacle_set_sleep(const struct device *dev, bool enabled);
int pinnacle_set_shutdown(const struct device *dev, bool enabled);