Skip to content
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

Jump to bottom

STM32: Add LP_TICKER and US_TICKER optimisation macro #14536

Merged
merged 1 commit into from
May 24, 2021
Merged

STM32: Add LP_TICKER and US_TICKER optimisation macro #14536

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
7 changes: 7 additions & 0 deletions targets/TARGET_STM/device.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,11 @@

#include "objects.h"

#if DEVICE_USTICKER
#include "us_ticker_defines.h"
#endif
#if DEVICE_LPTICKER
#include "lp_ticker_defines.h"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you make this include, or the contents of the header file, conditional on DEVICE_LPTICKER or whatever it is?

I'd rather the macros were left undefined in that case. There may not be any problem in the Mbed OS tree, but it's possible someone might assume LPTICKER is available if LP_TICKER_PERIOD_NUM is defined.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kjbracey-arm
Done

#endif

#endif
41 changes: 41 additions & 0 deletions targets/TARGET_STM/lp_ticker_defines.h
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
/* mbed Microcontroller Library
* SPDX-License-Identifier: BSD-3-Clause
******************************************************************************
*
* Copyright (c) 2015-2021 STMicroelectronics.
* All rights reserved.
*
* This software component is licensed by ST under BSD 3-Clause license,
* the "License"; You may not use this file except in compliance with the
* License. You may obtain a copy of the License at:
* opensource.org/licenses/BSD-3-Clause
*
******************************************************************************
*/

#ifndef __LP_TICKER_DEFINES_H
#define __LP_TICKER_DEFINES_H

#define LP_TICKER_PERIOD_NUM 1000000

#if MBED_CONF_TARGET_LPTICKER_LPTIM

#if MBED_CONF_TARGET_LSE_AVAILABLE
#define LP_TICKER_PERIOD_DEN (LSE_VALUE / MBED_CONF_TARGET_LPTICKER_LPTIM_CLOCK)
#else
#define LP_TICKER_PERIOD_DEN (LSI_VALUE / MBED_CONF_TARGET_LPTICKER_LPTIM_CLOCK)
#endif
#define LP_TICKER_MASK 0xFFFF // 16 bits

#else // MBED_CONF_TARGET_LPTICKER_LPTIM == 0 => Use RTC WAKEUP timer

#if MBED_CONF_TARGET_LSE_AVAILABLE
#define LP_TICKER_PERIOD_DEN (LSE_VALUE / 4) // RTC_CLOCK / RTC_WAKEUPCLOCK_RTCCLK_DIV4
#else
#define LP_TICKER_PERIOD_DEN (LSI_VALUE / 4) // RTC_CLOCK / RTC_WAKEUPCLOCK_RTCCLK_DIV4
#endif
#define LP_TICKER_MASK 0xFFFFFFFF // 32 bits

#endif // MBED_CONF_TARGET_LPTICKER_LPTIM

#endif // __LP_TICKER_DEFINES_H