Page 1 of 1

Small correction in hal_xsnor_base.c

Posted: Sun Feb 22, 2026 12:41 pm
by wolf
chibios-code-r17707-trunk

Hi all,

compiling hal_xsnor_base.c gives below warning:

Code: Select all

Compiling hal_xsnor_base.c
../../chibios_trunk/os/hal/lib/complex/xsnor/src/hal_xsnor_base.c: In function 'xsnorStart':
../../chibios_trunk/os/hal/lib/complex/xsnor/src/hal_xsnor_base.c:909:10: warning: 'err' may be used uninitialized in this function [-Wmaybe-uninitialized]
  909 |   return err;
      |          ^~~
I think err should be initialized as shown below:

Code: Select all

/**
 * @brief       Configures and activates a SNOR driver.
 *
 * @param[in,out] ip            Pointer to a @p hal_xsnor_base_c instance.
 * @param[in]     config        Pointer to the configuration.
 * @return                      An error code.
 * @retval FLASH_NO_ERROR       Operation successful.
 * @retval FLASH_ERROR_HW_FAILURE If initialization failed.
 *
 * @api
 */
flash_error_t xsnorStart(void *ip, const xsnor_config_t *config) {
  hal_xsnor_base_c *self = (hal_xsnor_base_c *)ip;
  flash_error_t err = FLASH_NO_ERROR;
Best regards,
Wolf

Re: Small correction in hal_xsnor_base.c

Posted: Wed Feb 25, 2026 9:57 am
by Giovanni
Hi,

Fixed in trunk, thanks.

Giovanni