Patchwork PONY_SPI, the latest patch

login
register
about
Submitter Virgil-Adrian Teaca
Date 2012-04-23 22:08:00
Message ID <CAOfppk50TD7PKZwTA6G_HjoU+q7mcSZZpiY092DtDiDPDfuKYg@mail.gmail.com>
Download mbox | patch
Permalink /patch/3604/
State Superseded
Headers show

Comments

Virgil-Adrian Teaca - 2012-04-23 22:08:00
Hello,

there is the latest patch for PONY_SPI.

So, pony_spi accept these parameters:

dev=/dev/ttyUSB0

"dev" is an required parameter.

type=si_prog -> default Si-Prog / Ponyprog compat
type=serbang -> my little programmer

"type" is an optional parameter, with default to "si_prog".

Also, now pony_spi is able to detect if it have on serial port an SI-Prog
compatible programmer. And it insist to have it, to work.

The detection is simple, using the standard on SI-Prog, RTS/DSR ping-back.

All the best,
Virgil.
Signed-off-by: Virgil-Adrian Teaca <darkstarlinux@gmail.com>

Patch

diff -urN flashrom.orig/Makefile flashrom/Makefile
--- flashrom.orig/Makefile	2012-04-20 12:17:31.000000000 +0200
+++ flashrom/Makefile	2012-04-20 21:46:07.000000000 +0200
@@ -122,6 +122,11 @@ 
 else
 override CONFIG_RAYER_SPI = no
 endif
+ifeq ($(CONFIG_PONY_SPI), yes)
+UNSUPPORTED_FEATURES += CONFIG_PONY_SPI=yes
+else
+override CONFIG_PONY_SPI = no
+endif
 ifeq ($(CONFIG_NIC3COM), yes)
 UNSUPPORTED_FEATURES += CONFIG_NIC3COM=yes
 else
@@ -246,6 +251,11 @@ 
 else
 override CONFIG_RAYER_SPI = no
 endif
+ifeq ($(CONFIG_PONY_SPI), yes)
+UNSUPPORTED_FEATURES += CONFIG_PONY_SPI=yes
+else
+override CONFIG_PONY_SPI = no
+endif
 ifeq ($(CONFIG_ATAHPT), yes)
 UNSUPPORTED_FEATURES += CONFIG_ATAHPT=yes
 else
@@ -292,6 +302,9 @@ 
 # RayeR SPIPGM hardware support
 CONFIG_RAYER_SPI ?= yes
 
+# PonyProg2000 SPI hardware support
+CONFIG_PONY_SPI ?= yes
+
 # Always enable 3Com NICs for now.
 CONFIG_NIC3COM ?= yes
 
@@ -348,6 +361,9 @@ 
 ifeq ($(CONFIG_RAYER_SPI), yes)
 override CONFIG_BITBANG_SPI = yes
 else
+ifeq ($(CONFIG_PONY_SPI), yes)
+override CONFIG_BITBANG_SPI = yes
+else
 ifeq ($(CONFIG_INTERNAL), yes)
 override CONFIG_BITBANG_SPI = yes
 else
@@ -362,6 +378,7 @@ 
 endif
 endif
 endif
+endif
 
 ifeq ($(CONFIG_INTERNAL), yes)
 FEATURE_CFLAGS += -D'CONFIG_INTERNAL=1'
@@ -388,6 +405,11 @@ 
 NEED_PCI := yes
 endif
 
+ifeq ($(CONFIG_PONY_SPI), yes)
+FEATURE_CFLAGS += -D'CONFIG_PONY_SPI=1'
+PROGRAMMER_OBJS += pony_spi.o
+endif
+
 ifeq ($(CONFIG_BITBANG_SPI), yes)
 FEATURE_CFLAGS += -D'CONFIG_BITBANG_SPI=1'
 PROGRAMMER_OBJS += bitbang_spi.o
diff -urN flashrom.orig/flashrom.c flashrom/flashrom.c
--- flashrom.orig/flashrom.c	2012-02-26 11:12:57.000000000 +0100
+++ flashrom/flashrom.c	2012-04-20 18:07:32.000000000 +0200
@@ -201,6 +201,16 @@ 
 	},
 #endif
 
+#if CONFIG_PONY_SPI == 1
+	{
+		.name			= "pony_spi",
+		.init			= pony_spi_init,
+		.map_flash_region	= fallback_map,
+		.unmap_flash_region	= fallback_unmap,
+		.delay			= internal_delay,
+},
+#endif
+
 #if CONFIG_NICINTEL == 1
 	{
 		.name			= "nicintel",
diff -urN flashrom.orig/pony_spi.c flashrom/pony_spi.c
--- flashrom.orig/pony_spi.c	1970-01-01 01:00:00.000000000 +0100
+++ flashrom/pony_spi.c	2012-04-23 23:43:08.436026558 +0200
@@ -0,0 +1,222 @@ 
+/*
+ * This file is part of the flashrom project.
+ *
+ * Copyright (C) 2012 Virgil-Adrian Teaca
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
+ */
+
+/* Driver for the SI-Prog like hardware by Lancos.com.
+ * See http://www.lancos.com/siprogsch.html for schematics and instructions.
+ */
+
+#include <stdlib.h>
+#include <string.h>
+#include <termios.h>
+#include <unistd.h>
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <fcntl.h>
+#include <sys/ioctl.h>
+
+#include "flash.h"
+#include "programmer.h"
+
+/* We have:
+ *
+ * MOSI -----> DTR
+ * MISO -----> CTS
+ * SCK  --+--> RTS
+ *        +--> DSR
+ * CS#  -----> TXD
+ *
+ */
+
+enum pony_type {
+	TYPE_SI_PROG,
+	TYPE_SERBANG,
+};
+
+/* The hardware programmer used. */
+static enum pony_type pony_type = TYPE_SI_PROG;
+
+static void pony_bitbang_set_cs( int val )
+{
+	if( pony_type == TYPE_SI_PROG )
+	{
+		/* CS# is negated by the SI-Prog programmer. */
+		val =  val ? 0 : 1;
+	}
+
+	ioctl( sp_fd, val ? TIOCSBRK : TIOCCBRK, 0 );
+}
+
+static void pony_bitbang_set_sck( int val )
+{
+	int s;
+
+	ioctl( sp_fd, TIOCMGET, &s );
+
+	if( val ) {
+	  	s |= TIOCM_RTS;
+	}
+	else {
+	  	s &= ~TIOCM_RTS;
+	}
+
+	ioctl( sp_fd, TIOCMSET, &s );
+}
+
+static void pony_bitbang_set_mosi( int val )
+{
+	int s;
+
+	ioctl( sp_fd, TIOCMGET, &s );
+
+	if( val ) {
+	  	s |= TIOCM_DTR;
+	}
+	else {
+	  	s &= ~TIOCM_DTR;
+	}
+
+	ioctl( sp_fd, TIOCMSET, &s );
+}
+
+static int pony_bitbang_get_miso(void)
+{
+	int s, tmp;
+
+	ioctl( sp_fd, TIOCMGET, &s );
+
+	tmp = ( s & TIOCM_CTS ) ? 1 : 0;
+
+	if( pony_type == TYPE_SERBANG )
+	{
+		/* MISO is negated by the SERBANG programmer. */
+		tmp = tmp ? 0 : 1;
+	}
+
+	return tmp;
+}
+
+static int pony_bitbang_get_dsr(void)
+{
+	int s, tmp;
+
+	ioctl( sp_fd, TIOCMGET, &s );
+
+	tmp = ( s & TIOCM_DSR ) ? 1 : 0;
+
+	return tmp;
+}
+
+static const struct bitbang_spi_master bitbang_spi_master_pony = {
+	.type = BITBANG_SPI_MASTER_PONY,
+	.set_cs = pony_bitbang_set_cs,
+	.set_sck = pony_bitbang_set_sck,
+	.set_mosi = pony_bitbang_set_mosi,
+	.get_miso = pony_bitbang_get_miso,
+	.half_period = 0,
+};
+
+int pony_spi_init(void)
+{
+	char *arg = NULL;
+
+	int i, data_in, data_out, have_device = 0;
+
+
+	/* The parameter is on format "dev=/dev/device:type=serbang" */
+
+	arg = extract_programmer_param( "dev" );
+
+	if ( arg && strlen( arg ) ) {
+		sp_fd = sp_openserport( arg, 9600 );
+
+		have_device++;
+	}
+
+	free( arg );
+
+	if ( ! have_device ) {
+		msg_perr("Error: No valid device specified.\n"
+			 "Use flashrom -p pony_spi:dev=/dev/device\n");
+
+		return 1;
+	}
+
+	arg = extract_programmer_param( "type" );
+
+	if ( arg ) {
+		if ( ! strcasecmp( arg, "serbang" ) ) {
+			pony_type = TYPE_SERBANG;
+		} else if ( ! strcasecmp( arg, "si_prog" ) ) {
+			pony_type = TYPE_SI_PROG;
+		} else {
+			msg_perr("Error: Invalid programmer type specified.\n");
+
+			free( arg );
+
+			serialport_shutdown( NULL );
+
+			return 1;
+		}
+	}
+
+	free( arg );
+
+	msg_pdbg( "Using the %s programmer.\n", ( ( pony_type == TYPE_SI_PROG ) ? "SI-Prog" : "SERBANG" ) );
+
+	/*
+	 * Detect if there is a SI-Prog compatible programmer connected.
+	 */
+
+	pony_bitbang_set_cs( 1 );
+	pony_bitbang_set_mosi( 1 );
+
+	/* We toggle SCK while we keep MOSI and CS# on. */
+
+	for ( i = 1; i <= 10; ++i ) {
+		data_out = i & 1;
+
+		pony_bitbang_set_sck( data_out );
+
+		programmer_delay( 10000 );
+
+		data_in = pony_bitbang_get_dsr();
+
+		if ( data_out != data_in ) {
+			msg_perr( "No SI-Prog compatible hardware detected.\n" );
+
+			serialport_shutdown( NULL );
+
+			return 1;
+		}
+	}
+
+	/* Register the shutdown callback. */
+
+	if ( register_shutdown( serialport_shutdown, NULL ) ) {
+		return 1;
+	}
+
+	/* Get the initial value before writing to any line. */
+
+	if ( bitbang_spi_init( &bitbang_spi_master_pony ) ) {
+		return 1;
+	}
+
+	return 0;
+}
diff -urN flashrom.orig/programmer.h flashrom/programmer.h
--- flashrom.orig/programmer.h	2012-03-13 16:52:50.000000000 +0100
+++ flashrom/programmer.h	2012-04-20 18:27:12.000000000 +0200
@@ -69,6 +69,9 @@ 
 #if CONFIG_RAYER_SPI == 1
 	PROGRAMMER_RAYER_SPI,
 #endif
+#if CONFIG_PONY_SPI == 1
+	PROGRAMMER_PONY_SPI,
+#endif
 #if CONFIG_NICINTEL == 1
 	PROGRAMMER_NICINTEL,
 #endif
@@ -110,6 +113,9 @@ 
 #if CONFIG_RAYER_SPI == 1
 	BITBANG_SPI_MASTER_RAYER,
 #endif
+#if CONFIG_PONY_SPI == 1
+	BITBANG_SPI_MASTER_PONY,
+#endif
 #if CONFIG_NICINTEL_SPI == 1
 	BITBANG_SPI_MASTER_NICINTEL,
 #endif
@@ -430,6 +436,11 @@ 
 int rayer_spi_init(void);
 #endif
 
+/* pony_spi.c */
+#if CONFIG_PONY_SPI == 1
+int pony_spi_init(void);
+#endif
+
 /* bitbang_spi.c */
 int bitbang_spi_init(const struct bitbang_spi_master *master);