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.
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
drivers: kscan: mchp: low power mode enabled #60806
drivers: kscan: mchp: low power mode enabled #60806
Conversation
@albertofloyd could you please help to review? |
@jvasanth1 @albertofloyd will you please take a look? |
drivers/kscan/kscan_mchp_xec.c
Outdated
@@ -411,6 +418,9 @@ void polling_task(const struct device *dev, void *dummy2, void *dummy3) | |||
/* Allow other threads to run while we sleep */ | |||
k_usleep(wait_period); | |||
} | |||
#ifdef CONFIG_PM_DEVICE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit, do we need to space ifdef like this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, removed the space before #ifdef.
@Manimaran-A could you rebase please? |
pinctrl and dts updated to support low power feature Signed-off-by: Manimaran A <manimaran.a@microchip.com>
KSCAN driver updated to support low power feature Signed-off-by: Manimaran A <manimaran.a@microchip.com>
8207b3d
96b2d05
to
8207b3d
Compare
@albertofloyd @jvasanth1 Could you please re-approve the PR |
Updated KSCAN driver and dts files to support low power feature.