Darian Leung
14fe6dcaaf
When using the Force U32 macros in C++, the peripheral structs will not have copy constructors due to them being volatile. Thus, doing temp_reg = reg via typeof() will not work and cause a "ambiguous overload of operator=" error. This commit fixes the macros by reading the reg into a uint32_t value first.
60 lines
2.7 KiB
C
60 lines
2.7 KiB
C
// Copyright 2021 Espressif Systems (Shanghai) PTE LTD
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
#pragma once
|
|
|
|
#define HAL_SWAP16(d) __builtin_bswap16((d))
|
|
#define HAL_SWAP32(d) __builtin_bswap32((d))
|
|
#define HAL_SWAP64(d) __builtin_bswap64((d))
|
|
|
|
/** @cond */ //Doxy command to hide preprocessor definitions from docs */
|
|
|
|
/**
|
|
* @brief Macro to force a 32-bit read, modify, then write on a peripheral register
|
|
*
|
|
* Due to a GCC bug, the compiler may still try to optimize read/writes to peripheral register fields by using 8/16 bit
|
|
* access, even if they are marked volatile (i.e., -fstrict-volatile-bitfields has no effect).
|
|
*
|
|
* For ESP chips, the peripheral bus only allows 32-bit read/writes. The following macro works around the compiler issue
|
|
* by forcing a 32-bit read/modify/write.
|
|
*
|
|
* @note This macro should only be called on register fields of xxx_struct.h type headers, as it depends on the presence
|
|
* of a 'val' field of the register union.
|
|
* @note Current implementation reads into a uint32_t instead of copy base_reg direclty to temp_reg. The reason being
|
|
* that C++ does not create a copy constructor for volatile structs.
|
|
*/
|
|
#define HAL_FORCE_MODIFY_U32_REG_FIELD(base_reg, reg_field, field_val) \
|
|
{ \
|
|
uint32_t temp_val = base_reg.val; \
|
|
typeof(base_reg) temp_reg; \
|
|
temp_reg.val = temp_val; \
|
|
temp_reg.reg_field = (field_val); \
|
|
(base_reg).val = temp_reg.val; \
|
|
}
|
|
|
|
/**
|
|
* @brief Macro to force a 32-bit read on a peripheral register
|
|
*
|
|
* @note This macro should only be called on register fields of xxx_struct.h type headers. See description above for
|
|
* more details.
|
|
* @note Current implementation reads into a uint32_t. See description above for more details.
|
|
*/
|
|
#define HAL_FORCE_READ_U32_REG_FIELD(base_reg, reg_field) ({ \
|
|
uint32_t temp_val = base_reg.val; \
|
|
typeof(base_reg) temp_reg; \
|
|
temp_reg.val = temp_val; \
|
|
temp_reg.reg_field; \
|
|
})
|
|
|
|
/** @endcond */
|