Hello Maurus, Glad to see you're experimenting what you've seen in the kernel training :-) On Wed, 2 Mar 2011 14:08:34 +0100 "Frey ext-FA, Maurus" <maurus.frey.ext@siemens.com> wrote:
I wrote a simple module, which should initialise an USART in RS485-mode (RTS stays low after initialisation) and register it as platform_device. Its on an Atmel at91sam9260 board, so I can re-use most of the mach arm-sources (atmel_serial.c etc.)
When the module gets loaded I got the following output
$ modprobe axm_rs485_uart Original ATMEL_US_MR=0 Write 1 to ATMEL_US_MR After Write ATMEL_US_MR=0 axm_rs485_uart: registering uart port #3 in RS485 mode as atmel_usart3
- read the actual state from the USART Mode Register - Put the mode into RS485 mode ("or" a bit) - write the new mode into the register - read the register again...and (what a surprise) see, that the write operation didn't succeed
A simple write to a register seems to fail. In consequence the RTS line gets high after the Pin has been "muxed" to its RTS-Rolle with at91_set_B_periph().
I think, that I'm doing something wrong with request_mem() and ioremap() No clue what? Can you help me with this?
I haven't look into many details, but I don't see why you need a specific driver to do this. The Atmel serial driver in drivers/serial/atmel_serial.c already supports RS485. So from what I can see, all you need to do is to add some details in the platform_data structure relative to the serial port. So probably you need to change the uart3_data structure in arch/arm/mach-at91/at91sam9260_devices.c, by adding a .rs485 field : static struct atmel_uart_data uart3_data = { .use_dma_tx = 1, .use_dma_rx = 1, .rs485 = { /* Fill the fields of the serial_rs485 structure defined in include/linux/serial.h */ }, }; To configure the pins, you have to call at91_register_uart() in the ek_map_io() function of your board file (presumably arch/arm/mach-at91/board-at91sam9261ek.c). The third parameter allows to tell which pins should be configured. For example, if you pass ATMEL_UART_RTS, then the code in arch/arm/mach-at91/at91sam9260_devices.c will do: if (pins & ATMEL_UART_RTS) at91_set_B_periph(AT91_PIN_PC8, 0); /* RTS3 */ see the function configure_usart3_pins().
#include <linux/errno.h> #include <linux/init.h> #include <linux/module.h> #include <linux/platform_device.h> #include <linux/gpio.h> #include <linux/atmel_serial.h> #include <mach/board.h>
#define UART_PORT_WITH_RTS_MAX 4 #define UART_PORT_DEFAULT 3 #define UART_PORT_DEFAULT_DEVICE_NAME "atmel_usart3"
// USART ids / interrupts static u32 at91sam9260_id_us[UART_PORT_WITH_RTS_MAX] = { AT91SAM9260_ID_US0, AT91SAM9260_ID_US1, AT91SAM9260_ID_US2, AT91SAM9260_ID_US3 };
#define A_PERIPHERAL 0 #define B_PERIPHERAL 1
struct io_mux_pin { u32 io_line_addr; // I/O Line address uint peripheral; // A or B Peripheral } io_mux_pin;
struct io_mux_pin at91_rts_io_pin[UART_PORT_WITH_RTS_MAX] = { { AT91_PIN_PB26, A_PERIPHERAL }, //RTS0 { AT91_PIN_PB28, A_PERIPHERAL }, //RTS1 { AT91_PIN_PA4, A_PERIPHERAL }, //RTS2 { AT91_PIN_PC8, B_PERIPHERAL } //RTS3 };
//Define some module information. MODULE_AUTHOR("Maus"); MODULE_LICENSE("GPL"); MODULE_DESCRIPTION("Initialize UART port in RS485 mode. RTS-Line stays low.") ;
// Stores our pointer to the platform device, needed for release during exit. static struct platform_device *pdev; // Remapped virtual base address static void *vbaseaddr;
// get port_number as parameter, available in sysfs // valid value from 0 to UART_PORT_MAX static uint uart_port = UART_PORT_DEFAULT; module_param( uart_port, uint, 0400 ); MODULE_PARM_DESC(uart_port, "UART port number to initialize in RS485 mode [0-" __MODULE_STRING(UART_PORT_WITH_RTS_MAX)"]");
// get device_name as parameter, available in sysfs static char device_name[30] = UART_PORT_DEFAULT_DEVICE_NAME; module_param_string( device_name, device_name, sizeof(device_name), 0400 );
/* * Register UARTX as platform device. */ static int __init axm_rs485_uart_init(void) { int retValue = 0; unsigned mode;
//check parameter input if (uart_port >= UART_PORT_WITH_RTS_MAX) { printk(KERN_ERR "%s: uart_port=%u must be below %u\n", THIS_MODULE->name, uart_port, UART_PORT_WITH_RTS_MAX); return -EINVAL; }
/* * Get platform_device struct from exported at91_register_uart * * - Don't configure RTS pin. * - portnr = uart_port+1 (not necessary due call after boot-up) */ pdev = at91_register_uart(at91sam9260_id_us[uart_port], uart_port + 1, ATMEL_UART_CTS);
I don't know how this can even compile: at91_register_uart() returns an int, and pdev is a platform_device * is your code. I'm sorry but this code really isn't implemented with the driver model in mind. That's not how things are supposed to work. And also, your code is completely wrapped, which makes it impossible to read. It's the board/SoC code (in your case arch/arm/mach-at91/board-at91sam9261ek.c and arch/arm/mach-at91/at91sam9260_devices.c) that instantiates the platform_device structure and does the pin muxing. Each platform_device structure can receive driver-specific information in the platform_data field, such as the ones needed here for RS485. You really don't need a separate kernel module to do this, just modify your board/SoC file (in this case you'll have to modify the SoC file, as the at91_add_device_serial() function doesn't allow to pass the UART-specific platform_data, unlike what is done with the other at91_add_device_*() functions). Regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com