STMicroelectronics / STM32CubeG0

STM32Cube MCU Full Package for the STM32G0 series - (HAL + LL Drivers, CMSIS Core, CMSIS Device, MW libraries plus a set of Projects running on all boards provided by ST (Nucleo, Evaluation and Discovery Kits))
Other
165 stars 75 forks source link

Naming inconsistency for Flash Busy flag #5

Closed psarvesh closed 4 years ago

psarvesh commented 4 years ago

Caution The Issues are strictly limited for the reporting of problem encountered with the software provided in this project. For any other problem related to the STM32 product, the performance, the hardware characteristics and boards, the tools the environment in general, please post a topic in the ST Community/STM32 MCUs forum

Describe the set-up Discovered an inconsistency in naming when making a pull request in some other project (https://github.com/zephyrproject-rtos/zephyr/pull/20754).

The FLASH_SR_BSY1 flag for flash in STM32CubeG0 package should be called FLASH_SR_BSY to maintain consistency with other micros in STM32 series

ASELSTM commented 4 years ago

Hi Sarvesh,

The issue you pointed out has been communicated to our development teams. It will be investigated soon. We will be back to you as soon as we get an answer. Thank you for your patience.

With regards,

ALABSTM commented 4 years ago

Hi Sarvesh,

Thank you for having pointed out this aspect. Actually, the flag has been named FLASH_SR_BSY1 to be compliant with the reference manual.

However, it would enhance portability if this flag is also aliased FLASH_SR_BSY. This will be done in a future release. We cannot share a date for the moment but we hope this is done soon enough.

With regards,

ALABSTM commented 4 years ago

ST Internal Reference: 82764

ALABSTM commented 4 years ago

Hi Sarvesh,

I hope you are doing well. Your request has been submitted to our technical committee who said it is more appropriate to keep the flag name unchanged (i.e. FLASH_SR_BSY1).

Indeed, beyond the fact this name is compliant with the reference manual, it shall be left as it is for the need of a future use.

Unfortunately, your request could not be addressed this time. Thank you for your comprehension and thank you again for having reported this point.

Please, allow me to close this issue now. Take care and stay safe.

With regards,