Skip to content

Commit

Permalink
fix: Defer auto enabling of kscan sideband.
Browse files Browse the repository at this point in the history
* Now that device init of kscan sideband is in POST_KERNEL stage,
  use a separate SYS_INIT for auto enabling the device so processing
  of early/initial presses from referenced toggle mode kscan devices
  occurs at the proper time during init.
  • Loading branch information
petejohanson committed Oct 8, 2024
1 parent 29b39f2 commit 2e03038
Showing 1 changed file with 13 additions and 8 deletions.
21 changes: 13 additions & 8 deletions app/src/kscan_sideband_behaviors.c
Original file line number Diff line number Diff line change
Expand Up @@ -168,16 +168,11 @@ static int ksbb_init(const struct device *dev) {
return -ENODEV;
}

if (config->auto_enable) {
#if !IS_ENABLED(CONFIG_PM_DEVICE)
kscan_config(config->kscan, &ksbb_inner_kscan_callback);
kscan_enable_callback(config->kscan);
#else
ksbb_pm_action(dev, PM_DEVICE_ACTION_RESUME);
} else {
#if IS_ENABLED(CONFIG_PM_DEVICE)
if (!config->auto_enable) {
pm_device_init_suspended(dev);
#endif
}
#endif

return 0;
}
Expand All @@ -196,6 +191,16 @@ static const struct kscan_driver_api ksbb_api = {
}

#define KSBB_INST(n) \
COND_CODE_1(DT_INST_PROP_OR(n, auto_enable, false), (static int ksbb_auto_enable_##n(void) { \
const struct device *dev = DEVICE_DT_GET(DT_DRV_INST(n)); \
COND_CODE_1(IS_ENABLED(CONFIG_PM_DEVICE), \
(ksbb_pm_action(dev, PM_DEVICE_ACTION_RESUME);), \
(const struct ksbb_config *config = dev->config; \
kscan_config(config->kscan, &ksbb_inner_kscan_callback); \
kscan_enable_callback(config->kscan);)) \
return 0; \
} SYS_INIT(ksbb_auto_enable_##n, APPLICATION, CONFIG_APPLICATION_INIT_PRIORITY);), \
()) \
static struct ksbb_entry entries_##n[] = { \
DT_INST_FOREACH_CHILD_STATUS_OKAY_SEP(n, ENTRY, (, ))}; \
const struct ksbb_config ksbb_config_##n = { \
Expand Down

0 comments on commit 2e03038

Please sign in to comment.