Re: [PATCH v1] iio: accel: bma400: remove completed tasks from TODO list

On Thu, 13 Aug 2026 11:01:03 -0400
Marco Chen <[email protected]> wrote:

> Two of the four tasks in the TODO comment have been implemented.
> 
> Interrupts were added by commit ffe0ab6a9698 ("iio: accel:
> bma400: Add triggered buffer support"). Support for events was added by
> commit 961db2da159d ("iio: accel: bma400: Add support for single and
> double tap events"), among others. The driver now has an interrupt
> handler pushing IIO events, and event configuration from
> read_event_config(), write_event_config(), read_event_value(), and
> write_event_value().
> 
> A step count channel was also added by commit d221de60eee3 ("iio: accel:
> bma400: Add separate channel for step counter").
> 
> The power management and sensor time channel TODOs remain unimplemented.
> 
> Signed-off-by: Marco Chen <[email protected]>
> ---
> I would like to take on the remaining power management TODO. The driver
> has bma400_set_power_mode() but neither bus assigns a dev_pm_ops
> struct. POWER_MODE_LOW is also never entered. I don't have a BMA400 but
> I would definitely buy one to develop and test on. I also have access to
> a logic analyzer for testing. Is this a patch that is desirable?

Strange that these weren't removed in the commits that introduce said
changes... Anyhow, not sure if this patch on its own is too much churn,
Jonathan and the other usually prefer these to go in a series of more
changes (i.e. if you do the power management TODO, you send this patch
in the series). This is up to Jonathan however.

-- 
Kind regards,
Joshua Crofts