Skip to content

Commit

Permalink
configs/stm32f3discovery: Board support did not conform to standard f…
Browse files Browse the repository at this point in the history
…ile layout and specifically did not include board_initialize.
  • Loading branch information
gregory-nutt committed Jun 25, 2018
1 parent 872a11b commit c4de9f1
Show file tree
Hide file tree
Showing 5 changed files with 203 additions and 85 deletions.
12 changes: 6 additions & 6 deletions configs/stm32f3discovery/src/Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
############################################################################
# configs/stm32f3discovery/src/Makefile
#
# Copyright (C) 2013 Gregory Nutt. All rights reserved.
# Copyright (C) 2013, 2018 Gregory Nutt. All rights reserved.
# Author: Gregory Nutt <gnutt@nuttx.org>
#
# Redistribution and use in source and binary forms, with or without
Expand Down Expand Up @@ -36,7 +36,11 @@
-include $(TOPDIR)/Make.defs

ASRCS =
CSRCS = stm32_boot.c stm32_spi.c
CSRCS = stm32_boot.c stm32_bringup.c stm32_spi.c

ifeq ($(CONFIG_LIB_BOARDCTL),y)
CSRCS += stm32_appinit.c
endif

ifeq ($(CONFIG_ARCH_LEDS),y)
CSRCS += stm32_autoleds.c
Expand All @@ -60,8 +64,4 @@ ifeq ($(CONFIG_SENSORS_QENCODER),y)
CSRCS += stm32_qencoder.c
endif

ifeq ($(CONFIG_LIB_BOARDCTL),y)
CSRCS += stm32_appinit.c
endif

include $(TOPDIR)/configs/Board.mk
80 changes: 10 additions & 70 deletions configs/stm32f3discovery/src/stm32_appinit.c
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
/****************************************************************************
* config/stm32f3discovery/src/stm32_appinit.c
*
* Copyright (C) 2013, 2016 Gregory Nutt. All rights reserved.
* Copyright (C) 2013, 2016, 2018 Gregory Nutt. All rights reserved.
* Author: Gregory Nutt <gnutt@nuttx.org>
*
* Redistribution and use in source and binary forms, with or without
Expand Down Expand Up @@ -39,49 +39,16 @@

#include <nuttx/config.h>

#include <stdbool.h>
#include <stdio.h>
#include <syslog.h>
#include <errno.h>

#include <nuttx/board.h>

#ifdef CONFIG_USBMONITOR
# include <nuttx/usb/usbmonitor.h>
#endif

#include "stm32.h"
#include "stm32f3discovery.h"

/****************************************************************************
* Pre-processor Definitions
****************************************************************************/

/* Configuration ************************************************************/

#define HAVE_USBDEV 1
#define HAVE_USBMONITOR 1

/* Can't support USB device features if the STM32 USB peripheral is not
* enabled.
*/

#ifndef CONFIG_STM32_USB
# undef HAVE_USBDEV
# undef HAVE_USBMONITOR
#endif

/* Can't support USB device is USB device is not enabled */

#ifndef CONFIG_USBDEV
# undef HAVE_USBDEV
# undef HAVE_USBMONITOR
#endif

/* Check if we should enable the USB monitor before starting NSH */

#if !defined(CONFIG_USBDEV_TRACE) || !defined(CONFIG_USBMONITOR)
# undef HAVE_USBMONITOR
#ifndef OK
# define OK 0
#endif

/****************************************************************************
Expand All @@ -100,7 +67,7 @@
* arg - The boardctl() argument is passed to the board_app_initialize()
* implementation without modification. The argument has no
* meaning to NuttX; the meaning of the argument is a contract
* between the board-specific initalization logic and the
* between the board-specific initialization logic and the
* matching application logic. The value cold be such things as a
* mode enumeration value, a set of DIP switch switch settings, a
* pointer to configuration data read from a file or serial FLASH,
Expand All @@ -115,40 +82,13 @@

int board_app_initialize(uintptr_t arg)
{
int ret = OK;

#ifdef HAVE_USBMONITOR
/* Start the USB Monitor */

ret = usbmonitor_start();
if (ret != OK)
{
syslog(LOG_ERR, "ERROR: Failed to start USB monitor: %d\n", ret);
}
#endif
#ifdef CONFIG_BOARD_INITIALIZE
/* Board initialization already performed by board_initialize() */

#ifdef CONFIG_PWM
/* Initialize PWM and register the PWM device. */
return OK;
#else
/* Perform board-specific initialization */

ret = stm32_pwm_setup();
if (ret < 0)
{
syslog(LOG_ERR, "ERROR: stm32_pwm_setup() failed: %d\n", ret);
}
return stm32_bringup();
#endif

#ifdef CONFIG_SENSORS_QENCODER
/* Initialize and register the qencoder driver */

ret = stm32_qencoder_initialize("/dev/qe0", CONFIG_STM32F3DISCO_QETIMER);
if (ret != OK)
{
syslog(LOG_ERR,
"ERROR: Failed to register the qencoder: %d\n",
ret);
return ret;
}
#endif

return ret;
}
31 changes: 23 additions & 8 deletions configs/stm32f3discovery/src/stm32_boot.c
Original file line number Diff line number Diff line change
Expand Up @@ -47,14 +47,6 @@
#include "up_arch.h"
#include "stm32f3discovery.h"

/************************************************************************************
* Pre-processor Definitions
************************************************************************************/

/************************************************************************************
* Private Functions
************************************************************************************/

/************************************************************************************
* Public Functions
************************************************************************************/
Expand Down Expand Up @@ -100,3 +92,26 @@ void stm32_boardinitialize(void)
board_autoled_initialize();
#endif
}


/****************************************************************************
* Name: board_initialize
*
* Description:
* If CONFIG_BOARD_INITIALIZE is selected, then an additional
* initialization call will be performed in the boot-up sequence to a
* function called board_initialize(). board_initialize() will be
* called immediately after up_initialize() is called and just before the
* initial application is started. This additional initialization phase
* may be used, for example, to initialize board-specific device drivers.
*
****************************************************************************/

#ifdef CONFIG_BOARD_INITIALIZE
void board_initialize(void)
{
/* Perform board-specific initialization */

(void)stm32_bringup();
}
#endif
143 changes: 143 additions & 0 deletions configs/stm32f3discovery/src/stm32_bringup.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,143 @@
/****************************************************************************
* config/stm32f3discovery/src/stm32_bringup.c
*
* Copyright (C) 2013, 2016, 2018 Gregory Nutt. All rights reserved.
* Author: Gregory Nutt <gnutt@nuttx.org>
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
* are met:
*
* 1. Redistributions of source code must retain the above copyright
* notice, this list of conditions and the following disclaimer.
* 2. Redistributions in binary form must reproduce the above copyright
* notice, this list of conditions and the following disclaimer in
* the documentation and/or other materials provided with the
* distribution.
* 3. Neither the name NuttX nor the names of its contributors may be
* used to endorse or promote products derived from this software
* without specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
* FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
* COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
* INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
* BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS
* OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
* AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
* ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*
****************************************************************************/

/****************************************************************************
* Included Files
****************************************************************************/

#include <nuttx/config.h>

#include <stdbool.h>
#include <stdio.h>
#include <syslog.h>
#include <errno.h>

#include <nuttx/board.h>

#ifdef CONFIG_USBMONITOR
# include <nuttx/usb/usbmonitor.h>
#endif

#include "stm32.h"
#include "stm32f3discovery.h"

/****************************************************************************
* Pre-processor Definitions
****************************************************************************/

/* Configuration ************************************************************/

#define HAVE_USBDEV 1
#define HAVE_USBMONITOR 1

/* Can't support USB device features if the STM32 USB peripheral is not
* enabled.
*/

#ifndef CONFIG_STM32_USB
# undef HAVE_USBDEV
# undef HAVE_USBMONITOR
#endif

/* Can't support USB device is USB device is not enabled */

#ifndef CONFIG_USBDEV
# undef HAVE_USBDEV
# undef HAVE_USBMONITOR
#endif

/* Check if we should enable the USB monitor before starting NSH */

#if !defined(CONFIG_USBDEV_TRACE) || !defined(CONFIG_USBMONITOR)
# undef HAVE_USBMONITOR
#endif

/****************************************************************************
* Public Functions
****************************************************************************/

/****************************************************************************
* Name: stm32_bringup
*
* Description:
* Perform architecture-specific initialization
*
* CONFIG_BOARD_INITIALIZE=y :
* Called from board_initialize().
*
* CONFIG_BOARD_INITIALIZE=y && CONFIG_LIB_BOARDCTL=y :
* Called from the NSH library
*
****************************************************************************/

int stm32_bringup(void)
{
int ret = OK;

#ifdef HAVE_USBMONITOR
/* Start the USB Monitor */

ret = usbmonitor_start();
if (ret != OK)
{
syslog(LOG_ERR, "ERROR: Failed to start USB monitor: %d\n", ret);
}
#endif

#ifdef CONFIG_PWM
/* Initialize PWM and register the PWM device. */

ret = stm32_pwm_setup();
if (ret < 0)
{
syslog(LOG_ERR, "ERROR: stm32_pwm_setup() failed: %d\n", ret);
}
#endif

#ifdef CONFIG_SENSORS_QENCODER
/* Initialize and register the qencoder driver */

ret = stm32_qencoder_initialize("/dev/qe0", CONFIG_STM32F3DISCO_QETIMER);
if (ret != OK)
{
syslog(LOG_ERR,
"ERROR: Failed to register the qencoder: %d\n",
ret);
return ret;
}
#endif

return ret;
}
22 changes: 21 additions & 1 deletion configs/stm32f3discovery/src/stm32f3discovery.h
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,13 @@
****************************************************************************************************/

#include <nuttx/config.h>
#include <nuttx/compiler.h>

#include <stdint.h>

#include <nuttx/compiler.h>

#include <arch/stm32/chip.h>

/****************************************************************************************************
* Pre-processor Definitions
****************************************************************************************************/
Expand Down Expand Up @@ -138,6 +142,22 @@
* Public Functions
****************************************************************************************************/

/****************************************************************************************************
* Name: stm32_bringup
*
* Description:
* Perform architecture-specific initialization
*
* CONFIG_BOARD_INITIALIZE=y :
* Called from board_initialize().
*
* CONFIG_BOARD_INITIALIZE=y && CONFIG_LIB_BOARDCTL=y :
* Called from the NSH library
*
****************************************************************************************************/

int stm32_bringup(void);

/****************************************************************************************************
* Name: stm32_spidev_initialize
*
Expand Down

0 comments on commit c4de9f1

Please sign in to comment.