diff --git a/README b/README
index 60338d0763b25b5a4f0b7b30b7956bfa4f15ac92..a9ccd0018d8da0a12aa76818332b749748f380f5 100644
--- a/README
+++ b/README
@@ -117,7 +117,7 @@ Debug targets
 
 ARM11, ARM7, ARM9, AVR32, Cortex-A, Cortex-R, Cortex-M,
 Feroceon/Dragonite, DSP563xx, DSP5680xx, FA526, MIPS EJTAG, NDS32,
-XScale.
+XScale, Intel Quark.
 
 Flash drivers
 -------------
diff --git a/doc/openocd.texi b/doc/openocd.texi
index f614c62949add89d247eb440858b36cb0d6add33..76e335b8f23c2466606b37baaa65a18b4b759223 100644
--- a/doc/openocd.texi
+++ b/doc/openocd.texi
@@ -156,9 +156,9 @@ USB-based, parallel port-based, and other standalone boxes that run
 OpenOCD internally. @xref{Debug Adapter Hardware}.
 
 @b{GDB Debug:} It allows ARM7 (ARM7TDMI and ARM720t), ARM9 (ARM920T,
-ARM922T, ARM926EJ--S, ARM966E--S), XScale (PXA25x, IXP42x) and
-Cortex-M3 (Stellaris LM3, ST STM32 and Energy Micro EFM32) based cores to be
-debugged via the GDB protocol.
+ARM922T, ARM926EJ--S, ARM966E--S), XScale (PXA25x, IXP42x), Cortex-M3
+(Stellaris LM3, ST STM32 and Energy Micro EFM32) and Intel Quark (x10xx)
+based cores to be debugged via the GDB protocol.
 
 @b{Flash Programming:} Flash writing is supported for external
 CFI-compatible NOR flashes (Intel and AMD/Spansion command set) and several
@@ -7542,6 +7542,47 @@ the peripherals.
 @xref{targetevents,,Target Events}.
 @end deffn
 
+@section Intel Architecture
+
+Intel Quark X10xx is the first product in the Quark family of SoCs. It is an IA-32
+(Pentium x86 ISA) compatible SoC. The core CPU in the X10xx is codenamed Lakemont.
+Lakemont version 1 (LMT1) is used in X10xx. The CPU TAP (Lakemont TAP) is used for
+software debug and the CLTAP is used for SoC level operations.
+Useful docs are here: https://communities.intel.com/community/makers/documentation
+@itemize
+@item Intel Quark SoC X1000 OpenOCD/GDB/Eclipse App Note (web search for doc num 330015)
+@item Intel Quark SoC X1000 Debug Operations User Guide (web search for doc num 329866)
+@item Intel Quark SoC X1000 Datasheet (web search for doc num 329676)
+@end itemize
+
+@subsection x86 32-bit specific commands
+The three main address spaces for x86 are memory, I/O and configuration space.
+These commands allow a user to read and write to the 64Kbyte I/O address space.
+
+@deffn Command {x86_32 idw} address
+Display the contents of a 32-bit I/O port from address range 0x0000 - 0xffff.
+@end deffn
+
+@deffn Command {x86_32 idh} address
+Display the contents of a 16-bit I/O port from address range 0x0000 - 0xffff.
+@end deffn
+
+@deffn Command {x86_32 idb} address
+Display the contents of a 8-bit I/O port from address range 0x0000 - 0xffff.
+@end deffn
+
+@deffn Command {x86_32 iww} address
+Write the contents of a 32-bit I/O port to address range 0x0000 - 0xffff.
+@end deffn
+
+@deffn Command {x86_32 iwh} address
+Write the contents of a 16-bit I/O port to address range 0x0000 - 0xffff.
+@end deffn
+
+@deffn Command {x86_32 iwb} address
+Write the contents of a 8-bit I/O port to address range 0x0000 - 0xffff.
+@end deffn
+
 @section OpenRISC Architecture
 
 The OpenRISC CPU is a soft core. It is used in a programmable SoC which can be
diff --git a/src/target/Makefile.am b/src/target/Makefile.am
index 4d9ea418a39a9319ecdeaf90ef013a40e153979c..159f0fa4307c426935c971445caa7178519d15ab 100644
--- a/src/target/Makefile.am
+++ b/src/target/Makefile.am
@@ -35,6 +35,7 @@ libtarget_la_SOURCES = \
 	$(AVR32_SRC) \
 	$(MIPS32_SRC) \
 	$(NDS32_SRC) \
+	$(INTEL_IA32_SRC) \
 	avrt.c \
 	dsp563xx.c \
 	dsp563xx_once.c \
@@ -124,6 +125,10 @@ NDS32_SRC = \
 	nds32_v3m.c \
 	nds32_aice.c
 
+INTEL_IA32_SRC = \
+	quark_x10xx.c \
+	lakemont.c \
+	x86_32_common.c
 
 noinst_HEADERS = \
 	algorithm.h \
@@ -191,7 +196,9 @@ noinst_HEADERS = \
 	nds32_v3_common.h \
 	nds32_v3.h \
 	nds32_v3m.h \
-	nds32_aice.h
+	nds32_aice.h \
+	lakemont.h \
+	x86_32_common.h
 
 ocddatadir = $(pkglibdir)
 nobase_dist_ocddata_DATA =
diff --git a/src/target/lakemont.c b/src/target/lakemont.c
new file mode 100644
index 0000000000000000000000000000000000000000..70c785c6a55cc9453e527d0445dc5590317349d2
--- /dev/null
+++ b/src/target/lakemont.c
@@ -0,0 +1,1111 @@
+/*
+ * Copyright(c) 2013 Intel Corporation.
+ *
+ * Adrian Burns (adrian.burns@intel.com)
+ * Thomas Faust (thomas.faust@intel.com)
+ * Ivan De Cesaris (ivan.de.cesaris@intel.com)
+ * Julien Carreno (julien.carreno@intel.com)
+ * Jeffrey Maxwell (jeffrey.r.maxwell@intel.com)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of version 2 of the GNU General Public License as
+ * published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * Contact Information:
+ * Intel Corporation
+ */
+
+/*
+ * @file
+ * This implements the probemode operations for Lakemont 1 (LMT1).
+ */
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <helper/log.h>
+
+#include "target.h"
+#include "target_type.h"
+#include "lakemont.h"
+#include "register.h"
+#include "breakpoints.h"
+#include "x86_32_common.h"
+
+static int irscan(struct target *t, uint8_t *out,
+			uint8_t *in, uint8_t ir_len);
+static int drscan(struct target *t, uint8_t *out, uint8_t *in, uint8_t len);
+static int save_context(struct target *target);
+static int restore_context(struct target *target);
+static uint32_t get_tapstatus(struct target *t);
+static int enter_probemode(struct target *t);
+static int exit_probemode(struct target *t);
+static int halt_prep(struct target *t);
+static int do_halt(struct target *t);
+static int do_resume(struct target *t);
+static int read_all_core_hw_regs(struct target *t);
+static int write_all_core_hw_regs(struct target *t);
+static int read_hw_reg(struct target *t,
+			int reg, uint32_t *regval, uint8_t cache);
+static int write_hw_reg(struct target *t,
+			int reg, uint32_t regval, uint8_t cache);
+static struct reg_cache *lakemont_build_reg_cache
+			(struct target *target);
+static int submit_reg_pir(struct target *t, int num);
+static int submit_instruction_pir(struct target *t, int num);
+static int submit_pir(struct target *t, uint64_t op);
+static int lakemont_get_core_reg(struct reg *reg);
+static int lakemont_set_core_reg(struct reg *reg, uint8_t *buf);
+
+static struct scan_blk scan;
+
+/* registers and opcodes for register access, pm_idx is used to identify the
+ * registers that are modified for lakemont probemode specific operations
+ */
+static const struct {
+	uint8_t id;
+	const char *name;
+	uint64_t op;
+	uint8_t pm_idx;
+	unsigned bits;
+	enum reg_type type;
+	const char *group;
+	const char *feature;
+} regs[] = {
+	/* general purpose registers */
+	{ EAX, "eax", 0x000000D01D660000, 0, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ECX, "ecx", 0x000000501D660000, 1, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ EDX, "edx", 0x000000901D660000, 2, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ EBX, "ebx", 0x000000101D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ESP, "esp", 0x000000E01D660000, NOT_PMREG, 32, REG_TYPE_DATA_PTR, "general", "org.gnu.gdb.i386.core" },
+	{ EBP, "ebp", 0x000000601D660000, NOT_PMREG, 32, REG_TYPE_DATA_PTR, "general", "org.gnu.gdb.i386.core" },
+	{ ESI, "esi", 0x000000A01D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ EDI, "edi", 0x000000201D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+
+	/* instruction pointer & flags */
+	{ EIP, "eip", 0x000000C01D660000, 3, 32, REG_TYPE_CODE_PTR, "general", "org.gnu.gdb.i386.core" },
+	{ EFLAGS, "eflags", 0x000000401D660000, 4, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+
+	/* segment registers */
+	{ CS, "cs", 0x000000281D660000, 5, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ SS, "ss", 0x000000C81D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ DS, "ds", 0x000000481D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ES, "es", 0x000000A81D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FS, "fs", 0x000000881D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ GS, "gs", 0x000000081D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+
+	/* floating point unit registers - not accessible via JTAG - here to satisfy GDB */
+	{ ST0, "st0", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST1, "st1", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST2, "st2", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST3, "st3", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST4, "st4", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST5, "st5", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST6, "st6", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ ST7, "st7", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FCTRL, "fctrl", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FSTAT, "fstat", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FTAG, "ftag", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FISEG, "fiseg", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FIOFF, "fioff", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FOSEG, "foseg", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FOOFF, "fooff", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+	{ FOP, "fop", 0x0, NOT_AVAIL_REG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.core" },
+
+	/* control registers */
+	{ CR0, "cr0", 0x000000001D660000, 6, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ CR2, "cr2", 0x000000BC1D660000, 7, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ CR3, "cr3", 0x000000801D660000, 8, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ CR4, "cr4", 0x0000002C1D660000, 9, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+
+	/* debug registers */
+	{ DR0, "dr0", 0x0000007C1D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DR1, "dr1", 0x000000FC1D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DR2, "dr2", 0x000000021D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DR3, "dr3", 0x000000821D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DR6, "dr6", 0x000000301D660000, 10, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DR7, "dr7", 0x000000B01D660000, 11, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+
+	/* descriptor tables */
+	{ IDTB, "idtbase", 0x000000581D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ IDTL, "idtlimit", 0x000000D81D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ IDTAR, "idtar", 0x000000981D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ GDTB, "gdtbase", 0x000000B81D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ GDTL, "gdtlimit", 0x000000781D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ GDTAR, "gdtar", 0x000000381D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ TR, "tr", 0x000000701D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ LDTR, "ldtr", 0x000000F01D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ LDTB, "ldbase", 0x000000041D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ LDTL, "ldlimit", 0x000000841D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ LDTAR, "ldtar", 0x000000F81D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+
+	/* segment registers */
+	{ CSB, "csbase", 0x000000F41D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ CSL, "cslimit", 0x0000000C1D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ CSAR, "csar", 0x000000741D660000, 12, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DSB, "dsbase", 0x000000941D660000, 13, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DSL, "dslimit", 0x000000541D660000, 14, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ DSAR, "dsar", 0x000000141D660000, 15, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ ESB, "esbase", 0x0000004C1D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ ESL, "eslimit", 0x000000CC1D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ ESAR, "esar", 0x0000008C1D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ FSB, "fsbase", 0x000000641D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ FSL, "fslimit", 0x000000E41D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ FSAR, "fsar", 0x000000A41D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ GSB, "gsbase", 0x000000C41D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ GSL, "gslimit", 0x000000241D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ GSAR, "gsar", 0x000000441D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ SSB, "ssbase", 0x000000341D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ SSL, "sslimit", 0x000000B41D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ SSAR, "ssar", 0x000000D41D660000, 16, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ TSSB, "tssbase", 0x000000E81D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ TSSL, "tsslimit", 0x000000181D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	{ TSSAR, "tssar", 0x000000681D660000, NOT_PMREG, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+	/* probemode control register */
+	{ PMCR, "pmcr", 0x000000421D660000, 17, 32, REG_TYPE_INT32, "general", "org.gnu.gdb.i386.sys" },
+};
+
+static const struct {
+	uint8_t id;
+	const char *name;
+	uint64_t op;
+} instructions[] = {
+	/* memory read/write */
+	{ MEMRDB32, "MEMRDB32", 0x0909090909090851 },
+	{ MEMRDB16, "MEMRDB16", 0x09090909090851E6 },
+	{ MEMRDH32, "MEMRDH32", 0x090909090908D166 },
+	{ MEMRDH16, "MEMRDH16", 0x090909090908D1E6 },
+	{ MEMRDW32, "MEMRDW32", 0x09090909090908D1 },
+	{ MEMRDW16, "MEMRDW16", 0x0909090908D1E666 },
+	{ MEMWRB32, "MEMWRB32", 0x0909090909090811 },
+	{ MEMWRB16, "MEMWRB16", 0x09090909090811E6 },
+	{ MEMWRH32, "MEMWRH32", 0x0909090909089166 },
+	{ MEMWRH16, "MEMWRH16", 0x09090909090891E6 },
+	{ MEMWRW32, "MEMWRW32", 0x0909090909090891 },
+	{ MEMWRW16, "MEMWRW16", 0x090909090891E666 },
+	/* IO read/write */
+	{ IORDB32, "IORDB32", 0x0909090909090937 },
+	{ IORDB16, "IORDB16", 0x09090909090937E6 },
+	{ IORDH32, "IORDH32", 0x090909090909B766 },
+	{ IORDH16, "IORDH16", 0x090909090909B7E6 },
+	{ IORDW32, "IORDW32", 0x09090909090909B7 },
+	{ IORDW16, "IORDW16", 0x0909090909B7E666 },
+	{ IOWRB32, "IOWRB32", 0x0909090909090977 },
+	{ IOWRB16, "IOWRB16", 0x09090909090977E6 },
+	{ IOWRH32, "IOWRH32", 0x090909090909F766 },
+	{ IOWRH16, "IOWRH16", 0x090909090909F7E6 },
+	{ IOWRW32, "IOWRW32", 0x09090909090909F7 },
+	{ IOWRW16, "IOWRW16", 0x0909090909F7E666 },
+	/* lakemont1 core shadow ram access opcodes */
+	{ SRAMACCESS, "SRAMACCESS", 0x0000000E9D660000 },
+	{ SRAM2PDR, "SRAM2PDR", 0x4CF0000000000000 },
+	{ PDR2SRAM, "PDR2SRAM", 0x0CF0000000000000 },
+	{ WBINVD, "WBINVD", 0x09090909090990F0 },
+};
+
+bool check_not_halted(const struct target *t)
+{
+	bool halted = t->state == TARGET_HALTED;
+	if (!halted)
+		LOG_ERROR("target running, halt it first");
+	return !halted;
+}
+
+static int irscan(struct target *t, uint8_t *out,
+			uint8_t *in, uint8_t ir_len)
+{
+	int retval = ERROR_OK;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	if (NULL == t->tap) {
+		retval = ERROR_FAIL;
+		LOG_ERROR("%s invalid target tap", __func__);
+		return retval;
+	}
+	if (ir_len != t->tap->ir_length) {
+		retval = ERROR_FAIL;
+		if (t->tap->enabled)
+			LOG_ERROR("%s tap enabled but tap irlen=%d",
+					__func__, t->tap->ir_length);
+		else
+			LOG_ERROR("%s tap not enabled and irlen=%d",
+					__func__, t->tap->ir_length);
+		return retval;
+	}
+	struct scan_field *fields = &scan.field;
+	fields->num_bits = ir_len;
+	fields->out_value = out;
+	fields->in_value = in;
+	jtag_add_ir_scan(x86_32->curr_tap, fields, TAP_IDLE);
+	if (x86_32->flush) {
+		retval = jtag_execute_queue();
+		if (retval != ERROR_OK)
+			LOG_ERROR("%s failed to execute queue", __func__);
+	}
+	return retval;
+}
+
+static int drscan(struct target *t, uint8_t *out, uint8_t *in, uint8_t len)
+{
+	int retval = ERROR_OK;
+	uint64_t data = 0;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	if (NULL == t->tap) {
+		retval = ERROR_FAIL;
+		LOG_ERROR("%s invalid target tap", __func__);
+		return retval;
+	}
+	if (len > MAX_SCAN_SIZE || 0 == len) {
+		retval = ERROR_FAIL;
+		LOG_ERROR("%s data len is %d bits, max is %d bits",
+				__func__, len, MAX_SCAN_SIZE);
+		return retval;
+	}
+	struct scan_field *fields = &scan.field;
+	fields->out_value = out;
+	fields->in_value = in;
+	fields->num_bits = len;
+	jtag_add_dr_scan(x86_32->curr_tap, 1, fields, TAP_IDLE);
+	if (x86_32->flush) {
+		retval = jtag_execute_queue();
+		if (retval != ERROR_OK) {
+			LOG_ERROR("%s drscan failed to execute queue", __func__);
+			return retval;
+		}
+	}
+	if (in != NULL) {
+		if (len >= 8) {
+			for (int n = (len / 8) - 1 ; n >= 0; n--)
+				data = (data << 8) + *(in+n);
+		} else
+			LOG_DEBUG("dr in 0x%02" PRIx8, *in);
+	} else {
+		LOG_ERROR("%s no drscan data", __func__);
+		retval = ERROR_FAIL;
+	}
+	return retval;
+}
+
+static int save_context(struct target *t)
+{
+	int err;
+	/* read core registers from lakemont sram */
+	err = read_all_core_hw_regs(t);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error reading regs", __func__);
+		return err;
+	}
+	return ERROR_OK;
+}
+
+static int restore_context(struct target *t)
+{
+	int err = ERROR_OK;
+	uint32_t i;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	/* write core regs into the core PM SRAM from the reg_cache */
+	err = write_all_core_hw_regs(t);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error writing regs", __func__);
+		return err;
+	}
+
+	for (i = 0; i < (x86_32->cache->num_regs); i++) {
+		x86_32->cache->reg_list[i].dirty = 0;
+		x86_32->cache->reg_list[i].valid = 0;
+	}
+	return err;
+}
+
+/*
+ * we keep reg_cache in sync with hardware at halt/resume time, we avoid
+ * writing to real hardware here bacause pm_regs reflects the hardware
+ * while we are halted then reg_cache syncs with hw on resume
+ * TODO - in order for "reg eip force" to work it assume get/set reads
+ * and writes from hardware, may be other reasons also because generally
+ * other openocd targets read/write from hardware in get/set - watch this!
+ */
+static int lakemont_get_core_reg(struct reg *reg)
+{
+	int retval = ERROR_OK;
+	struct lakemont_core_reg *lakemont_reg = reg->arch_info;
+	struct target *t = lakemont_reg->target;
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	LOG_DEBUG("reg=%s, value=%08" PRIx32, reg->name,
+			buf_get_u32(reg->value, 0, 32));
+	return retval;
+}
+
+static int lakemont_set_core_reg(struct reg *reg, uint8_t *buf)
+{
+	struct lakemont_core_reg *lakemont_reg = reg->arch_info;
+	struct target *t = lakemont_reg->target;
+	uint32_t value = buf_get_u32(buf, 0, 32);
+	LOG_DEBUG("reg=%s, newval=%08" PRIx32, reg->name, value);
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	buf_set_u32(reg->value, 0, 32, value);
+	reg->dirty = 1;
+	reg->valid = 1;
+	return ERROR_OK;
+}
+
+static const struct reg_arch_type lakemont_reg_type = {
+	/* these get called if reg_cache doesnt have a "valid" value
+	 * of an individual reg eg "reg eip" but not for "reg" block
+	 */
+	.get = lakemont_get_core_reg,
+	.set = lakemont_set_core_reg,
+};
+
+struct reg_cache *lakemont_build_reg_cache(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	int num_regs = ARRAY_SIZE(regs);
+	struct reg_cache **cache_p = register_get_last_cache_p(&t->reg_cache);
+	struct reg_cache *cache = malloc(sizeof(struct reg_cache));
+	struct reg *reg_list = malloc(sizeof(struct reg) * num_regs);
+	struct lakemont_core_reg *arch_info = malloc(sizeof(struct lakemont_core_reg) * num_regs);
+	struct reg_feature *feature;
+	int i;
+
+	if (cache == NULL || reg_list == NULL || arch_info == NULL) {
+		free(cache);
+		free(reg_list);
+		free(arch_info);
+		LOG_ERROR("%s out of memory", __func__);
+		return NULL;
+	}
+
+	/* Build the process context cache */
+	cache->name = "lakemont registers";
+	cache->next = NULL;
+	cache->reg_list = reg_list;
+	cache->num_regs = num_regs;
+	(*cache_p) = cache;
+	x86_32->cache = cache;
+
+	for (i = 0; i < num_regs; i++) {
+		arch_info[i].target = t;
+		arch_info[i].x86_32_common = x86_32;
+		arch_info[i].op = regs[i].op;
+		arch_info[i].pm_idx = regs[i].pm_idx;
+		reg_list[i].name = regs[i].name;
+		reg_list[i].size = 32;
+		reg_list[i].value = calloc(1, 4);
+		reg_list[i].dirty = 0;
+		reg_list[i].valid = 0;
+		reg_list[i].type = &lakemont_reg_type;
+		reg_list[i].arch_info = &arch_info[i];
+
+		reg_list[i].group = regs[i].group;
+		reg_list[i].number = i;
+		reg_list[i].exist = true;
+		reg_list[i].caller_save = true;	/* gdb defaults to true */
+
+		feature = calloc(1, sizeof(struct reg_feature));
+		if (feature) {
+			feature->name = regs[i].feature;
+			reg_list[i].feature = feature;
+		} else
+			LOG_ERROR("%s unable to allocate feature list", __func__);
+
+		reg_list[i].reg_data_type = calloc(1, sizeof(struct reg_data_type));
+		if (reg_list[i].reg_data_type)
+			reg_list[i].reg_data_type->type = regs[i].type;
+		else
+			LOG_ERROR("%s unable to allocate reg type list", __func__);
+	}
+	return cache;
+}
+
+static uint32_t get_tapstatus(struct target *t)
+{
+	scan.out[0] = TAPSTATUS;
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return 0;
+	if (drscan(t, NULL, scan.out, TS_SIZE) != ERROR_OK)
+		return 0;
+	return buf_get_u32(scan.out, 0, 32);
+}
+
+static int enter_probemode(struct target *t)
+{
+	uint32_t tapstatus = 0;
+	tapstatus = get_tapstatus(t);
+	LOG_DEBUG("TS before PM enter = %08" PRIx32, tapstatus);
+	if (tapstatus & TS_PM_BIT) {
+		LOG_DEBUG("core already in probemode");
+		return ERROR_OK;
+	}
+	scan.out[0] = PROBEMODE;
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return ERROR_FAIL;
+	scan.out[0] = 1;
+	if (drscan(t, scan.out, scan.in, 1) != ERROR_OK)
+		return ERROR_FAIL;
+	tapstatus = get_tapstatus(t);
+	LOG_DEBUG("TS after PM enter = %08" PRIx32, tapstatus);
+	if ((tapstatus & TS_PM_BIT) && (!(tapstatus & TS_EN_PM_BIT)))
+		return ERROR_OK;
+	else {
+		LOG_ERROR("%s PM enter error, tapstatus = %08" PRIx32
+				, __func__, tapstatus);
+		return ERROR_FAIL;
+	}
+}
+
+static int exit_probemode(struct target *t)
+{
+	uint32_t tapstatus = get_tapstatus(t);
+	LOG_DEBUG("TS before PM exit = %08" PRIx32, tapstatus);
+
+	if (!(tapstatus & TS_PM_BIT)) {
+		LOG_USER("core not in PM");
+		return ERROR_OK;
+	}
+	scan.out[0] = PROBEMODE;
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return ERROR_FAIL;
+	scan.out[0] = 0;
+	if (drscan(t, scan.out, scan.in, 1) != ERROR_OK)
+		return ERROR_FAIL;
+	return ERROR_OK;
+}
+
+/* do whats needed to properly enter probemode for debug on lakemont */
+static int halt_prep(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	if (write_hw_reg(t, DSB, PM_DSB, 0) != ERROR_OK)
+		return ERROR_FAIL;
+	LOG_DEBUG("write %s %08" PRIx32, regs[DSB].name, PM_DSB);
+	if (write_hw_reg(t, DSL, PM_DSL, 0) != ERROR_OK)
+		return ERROR_FAIL;
+	LOG_DEBUG("write %s %08" PRIx32, regs[DSL].name, PM_DSL);
+	if (write_hw_reg(t, DSAR, PM_DSAR, 0) != ERROR_OK)
+		return ERROR_FAIL;
+	LOG_DEBUG("write DSAR %08" PRIx32, PM_DSAR);
+	if (write_hw_reg(t, DR7, PM_DR7, 0) != ERROR_OK)
+		return ERROR_FAIL;
+	LOG_DEBUG("write DR7 %08" PRIx32, PM_DR7);
+
+	uint32_t eflags = buf_get_u32(x86_32->cache->reg_list[EFLAGS].value, 0, 32);
+	uint32_t csar = buf_get_u32(x86_32->cache->reg_list[CSAR].value, 0, 32);
+	uint32_t ssar = buf_get_u32(x86_32->cache->reg_list[SSAR].value, 0, 32);
+	uint32_t cr0 = buf_get_u32(x86_32->cache->reg_list[CR0].value, 0, 32);
+
+	/* clear VM86 and IF bits if they are set */
+	LOG_DEBUG("EFLAGS = %08" PRIx32 ", VM86 = %d, IF = %d", eflags,
+			eflags & EFLAGS_VM86 ? 1 : 0,
+			eflags & EFLAGS_IF ? 1 : 0);
+	if (eflags & EFLAGS_VM86
+		|| eflags & EFLAGS_IF) {
+		x86_32->pm_regs[I(EFLAGS)] = eflags & ~(EFLAGS_VM86 | EFLAGS_IF);
+		if (write_hw_reg(t, EFLAGS, x86_32->pm_regs[I(EFLAGS)], 0) != ERROR_OK)
+			return ERROR_FAIL;
+		LOG_DEBUG("EFLAGS now = %08" PRIx32 ", VM86 = %d, IF = %d",
+				x86_32->pm_regs[I(EFLAGS)],
+				x86_32->pm_regs[I(EFLAGS)] & EFLAGS_VM86 ? 1 : 0,
+				x86_32->pm_regs[I(EFLAGS)] & EFLAGS_IF ? 1 : 0);
+	}
+
+	/* set CPL to 0 for memory access */
+	if (csar & CSAR_DPL) {
+		x86_32->pm_regs[I(CSAR)] = csar & ~CSAR_DPL;
+		if (write_hw_reg(t, CSAR, x86_32->pm_regs[I(CSAR)], 0) != ERROR_OK)
+			return ERROR_FAIL;
+		LOG_DEBUG("write CSAR_CPL to 0 %08" PRIx32, x86_32->pm_regs[I(CSAR)]);
+	}
+	if (ssar & SSAR_DPL) {
+		x86_32->pm_regs[I(SSAR)] = ssar & ~CSAR_DPL;
+		if (write_hw_reg(t, SSAR, x86_32->pm_regs[I(SSAR)], 0) != ERROR_OK)
+			return ERROR_FAIL;
+		LOG_DEBUG("write SSAR_CPL to 0 %08" PRIx32, x86_32->pm_regs[I(SSAR)]);
+	}
+
+	/* if cache's are enabled, disable and flush */
+	if (!(cr0 & CR0_CD)) {
+		LOG_DEBUG("caching enabled CR0 = %08" PRIx32, cr0);
+		if (cr0 & CR0_PG) {
+			x86_32->pm_regs[I(CR0)] = cr0 & ~CR0_PG;
+			if (write_hw_reg(t, CR0, x86_32->pm_regs[I(CR0)], 0) != ERROR_OK)
+				return ERROR_FAIL;
+			LOG_DEBUG("cleared paging CR0_PG = %08" PRIx32, x86_32->pm_regs[I(CR0)]);
+			/* submit wbinvd to flush cache */
+			if (submit_reg_pir(t, WBINVD) != ERROR_OK)
+				return ERROR_FAIL;
+			x86_32->pm_regs[I(CR0)] =
+				x86_32->pm_regs[I(CR0)] | (CR0_CD | CR0_NW | CR0_PG);
+			if (write_hw_reg(t, CR0, x86_32->pm_regs[I(CR0)], 0) != ERROR_OK)
+				return ERROR_FAIL;
+			LOG_DEBUG("set CD, NW and PG, CR0 = %08" PRIx32, x86_32->pm_regs[I(CR0)]);
+		}
+	}
+	return ERROR_OK;
+}
+
+static int do_halt(struct target *t)
+{
+	/* needs proper handling later if doing a halt errors out */
+	t->state = TARGET_DEBUG_RUNNING;
+	if (enter_probemode(t) != ERROR_OK)
+		return ERROR_FAIL;
+	if (save_context(t) != ERROR_OK)
+		return ERROR_FAIL;
+	if (halt_prep(t) != ERROR_OK)
+		return ERROR_FAIL;
+	t->state = TARGET_HALTED;
+
+	return target_call_event_callbacks(t, TARGET_EVENT_HALTED);
+}
+
+static int do_resume(struct target *t)
+{
+	/* needs proper handling later */
+	t->state = TARGET_DEBUG_RUNNING;
+	if (restore_context(t) != ERROR_OK)
+		return ERROR_FAIL;
+	if (exit_probemode(t) != ERROR_OK)
+		return ERROR_FAIL;
+	t->state = TARGET_RUNNING;
+
+	t->debug_reason = DBG_REASON_NOTHALTED;
+	LOG_USER("target running");
+
+	return target_call_event_callbacks(t, TARGET_EVENT_RESUMED);
+}
+
+static int read_all_core_hw_regs(struct target *t)
+{
+	int err;
+	uint32_t regval, i;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	for (i = 0; i < (x86_32->cache->num_regs); i++) {
+		if (NOT_AVAIL_REG == regs[i].pm_idx)
+			continue;
+		err = read_hw_reg(t, regs[i].id, &regval, 1);
+		if (err != ERROR_OK) {
+			LOG_ERROR("%s error saving reg %s",
+					__func__, x86_32->cache->reg_list[i].name);
+			return err;
+		}
+	}
+	LOG_DEBUG("read_all_core_hw_regs read %d registers ok", i);
+	return ERROR_OK;
+}
+
+static int write_all_core_hw_regs(struct target *t)
+{
+	int err;
+	uint32_t i;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	for (i = 0; i < (x86_32->cache->num_regs); i++) {
+		if (NOT_AVAIL_REG == regs[i].pm_idx)
+			continue;
+		err = write_hw_reg(t, i, 0, 1);
+		if (err != ERROR_OK) {
+			LOG_ERROR("%s error restoring reg %s",
+					__func__, x86_32->cache->reg_list[i].name);
+			return err;
+		}
+	}
+	LOG_DEBUG("write_all_core_hw_regs wrote %d registers ok", i);
+	return ERROR_OK;
+}
+
+/* read reg from lakemont core shadow ram, update reg cache if needed */
+static int read_hw_reg(struct target *t, int reg, uint32_t *regval, uint8_t cache)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	struct lakemont_core_reg *arch_info;
+	arch_info = x86_32->cache->reg_list[reg].arch_info;
+	x86_32->flush = 0; /* dont flush scans till we have a batch */
+	if (submit_reg_pir(t, reg) != ERROR_OK)
+		return ERROR_FAIL;
+	if (submit_instruction_pir(t, SRAMACCESS) != ERROR_OK)
+		return ERROR_FAIL;
+	if (submit_instruction_pir(t, SRAM2PDR) != ERROR_OK)
+		return ERROR_FAIL;
+	x86_32->flush = 1;
+	scan.out[0] = RDWRPDR;
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return ERROR_FAIL;
+	if (drscan(t, NULL, scan.out, PDR_SIZE) != ERROR_OK)
+		return ERROR_FAIL;
+
+	jtag_add_sleep(DELAY_SUBMITPIR);
+	*regval = buf_get_u32(scan.out, 0, 32);
+	if (cache) {
+		buf_set_u32(x86_32->cache->reg_list[reg].value, 0, 32, *regval);
+		x86_32->cache->reg_list[reg].valid = 1;
+		x86_32->cache->reg_list[reg].dirty = 0;
+	}
+	LOG_DEBUG("reg=%s, op=0x%016" PRIx64 ", val=%08" PRIx32,
+			x86_32->cache->reg_list[reg].name,
+			arch_info->op,
+			*regval);
+	return ERROR_OK;
+}
+
+/* write lakemont core shadow ram reg, update reg cache if needed */
+static int write_hw_reg(struct target *t, int reg, uint32_t regval, uint8_t cache)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	struct lakemont_core_reg *arch_info;
+	arch_info = x86_32->cache->reg_list[reg].arch_info;
+
+	uint8_t reg_buf[4];
+	if (cache)
+		regval = buf_get_u32(x86_32->cache->reg_list[reg].value, 0, 32);
+	buf_set_u32(reg_buf, 0, 32, regval);
+	LOG_DEBUG("reg=%s, op=0x%016" PRIx64 ", val=%08" PRIx32,
+			x86_32->cache->reg_list[reg].name,
+			arch_info->op,
+			regval);
+
+	scan.out[0] = RDWRPDR;
+	x86_32->flush = 0; /* dont flush scans till we have a batch */
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return ERROR_FAIL;
+	if (drscan(t, reg_buf, scan.out, PDR_SIZE) != ERROR_OK)
+		return ERROR_FAIL;
+	if (submit_reg_pir(t, reg) != ERROR_OK)
+		return ERROR_FAIL;
+	if (submit_instruction_pir(t, SRAMACCESS) != ERROR_OK)
+		return ERROR_FAIL;
+	x86_32->flush = 1;
+	if (submit_instruction_pir(t, PDR2SRAM) != ERROR_OK)
+		return ERROR_FAIL;
+
+	/* we are writing from the cache so ensure we reset flags */
+	if (cache) {
+		x86_32->cache->reg_list[reg].dirty = 0;
+		x86_32->cache->reg_list[reg].valid = 0;
+	}
+	return ERROR_OK;
+}
+
+static bool is_paging_enabled(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	if (x86_32->pm_regs[I(CR0)] & CR0_PG)
+		return true;
+	else
+		return false;
+}
+
+static uint8_t get_num_user_regs(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	return x86_32->cache->num_regs;
+}
+/* value of the CR0.PG (paging enabled) bit influences memory reads/writes */
+static int disable_paging(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	x86_32->pm_regs[I(CR0)] = x86_32->pm_regs[I(CR0)] & ~CR0_PG;
+	int err = x86_32->write_hw_reg(t, CR0, x86_32->pm_regs[I(CR0)], 0);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error disabling paging", __func__);
+		return err;
+	}
+	return err;
+}
+
+static int enable_paging(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	x86_32->pm_regs[I(CR0)] = (x86_32->pm_regs[I(CR0)] | CR0_PG);
+	int err = x86_32->write_hw_reg(t, CR0, x86_32->pm_regs[I(CR0)], 0);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error enabling paging", __func__);
+		return err;
+	}
+	return err;
+}
+
+static bool sw_bpts_supported(struct target *t)
+{
+	uint32_t tapstatus = get_tapstatus(t);
+	if (tapstatus & TS_SBP_BIT)
+		return true;
+	else
+		return false;
+}
+
+static int transaction_status(struct target *t)
+{
+	uint32_t tapstatus = get_tapstatus(t);
+	if ((TS_EN_PM_BIT | TS_PRDY_BIT) & tapstatus) {
+		LOG_ERROR("%s transaction error tapstatus = %08" PRIx32
+				, __func__, tapstatus);
+		return ERROR_FAIL;
+	} else {
+		return ERROR_OK;
+	}
+}
+
+static int submit_instruction(struct target *t, int num)
+{
+	int err = submit_instruction_pir(t, num);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error submitting pir", __func__);
+		return err;
+	}
+	return err;
+}
+
+static int submit_reg_pir(struct target *t, int num)
+{
+	LOG_DEBUG("reg %s op=0x%016" PRIx64, regs[num].name, regs[num].op);
+	int err = submit_pir(t, regs[num].op);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error submitting pir", __func__);
+		return err;
+	}
+	return err;
+}
+
+static int submit_instruction_pir(struct target *t, int num)
+{
+	LOG_DEBUG("%s op=0x%016" PRIx64, instructions[num].name,
+			instructions[num].op);
+	int err = submit_pir(t, instructions[num].op);
+	if (err != ERROR_OK) {
+		LOG_ERROR("%s error submitting pir", __func__);
+		return err;
+	}
+	return err;
+}
+
+/*
+ * PIR (Probe Mode Instruction Register), SUBMITPIR is an "IR only" TAP
+ * command; there is no corresponding data register
+ */
+static int submit_pir(struct target *t, uint64_t op)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	uint8_t op_buf[8];
+	buf_set_u64(op_buf, 0, 64, op);
+	int flush = x86_32->flush;
+	x86_32->flush = 0;
+	scan.out[0] = WRPIR;
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return ERROR_FAIL;
+	if (drscan(t, op_buf, scan.out, PIR_SIZE) != ERROR_OK)
+		return ERROR_FAIL;
+	scan.out[0] = SUBMITPIR;
+	x86_32->flush = flush;
+	if (irscan(t, scan.out, NULL, LMT_IRLEN) != ERROR_OK)
+		return ERROR_FAIL;
+	jtag_add_sleep(DELAY_SUBMITPIR);
+	return ERROR_OK;
+}
+
+int lakemont_init_target(struct command_context *cmd_ctx, struct target *t)
+{
+	lakemont_build_reg_cache(t);
+	t->state = TARGET_RUNNING;
+	t->debug_reason = DBG_REASON_NOTHALTED;
+	return ERROR_OK;
+}
+
+int lakemont_init_arch_info(struct target *t, struct x86_32_common *x86_32)
+{
+	x86_32->submit_instruction = submit_instruction;
+	x86_32->transaction_status = transaction_status;
+	x86_32->read_hw_reg = read_hw_reg;
+	x86_32->write_hw_reg = write_hw_reg;
+	x86_32->sw_bpts_supported = sw_bpts_supported;
+	x86_32->get_num_user_regs = get_num_user_regs;
+	x86_32->is_paging_enabled = is_paging_enabled;
+	x86_32->disable_paging = disable_paging;
+	x86_32->enable_paging = enable_paging;
+	return ERROR_OK;
+}
+
+int lakemont_poll(struct target *t)
+{
+	/* LMT1 PMCR register currently allows code breakpoints, data breakpoints,
+	 * single stepping and shutdowns to be redirected to PM but does not allow
+	 * redirecting into PM as a result of SMM enter and SMM exit
+	 */
+	uint32_t ts = get_tapstatus(t);
+
+	if (ts == 0xFFFFFFFF && t->state != TARGET_DEBUG_RUNNING) {
+		/* something is wrong here */
+		LOG_ERROR("tapstatus invalid - scan_chain serialization or locked JTAG access issues");
+		/* TODO: Give a hint that unlocking is wrong or maybe a
+		 * 'jtag arp_init' helps
+		 */
+		t->state = TARGET_DEBUG_RUNNING;
+		return ERROR_OK;
+	}
+
+	if (t->state == TARGET_HALTED && (!(ts & TS_PM_BIT))) {
+		LOG_INFO("target running for unknown reason");
+		t->state = TARGET_RUNNING;
+	}
+
+	if (t->state == TARGET_RUNNING &&
+		t->state != TARGET_DEBUG_RUNNING) {
+
+		if ((ts & TS_PM_BIT) && (ts & TS_PMCR_BIT)) {
+
+			LOG_DEBUG("redirect to PM, tapstatus=%08" PRIx32, get_tapstatus(t));
+
+			t->state = TARGET_DEBUG_RUNNING;
+			if (save_context(t) != ERROR_OK)
+				return ERROR_FAIL;
+			if (halt_prep(t) != ERROR_OK)
+				return ERROR_FAIL;
+			t->state = TARGET_HALTED;
+			t->debug_reason = DBG_REASON_UNDEFINED;
+
+			struct x86_32_common *x86_32 = target_to_x86_32(t);
+			uint32_t eip = buf_get_u32(x86_32->cache->reg_list[EIP].value, 0, 32);
+			uint32_t dr6 = buf_get_u32(x86_32->cache->reg_list[DR6].value, 0, 32);
+			uint32_t hwbreakpoint = (uint32_t)-1;
+
+			if (dr6 & DR6_BRKDETECT_0)
+				hwbreakpoint = 0;
+			if (dr6 & DR6_BRKDETECT_1)
+				hwbreakpoint = 1;
+			if (dr6 & DR6_BRKDETECT_2)
+				hwbreakpoint = 2;
+			if (dr6 & DR6_BRKDETECT_3)
+				hwbreakpoint = 3;
+
+			if (hwbreakpoint != (uint32_t)-1) {
+				uint32_t dr7 = buf_get_u32(x86_32->cache->reg_list[DR7].value, 0, 32);
+				uint32_t type = dr7 & (0x03 << (DR7_RW_SHIFT + hwbreakpoint*DR7_RW_LEN_SIZE));
+				if (type == DR7_BP_EXECUTE) {
+					LOG_USER("hit hardware breakpoint (hwreg=%d) at 0x%08" PRIx32, hwbreakpoint, eip);
+				} else {
+					uint32_t address = 0;
+					switch (hwbreakpoint) {
+					default:
+					case 0:
+						address = buf_get_u32(x86_32->cache->reg_list[DR0].value, 0, 32);
+					break;
+					case 1:
+						address = buf_get_u32(x86_32->cache->reg_list[DR1].value, 0, 32);
+					break;
+					case 2:
+						address = buf_get_u32(x86_32->cache->reg_list[DR2].value, 0, 32);
+					break;
+					case 3:
+						address = buf_get_u32(x86_32->cache->reg_list[DR3].value, 0, 32);
+					break;
+					}
+					LOG_USER("hit '%s' watchpoint for 0x%08" PRIx32 " (hwreg=%d) at 0x%08" PRIx32,
+								type == DR7_BP_WRITE ? "write" : "access", address,
+								hwbreakpoint, eip);
+				}
+				t->debug_reason = DBG_REASON_BREAKPOINT;
+			} else {
+				/* Check if the target hit a software breakpoint.
+				 * ! Watch out: EIP is currently pointing after the breakpoint opcode
+				 */
+				struct breakpoint *bp = NULL;
+				bp = breakpoint_find(t, eip-1);
+				if (bp != NULL) {
+					t->debug_reason = DBG_REASON_BREAKPOINT;
+					if (bp->type == BKPT_SOFT) {
+						/* The EIP is now pointing the the next byte after the
+						 * breakpoint instruction. This needs to be corrected.
+						 */
+						buf_set_u32(x86_32->cache->reg_list[EIP].value, 0, 32, eip-1);
+						x86_32->cache->reg_list[EIP].dirty = 1;
+						x86_32->cache->reg_list[EIP].valid = 1;
+						LOG_USER("hit software breakpoint at 0x%08" PRIx32, eip-1);
+					} else {
+						/* it's not a hardware breakpoint (checked already in DR6 state)
+						 * and it's also not a software breakpoint ...
+						 */
+						LOG_USER("hit unknown breakpoint at 0x%08" PRIx32, eip);
+					}
+				} else {
+
+					/* There is also the case that we hit an breakpoint instruction,
+					 * which was not set by us. This needs to be handled be the
+					 * application that introduced the breakpoint.
+					 */
+
+					LOG_USER("unknown break reason at 0x%08" PRIx32, eip);
+				}
+			}
+
+			return target_call_event_callbacks(t, TARGET_EVENT_HALTED);
+		}
+	}
+	return ERROR_OK;
+}
+
+int lakemont_arch_state(struct target *t)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	LOG_USER("target halted due to %s at 0x%08" PRIx32 " in %s mode",
+			debug_reason_name(t),
+			buf_get_u32(x86_32->cache->reg_list[EIP].value, 0, 32),
+			(buf_get_u32(x86_32->cache->reg_list[CR0].value, 0, 32) & CR0_PE) ? "protected" : "real");
+
+	return ERROR_OK;
+}
+
+int lakemont_halt(struct target *t)
+{
+	if (t->state == TARGET_RUNNING) {
+		t->debug_reason = DBG_REASON_DBGRQ;
+		if (do_halt(t) != ERROR_OK)
+			return ERROR_FAIL;
+		return ERROR_OK;
+	} else {
+		LOG_ERROR("%s target not running", __func__);
+		return ERROR_FAIL;
+	}
+}
+
+int lakemont_resume(struct target *t, int current, uint32_t address,
+			int handle_breakpoints, int debug_execution)
+{
+	struct breakpoint *bp = NULL;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	/* TODO lakemont_enable_breakpoints(t); */
+	if (t->state == TARGET_HALTED) {
+
+		/* running away for a software breakpoint needs some special handling */
+		uint32_t eip = buf_get_u32(x86_32->cache->reg_list[EIP].value, 0, 32);
+		bp = breakpoint_find(t, eip);
+		if (bp != NULL /*&& bp->type == BKPT_SOFT*/) {
+			/* the step will step over the breakpoint */
+			if (lakemont_step(t, 0, 0, 1) != ERROR_OK) {
+				LOG_ERROR("%s stepping over a software breakpoint at 0x%08" PRIx32 " "
+						"failed to resume the target", __func__, eip);
+				return ERROR_FAIL;
+			}
+		}
+
+		/* if breakpoints are enabled, we need to redirect these into probe mode */
+		struct breakpoint *activeswbp = t->breakpoints;
+		while (activeswbp != NULL && activeswbp->set == 0)
+			activeswbp = activeswbp->next;
+		struct watchpoint *activehwbp = t->watchpoints;
+		while (activehwbp != NULL && activehwbp->set == 0)
+			activehwbp = activehwbp->next;
+		if (activeswbp != NULL || activehwbp != NULL)
+			buf_set_u32(x86_32->cache->reg_list[PMCR].value, 0, 32, 1);
+		if (do_resume(t) != ERROR_OK)
+			return ERROR_FAIL;
+	} else {
+		LOG_USER("target not halted");
+		return ERROR_FAIL;
+	}
+	return ERROR_OK;
+}
+
+int lakemont_step(struct target *t, int current,
+			uint32_t address, int handle_breakpoints)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	uint32_t eflags = buf_get_u32(x86_32->cache->reg_list[EFLAGS].value, 0, 32);
+	uint32_t eip = buf_get_u32(x86_32->cache->reg_list[EIP].value, 0, 32);
+	uint32_t pmcr = buf_get_u32(x86_32->cache->reg_list[PMCR].value, 0, 32);
+	struct breakpoint *bp = NULL;
+	int retval = ERROR_OK;
+	uint32_t tapstatus = 0;
+
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	bp = breakpoint_find(t, eip);
+	if (retval == ERROR_OK && bp != NULL/*&& bp->type == BKPT_SOFT*/) {
+		/* TODO: This should only be done for software breakpoints.
+		 * Stepping from hardware breakpoints should be possible with the resume flag
+		 * Needs testing.
+		 */
+		retval = x86_32_common_remove_breakpoint(t, bp);
+	}
+
+	/* Set EFLAGS[TF] and PMCR[IR], exit pm and wait for PRDY# */
+	LOG_DEBUG("modifying PMCR = %d and EFLAGS = %08" PRIx32, pmcr, eflags);
+	eflags = eflags | (EFLAGS_TF | EFLAGS_RF);
+	buf_set_u32(x86_32->cache->reg_list[EFLAGS].value, 0, 32, eflags);
+	buf_set_u32(x86_32->cache->reg_list[PMCR].value, 0, 32, 1);
+	LOG_DEBUG("EFLAGS [TF] [RF] bits set=%08" PRIx32 ", PMCR=%d, EIP=%08" PRIx32,
+			eflags, pmcr, eip);
+
+	tapstatus = get_tapstatus(t);
+
+	t->debug_reason = DBG_REASON_SINGLESTEP;
+	t->state = TARGET_DEBUG_RUNNING;
+	if (restore_context(t) != ERROR_OK)
+		return ERROR_FAIL;
+	if (exit_probemode(t) != ERROR_OK)
+		return ERROR_FAIL;
+
+	target_call_event_callbacks(t, TARGET_EVENT_RESUMED);
+
+	tapstatus = get_tapstatus(t);
+	if (tapstatus & (TS_PM_BIT | TS_EN_PM_BIT | TS_PRDY_BIT | TS_PMCR_BIT)) {
+		/* target has stopped */
+		if (save_context(t) != ERROR_OK)
+			return ERROR_FAIL;
+		if (halt_prep(t) != ERROR_OK)
+			return ERROR_FAIL;
+		t->state = TARGET_HALTED;
+
+		LOG_USER("step done from EIP 0x%08" PRIx32 " to 0x%08" PRIx32, eip,
+				buf_get_u32(x86_32->cache->reg_list[EIP].value, 0, 32));
+		target_call_event_callbacks(t, TARGET_EVENT_HALTED);
+	} else {
+		/* target didn't stop
+		 * I hope the poll() will catch it, but the deleted breakpoint is gone
+		 */
+		LOG_ERROR("%s target didn't stop after executing a single step", __func__);
+		t->state = TARGET_RUNNING;
+		return ERROR_FAIL;
+	}
+
+	/* try to re-apply the breakpoint, even of step failed
+	 * TODO: When a bp was set, we should try to stop the target - fix the return above
+	 */
+	if (bp != NULL/*&& bp->type == BKPT_SOFT*/) {
+		/* TODO: This should only be done for software breakpoints.
+		 * Stepping from hardware breakpoints should be possible with the resume flag
+		 * Needs testing.
+		 */
+		retval = x86_32_common_add_breakpoint(t, bp);
+	}
+
+	return retval;
+}
+
+/* TODO - implement resetbreak fully through CLTAP registers */
+int lakemont_reset_assert(struct target *t)
+{
+	LOG_DEBUG("-");
+	return ERROR_OK;
+}
+
+int lakemont_reset_deassert(struct target *t)
+{
+	LOG_DEBUG("-");
+	return ERROR_OK;
+}
diff --git a/src/target/lakemont.h b/src/target/lakemont.h
new file mode 100644
index 0000000000000000000000000000000000000000..e63cab027805a19f72db1979a6315dd5f86a405f
--- /dev/null
+++ b/src/target/lakemont.h
@@ -0,0 +1,103 @@
+/*
+ * Copyright(c) 2013 Intel Corporation.
+ *
+ * Adrian Burns (adrian.burns@intel.com)
+ * Thomas Faust (thomas.faust@intel.com)
+ * Ivan De Cesaris (ivan.de.cesaris@intel.com)
+ * Julien Carreno (julien.carreno@intel.com)
+ * Jeffrey Maxwell (jeffrey.r.maxwell@intel.com)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of version 2 of the GNU General Public License as
+ * published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * Contact Information:
+ * Intel Corporation
+ */
+
+/*
+ * @file
+ * This is the interface to the probemode operations for Lakemont 1 (LMT1).
+ */
+
+#ifndef LAKEMONT_H
+#define LAKEMONT_H
+#include <jtag/jtag.h>
+
+/* The Intel Quark SoC X1000 Core is codenamed lakemont */
+
+#define LMT_IRLEN		8
+
+/* lakemont tap instruction opcodes */
+#define IDCODE			2
+#define SUBMITPIR		3
+#define PROBEMODE		4
+#define WRPIR			6
+#define RDWRPDR			8
+#define TAPSTATUS		11
+#define BYPASS			255
+#define NOT_NULL		2
+
+/* DR sizes */
+#define ID_SIZE			32
+#define PM_SIZE			1
+#define PIR_SIZE		64
+#define PDR_SIZE		32
+#define TS_SIZE			32
+#define BP_SIZE			1
+#define MAX_SCAN_SIZE	PIR_SIZE
+
+/* needed during lakemont probemode */
+#define NOT_PMREG		0xfe
+#define NOT_AVAIL_REG		0xff
+#define PM_DSB			0x00000000
+#define PM_DSL			0xFFFFFFFF
+#define PM_DSAR			0x004F9300
+#define PM_DR7			0x00000400
+#define DELAY_SUBMITPIR		0 /* for now 0 is working */
+
+/* lakemont tapstatus bits */
+#define TS_PRDY_BIT		0x00000001
+#define TS_EN_PM_BIT		0x00000002
+#define TS_PM_BIT		0x00000004
+#define TS_PMCR_BIT		0x00000008
+#define TS_SBP_BIT		0x00000010
+
+struct lakemont_core_reg {
+	uint32_t num;
+	struct target *target;
+	struct x86_32_common *x86_32_common;
+	uint64_t op;
+	uint8_t pm_idx;
+};
+
+struct scan_blk {
+	uint8_t out[MAX_SCAN_SIZE]; /* scanned out to the tap */
+	uint8_t in[MAX_SCAN_SIZE]; /* in to our capture buf */
+	struct scan_field field;
+};
+
+#define I(name) (((struct lakemont_core_reg *)x86_32->cache->reg_list[name].arch_info)->pm_idx)
+
+int lakemont_init_target(struct command_context *cmd_ctx, struct target *t);
+int lakemont_init_arch_info(struct target *t, struct x86_32_common *x86_32);
+int lakemont_poll(struct target *t);
+int lakemont_arch_state(struct target *t);
+int lakemont_halt(struct target *t);
+int lakemont_resume(struct target *t, int current, uint32_t address,
+			int handle_breakpoints, int debug_execution);
+int lakemont_step(struct target *t, int current,
+			uint32_t address, int handle_breakpoints);
+int lakemont_reset_assert(struct target *t);
+int lakemont_reset_deassert(struct target *t);
+
+#endif /* LAKEMONT_H */
diff --git a/src/target/quark_x10xx.c b/src/target/quark_x10xx.c
new file mode 100644
index 0000000000000000000000000000000000000000..05818b7c109e9b329f3f278a5808347bd8e00db5
--- /dev/null
+++ b/src/target/quark_x10xx.c
@@ -0,0 +1,97 @@
+/*
+ * Copyright(c) 2013 Intel Corporation.
+ *
+ * Adrian Burns (adrian.burns@intel.com)
+ * Thomas Faust (thomas.faust@intel.com)
+ * Ivan De Cesaris (ivan.de.cesaris@intel.com)
+ * Julien Carreno (julien.carreno@intel.com)
+ * Jeffrey Maxwell (jeffrey.r.maxwell@intel.com)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of version 2 of the GNU General Public License as
+ * published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * Contact Information:
+ * Intel Corporation
+ */
+
+/*
+ * @file
+ * Debugger for Intel Quark SoC X1000
+ * Intel Quark X10xx is the first product in the Quark family of SoCs.
+ * It is an IA-32 (Pentium x86 ISA) compatible SoC. The core CPU in the
+ * X10xx is codenamed Lakemont. Lakemont version 1 (LMT1) is used in X10xx.
+ * The CPU TAP (Lakemont TAP) is used for software debug and the CLTAP is
+ * used for SoC level operations.
+ * Useful docs are here: https://communities.intel.com/community/makers/documentation
+ * Intel Quark SoC X1000 OpenOCD/GDB/Eclipse App Note (web search for doc num 330015)
+ * Intel Quark SoC X1000 Debug Operations User Guide (web search for doc num 329866)
+ * Intel Quark SoC X1000 Datasheet (web search for doc num 329676)
+ *
+ * This file implements any Quark SoC specific features such as resetbreak (TODO)
+ */
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <helper/log.h>
+
+#include "target.h"
+#include "target_type.h"
+#include "lakemont.h"
+#include "x86_32_common.h"
+
+int quark_x10xx_target_create(struct target *t, Jim_Interp *interp)
+{
+	struct x86_32_common *x86_32 = calloc(1, sizeof(struct x86_32_common));
+	if (x86_32 == NULL) {
+		LOG_ERROR("%s out of memory", __func__);
+		return ERROR_FAIL;
+	}
+	x86_32_common_init_arch_info(t, x86_32);
+	lakemont_init_arch_info(t, x86_32);
+	return ERROR_OK;
+}
+
+int quark_x10xx_init_target(struct command_context *cmd_ctx, struct target *t)
+{
+	return lakemont_init_target(cmd_ctx, t);
+}
+
+struct target_type quark_x10xx_target = {
+	.name = "quark_x10xx",
+	/* Quark X1000 SoC */
+	.target_create = quark_x10xx_target_create,
+	.init_target = quark_x10xx_init_target,
+	/* lakemont probemode specific code */
+	.poll = lakemont_poll,
+	.arch_state = lakemont_arch_state,
+	.halt = lakemont_halt,
+	.resume = lakemont_resume,
+	.step = lakemont_step,
+	.assert_reset = lakemont_reset_assert,
+	.deassert_reset = lakemont_reset_deassert,
+	/* common x86 code */
+	.commands = x86_32_command_handlers,
+	.get_gdb_reg_list = x86_32_get_gdb_reg_list,
+	.read_memory = x86_32_common_read_memory,
+	.write_memory = x86_32_common_write_memory,
+	.add_breakpoint = x86_32_common_add_breakpoint,
+	.remove_breakpoint = x86_32_common_remove_breakpoint,
+	.add_watchpoint = x86_32_common_add_watchpoint,
+	.remove_watchpoint = x86_32_common_remove_watchpoint,
+	.virt2phys = x86_32_common_virt2phys,
+	.read_phys_memory = x86_32_common_read_phys_mem,
+	.write_phys_memory = x86_32_common_write_phys_mem,
+	.mmu = x86_32_common_mmu,
+};
diff --git a/src/target/target.c b/src/target/target.c
index 43d2c6a7a737a441ac6d89e9ebcad10ce94e3db9..b2af96a3144d188cb3d9056c7c67e85b9ee4ef32 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -102,6 +102,7 @@ extern struct target_type nds32_v2_target;
 extern struct target_type nds32_v3_target;
 extern struct target_type nds32_v3m_target;
 extern struct target_type or1k_target;
+extern struct target_type quark_x10xx_target;
 
 static struct target_type *target_types[] = {
 	&arm7tdmi_target,
@@ -130,6 +131,7 @@ static struct target_type *target_types[] = {
 	&nds32_v3_target,
 	&nds32_v3m_target,
 	&or1k_target,
+	&quark_x10xx_target,
 	NULL,
 };
 
diff --git a/src/target/x86_32_common.c b/src/target/x86_32_common.c
new file mode 100644
index 0000000000000000000000000000000000000000..34da39b9fb23f1cbc3efbf707e184dfc4ca17681
--- /dev/null
+++ b/src/target/x86_32_common.c
@@ -0,0 +1,1443 @@
+/*
+ * Copyright(c) 2013 Intel Corporation.
+ *
+ * Adrian Burns (adrian.burns@intel.com)
+ * Thomas Faust (thomas.faust@intel.com)
+ * Ivan De Cesaris (ivan.de.cesaris@intel.com)
+ * Julien Carreno (julien.carreno@intel.com)
+ * Jeffrey Maxwell (jeffrey.r.maxwell@intel.com)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of version 2 of the GNU General Public License as
+ * published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * Contact Information:
+ * Intel Corporation
+ */
+
+/*
+ * @file
+ * This implements generic x86 32 bit memory and breakpoint operations.
+ */
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <helper/log.h>
+
+#include "target.h"
+#include "target_type.h"
+#include "register.h"
+#include "breakpoints.h"
+#include "x86_32_common.h"
+
+static int set_debug_regs(struct target *t, uint32_t address,
+			uint8_t bp_num, uint8_t bp_type, uint8_t bp_length);
+static int unset_debug_regs(struct target *t, uint8_t bp_num);
+static int read_mem(struct target *t, uint32_t size,
+			uint32_t addr, uint8_t *buf);
+static int write_mem(struct target *t, uint32_t size,
+			uint32_t addr, const uint8_t *buf);
+static int calcaddr_pyhsfromlin(struct target *t, uint32_t addr,
+			uint32_t *physaddr);
+static int read_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, uint8_t *buffer);
+static int write_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, const uint8_t *buffer);
+static int set_breakpoint(struct target *target,
+			struct breakpoint *breakpoint);
+static int unset_breakpoint(struct target *target,
+			struct breakpoint *breakpoint);
+static int set_watchpoint(struct target *target,
+			struct watchpoint *watchpoint);
+static int unset_watchpoint(struct target *target,
+			struct watchpoint *watchpoint);
+static int read_hw_reg_to_cache(struct target *t, int num);
+static int write_hw_reg_from_cache(struct target *t, int num);
+
+int x86_32_get_gdb_reg_list(struct target *t,
+			struct reg **reg_list[], int *reg_list_size,
+			enum target_register_class reg_class)
+{
+
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	int i;
+	*reg_list_size = x86_32->cache->num_regs;
+	LOG_DEBUG("num_regs=%d, reg_class=%d", (*reg_list_size), reg_class);
+	*reg_list = malloc(sizeof(struct reg *) * (*reg_list_size));
+	if (*reg_list == NULL) {
+		LOG_ERROR("%s out of memory", __func__);
+		return ERROR_FAIL;
+	}
+	/* this will copy the values from our reg list to gdbs */
+	for (i = 0; i < (*reg_list_size); i++) {
+		(*reg_list)[i] = &x86_32->cache->reg_list[i];
+		LOG_DEBUG("value %s = %08" PRIx32, x86_32->cache->reg_list[i].name,
+				buf_get_u32(x86_32->cache->reg_list[i].value, 0, 32));
+	}
+	return ERROR_OK;
+}
+
+int x86_32_common_init_arch_info(struct target *t, struct x86_32_common *x86_32)
+{
+	t->arch_info = x86_32;
+	x86_32->common_magic = X86_32_COMMON_MAGIC;
+	x86_32->num_hw_bpoints = MAX_DEBUG_REGS;
+	x86_32->hw_break_list = calloc(x86_32->num_hw_bpoints,
+				sizeof(struct x86_32_dbg_reg));
+	if (x86_32->hw_break_list == NULL) {
+		LOG_ERROR("%s out of memory", __func__);
+		return ERROR_FAIL;
+	}
+	x86_32->curr_tap = t->tap;
+	x86_32->fast_data_area = NULL;
+	x86_32->flush = 1;
+	x86_32->read_hw_reg_to_cache = read_hw_reg_to_cache;
+	x86_32->write_hw_reg_from_cache = write_hw_reg_from_cache;
+	return ERROR_OK;
+}
+
+int x86_32_common_mmu(struct target *t, int *enabled)
+{
+	*enabled = true;
+	return ERROR_OK;
+}
+
+int x86_32_common_virt2phys(struct target *t, uint32_t address, uint32_t *physical)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	/*
+	 * We need to ignore 'segmentation' for now, as OpenOCD can't handle
+	 * segmented addresses.
+	 * In protected mode that is almost OK, as (almost) any known OS is using
+	 * flat segmentation. In real mode we use use the base of the DS segment,
+	 * as we don't know better ...
+	 */
+
+	uint32_t cr0 = buf_get_u32(x86_32->cache->reg_list[CR0].value, 0, 32);
+	if (!(cr0 & CR0_PG)) {
+		/* target halted in real mode */
+		/* TODO: needs validation !!! */
+		uint32_t dsb = buf_get_u32(x86_32->cache->reg_list[DSB].value, 0, 32);
+		*physical = dsb + address;
+
+	} else {
+		/* target halted in protected mode */
+		if (calcaddr_pyhsfromlin(t, address, physical) != ERROR_OK) {
+			LOG_ERROR("%s failed to calculate physical address from 0x%08" PRIx32,
+					__func__, address);
+			return ERROR_FAIL;
+		}
+	}
+	return ERROR_OK;
+}
+
+int x86_32_common_read_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, uint8_t *buffer)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	int error;
+
+	error = read_phys_mem(t, phys_address, size, count, buffer);
+	if (error != ERROR_OK)
+		return error;
+
+	/* After reading memory from target, we must replace software breakpoints
+	 * with the original instructions again.
+	 */
+	struct swbp_mem_patch *iter = x86_32->swbbp_mem_patch_list;
+	while (iter != NULL) {
+		if (iter->physaddr >= phys_address && iter->physaddr < phys_address+(size*count)) {
+			uint32_t offset = iter->physaddr - phys_address;
+			buffer[offset] = iter->orig_byte;
+		}
+		iter = iter->next;
+	}
+	return ERROR_OK;
+}
+
+static int read_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, uint8_t *buffer)
+{
+	int retval = ERROR_OK;
+	bool pg_disabled = false;
+	LOG_DEBUG("addr=%08" PRIx32 ", size=%d, count=%d, buf=%p",
+			phys_address, size, count, buffer);
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	if (!count || !buffer || !phys_address) {
+		LOG_ERROR("%s invalid params count=%d, buf=%p, addr=%08" PRIx32,
+				__func__, count, buffer, phys_address);
+		return ERROR_COMMAND_ARGUMENT_INVALID;
+	}
+
+	/* to access physical memory, switch off the CR0.PG bit */
+	if (x86_32->is_paging_enabled(t)) {
+		retval = x86_32->disable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+		pg_disabled = true;
+	}
+
+	for (uint32_t i = 0; i < count; i++) {
+		switch (size) {
+		case BYTE:
+			retval = read_mem(t, size, phys_address + i, buffer + i);
+			break;
+		case WORD:
+			retval = read_mem(t, size, phys_address + i * 2, buffer + i * 2);
+			break;
+		case DWORD:
+			retval = read_mem(t, size, phys_address + i * 4, buffer + i * 4);
+			break;
+		default:
+			LOG_ERROR("%s invalid read size", __func__);
+			break;
+		}
+	}
+	/* restore CR0.PG bit if needed (regardless of retval) */
+	if (pg_disabled) {
+		retval = x86_32->enable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+		pg_disabled = true;
+	}
+	/* TODO: After reading memory from target, we must replace
+	 * software breakpoints with the original instructions again.
+	 * Solve this with the breakpoint fix
+	 */
+	return retval;
+}
+
+int x86_32_common_write_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, const uint8_t *buffer)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	int error = ERROR_OK;
+	uint8_t *newbuffer = NULL;
+
+	check_not_halted(t);
+	if (!count || !buffer || !phys_address) {
+		LOG_ERROR("%s invalid params count=%d, buf=%p, addr=%08" PRIx32,
+				__func__, count, buffer, phys_address);
+		return ERROR_COMMAND_ARGUMENT_INVALID;
+	}
+	/* Before writing memory to target, we must update software breakpoints
+	 * with the new instructions and patch the memory buffer with the
+	 * breakpoint instruction.
+	 */
+	newbuffer = malloc(size*count);
+	if (newbuffer == NULL) {
+		LOG_ERROR("%s out of memory", __func__);
+		return ERROR_FAIL;
+	}
+	memcpy(newbuffer, buffer, size*count);
+	struct swbp_mem_patch *iter = x86_32->swbbp_mem_patch_list;
+	while (iter != NULL) {
+		if (iter->physaddr >= phys_address && iter->physaddr < phys_address+(size*count)) {
+			uint32_t offset = iter->physaddr - phys_address;
+			newbuffer[offset] = SW_BP_OPCODE;
+
+			/* update the breakpoint */
+			struct breakpoint *pbiter = t->breakpoints;
+			while (pbiter != NULL && pbiter->unique_id != iter->swbp_unique_id)
+				pbiter = pbiter->next;
+			if (pbiter)
+				pbiter->orig_instr[0] = buffer[offset];
+		}
+		iter = iter->next;
+	}
+
+	error = write_phys_mem(t, phys_address, size, count, newbuffer);
+	free(newbuffer);
+	return error;
+}
+
+static int write_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, const uint8_t *buffer)
+{
+	int retval = ERROR_OK;
+	bool pg_disabled = false;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("addr=%08" PRIx32 ", size=%d, count=%d, buf=%p",
+			phys_address, size, count, buffer);
+
+	check_not_halted(t);
+	if (!count || !buffer || !phys_address) {
+		LOG_ERROR("%s invalid params count=%d, buf=%p, addr=%08" PRIx32,
+				__func__, count, buffer, phys_address);
+		return ERROR_COMMAND_ARGUMENT_INVALID;
+	}
+	/* TODO: Before writing memory to target, we must update
+	 * software breakpoints with the new instructions and
+	 * patch the memory buffer with the breakpoint instruction.
+	 * Solve this with the breakpoint fix
+	 */
+
+	/* to access physical memory, switch off the CR0.PG bit */
+	if (x86_32->is_paging_enabled(t)) {
+		retval = x86_32->disable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+		pg_disabled = true;
+	}
+	for (uint32_t i = 0; i < count; i++) {
+		switch (size) {
+		case BYTE:
+			retval = write_mem(t, size, phys_address + i, buffer + i);
+			break;
+		case WORD:
+			retval = write_mem(t, size, phys_address + i * 2, buffer + i * 2);
+			break;
+		case DWORD:
+			retval = write_mem(t, size, phys_address + i * 4, buffer + i * 4);
+			break;
+		default:
+			LOG_DEBUG("invalid read size");
+			break;
+		}
+	}
+	/* restore CR0.PG bit if needed (regardless of retval) */
+	if (pg_disabled) {
+		retval = x86_32->enable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+	}
+	return retval;
+}
+
+static int read_mem(struct target *t, uint32_t size,
+			uint32_t addr, uint8_t *buf)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	/* if CS.D bit=1 then its a 32 bit code segment, else 16 */
+	bool use32 = (buf_get_u32(x86_32->cache->reg_list[CSAR].value, 0, 32)) & CSAR_D;
+	int retval = x86_32->write_hw_reg(t, EAX, addr, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error write EAX", __func__);
+		return retval;
+	}
+
+	switch (size) {
+		case BYTE:
+			if (use32)
+				retval = x86_32->submit_instruction(t, MEMRDB32);
+			else
+				retval = x86_32->submit_instruction(t, MEMRDB16);
+			break;
+		case WORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, MEMRDH32);
+			else
+				retval = x86_32->submit_instruction(t, MEMRDH16);
+			break;
+		case DWORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, MEMRDW32);
+			else
+				retval = x86_32->submit_instruction(t, MEMRDW16);
+			break;
+		default:
+			LOG_ERROR("%s invalid read mem size", __func__);
+			break;
+	}
+
+	/* read_hw_reg() will write to 4 bytes (uint32_t)
+	 * Watch out, the buffer passed into read_mem() might be 1 or 2 bytes.
+	 */
+	uint32_t regval;
+	retval = x86_32->read_hw_reg(t, EDX, &regval, 0);
+
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error read EDX", __func__);
+		return retval;
+	}
+	for (uint8_t i = 0; i < size; i++)
+		buf[i] = (regval >> (i*8)) & 0x000000FF;
+
+	retval = x86_32->transaction_status(t);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on mem read", __func__);
+		return retval;
+	}
+	return retval;
+}
+
+static int write_mem(struct target *t, uint32_t size,
+			uint32_t addr, const uint8_t *buf)
+{
+	uint32_t i = 0;
+	uint32_t buf4bytes = 0;
+	int retval = ERROR_OK;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	for (i = 0; i < size; ++i) {
+		buf4bytes = buf4bytes << 8; /* first time we only shift 0s */
+		buf4bytes += buf[(size-1)-i]; /* it was hard to write, should be hard to read! */
+	}
+	/* if CS.D bit=1 then its a 32 bit code segment, else 16 */
+	bool use32 = (buf_get_u32(x86_32->cache->reg_list[CSAR].value, 0, 32)) & CSAR_D;
+	retval = x86_32->write_hw_reg(t, EAX, addr, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error write EAX", __func__);
+		return retval;
+	}
+
+	/* write_hw_reg() will write to 4 bytes (uint32_t)
+	 * Watch out, the buffer passed into write_mem() might be 1 or 2 bytes.
+	 */
+	retval = x86_32->write_hw_reg(t, EDX, buf4bytes, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error write EDX", __func__);
+		return retval;
+	}
+	switch (size) {
+		case BYTE:
+			if (use32)
+				retval = x86_32->submit_instruction(t, MEMWRB32);
+			else
+				retval = x86_32->submit_instruction(t, MEMWRB16);
+			break;
+		case WORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, MEMWRH32);
+			else
+				retval = x86_32->submit_instruction(t, MEMWRH16);
+			break;
+		case DWORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, MEMWRW32);
+			else
+				retval = x86_32->submit_instruction(t, MEMWRW16);
+			break;
+		default:
+			LOG_ERROR("%s invalid write mem size", __func__);
+			return ERROR_FAIL;
+	}
+	retval = x86_32->transaction_status(t);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on mem write", __func__);
+		return retval;
+	}
+	return retval;
+}
+
+int calcaddr_pyhsfromlin(struct target *t, uint32_t addr, uint32_t *physaddr)
+{
+	uint8_t entry_buffer[8];
+
+	if (physaddr == NULL || t == NULL)
+		return ERROR_FAIL;
+
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	/* The 'user-visible' CR0.PG should be set - otherwise the function shouldn't be called
+	 * (Don't check the CR0.PG on the target, this might be temporally disabled at this point)
+	 */
+	uint32_t cr0 = buf_get_u32(x86_32->cache->reg_list[CR0].value, 0, 32);
+	if (!(cr0 & CR0_PG)) {
+		/* you are wrong in this function, never mind */
+		*physaddr = addr;
+		return ERROR_OK;
+	}
+
+	uint32_t cr4 = buf_get_u32(x86_32->cache->reg_list[CR4].value, 0, 32);
+	bool isPAE = cr4 & 0x00000020; /* PAE - Physical Address Extension */
+
+	uint32_t cr3 = buf_get_u32(x86_32->cache->reg_list[CR3].value, 0, 32);
+	if (isPAE) {
+		uint32_t pdpt_base = cr3 & 0xFFFFF000; /* lower 12 bits of CR3 must always be 0 */
+		uint32_t pdpt_index = (addr & 0xC0000000) >> 30; /* A[31:30] index to PDPT */
+		uint32_t pdpt_addr = pdpt_base + (8 * pdpt_index);
+		if (x86_32_common_read_phys_mem(t, pdpt_addr, 4, 2, entry_buffer) != ERROR_OK) {
+			LOG_ERROR("%s couldn't read page directory pointer table entry at 0x%08" PRIx32,
+					__func__, pdpt_addr);
+			return ERROR_FAIL;
+		}
+		uint64_t pdpt_entry = target_buffer_get_u64(t, entry_buffer);
+		if (!(pdpt_entry & 0x0000000000000001)) {
+			LOG_ERROR("%s page directory pointer table entry at 0x%08" PRIx32 " is not present",
+					__func__, pdpt_addr);
+			return ERROR_FAIL;
+		}
+
+		uint32_t pd_base = pdpt_entry & 0xFFFFF000; /* A[31:12] is PageTable/Page Base Address */
+		uint32_t pd_index = (addr & 0x3FE00000) >> 21; /* A[29:21] index to PD entry with PAE */
+		uint32_t pd_addr = pd_base + (8 * pd_index);
+		if (x86_32_common_read_phys_mem(t, pd_addr, 4, 2, entry_buffer) != ERROR_OK) {
+			LOG_ERROR("%s couldn't read page directory entry at 0x%08" PRIx32,
+					__func__, pd_addr);
+			return ERROR_FAIL;
+		}
+		uint64_t pd_entry = target_buffer_get_u64(t, entry_buffer);
+		if (!(pd_entry & 0x0000000000000001)) {
+			LOG_ERROR("%s page directory entry at 0x%08" PRIx32 " is not present",
+					__func__, pd_addr);
+			return ERROR_FAIL;
+		}
+
+		/* PS bit in PD entry is indicating 4KB or 2MB page size */
+		if (pd_entry & 0x0000000000000080) {
+
+			uint32_t page_base = (uint32_t)(pd_entry & 0x00000000FFE00000); /* [31:21] */
+			uint32_t offset = addr & 0x001FFFFF; /* [20:0] */
+			*physaddr = page_base + offset;
+			return ERROR_OK;
+
+		} else {
+
+			uint32_t pt_base = (uint32_t)(pd_entry & 0x00000000FFFFF000); /*[31:12]*/
+			uint32_t pt_index = (addr & 0x001FF000) >> 12; /*[20:12]*/
+			uint32_t pt_addr = pt_base + (8 * pt_index);
+			if (x86_32_common_read_phys_mem(t, pt_addr, 4, 2, entry_buffer) != ERROR_OK) {
+				LOG_ERROR("%s couldn't read page table entry at 0x%08" PRIx32, __func__, pt_addr);
+				return ERROR_FAIL;
+			}
+			uint64_t pt_entry = target_buffer_get_u64(t, entry_buffer);
+			if (!(pt_entry & 0x0000000000000001)) {
+				LOG_ERROR("%s page table entry at 0x%08" PRIx32 " is not present", __func__, pt_addr);
+				return ERROR_FAIL;
+			}
+
+			uint32_t page_base = (uint32_t)(pt_entry & 0x00000000FFFFF000); /*[31:12]*/
+			uint32_t offset =  addr & 0x00000FFF; /*[11:0]*/
+			*physaddr = page_base + offset;
+			return ERROR_OK;
+		}
+	} else {
+		uint32_t pd_base = cr3 & 0xFFFFF000; /* lower 12 bits of CR3 must always be 0 */
+		uint32_t pd_index = (addr & 0xFFC00000) >> 22; /* A[31:22] index to PD entry */
+		uint32_t pd_addr = pd_base + (4 * pd_index);
+		if (x86_32_common_read_phys_mem(t, pd_addr, 4, 1, entry_buffer) != ERROR_OK) {
+			LOG_ERROR("%s couldn't read page directory entry at 0x%08" PRIx32, __func__, pd_addr);
+			return ERROR_FAIL;
+		}
+		uint32_t pd_entry = target_buffer_get_u32(t, entry_buffer);
+		if (!(pd_entry & 0x00000001)) {
+			LOG_ERROR("%s page directory entry at 0x%08" PRIx32 " is not present", __func__, pd_addr);
+			return ERROR_FAIL;
+		}
+
+		/* Bit 7 in page directory entry is page size.
+		 */
+		if (pd_entry & 0x00000080) {
+			/* 4MB pages */
+			uint32_t page_base = pd_entry & 0xFFC00000;
+			*physaddr = page_base + (addr & 0x003FFFFF);
+
+		} else {
+			/* 4KB pages */
+			uint32_t pt_base = pd_entry & 0xFFFFF000; /* A[31:12] is PageTable/Page Base Address */
+			uint32_t pt_index = (addr & 0x003FF000) >> 12; /* A[21:12] index to page table entry */
+			uint32_t pt_addr = pt_base + (4 * pt_index);
+			if (x86_32_common_read_phys_mem(t, pt_addr, 4, 1, entry_buffer) != ERROR_OK) {
+				LOG_ERROR("%s couldn't read page table entry at 0x%08" PRIx32, __func__, pt_addr);
+				return ERROR_FAIL;
+			}
+			uint32_t pt_entry = target_buffer_get_u32(t, entry_buffer);
+			if (!(pt_entry & 0x00000001)) {
+				LOG_ERROR("%s page table entry at 0x%08" PRIx32 " is not present", __func__, pt_addr);
+				return ERROR_FAIL;
+			}
+			uint32_t page_base = pt_entry & 0xFFFFF000; /* A[31:12] is PageTable/Page Base Address */
+			*physaddr = page_base + (addr & 0x00000FFF); /* A[11:0] offset to 4KB page in linear address */
+		}
+	}
+	return ERROR_OK;
+}
+
+int x86_32_common_read_memory(struct target *t, uint32_t addr,
+			uint32_t size, uint32_t count, uint8_t *buf)
+{
+	int retval = ERROR_OK;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("addr=%08" PRIx32 ", size=%d, count=%d, buf=%p",
+			addr, size, count, buf);
+	check_not_halted(t);
+	if (!count || !buf || !addr) {
+		LOG_ERROR("%s invalid params count=%d, buf=%p, addr=%08" PRIx32,
+				__func__, count, buf, addr);
+		return ERROR_COMMAND_ARGUMENT_INVALID;
+	}
+
+	if (x86_32->is_paging_enabled(t)) {
+		/* all memory accesses from debugger must be physical (CR0.PG == 0)
+		 * conversion to physical address space needed
+		 */
+		retval = x86_32->disable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+		uint32_t physaddr = 0;
+		if (calcaddr_pyhsfromlin(t, addr, &physaddr) != ERROR_OK) {
+			LOG_ERROR("%s failed to calculate physical address from 0x%08" PRIx32, __func__, addr);
+			retval = ERROR_FAIL;
+		}
+		/* TODO: !!! Watch out for page boundaries
+		 * for every 4kB, the physical address has to be re-calculated
+		 * This should be fixed together with bulk memory reads
+		 */
+
+		if (retval == ERROR_OK
+			&& x86_32_common_read_phys_mem(t, physaddr, size, count, buf) != ERROR_OK) {
+			LOG_ERROR("%s failed to read memory from physical address 0x%08" PRIx32, __func__, physaddr);
+			retval = ERROR_FAIL;
+		}
+		/* restore PG bit if it was cleared prior (regardless of retval) */
+		retval = x86_32->enable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+	} else {
+		/* paging is off - linear address is physical address */
+		if (x86_32_common_read_phys_mem(t, addr, size, count, buf) != ERROR_OK) {
+			LOG_ERROR("%s failed to read memory from address 0%08" PRIx32, __func__, addr);
+			retval = ERROR_FAIL;
+		}
+	}
+
+	return retval;
+}
+
+int x86_32_common_write_memory(struct target *t, uint32_t addr,
+			uint32_t size, uint32_t count, const uint8_t *buf)
+{
+	int retval = ERROR_OK;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("addr=%08" PRIx32 ", size=%d, count=%d, buf=%p",
+			addr, size, count, buf);
+	check_not_halted(t);
+	if (!count || !buf || !addr) {
+		LOG_ERROR("%s invalid params count=%d, buf=%p, addr=%08" PRIx32,
+					__func__, count, buf, addr);
+		return ERROR_COMMAND_ARGUMENT_INVALID;
+	}
+	if (x86_32->is_paging_enabled(t)) {
+		/* all memory accesses from debugger must be physical (CR0.PG == 0)
+		 * conversion to physical address space needed
+		 */
+		retval = x86_32->disable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+		uint32_t physaddr = 0;
+		if (calcaddr_pyhsfromlin(t, addr, &physaddr) != ERROR_OK) {
+			LOG_ERROR("%s failed to calculate physical address from 0x%08" PRIx32,
+					__func__, addr);
+			retval = ERROR_FAIL;
+		}
+		/* TODO: !!! Watch out for page boundaries
+		 * for every 4kB, the physical address has to be re-calculated
+		 * This should be fixed together with bulk memory reads
+		 */
+		if (retval == ERROR_OK
+			&& x86_32_common_write_phys_mem(t, physaddr, size, count, buf) != ERROR_OK) {
+			LOG_ERROR("%s failed to write memory to physical address 0x%08" PRIx32,
+					__func__, physaddr);
+			retval = ERROR_FAIL;
+		}
+		/* restore PG bit if it was cleared prior (regardless of retval) */
+		retval = x86_32->enable_paging(t);
+		if (retval != ERROR_OK)
+			return retval;
+	} else {
+
+		/* paging is off - linear address is physical address */
+		if (x86_32_common_write_phys_mem(t, addr, size, count, buf) != ERROR_OK) {
+			LOG_ERROR("%s failed to write memory to address 0x%08" PRIx32,
+					__func__, addr);
+			retval = ERROR_FAIL;
+		}
+	}
+	return retval;
+}
+
+int x86_32_common_read_io(struct target *t, uint32_t addr,
+			uint32_t size, uint8_t *buf)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	/* if CS.D bit=1 then its a 32 bit code segment, else 16 */
+	bool use32 = (buf_get_u32(x86_32->cache->reg_list[CSAR].value, 0, 32)) & CSAR_D;
+	int retval = ERROR_FAIL;
+	LOG_DEBUG("addr=%08" PRIx32 ", size=%d, buf=%p", addr, size, buf);
+	check_not_halted(t);
+	if (!buf || !addr) {
+		LOG_ERROR("%s invalid params buf=%p, addr=%08" PRIx32, __func__, buf, addr);
+		return retval;
+	}
+	retval = x86_32->write_hw_reg(t, EDX, addr, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error EDX write", __func__);
+		return retval;
+	}
+	switch (size) {
+		case BYTE:
+			if (use32)
+				retval = x86_32->submit_instruction(t, IORDB32);
+			else
+				retval = x86_32->submit_instruction(t, IORDB16);
+			break;
+		case WORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, IORDH32);
+			else
+				retval = x86_32->submit_instruction(t, IORDH16);
+			break;
+		case DWORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, IORDW32);
+			else
+				retval = x86_32->submit_instruction(t, IORDW16);
+			break;
+		default:
+			LOG_ERROR("%s invalid read io size", __func__);
+			return ERROR_FAIL;
+	}
+	uint32_t regval = 0;
+	retval = x86_32->read_hw_reg(t, EAX, &regval, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on read EAX", __func__);
+		return retval;
+	}
+	for (uint8_t i = 0; i < size; i++)
+		buf[i] = (regval >> (i*8)) & 0x000000FF;
+	retval = x86_32->transaction_status(t);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on io read", __func__);
+		return retval;
+	}
+	return retval;
+}
+
+int x86_32_common_write_io(struct target *t, uint32_t addr,
+			uint32_t size, const uint8_t *buf)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	/* if CS.D bit=1 then its a 32 bit code segment, else 16 */
+	bool use32 = (buf_get_u32(x86_32->cache->reg_list[CSAR].value, 0, 32)) & CSAR_D;
+	LOG_DEBUG("addr=%08" PRIx32 ", size=%d, buf=%p", addr, size, buf);
+	check_not_halted(t);
+	int retval = ERROR_FAIL;
+	if (!buf || !addr) {
+		LOG_ERROR("%s invalid params buf=%p, addr=%08" PRIx32, __func__, buf, addr);
+		return retval;
+	}
+	/* no do the write */
+	retval = x86_32->write_hw_reg(t, EDX, addr, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on EDX write", __func__);
+		return retval;
+	}
+	uint32_t regval = 0;
+	for (uint8_t i = 0; i < size; i++)
+		regval += (buf[i] << (i*8));
+	retval = x86_32->write_hw_reg(t, EAX, regval, 0);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on EAX write", __func__);
+		return retval;
+	}
+	switch (size) {
+		case BYTE:
+			if (use32)
+				retval = x86_32->submit_instruction(t, IOWRB32);
+			else
+				retval = x86_32->submit_instruction(t, IOWRB16);
+			break;
+		case WORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, IOWRH32);
+			else
+				retval = x86_32->submit_instruction(t, IOWRH16);
+			break;
+		case DWORD:
+			if (use32)
+				retval = x86_32->submit_instruction(t, IOWRW32);
+			else
+				retval = x86_32->submit_instruction(t, IOWRW16);
+			break;
+		default:
+			LOG_ERROR("%s invalid write io size", __func__);
+			return ERROR_FAIL;
+	}
+	retval = x86_32->transaction_status(t);
+	if (retval != ERROR_OK) {
+		LOG_ERROR("%s error on io write", __func__);
+		return retval;
+	}
+	return retval;
+}
+
+int x86_32_common_add_watchpoint(struct target *t, struct watchpoint *wp)
+{
+	check_not_halted(t);
+	/* set_watchpoint() will return ERROR_TARGET_RESOURCE_NOT_AVAILABLE if all
+	 * hardware registers are gone
+	 */
+	return set_watchpoint(t, wp);
+}
+
+int x86_32_common_remove_watchpoint(struct target *t, struct watchpoint *wp)
+{
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	if (wp->set)
+		unset_watchpoint(t, wp);
+	return ERROR_OK;
+}
+
+int x86_32_common_add_breakpoint(struct target *t, struct breakpoint *bp)
+{
+	LOG_DEBUG("type=%d, addr=%08" PRIx32, bp->type, bp->address);
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	/* set_breakpoint() will return ERROR_TARGET_RESOURCE_NOT_AVAILABLE if all
+	 * hardware registers are gone (for hardware breakpoints)
+	 */
+	return set_breakpoint(t, bp);
+}
+
+int x86_32_common_remove_breakpoint(struct target *t, struct breakpoint *bp)
+{
+	LOG_DEBUG("type=%d, addr=%08" PRIx32, bp->type, bp->address);
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	if (bp->set)
+		unset_breakpoint(t, bp);
+
+	return ERROR_OK;
+}
+
+static int set_debug_regs(struct target *t, uint32_t address,
+			uint8_t bp_num, uint8_t bp_type, uint8_t bp_length)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("addr=%08" PRIx32 ", bp_num=%d, bp_type=%d, pb_length=%d",
+			address, bp_num, bp_type, bp_length);
+
+	/* DR7 - set global enable */
+	uint32_t dr7 = buf_get_u32(x86_32->cache->reg_list[DR7].value, 0, 32);
+
+	if (bp_length != 1 && bp_length != 2 && bp_length != 4)
+		return ERROR_FAIL;
+
+	if (DR7_BP_FREE(dr7, bp_num))
+		DR7_GLOBAL_ENABLE(dr7, bp_num);
+	else {
+		LOG_ERROR("%s dr7 error, already enabled, val=%08" PRIx32, __func__, dr7);
+		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
+	}
+
+	switch (bp_type) {
+		case 0:
+			/* 00 - only on instruction execution */
+			DR7_SET_EXE(dr7, bp_num);
+			DR7_SET_LENGTH(dr7, bp_num, bp_length);
+		break;
+		case 1:
+			/* 01 - only on data writes */
+			DR7_SET_WRITE(dr7, bp_num);
+			DR7_SET_LENGTH(dr7, bp_num, bp_length);
+		break;
+		case 2:
+			/* 10 UNSUPPORTED - an I/O read and I/O write */
+			LOG_ERROR("%s unsupported feature bp_type=%d", __func__, bp_type);
+			return ERROR_FAIL;
+		break;
+		case 3:
+			/* on data read or data write */
+			DR7_SET_ACCESS(dr7, bp_num);
+			DR7_SET_LENGTH(dr7, bp_num, bp_length);
+		break;
+		default:
+			LOG_ERROR("%s invalid request [only 0-3] bp_type=%d", __func__, bp_type);
+			return ERROR_FAIL;
+	}
+
+	/* update regs in the reg cache ready to be written to hardware
+	 * when we exit PM
+	*/
+	buf_set_u32(x86_32->cache->reg_list[bp_num+DR0].value, 0, 32, address);
+	x86_32->cache->reg_list[bp_num+DR0].dirty = 1;
+	x86_32->cache->reg_list[bp_num+DR0].valid = 1;
+	buf_set_u32(x86_32->cache->reg_list[DR6].value, 0, 32, PM_DR6);
+	x86_32->cache->reg_list[DR6].dirty = 1;
+	x86_32->cache->reg_list[DR6].valid = 1;
+	buf_set_u32(x86_32->cache->reg_list[DR7].value, 0, 32, dr7);
+	x86_32->cache->reg_list[DR7].dirty = 1;
+	x86_32->cache->reg_list[DR7].valid = 1;
+	return ERROR_OK;
+}
+
+static int unset_debug_regs(struct target *t, uint8_t bp_num)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("bp_num=%d", bp_num);
+
+	uint32_t dr7 = buf_get_u32(x86_32->cache->reg_list[DR7].value, 0, 32);
+
+	if (!(DR7_BP_FREE(dr7, bp_num))) {
+		DR7_GLOBAL_DISABLE(dr7, bp_num);
+	} else {
+		LOG_ERROR("%s dr7 error, not enabled, val=%08" PRIx32, __func__, dr7);
+		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
+	}
+	/* this will clear rw and len bits */
+	DR7_RESET_RWLEN_BITS(dr7, bp_num);
+
+	/* update regs in the reg cache ready to be written to hardware
+	 * when we exit PM
+	*/
+	buf_set_u32(x86_32->cache->reg_list[bp_num+DR0].value, 0, 32, 0);
+	x86_32->cache->reg_list[bp_num+DR0].dirty = 1;
+	x86_32->cache->reg_list[bp_num+DR0].valid = 1;
+	buf_set_u32(x86_32->cache->reg_list[DR6].value, 0, 32, PM_DR6);
+	x86_32->cache->reg_list[DR6].dirty = 1;
+	x86_32->cache->reg_list[DR6].valid = 1;
+	buf_set_u32(x86_32->cache->reg_list[DR7].value, 0, 32, dr7);
+	x86_32->cache->reg_list[DR7].dirty = 1;
+	x86_32->cache->reg_list[DR7].valid = 1;
+	return ERROR_OK;
+}
+
+static int set_hwbp(struct target *t, struct breakpoint *bp)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	struct x86_32_dbg_reg *debug_reg_list = x86_32->hw_break_list;
+	uint8_t hwbp_num = 0;
+
+	while (debug_reg_list[hwbp_num].used && (hwbp_num < x86_32->num_hw_bpoints))
+		hwbp_num++;
+	if (hwbp_num >= x86_32->num_hw_bpoints) {
+		LOG_ERROR("%s no free hw breakpoint bpid=%d", __func__, bp->unique_id);
+		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
+	}
+	if (set_debug_regs(t, bp->address, hwbp_num, DR7_BP_EXECUTE, 1) != ERROR_OK)
+		return ERROR_FAIL;
+	bp->set = hwbp_num + 1;
+	debug_reg_list[hwbp_num].used = 1;
+	debug_reg_list[hwbp_num].bp_value = bp->address;
+	LOG_USER("%s hardware breakpoint %d set at 0x%08" PRIx32 " (hwreg=%d)", __func__,
+			bp->unique_id, debug_reg_list[hwbp_num].bp_value, hwbp_num);
+	return ERROR_OK;
+}
+
+static int unset_hwbp(struct target *t, struct breakpoint *bp)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	struct x86_32_dbg_reg *debug_reg_list = x86_32->hw_break_list;
+	int hwbp_num = bp->set - 1;
+
+	if ((hwbp_num < 0) || (hwbp_num >= x86_32->num_hw_bpoints)) {
+		LOG_ERROR("%s invalid breakpoint number=%d, bpid=%d",
+				__func__, hwbp_num, bp->unique_id);
+		return ERROR_OK;
+	}
+
+	if (unset_debug_regs(t, hwbp_num) != ERROR_OK)
+		return ERROR_FAIL;
+	debug_reg_list[hwbp_num].used = 0;
+	debug_reg_list[hwbp_num].bp_value = 0;
+
+	LOG_USER("%s hardware breakpoint %d removed from 0x%08" PRIx32 " (hwreg=%d)",
+			__func__, bp->unique_id, bp->address, hwbp_num);
+	return ERROR_OK;
+}
+
+static int set_swbp(struct target *t, struct breakpoint *bp)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("id %d", bp->unique_id);
+	uint32_t physaddr;
+	uint8_t opcode = SW_BP_OPCODE;
+	uint8_t readback;
+
+	if (calcaddr_pyhsfromlin(t, bp->address, &physaddr) != ERROR_OK)
+		return ERROR_FAIL;
+	if (read_phys_mem(t, physaddr, 1, 1, bp->orig_instr))
+		return ERROR_FAIL;
+
+	LOG_DEBUG("set software breakpoint - orig byte=%02" PRIx8 "", *bp->orig_instr);
+
+	/* just write the instruction trap byte */
+	if (write_phys_mem(t, physaddr, 1, 1, &opcode))
+		return ERROR_FAIL;
+
+	/* verify that this is not invalid/read-only memory */
+	if (read_phys_mem(t, physaddr, 1, 1, &readback))
+		return ERROR_FAIL;
+
+	if (readback != SW_BP_OPCODE) {
+		LOG_ERROR("%s software breakpoint error at 0x%08" PRIx32 ", check memory",
+				__func__, bp->address);
+		LOG_ERROR("%s readback=%02" PRIx8 " orig=%02" PRIx8 "",
+				__func__, readback, *bp->orig_instr);
+		return ERROR_FAIL;
+	}
+	bp->set = SW_BP_OPCODE; /* just non 0 */
+
+	/* add the memory patch */
+	struct swbp_mem_patch *new_patch = malloc(sizeof(struct swbp_mem_patch));
+	if (new_patch == NULL) {
+		LOG_ERROR("%s out of memory", __func__);
+		return ERROR_FAIL;
+	}
+	new_patch->next = NULL;
+	new_patch->orig_byte = *bp->orig_instr;
+	new_patch->physaddr = physaddr;
+	new_patch->swbp_unique_id = bp->unique_id;
+
+	struct swbp_mem_patch *addto = x86_32->swbbp_mem_patch_list;
+	if (addto == NULL)
+		x86_32->swbbp_mem_patch_list = new_patch;
+	else {
+		while (addto->next != NULL)
+			addto = addto->next;
+		addto->next = new_patch;
+	}
+	LOG_USER("%s software breakpoint %d set at 0x%08" PRIx32,
+			__func__, bp->unique_id, bp->address);
+	return ERROR_OK;
+}
+
+static int unset_swbp(struct target *t, struct breakpoint *bp)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("id %d", bp->unique_id);
+	uint32_t physaddr;
+	uint8_t current_instr;
+
+	/* check that user program has not modified breakpoint instruction */
+	if (calcaddr_pyhsfromlin(t, bp->address, &physaddr) != ERROR_OK)
+		return ERROR_FAIL;
+	if (read_phys_mem(t, physaddr, 1, 1, &current_instr))
+		return ERROR_FAIL;
+
+	if (current_instr == SW_BP_OPCODE) {
+		if (write_phys_mem(t, physaddr, 1, 1, bp->orig_instr))
+			return ERROR_FAIL;
+	} else {
+		LOG_ERROR("%s software breakpoint remove error at 0x%08" PRIx32 ", check memory",
+				__func__, bp->address);
+		LOG_ERROR("%s current=%02" PRIx8 " orig=%02" PRIx8 "",
+				__func__, current_instr, *bp->orig_instr);
+		return ERROR_FAIL;
+	}
+
+	/* remove from patch */
+	struct swbp_mem_patch *iter = x86_32->swbbp_mem_patch_list;
+	if (iter != NULL) {
+		if (iter->swbp_unique_id == bp->unique_id) {
+			/* it's the first item */
+			x86_32->swbbp_mem_patch_list = iter->next;
+			free(iter);
+		} else {
+			while (iter->next != NULL && iter->next->swbp_unique_id != bp->unique_id)
+				iter = iter->next;
+			if (iter->next != NULL) {
+				/* it's the next one */
+				struct swbp_mem_patch *freeme = iter->next;
+				iter->next = iter->next->next;
+				free(freeme);
+			}
+		}
+	}
+
+	LOG_USER("%s software breakpoint %d removed from 0x%08" PRIx32,
+			__func__, bp->unique_id, bp->address);
+	return ERROR_OK;
+}
+
+static int set_breakpoint(struct target *t, struct breakpoint *bp)
+{
+	int error = ERROR_OK;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	LOG_DEBUG("type=%d, addr=%08" PRIx32, bp->type, bp->address);
+	if (bp->set) {
+		LOG_ERROR("breakpoint already set");
+		return error;
+	}
+	if (bp->type == BKPT_HARD) {
+		error = set_hwbp(t, bp);
+		if (error != ERROR_OK) {
+			LOG_ERROR("%s error setting hardware breakpoint at 0x%08" PRIx32,
+					__func__, bp->address);
+			return error;
+		}
+	} else {
+		if (x86_32->sw_bpts_supported(t)) {
+			error = set_swbp(t, bp);
+			if (error != ERROR_OK) {
+				LOG_ERROR("%s error setting software breakpoint at 0x%08" PRIx32,
+						__func__, bp->address);
+				return error;
+			}
+		} else {
+			LOG_ERROR("%s core doesn't support SW breakpoints", __func__);
+			error = ERROR_FAIL;
+			return ERROR_FAIL;
+		}
+	}
+	return error;
+}
+
+static int unset_breakpoint(struct target *t, struct breakpoint *bp)
+{
+	LOG_DEBUG("type=%d, addr=%08" PRIx32, bp->type, bp->address);
+	if (!bp->set) {
+		LOG_WARNING("breakpoint not set");
+		return ERROR_OK;
+	}
+
+	if (bp->type == BKPT_HARD) {
+		if (unset_hwbp(t, bp) != ERROR_OK) {
+			LOG_ERROR("%s error removing hardware breakpoint at 0x%08" PRIx32,
+					__func__, bp->address);
+			return ERROR_FAIL;
+		}
+	} else {
+		if (unset_swbp(t, bp) != ERROR_OK) {
+			LOG_ERROR("%s error removing software breakpoint at 0x%08" PRIx32,
+					__func__, bp->address);
+			return ERROR_FAIL;
+		}
+	}
+	bp->set = 0;
+	return ERROR_OK;
+}
+
+static int set_watchpoint(struct target *t, struct watchpoint *wp)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	struct x86_32_dbg_reg *debug_reg_list = x86_32->hw_break_list;
+	int wp_num = 0;
+	LOG_DEBUG("type=%d, addr=%08" PRIx32, wp->rw, wp->address);
+
+	if (wp->set) {
+		LOG_ERROR("%s watchpoint already set", __func__);
+		return ERROR_OK;
+	}
+
+	if (wp->rw == WPT_READ) {
+		LOG_ERROR("%s no support for 'read' watchpoints, use 'access' or 'write'"
+				, __func__);
+		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
+	}
+
+	while (debug_reg_list[wp_num].used && (wp_num < x86_32->num_hw_bpoints))
+		wp_num++;
+	if (wp_num >= x86_32->num_hw_bpoints) {
+		LOG_ERROR("%s no debug registers left", __func__);
+		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
+	}
+
+	if (wp->length != 4 && wp->length != 2 && wp->length != 1) {
+		LOG_ERROR("%s only watchpoints of length 1, 2 or 4 are supported", __func__);
+		return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
+	}
+
+	switch (wp->rw) {
+		case WPT_WRITE:
+			if (set_debug_regs(t, wp->address, wp_num,
+						DR7_BP_WRITE, wp->length) != ERROR_OK) {
+				return ERROR_FAIL;
+			}
+			break;
+		case WPT_ACCESS:
+			if (set_debug_regs(t, wp->address, wp_num, DR7_BP_READWRITE,
+						wp->length) != ERROR_OK) {
+				return ERROR_FAIL;
+			}
+			break;
+		default:
+			LOG_ERROR("%s only 'access' or 'write' watchpoints are supported", __func__);
+			break;
+	}
+	wp->set = wp_num + 1;
+	debug_reg_list[wp_num].used = 1;
+	debug_reg_list[wp_num].bp_value = wp->address;
+	LOG_USER("'%s' watchpoint %d set at 0x%08" PRIx32 " with length %d (hwreg=%d)",
+			wp->rw == WPT_READ ? "read" : wp->rw == WPT_WRITE ?
+			"write" : wp->rw == WPT_ACCESS ? "access" : "?",
+			wp->unique_id, wp->address, wp->length, wp_num);
+	return ERROR_OK;
+}
+
+static int unset_watchpoint(struct target *t, struct watchpoint *wp)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	struct x86_32_dbg_reg *debug_reg_list = x86_32->hw_break_list;
+	LOG_DEBUG("type=%d, addr=%08" PRIx32, wp->rw, wp->address);
+	if (!wp->set) {
+		LOG_WARNING("watchpoint not set");
+		return ERROR_OK;
+	}
+
+	int wp_num = wp->set - 1;
+	if ((wp_num < 0) || (wp_num >= x86_32->num_hw_bpoints)) {
+		LOG_DEBUG("Invalid FP Comparator number in watchpoint");
+		return ERROR_OK;
+	}
+	if (unset_debug_regs(t, wp_num) != ERROR_OK)
+		return ERROR_FAIL;
+
+	debug_reg_list[wp_num].used = 0;
+	debug_reg_list[wp_num].bp_value = 0;
+	wp->set = 0;
+
+	LOG_USER("'%s' watchpoint %d removed from 0x%08" PRIx32 " with length %d (hwreg=%d)",
+			wp->rw == WPT_READ ? "read" : wp->rw == WPT_WRITE ?
+			"write" : wp->rw == WPT_ACCESS ? "access" : "?",
+			wp->unique_id, wp->address, wp->length, wp_num);
+
+	return ERROR_OK;
+}
+
+static int read_hw_reg_to_cache(struct target *t, int num)
+{
+	uint32_t reg_value;
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	if ((num < 0) || (num >= x86_32->get_num_user_regs(t)))
+		return ERROR_COMMAND_SYNTAX_ERROR;
+	if (x86_32->read_hw_reg(t, num, &reg_value, 1) != ERROR_OK) {
+		LOG_ERROR("%s fail for %s", x86_32->cache->reg_list[num].name, __func__);
+		return ERROR_FAIL;
+	}
+	LOG_DEBUG("reg %s value 0x%08" PRIx32,
+			x86_32->cache->reg_list[num].name, reg_value);
+	return ERROR_OK;
+}
+
+static int write_hw_reg_from_cache(struct target *t, int num)
+{
+	struct x86_32_common *x86_32 = target_to_x86_32(t);
+	if (check_not_halted(t))
+		return ERROR_TARGET_NOT_HALTED;
+	if ((num < 0) || (num >= x86_32->get_num_user_regs(t)))
+		return ERROR_COMMAND_SYNTAX_ERROR;
+	if (x86_32->write_hw_reg(t, num, 0, 1) != ERROR_OK) {
+		LOG_ERROR("%s fail for %s", x86_32->cache->reg_list[num].name, __func__);
+		return ERROR_FAIL;
+	}
+	LOG_DEBUG("reg %s value 0x%08" PRIx32, x86_32->cache->reg_list[num].name,
+			buf_get_u32(x86_32->cache->reg_list[num].value, 0, 32));
+	return ERROR_OK;
+}
+
+/* x86 32 commands */
+static void handle_iod_output(struct command_context *cmd_ctx,
+		struct target *target, uint32_t address, unsigned size,
+		unsigned count, const uint8_t *buffer)
+{
+	const unsigned line_bytecnt = 32;
+	unsigned line_modulo = line_bytecnt / size;
+
+	char output[line_bytecnt * 4 + 1];
+	unsigned output_len = 0;
+
+	const char *value_fmt;
+	switch (size) {
+	case 4:
+		value_fmt = "%8.8x ";
+		break;
+	case 2:
+		value_fmt = "%4.4x ";
+		break;
+	case 1:
+		value_fmt = "%2.2x ";
+		break;
+	default:
+		/* "can't happen", caller checked */
+		LOG_ERROR("%s invalid memory read size: %u", __func__, size);
+		return;
+	}
+
+	for (unsigned i = 0; i < count; i++) {
+		if (i % line_modulo == 0) {
+			output_len += snprintf(output + output_len,
+					sizeof(output) - output_len,
+					"0x%8.8x: ",
+					(unsigned)(address + (i*size)));
+		}
+
+		uint32_t value = 0;
+		const uint8_t *value_ptr = buffer + i * size;
+		switch (size) {
+		case 4:
+			value = target_buffer_get_u32(target, value_ptr);
+			break;
+		case 2:
+			value = target_buffer_get_u16(target, value_ptr);
+			break;
+		case 1:
+			value = *value_ptr;
+		}
+		output_len += snprintf(output + output_len,
+				sizeof(output) - output_len,
+				value_fmt, value);
+
+		if ((i % line_modulo == line_modulo - 1) || (i == count - 1)) {
+			command_print(cmd_ctx, "%s", output);
+			output_len = 0;
+		}
+	}
+}
+
+COMMAND_HANDLER(handle_iod_command)
+{
+	if (CMD_ARGC != 1)
+		return ERROR_COMMAND_SYNTAX_ERROR;
+
+	uint32_t address;
+	COMMAND_PARSE_NUMBER(u32, CMD_ARGV[0], address);
+	if (address > 0xffff) {
+		LOG_ERROR("%s IA-32 I/O space is 2^16, %08" PRIx32 " exceeds max", __func__, address);
+		return ERROR_COMMAND_SYNTAX_ERROR;
+	}
+
+	unsigned size = 0;
+	switch (CMD_NAME[2]) {
+	case 'w':
+		size = 4;
+		break;
+	case 'h':
+		size = 2;
+		break;
+	case 'b':
+		size = 1;
+		break;
+	default:
+		return ERROR_COMMAND_SYNTAX_ERROR;
+	}
+	unsigned count = 1;
+	uint8_t *buffer = calloc(count, size);
+	struct target *target = get_current_target(CMD_CTX);
+	int retval = x86_32_common_read_io(target, address, size, buffer);
+	if (ERROR_OK == retval)
+		handle_iod_output(CMD_CTX, target, address, size, count, buffer);
+	free(buffer);
+	return retval;
+}
+
+static int target_fill_io(struct target *target,
+		uint32_t address,
+		unsigned data_size,
+		/* value */
+		uint32_t b)
+{
+	LOG_DEBUG("address=%08X, data_size=%d, b=%08X",
+			address, data_size, b);
+	uint8_t target_buf[data_size];
+	switch (data_size) {
+	case 4:
+		target_buffer_set_u32(target, target_buf, b);
+		break;
+	case 2:
+		target_buffer_set_u16(target, target_buf, b);
+		break;
+	case 1:
+		target_buf[0] = (b & 0x0ff);
+		break;
+	default:
+		exit(-1);
+	}
+	return x86_32_common_write_io(target, address, data_size, target_buf);
+}
+
+COMMAND_HANDLER(handle_iow_command)
+{
+	if (CMD_ARGC != 2)
+		return ERROR_COMMAND_SYNTAX_ERROR;
+	uint32_t address;
+	COMMAND_PARSE_NUMBER(u32, CMD_ARGV[0], address);
+	uint32_t value;
+	COMMAND_PARSE_NUMBER(u32, CMD_ARGV[1], value);
+	struct target *target = get_current_target(CMD_CTX);
+
+	unsigned wordsize;
+	switch (CMD_NAME[2]) {
+		case 'w':
+			wordsize = 4;
+			break;
+		case 'h':
+			wordsize = 2;
+			break;
+		case 'b':
+			wordsize = 1;
+			break;
+		default:
+			return ERROR_COMMAND_SYNTAX_ERROR;
+	}
+	return target_fill_io(target, address, wordsize, value);
+}
+
+static const struct command_registration x86_32_exec_command_handlers[] = {
+	{
+		.name = "iww",
+		.mode = COMMAND_EXEC,
+		.handler = handle_iow_command,
+		.help = "write I/O port word",
+		.usage = "port data[word]",
+	},
+	{
+		.name = "iwh",
+		.mode = COMMAND_EXEC,
+		.handler = handle_iow_command,
+		.help = "write I/O port halfword",
+		.usage = "port data[halfword]",
+	},
+	{
+		.name = "iwb",
+		.mode = COMMAND_EXEC,
+		.handler = handle_iow_command,
+		.help = "write I/O port byte",
+		.usage = "port data[byte]",
+	},
+	{
+		.name = "idw",
+		.mode = COMMAND_EXEC,
+		.handler = handle_iod_command,
+		.help = "display I/O port word",
+		.usage = "port",
+	},
+	{
+		.name = "idh",
+		.mode = COMMAND_EXEC,
+		.handler = handle_iod_command,
+		.help = "display I/O port halfword",
+		.usage = "port",
+	},
+	{
+		.name = "idb",
+		.mode = COMMAND_EXEC,
+		.handler = handle_iod_command,
+		.help = "display I/O port byte",
+		.usage = "port",
+	},
+
+	COMMAND_REGISTRATION_DONE
+};
+
+const struct command_registration x86_32_command_handlers[] = {
+	{
+		.name = "x86_32",
+		.mode = COMMAND_ANY,
+		.help = "x86_32 target commands",
+		.usage = "",
+		.chain = x86_32_exec_command_handlers,
+	},
+	COMMAND_REGISTRATION_DONE
+};
diff --git a/src/target/x86_32_common.h b/src/target/x86_32_common.h
new file mode 100644
index 0000000000000000000000000000000000000000..ef5a9ccae18b61a68a621ec397542264305603e1
--- /dev/null
+++ b/src/target/x86_32_common.h
@@ -0,0 +1,323 @@
+/*
+ * Copyright(c) 2013 Intel Corporation.
+ *
+ * Adrian Burns (adrian.burns@intel.com)
+ * Thomas Faust (thomas.faust@intel.com)
+ * Ivan De Cesaris (ivan.de.cesaris@intel.com)
+ * Julien Carreno (julien.carreno@intel.com)
+ * Jeffrey Maxwell (jeffrey.r.maxwell@intel.com)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of version 2 of the GNU General Public License as
+ * published by the Free Software Foundation.
+ *
+ * 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.
+ *
+ * Contact Information:
+ * Intel Corporation
+ */
+
+/*
+ * @file
+ * This is the interface to the x86 32 bit memory and breakpoint operations.
+ */
+
+#ifndef X86_32_COMMON_H
+#define X86_32_COMMON_H
+
+#include <jtag/jtag.h>
+#include <helper/command.h>
+
+extern const struct command_registration x86_32_command_handlers[];
+
+/* for memory access */
+#define BYTE			1
+#define WORD			2
+#define DWORD			4
+
+#define EFLAGS_TF		0x00000100 /* Trap Flag */
+#define EFLAGS_IF		0x00000200 /* Interrupt Flag */
+#define EFLAGS_RF		0x00010000 /* Resume Flag */
+#define EFLAGS_VM86		0x00020000 /* Virtual 8086 Mode */
+
+#define CSAR_DPL		0x00006000
+#define CSAR_D			0x00400000
+#define SSAR_DPL		0x00006000
+
+#define CR0_PE			0x00000001 /* Protected Mode Enable */
+#define CR0_NW			0x20000000 /* Non Write-Through */
+#define CR0_CD			0x40000000 /* Cache Disable */
+#define CR0_PG			0x80000000 /* Paging Enable */
+
+/* TODO - move back to PM specific file */
+#define PM_DR6			0xFFFF0FF0
+
+#define DR6_BRKDETECT_0		0x00000001 /* B0 through B3 */
+#define DR6_BRKDETECT_1		0x00000002 /* breakpoint condition detected */
+#define DR6_BRKDETECT_2		0x00000004
+#define DR6_BRKDETECT_3		0x00000008
+
+enum {
+	/* general purpose registers */
+	EAX = 0,
+	ECX,
+	EDX,
+	EBX,
+	ESP,
+	EBP,
+	ESI,
+	EDI,
+	/* instruction pointer & flags */
+	EIP,
+	EFLAGS,
+
+	/* segment registers */
+	CS,
+	SS,
+	DS,
+	ES,
+	FS,
+	GS,
+
+	/* floating point unit registers */
+	ST0,
+	ST1,
+	ST2,
+	ST3,
+	ST4,
+	ST5,
+	ST6,
+	ST7,
+	FCTRL,
+	FSTAT,
+	FTAG,
+	FISEG,
+	FIOFF,
+	FOSEG,
+	FOOFF,
+	FOP,
+
+	/* control registers */
+	CR0,
+	CR2,
+	CR3,
+	CR4,
+
+	/* debug registers */
+	DR0,
+	DR1,
+	DR2,
+	DR3,
+	DR6,
+	DR7,
+
+	/* descriptor tables */
+	IDTB,
+	IDTL,
+	IDTAR,
+	GDTB,
+	GDTL,
+	GDTAR,
+	TR,
+	LDTR,
+	LDTB,
+	LDTL,
+	LDTAR,
+
+	/* segment registers */
+	CSB,
+	CSL,
+	CSAR,
+	DSB,
+	DSL,
+	DSAR,
+	ESB,
+	ESL,
+	ESAR,
+	FSB,
+	FSL,
+	FSAR,
+	GSB,
+	GSL,
+	GSAR,
+	SSB,
+	SSL,
+	SSAR,
+	TSSB,
+	TSSL,
+	TSSAR,
+
+	/* PM control reg */
+	PMCR,
+};
+
+#define X86_32_COMMON_MAGIC 0x86328632
+
+enum {
+	/* memory read/write */
+	MEMRDB32 = 0,
+	MEMRDB16,
+	MEMRDH32,
+	MEMRDH16,
+	MEMRDW32,
+	MEMRDW16,
+	MEMWRB32,
+	MEMWRB16,
+	MEMWRH32,
+	MEMWRH16,
+	MEMWRW32,
+	MEMWRW16,
+	/* IO read/write */
+	IORDB32,
+	IORDB16,
+	IORDH32,
+	IORDH16,
+	IORDW32,
+	IORDW16,
+	IOWRB32,
+	IOWRB16,
+	IOWRH32,
+	IOWRH16,
+	IOWRW32,
+	IOWRW16,
+	/* lakemont1 core shadow ram access opcodes */
+	SRAMACCESS,
+	SRAM2PDR,
+	PDR2SRAM,
+	WBINVD,
+};
+
+struct swbp_mem_patch {
+	uint8_t orig_byte;
+	uint32_t swbp_unique_id;
+	uint32_t physaddr;
+	struct swbp_mem_patch *next;
+};
+
+/* TODO - probemode specific - consider removing */
+#define NUM_PM_REGS		18 /* regs used in save/restore */
+
+struct x86_32_common {
+	uint32_t common_magic;
+	void *arch_info;
+	struct reg_cache *cache;
+	struct jtag_tap *curr_tap;
+	uint32_t stored_pc;
+	int flush;
+
+	/* pm_regs are for probemode save/restore state */
+	uint32_t pm_regs[NUM_PM_REGS];
+
+	/* working area for fastdata access */
+	struct working_area *fast_data_area;
+
+	int num_hw_bpoints;
+	struct x86_32_dbg_reg *hw_break_list;
+	struct swbp_mem_patch *swbbp_mem_patch_list;
+
+	/* core probemode implementation dependent functions */
+	uint8_t (*get_num_user_regs)(struct target *t);
+	bool (*is_paging_enabled)(struct target *t);
+	int (*disable_paging)(struct target *t);
+	int (*enable_paging)(struct target *t);
+	bool (*sw_bpts_supported)(struct target *t);
+	int (*transaction_status)(struct target *t);
+	int (*submit_instruction)(struct target *t, int num);
+	int (*read_hw_reg)(struct target *t, int reg, uint32_t *regval, uint8_t cache);
+	int (*write_hw_reg)(struct target *t, int reg,
+				uint32_t regval, uint8_t cache);
+
+	/* register cache to processor synchronization */
+	int (*read_hw_reg_to_cache)(struct target *target, int num);
+	int (*write_hw_reg_from_cache)(struct target *target, int num);
+};
+
+static inline struct x86_32_common *
+target_to_x86_32(struct target *target)
+{
+	return target->arch_info;
+}
+bool check_not_halted(const struct target *t);
+
+/* breakpoint defines */
+#define MAX_DEBUG_REGS		4
+#define SW_BP_OPCODE		0xf1
+#define MAX_SW_BPTS		20
+
+struct x86_32_dbg_reg {
+	int used;
+	uint32_t bp_value;
+};
+
+#define DR7_G_ENABLE_SHIFT		1
+#define DR7_ENABLE_SIZE			2 /* 2 bits per debug reg */
+#define DR7_RW_SHIFT			16
+#define DR7_LENGTH_SHIFT		18
+#define DR7_RW_LEN_SIZE			4
+#define DR7_BP_EXECUTE			0 /* 00 - only on instruction execution*/
+#define DR7_BP_WRITE			1 /* 01 - only on data writes */
+/*#define DR7_RW_IORW			2 UNSUPPORTED 10 - an I/O read and I/O write */
+#define DR7_BP_READWRITE		3 /* on data read or data write */
+#define DR7_BP_LENGTH_1			0 /* 00 - 1 byte length */
+#define DR7_BP_LENGTH_2			1 /* 01 - 2 byte length */
+#define DR7_BP_LENGTH_4			3 /* 11 - 4 byte length */
+
+#define DR7_GLOBAL_ENABLE(val, regnum) \
+	(val |= (1 << (DR7_G_ENABLE_SHIFT + (DR7_ENABLE_SIZE * (regnum)))))
+
+#define DR7_GLOBAL_DISABLE(val, regnum) \
+	(val &= ~(3 << (DR7_ENABLE_SIZE * (regnum))))
+
+#define DR7_BP_FREE(val, regnum) \
+	((val & (3 << (DR7_ENABLE_SIZE * (regnum)))) == 0)
+
+#define DR7_RESET_RWLEN_BITS(val, regnum) \
+	(val &= ~(0x0f << (DR7_RW_SHIFT + DR7_RW_LEN_SIZE * (regnum))))
+
+#define DR7_SET_EXE(val, regnum) \
+	(val &= ~(0x0f << (DR7_RW_SHIFT + DR7_RW_LEN_SIZE * (regnum))))
+
+#define DR7_SET_WRITE(val, regnum) \
+	(val |= (DR7_BP_WRITE << (DR7_RW_SHIFT + DR7_RW_LEN_SIZE * (regnum))))
+
+#define DR7_SET_ACCESS(val, regnum) \
+	(val |= (DR7_BP_READWRITE << (DR7_RW_SHIFT + DR7_RW_LEN_SIZE * (regnum))))
+
+#define DR7_SET_LENGTH(val, regnum, len) \
+	(val |= (len == 1) ? (DR7_BP_LENGTH_1 << (DR7_LENGTH_SHIFT + DR7_RW_LEN_SIZE * (regnum))) : \
+	(len == 2) ? (DR7_BP_LENGTH_2 << (DR7_LENGTH_SHIFT + DR7_RW_LEN_SIZE * (regnum))) : \
+	(DR7_BP_LENGTH_4 << (DR7_LENGTH_SHIFT + DR7_RW_LEN_SIZE * (regnum))))
+
+/* public interface */
+int x86_32_get_gdb_reg_list(struct target *t,
+			struct reg **reg_list[], int *reg_list_size,
+			enum target_register_class reg_class);
+int x86_32_common_init_arch_info(struct target *target,
+			struct x86_32_common *x86_32);
+int x86_32_common_mmu(struct target *t, int *enabled);
+int x86_32_common_virt2phys(struct target *t, uint32_t address, uint32_t *physical);
+int x86_32_common_read_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, uint8_t *buffer);
+int x86_32_common_write_phys_mem(struct target *t, uint32_t phys_address,
+			uint32_t size, uint32_t count, const uint8_t *buffer);
+int x86_32_common_read_memory(struct target *t, uint32_t addr,
+			uint32_t size, uint32_t count, uint8_t *buf);
+int x86_32_common_write_memory(struct target *t, uint32_t addr,
+			uint32_t size, uint32_t count, const uint8_t *buf);
+int x86_32_common_read_io(struct target *t, uint32_t addr,
+			uint32_t size, uint8_t *buf);
+int x86_32_common_write_io(struct target *t, uint32_t addr,
+			uint32_t size, const uint8_t *buf);
+int x86_32_common_add_breakpoint(struct target *t, struct breakpoint *bp);
+int x86_32_common_remove_breakpoint(struct target *t, struct breakpoint *bp);
+int x86_32_common_add_watchpoint(struct target *t, struct watchpoint *wp);
+int x86_32_common_remove_watchpoint(struct target *t, struct watchpoint *wp);
+
+#endif /* X86_32_COMMON_H */
diff --git a/tcl/board/quark_x10xx_board.cfg b/tcl/board/quark_x10xx_board.cfg
new file mode 100644
index 0000000000000000000000000000000000000000..8dc600b804d386215990e7f42cbf3e756e6d7513
--- /dev/null
+++ b/tcl/board/quark_x10xx_board.cfg
@@ -0,0 +1,9 @@
+# There are many Quark boards that can host the quark_x10xx SoC
+# Galileo is an example board
+
+source [find target/quark_x10xx.cfg]
+
+#default frequency but this can be adjusted at runtime
+adapter_khz 4000
+
+reset_config trst_only
diff --git a/tcl/target/quark_x10xx.cfg b/tcl/target/quark_x10xx.cfg
new file mode 100644
index 0000000000000000000000000000000000000000..a5bbfb497158a68de4c25bdb4777ba2e9b2ef8b6
--- /dev/null
+++ b/tcl/target/quark_x10xx.cfg
@@ -0,0 +1,52 @@
+if { [info exists CHIPNAME] } {
+   set _CHIPNAME $CHIPNAME
+} else {
+   set _CHIPNAME quark_x10xx
+}
+
+if { [info exists ENDIAN] } {
+   set _ENDIAN $ENDIAN
+} else {
+   set _ENDIAN little
+}
+
+
+if { [info exists CPUTAPID] } {
+   set _CPUTAPID $CPUTAPID
+} else {
+   set _CPUTAPID 0x18289013
+}
+
+jtag newtap quark_x10xx cpu   -irlen 8  -irmask 0xff  -expected-id   $_CPUTAPID  -disable
+jtag newtap quark_x10xx cltap -irlen 8  -irmask 0xff  -expected-id   0x0e681013  -enable
+
+#openocd puts tap at front of chain not end of chain
+proc quark_x10xx_tapenable {} {
+	echo "enabling core tap"
+	irscan quark_x10xx.cltap 0x11
+	drscan quark_x10xx.cltap 64 1
+	runtest 10
+}
+
+proc quark_x10xx_tapdisable {} {
+	echo "disabling core tap"
+	irscan quark_x10xx.cltap 0x11
+	drscan quark_x10xx.cltap 64 0
+	runtest 10
+}
+
+proc quark_x10xx_setup {} {
+	jtag tapenable quark_x10xx.cpu
+}
+
+jtag configure $_CHIPNAME.cpu -event tap-enable \
+   "quark_x10xx_tapenable"
+
+jtag configure $_CHIPNAME.cpu -event tap-disable \
+   "quark_x10xx_tapdisable"
+
+set _TARGETNAME $_CHIPNAME.cpu
+target create quark_x10xx.cpu quark_x10xx -endian $_ENDIAN -chain-position quark_x10xx.cpu
+
+jtag configure $_CHIPNAME.cpu -event setup \
+   "quark_x10xx_setup"