diff options
author | Gregory Nutt <gnutt@nuttx.org> | 2014-05-14 10:56:03 -0600 |
---|---|---|
committer | Gregory Nutt <gnutt@nuttx.org> | 2014-05-14 10:56:03 -0600 |
commit | 347f9563aaf31fc4589913bbffda1108235475e9 (patch) | |
tree | 8a12f5f34a0ae6d8709e414f804620fc23bb42f5 | |
parent | f956d9f53487d2ab9a7d0cf12e3b5ce9dc14b2aa (diff) | |
download | nuttx-347f9563aaf31fc4589913bbffda1108235475e9.tar.gz nuttx-347f9563aaf31fc4589913bbffda1108235475e9.tar.bz2 nuttx-347f9563aaf31fc4589913bbffda1108235475e9.zip |
SAM4E-EK: Ooops.. two touchscreen drivers(?)
-rw-r--r-- | nuttx/arch/arm/src/stm32/Kconfig | 1 | ||||
-rw-r--r-- | nuttx/configs/sam4e-ek/README.txt | 8 | ||||
-rw-r--r-- | nuttx/configs/sam4e-ek/src/Makefile | 4 | ||||
-rw-r--r-- | nuttx/configs/sam4e-ek/src/sam_touchscreen.c | 294 |
4 files changed, 7 insertions, 300 deletions
diff --git a/nuttx/arch/arm/src/stm32/Kconfig b/nuttx/arch/arm/src/stm32/Kconfig index ba9c4ec65..47d3a93b4 100644 --- a/nuttx/arch/arm/src/stm32/Kconfig +++ b/nuttx/arch/arm/src/stm32/Kconfig @@ -3056,6 +3056,7 @@ config STM32_I2C_ALT bool "Alternate I2C implementation" default n if !STM32_PERFORMANCELINE default y if STM32_PERFORMANCELINE + depends on !STM32_STM32F30XX ---help--- This selection enables an alternative I2C driver. This alternate driver implements some rather complex workarounds for errata against diff --git a/nuttx/configs/sam4e-ek/README.txt b/nuttx/configs/sam4e-ek/README.txt index f4f30db62..3abf854a7 100644 --- a/nuttx/configs/sam4e-ek/README.txt +++ b/nuttx/configs/sam4e-ek/README.txt @@ -889,6 +889,8 @@ Touchscreen CONFIG_DEBUG_VERBOSE=y : Enable verbose debug output CONFIG_DEBUG_INPUT=y : Enable debug output from input devices + STATUS: Verified 2014-05-14 + ILI9325-Based LCD ================= @@ -1379,6 +1381,8 @@ Configurations CONFIG_DEBUG_VERBOSE=y : Enable verbose debug output CONFIG_DEBUG_INPUT=y : Enable debug output from input devices + STATUS: Verified 2014-05-14 + 11. This configuration can be re-configured to test the on-board LCD module. @@ -1387,7 +1391,6 @@ Configurations Device Drivers -> LCD Driver Support CONFIG_LCD=y : Enable LCD support - CONFIG_NX_LCDDRIVER=y : LCD graphics device CONFIG_LCD_MAXCONTRAST=1 : Value should not matter CONFIG_LCD_MAXPOWER=64 : Must be > 16 CONFIG_LCD_LANDSCAPE=y : Landscape orientation @@ -1397,7 +1400,8 @@ Configurations CONFIG_SAM4EEK_LCD_BGCOLOR=0x00 : Initial background color Graphics Support - CONFIG_NX=y + CONFIG_NX=y : Enable Graphics support + CONFIG_NX_LCDDRIVER=y : LCD graphics device Graphics Support -> Supported Pixel Depths CONFIG_NX_DISABLE_1BPP=y : Only 16BPP supported diff --git a/nuttx/configs/sam4e-ek/src/Makefile b/nuttx/configs/sam4e-ek/src/Makefile index c16a4b7c2..df53b5917 100644 --- a/nuttx/configs/sam4e-ek/src/Makefile +++ b/nuttx/configs/sam4e-ek/src/Makefile @@ -73,10 +73,6 @@ ifeq ($(CONFIG_USBMSC),y) CSRCS += sam_usbmsc.c endif -ifeq ($(CONFIG_INPUT_ADS7843E),y) -CSRCS += sam_touchscreen.c -endif - COBJS = $(CSRCS:.c=$(OBJEXT)) SRCS = $(ASRCS) $(CSRCS) diff --git a/nuttx/configs/sam4e-ek/src/sam_touchscreen.c b/nuttx/configs/sam4e-ek/src/sam_touchscreen.c deleted file mode 100644 index 7965ad549..000000000 --- a/nuttx/configs/sam4e-ek/src/sam_touchscreen.c +++ /dev/null @@ -1,294 +0,0 @@ -/************************************************************************************ - * configs/sam4e-ek/src/sam_touchscreen.c - * - * Copyright (C) 2014 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 <debug.h> -#include <assert.h> -#include <errno.h> - -#include <nuttx/spi/spi.h> -#include <nuttx/input/touchscreen.h> -#include <nuttx/input/ads7843e.h> - -#include "sam_gpio.h" -#include "sam4e-ek.h" - -/**************************************************************************** - * Pre-Processor Definitions - ****************************************************************************/ -/* Configuration ************************************************************/ - -#ifdef CONFIG_INPUT_ADS7843E -#ifndef CONFIG_INPUT -# error "Touchscreen support requires CONFIG_INPUT" -#endif - -#ifndef CONFIG_SAM34_SPI0 -# error "Touchscreen support requires CONFIG_SAM34_SPI0" -#endif - -#ifndef CONFIG_SAM34_GPIOA_IRQ -# error "Touchscreen support requires CONFIG_SAM34_GPIOA_IRQ" -#endif - -#ifndef CONFIG_ADS7843E_FREQUENCY -# define CONFIG_ADS7843E_FREQUENCY 500000 -#endif - -#ifndef CONFIG_ADS7843E_SPIDEV -# define CONFIG_ADS7843E_SPIDEV TSC_CSNUM -#endif - -#if CONFIG_ADS7843E_SPIDEV != TSC_CSNUM -# error "CONFIG_ADS7843E_SPIDEV must have the same value as TSC_CSNUM" -#endif - -#ifndef CONFIG_ADS7843E_DEVMINOR -# define CONFIG_ADS7843E_DEVMINOR 0 -#endif - -/**************************************************************************** - * Static Function Prototypes - ****************************************************************************/ - -/* IRQ/GPIO access callbacks. These operations all hidden behind - * callbacks to isolate the ADS7843E driver from differences in GPIO - * interrupt handling by varying boards and MCUs. If possible, - * interrupts should be configured on both rising and falling edges - * so that contact and loss-of-contact events can be detected. - * - * attach - Attach the ADS7843E interrupt handler to the GPIO interrupt - * enable - Enable or disable the GPIO interrupt - * clear - Acknowledge/clear any pending GPIO interrupt - * pendown - Return the state of the pen down GPIO input - */ - -static int tsc_attach(FAR struct ads7843e_config_s *state, xcpt_t isr); -static void tsc_enable(FAR struct ads7843e_config_s *state, bool enable); -static void tsc_clear(FAR struct ads7843e_config_s *state); -static bool tsc_busy(FAR struct ads7843e_config_s *state); -static bool tsc_pendown(FAR struct ads7843e_config_s *state); - -/**************************************************************************** - * Private Data - ****************************************************************************/ - -/* A reference to a structure of this type must be passed to the ADS7843E - * driver. This structure provides information about the configuration - * of the ADS7843E and provides some board-specific hooks. - * - * Memory for this structure is provided by the caller. It is not copied - * by the driver and is presumed to persist while the driver is active. The - * memory must be writable because, under certain circumstances, the driver - * may modify frequency or X plate resistance values. - */ - -static struct ads7843e_config_s g_tscinfo = -{ - .frequency = CONFIG_ADS7843E_FREQUENCY, - - .attach = tsc_attach, - .enable = tsc_enable, - .clear = tsc_clear, - .busy = tsc_busy, - .pendown = tsc_pendown, -}; - -/**************************************************************************** - * Private Functions - ****************************************************************************/ - -/* IRQ/GPIO access callbacks. These operations all hidden behind - * callbacks to isolate the ADS7843E driver from differences in GPIO - * interrupt handling by varying boards and MCUs. If possible, - * interrupts should be configured on both rising and falling edges - * so that contact and loss-of-contact events can be detected. - * - * attach - Attach the ADS7843E interrupt handler to the GPIO interrupt - * enable - Enable or disable the GPIO interrupt - * clear - Acknowledge/clear any pending GPIO interrupt - * pendown - Return the state of the pen down GPIO input - */ - -static int tsc_attach(FAR struct ads7843e_config_s *state, xcpt_t isr) -{ - /* Attach the ADS7843E interrupt */ - - ivdbg("Attaching %p to IRQ %d\n", isr, SAM_TCS_IRQ); - return irq_attach(SAM_TCS_IRQ, isr); -} - -static void tsc_enable(FAR struct ads7843e_config_s *state, bool enable) -{ - /* Attach and enable, or detach and disable */ - - ivdbg("IRQ:%d enable:%d\n", SAM_TCS_IRQ, enable); - if (enable) - { - sam_gpioirqenable(SAM_TCS_IRQ); - } - else - { - sam_gpioirqdisable(SAM_TCS_IRQ); - } -} - -static void tsc_clear(FAR struct ads7843e_config_s *state) -{ - /* Does nothing */ -} - -static bool tsc_busy(FAR struct ads7843e_config_s *state) -{ -#if defined(CONFIG_DEBUG_INPUT) && defined(CONFIG_DEBUG_VERBOSE) - static bool last = (bool)-1; -#endif - - /* BUSY is high impedance when CS is high (not selected). When CS is - * is low, BUSY is active high. - */ - - bool busy = sam_gpioread(GPIO_TCS_BUSY); -#if defined(CONFIG_DEBUG_INPUT) && defined(CONFIG_DEBUG_VERBOSE) - if (busy != last) - { - ivdbg("busy:%d\n", busy); - last = busy; - } -#endif - - return busy; -} - -static bool tsc_pendown(FAR struct ads7843e_config_s *state) -{ - /* The /PENIRQ value is active low */ - - bool pendown = !sam_gpioread(GPIO_TCS_IRQ); - ivdbg("pendown:%d\n", pendown); - return pendown; -} - -/**************************************************************************** - * Public Functions - ****************************************************************************/ - -/**************************************************************************** - * Name: arch_tcinitialize - * - * Description: - * Each board that supports a touchscreen device must provide this function. - * This function is called by application-specific, setup logic to - * configure the touchscreen device. This function will register the driver - * as /dev/inputN where N is the minor device number. - * - * Input Parameters: - * minor - The input device minor number - * - * Returned Value: - * Zero is returned on success. Otherwise, a negated errno value is - * returned to indicate the nature of the failure. - * - ****************************************************************************/ - -int arch_tcinitialize(int minor) -{ - FAR struct spi_dev_s *dev; - int ret; - - idbg("minor %d\n", minor); - DEBUGASSERT(minor == 0); - - /* Configure and enable the ADS7843E interrupt pin as an input */ - - (void)sam_configgpio(GPIO_TCS_BUSY); - (void)sam_configgpio(GPIO_TCS_IRQ); - - /* Configure the PIO interrupt */ - - sam_gpioirq(GPIO_TCS_IRQ); - - /* Get an instance of the SPI interface for the touchscreen chip select */ - - dev = up_spiinitialize(TSC_CSNUM); - if (!dev) - { - idbg("Failed to initialize SPI chip select %d\n", TSC_CSNUM); - return -ENODEV; - } - - /* Initialize and register the SPI touschscreen device */ - - ret = ads7843e_register(dev, &g_tscinfo, CONFIG_ADS7843E_DEVMINOR); - if (ret < 0) - { - idbg("Failed to initialize SPI chip select %d\n", TSC_CSNUM); - /* up_spiuninitialize(dev); */ - return -ENODEV; - } - - return OK; -} - -/**************************************************************************** - * Name: arch_tcuninitialize - * - * Description: - * Each board that supports a touchscreen device must provide this function. - * This function is called by application-specific, setup logic to - * uninitialized the touchscreen device. - * - * Input Parameters: - * None - * - * Returned Value: - * None. - * - ****************************************************************************/ - -void arch_tcuninitialize(void) -{ - /* No support for un-initializing the touchscreen ADS7843E device yet */ -} - -#endif /* CONFIG_INPUT_ADS7843E */ - |