Skip to contentSkip to conversation

Fixes on gdsys boards and some new functionality

PATCH Superseded dirk.eibach at gdsys.cc 2 participants 15 messages Apr 30, 2014

v1 of 4 Superseded Latest is v4 →

lore
Share & tools
lore

dirk.eibach at gdsys.cc

author
#0 lore raw

cover Fixes on gdsys boards and some new functionality

From: Dirk Eibach <[email protected]>

Dirk Eibach (11):
board: controlcenterd: Fix pci access
board: gdsys: Adapt sdhc_boot.c to mmc_get_env_addr API change
board: controlcenterd: Use new API for setting i2c bus
board: iocon: Support DisplayPort hardware
board: gdsys: IHS I2C master driver
board: gdsys: Increase iocon and dlv10g version string
board: gdsys: Configure bridge on DP501 to support DDC only
board: gdsys: Make gdsys osd hardware detection more robust
board: gdsys: Enable scrambling on DP501
board: iocon: Modify iocon hardware startup
board: gdsys: Remove commands to reduce footprint

 board/gdsys/405ep/iocon.c             |  14 ++-
 board/gdsys/common/Makefile           |   4 +-
 board/gdsys/common/dp501.c            |  35 ++++--
 board/gdsys/common/ihs_i2c.c          | 195 ++++++++++++++++++++++++++++++++++
 board/gdsys/common/osd.c              | 189 ++++++++++++++++----------------
 board/gdsys/p1022/controlcenterd-id.c |   6 +-
 board/gdsys/p1022/controlcenterd.c    |   6 +-
 board/gdsys/p1022/sdhc_boot.c         |   2 +-
 include/configs/controlcenterd.h      |   7 +-
 include/configs/dlvision-10g.h        |  27 ++++-
 include/configs/io.h                  |   7 +-
 include/configs/iocon.h               |  37 +++++--
 include/configs/neo.h                 |   8 +-
 include/gdsys_fpga.h                  |  25 +++--
 14 files changed, 411 insertions(+), 151 deletions(-)
 create mode 100644 board/gdsys/common/ihs_i2c.c
Signature · 1 line
1.8.3
author
#1 lore raw

PATCH 1/11 board: controlcenterd: Fix pci access

From: Dirk Eibach <[email protected]>

readl was called with values instead of pointers to these values. Why this ever did work is a mystery...

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/p1022/controlcenterd.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
board/gdsys/p1022/controlcenterd.c +3 -3
index 8ccd9ce..0a3517d 100644
@@ -386,9 +386,9 @@ static void hydra_initialize(void)
 		fpga = pci_map_bar(devno, PCI_BASE_ADDRESS_0,
 			PCI_REGION_MEM);
 
-		versions = readl(fpga->versions);
-		fpga_version = readl(fpga->fpga_version);
-		fpga_features = readl(fpga->fpga_features);
+		versions = readl(&fpga->versions);
+		fpga_version = readl(&fpga->fpga_version);
+		fpga_features = readl(&fpga->fpga_features);
 
 		hardware_version = versions & 0xf;
 		feature_uart_channels = (fpga_features >> 6) & 0x1f;
Signature · 1 line
1.8.3
author
#2 lore raw

PATCH 2/11 board: gdsys: Adapt sdhc_boot.c to mmc_get_env_addr API change

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/p1022/sdhc_boot.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
board/gdsys/p1022/sdhc_boot.c +1 -1
index e432318..fd0e910 100644
@@ -32,7 +32,7 @@
 #define ESDHC_BOOT_IMAGE_SIZE	0x48
 #define ESDHC_BOOT_IMAGE_ADDR	0x50
 
-int mmc_get_env_addr(struct mmc *mmc, u32 *env_addr)
+int mmc_get_env_addr(struct mmc *mmc, int copy, u32 *env_addr)
 {
 	u8 *tmp_buf;
 	u32 blklen, code_offset, code_len, n;
Signature · 1 line
1.8.3
author
#3 lore raw

PATCH 3/11 board: controlcenterd: Use new API for setting i2c bus

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/p1022/controlcenterd-id.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
board/gdsys/p1022/controlcenterd-id.c +3 -3
index 3fca3c5..019df5e 100644
@@ -1189,8 +1189,8 @@ int startup_ccdm_id_module(void)
 	int result = 0;
 	unsigned int orig_i2c_bus;
 
-	orig_i2c_bus = I2C_GET_BUS();
-	I2C_SET_BUS(1);
+	orig_i2c_bus = i2c_get_bus_num();
+	i2c_set_bus_num(1);
 
 	/* goto end; */
 

@@ -1216,7 +1216,7 @@ int startup_ccdm_id_module(void)
 failure:
 	result = 1;
 end:
-	I2C_SET_BUS(orig_i2c_bus);
+	i2c_set_bus_num(orig_i2c_bus);
 	if (result)
 		ccdm_hang();
Signature · 1 line
1.8.3
#4 lore raw

PATCH 3/11 board: controlcenterd: Use new API for setting i2c bus

Hello Dirk,

Am 30.04.2014 15:49, schrieb dirk.eibach at gdsys.cc:

Quoted 13 lines

From: Dirk Eibach<[email protected]>

Signed-off-by Dirk Eibach<[email protected]>

---

  board/gdsys/p1022/controlcenterd-id.c | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)
board/gdsys/p1022/controlcenterd-id.c +0 -0
index 3fca3c5..019df5e 100644

board/gdsys/p1022/controlcenterd-id.c int startup_ccdm_id_module(void) · line 1193

Quoted 7 lines
board/gdsys/p1022/controlcenterd-id.c +2 -2
@@ -1189,8 +1189,8 @@ int startup_ccdm_id_module(void)
  	int result = 0;
  	unsigned int orig_i2c_bus;
 
-	orig_i2c_bus = I2C_GET_BUS();
-	I2C_SET_BUS(1);
+	orig_i2c_bus = i2c_get_bus_num();
+	i2c_set_bus_num(1);

Nitpicking: Can you replace the "1" with a define?

If all i2c drivers switched to the new API, we can get rid of this "storing from old i2c bus" mechanism complete ... but as this is not done yet:

Acked-by Heiko Schocher <[email protected]>

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
author
#5 lore raw

PATCH 4/11 board: iocon: Support DisplayPort hardware

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

There is a new iocon hardware flavor, supporting DisplayPort finally.

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/405ep/iocon.c   |  12 +++++
 board/gdsys/common/Makefile |   2 +-
 board/gdsys/common/dp501.c  |  33 +++++++++----
 board/gdsys/common/osd.c    | 111 ++++++++++++++++----------------------------
 include/configs/iocon.h     |   9 +++-
 5 files changed, 83 insertions(+), 84 deletions(-)
board/gdsys/405ep/iocon.c +12 -0
index 7a98e41..079dfb2 100644
@@ -15,6 +15,7 @@
 #include "405ep.h"
 #include <gdsys_fpga.h>
 
+#include "../common/dp501.h"
 #include "../common/osd.h"
 #include "../common/mclink.h"
 

@@ -98,6 +99,8 @@ enum {
 unsigned int mclink_fpgacount;
 struct ihs_fpga *fpga_ptr[] = CONFIG_SYS_FPGA_PTR;
 
+int dp501_i2c[] = CONFIG_SYS_DP501_I2C;
+
 static int setup_88e1518(const char *bus, unsigned char addr);
 
 int fpga_set_reg(u32 fpga, u16 *reg, off_t regoff, u16 data)

@@ -371,9 +374,18 @@ int last_stage_init(void)
 	u16 fpga_features;
 	int feature_carrier_speed = fpga_features & (1<<4);
 	bool ch0_rgmii2_present = false;
+	int old_bus = i2c_get_bus_num();
 
 	FPGA_GET_REG(0, fpga_features, &fpga_features);
 
+	/* Turn on Parade DP501 */
+	pca9698_direction_output(0x20, 9, 1);
+	udelay(500000);
+
+	i2c_set_bus_num(dp501_i2c[0]);
+	dp501_powerup(0x08);
+	i2c_set_bus_num(old_bus);
+
 	if (!legacy)
 		ch0_rgmii2_present = !pca9698_get_value(0x20, 30);
board/gdsys/common/Makefile +1 -1
index fb841e0..7f8b427 100644
@@ -8,6 +8,6 @@
 obj-$(CONFIG_SYS_FPGA_COMMON) += fpga.o
 obj-$(CONFIG_IO) += miiphybb.o
 obj-$(CONFIG_IO64) += miiphybb.o
-obj-$(CONFIG_IOCON) += osd.o mclink.o
+obj-$(CONFIG_IOCON) += osd.o mclink.o dp501.o
 obj-$(CONFIG_DLVISION_10G) += osd.o
 obj-$(CONFIG_CONTROLCENTERD) += dp501.o
board/gdsys/common/dp501.c +23 -10
index 52f3ea1..e00e589 100644
@@ -60,8 +60,31 @@ void dp501_powerup(u8 addr)
 	i2c_reg_write(addr, 0x71, 0x20); /* Enable Aux burst write */
 	dp501_setbits(addr, 0x78, 0x30); /* Disable HPD2 IRQ */
 	dp501_clrbits(addr, 0x2f, 0x40); /* Link FIFO reset selection */
+
+#ifdef CONFIG_SYS_DP501_VCAPCTRL0
+	i2c_reg_write(addr, 0x24, CONFIG_SYS_DP501_VCAPCTRL0);
+#else
 	i2c_reg_write(addr, 0x24, 0xc0); /* SDR mode 0, ext. H/VSYNC */
+#endif
+
+#ifdef CONFIG_SYS_DP501_DIFFERENTIAL
+	i2c_reg_write(addr + 2, 0x24, 0x10); /* clock input differential */
+	i2c_reg_write(addr + 2, 0x25, 0x04);
+	i2c_reg_write(addr + 2, 0x26, 0x10);
+#else
 	i2c_reg_write(addr + 2, 0x24, 0x02); /* clock input single ended */
+#endif
+
+	i2c_reg_write(addr + 2, 0x00, 0x18); /* driving strength */
+	i2c_reg_write(addr + 2, 0x03, 0x06); /* driving strength */
+	i2c_reg_write(addr, 0x2c, 0x00); /* configure N value */
+	i2c_reg_write(addr, 0x2d, 0x00); /* configure N value */
+	i2c_reg_write(addr, 0x2e, 0x0c); /* configure N value */
+	i2c_reg_write(addr, 0x76, 0xff); /* clear all interrupt */
+	dp501_setbits(addr, 0x78, 0x03); /* clear all interrupt */
+	i2c_reg_write(addr, 0x75, 0xf8); /* aux channel reset */
+	i2c_reg_write(addr, 0x75, 0x00); /* clear aux channel reset */
+	i2c_reg_write(addr, 0x87, 0x70); /* set retry counter as 7 */
 
 	if (dp501_detect_cable_adapter(addr)) {
 		printf("DVI/HDMI cable adapter detected\n");

@@ -69,16 +92,6 @@ void dp501_powerup(u8 addr)
 		dp501_clrbits(addr, 0x00, 0x08); /* DVI/HDMI HDCP operation */
 	} else {
 		printf("no DVI/HDMI cable adapter detected\n");
-		i2c_reg_write(addr + 2, 0x00, 0x18); /* driving strength */
-		i2c_reg_write(addr + 2, 0x03, 0x06); /* driving strength */
-		i2c_reg_write(addr, 0x2c, 0x00); /* configure N value */
-		i2c_reg_write(addr, 0x2d, 0x00); /* configure N value */
-		i2c_reg_write(addr, 0x2e, 0x0c); /* configure N value */
-		i2c_reg_write(addr, 0x76, 0xff); /* clear all interrupt */
-		dp501_setbits(addr, 0x78, 0x03); /* clear all interrupt */
-		i2c_reg_write(addr, 0x75, 0xf8); /* aux channel reset */
-		i2c_reg_write(addr, 0x75, 0x00); /* clear aux channel reset */
-		i2c_reg_write(addr, 0x87, 0x70); /* set retry counter as 7 */
 		dp501_setbits(addr, 0x00, 0x08); /* for DP HDCP operation */
 
 		dp501_link_training(addr);
board/gdsys/common/osd.c +40 -71

193 lines not on this page · raw message

include/configs/iocon.h +7 -2
index f36c2a3..5636f38 100644
@@ -121,7 +121,9 @@
 #define CONFIG_SYS_I2C_SOFT_SPEED_4		50000
 #define CONFIG_SYS_I2C_SOFT_SLAVE_4		0x7F
 
+#define CONFIG_SYS_ICS8N3QV01_I2C		{1, 2, 3, 4}
 #define CONFIG_SYS_CH7301_I2C			{1, 2, 3, 4}
+#define CONFIG_SYS_DP501_I2C			{1, 2, 3, 4}
 
 #ifndef __ASSEMBLY__
 void fpga_gpio_set(unsigned int bus, int pin);

@@ -146,6 +148,8 @@ int fpga_gpio_get(unsigned int bus, int pin);
 			fpga_gpio_set(I2C_ADAP_HWNR, 0x0020); \
 		else \
 			fpga_gpio_clear(I2C_ADAP_HWNR, 0x0020); \
+		while (!!fpga_gpio_get(I2C_ADAP_HWNR, 0x0020) != !!bit) \
+			; \
 	} while (0)
 #define I2C_DELAY	udelay(25)	/* 1/4 I2C clock duration */
 

@@ -153,7 +157,6 @@ int fpga_gpio_get(unsigned int bus, int pin);
  * OSD hardware
  */
 #define CONFIG_SYS_MPC92469AC
-#define CONFIG_SYS_CH7301
 
 /*
  * FLASH organization

@@ -282,9 +285,11 @@ int fpga_gpio_get(unsigned int bus, int pin);
 /*
  * OSD Setup
  */
+#define CONFIG_SYS_ICS8N3QV01
 #define CONFIG_SYS_MPC92469AC
-#define CONFIG_SYS_CH7301
 #define CONFIG_SYS_OSD_SCREENS		1
+#define CONFIG_SYS_DP501_DIFFERENTIAL
+#define CONFIG_SYS_DP501_VCAPCTRL0	0x01 /* DDR mode 0, DE for H/VSYNC */
 
 #define CONFIG_BITBANGMII		/* bit-bang MII PHY management */
 #define CONFIG_BITBANGMII_MULTI
Signature · 1 line
1.8.3
author
#6 lore raw

PATCH 5/11 board: gdsys: IHS I2C master driver

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

IHS I2C master support was merely a hack in the osd driver. Now it is a proper u-boot I2C framework driver, supporting the v2.00 master features.

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/common/Makefile    |   4 +-
 board/gdsys/common/ihs_i2c.c   | 196 +++++++++++++++++++++++++++++++++++++++++
 board/gdsys/common/osd.c       |   1 -
 include/configs/dlvision-10g.h |  20 ++++-
 include/configs/iocon.h        |  22 +++--
 include/gdsys_fpga.h           |  25 +++---
 6 files changed, 245 insertions(+), 23 deletions(-)
 create mode 100644 board/gdsys/common/ihs_i2c.c
board/gdsys/common/Makefile +2 -2
index 7f8b427..a5ee016 100644
@@ -8,6 +8,6 @@
 obj-$(CONFIG_SYS_FPGA_COMMON) += fpga.o
 obj-$(CONFIG_IO) += miiphybb.o
 obj-$(CONFIG_IO64) += miiphybb.o
-obj-$(CONFIG_IOCON) += osd.o mclink.o dp501.o
-obj-$(CONFIG_DLVISION_10G) += osd.o
+obj-$(CONFIG_IOCON) += osd.o mclink.o dp501.o ihs_i2c.o
+obj-$(CONFIG_DLVISION_10G) += osd.o ihs_i2c.o
 obj-$(CONFIG_CONTROLCENTERD) += dp501.o
board/gdsys/common/ihs_i2c.c new file +196 -0

196 lines not on this page · raw message

board/gdsys/common/osd.c +0 -1
index 0290949..a839a4e 100644
@@ -342,7 +342,6 @@ int osd_probe(unsigned screen)
 	i2c_reg_write(SIL1178_MASTER_I2C_ADDRESS, 0x08, 0x37);
 #endif
 
-	FPGA_SET_REG(screen, videocontrol, 0x0002);
 	FPGA_SET_REG(screen, osd.control, 0x0049);
 
 	FPGA_SET_REG(screen, osd.xy_size, ((32 - 1) << 8) | (16 - 1));
include/configs/dlvision-10g.h +18 -2
index 7877897..d886740 100644
@@ -97,9 +97,22 @@
 /*
  * I2C stuff
  */
+#define CONFIG_SYS_I2C_PPC4XX
+#define CONFIG_SYS_I2C_PPC4XX_CH0
 #define CONFIG_SYS_I2C_PPC4XX_SPEED_0		100000
+#define CONFIG_SYS_I2C_PPC4XX_SLAVE_0		0x7F
+
+#define CONFIG_SYS_I2C_IHS_CH0
+#define CONFIG_SYS_I2C_IHS_SPEED_0		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_0		0x7F
+#define CONFIG_SYS_I2C_IHS_CH1
+#define CONFIG_SYS_I2C_IHS_SPEED_1		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_1		0x7F
+
+#define CONFIG_SYS_SPD_BUS_NUM		2
 
 /* Temp sensor/hwmon/dtt */
+#define CONFIG_SYS_DTT_BUS_NUM	2
 #define CONFIG_DTT_LM63		1	/* National LM63	*/
 #define CONFIG_DTT_SENSORS	{ 0x4c, 0x4e, 0x18 } /* Sensor addresses */
 #define CONFIG_DTT_PWM_LOOKUPTABLE	\

@@ -107,6 +120,11 @@
 		  { 54, 27 }, { 56, 31 }, { 58, 36 }, { 60, 40 } }
 #define CONFIG_DTT_TACH_LIMIT	0xa10
 
+#define CONFIG_SYS_ICS8N3QV01
+#define CONFIG_SYS_ICS8N3QV01_I2C	{0, 1}
+#define CONFIG_SYS_SIL1178
+#define CONFIG_SYS_SIL1178_I2C		{0, 1}
+
 /* EBC peripherals */
 
 #define CONFIG_SYS_FLASH_BASE		0xFC000000

@@ -306,9 +324,7 @@
 /*
  * OSD Setup
  */
-#define CONFIG_SYS_ICS8N3QV01
 #define CONFIG_SYS_MPC92469AC
-#define CONFIG_SYS_SIL1178
 #define CONFIG_SYS_OSD_SCREENS		CONFIG_SYS_FPGA_COUNT
 
 #endif	/* __CONFIG_H */
include/configs/iocon.h +17 -5
index 5636f38..7fa41ee 100644
@@ -101,10 +101,24 @@
 #define CONFIG_SYS_I2C_PPC4XX_SLAVE_0		0x7F
 
 #define CONFIG_SYS_I2C_SPEED		400000
+#define CONFIG_SYS_SPD_BUS_NUM		4
 
 #define CONFIG_PCA953X			/* NXP PCA9554 */
 #define CONFIG_PCA9698			/* NXP PCA9698 */
 
+#define CONFIG_SYS_I2C_IHS_CH0
+#define CONFIG_SYS_I2C_IHS_SPEED_0		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_0		0x7F
+#define CONFIG_SYS_I2C_IHS_CH1
+#define CONFIG_SYS_I2C_IHS_SPEED_1		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_1		0x7F
+#define CONFIG_SYS_I2C_IHS_CH2
+#define CONFIG_SYS_I2C_IHS_SPEED_2		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_2		0x7F
+#define CONFIG_SYS_I2C_IHS_CH3
+#define CONFIG_SYS_I2C_IHS_SPEED_3		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_3		0x7F
+
 /*
  * Software (bit-bang) I2C driver configuration
  */

@@ -121,9 +135,9 @@
 #define CONFIG_SYS_I2C_SOFT_SPEED_4		50000
 #define CONFIG_SYS_I2C_SOFT_SLAVE_4		0x7F
 
-#define CONFIG_SYS_ICS8N3QV01_I2C		{1, 2, 3, 4}
-#define CONFIG_SYS_CH7301_I2C			{1, 2, 3, 4}
-#define CONFIG_SYS_DP501_I2C			{1, 2, 3, 4}
+#define CONFIG_SYS_ICS8N3QV01_I2C		{5, 6, 7, 8}
+#define CONFIG_SYS_CH7301_I2C			{5, 6, 7, 8}
+#define CONFIG_SYS_DP501_I2C			{0, 1, 2, 3}
 
 #ifndef __ASSEMBLY__
 void fpga_gpio_set(unsigned int bus, int pin);

@@ -148,8 +162,6 @@ int fpga_gpio_get(unsigned int bus, int pin);
 			fpga_gpio_set(I2C_ADAP_HWNR, 0x0020); \
 		else \
 			fpga_gpio_clear(I2C_ADAP_HWNR, 0x0020); \
-		while (!!fpga_gpio_get(I2C_ADAP_HWNR, 0x0020) != !!bit) \
-			; \
 	} while (0)
 #define I2C_DELAY	udelay(25)	/* 1/4 I2C clock duration */
include/gdsys_fpga.h +12 -13
index 85ddbcb..276a01e 100644
@@ -43,10 +43,12 @@ struct ihs_gpio {
 };
 
 struct ihs_i2c {
-	u16 write_mailbox;
+	u16 interrupt_status;
+	u16 interrupt_enable;
 	u16 write_mailbox_ext;
-	u16 read_mailbox;
+	u16 write_mailbox;
 	u16 read_mailbox_ext;
+	u16 read_mailbox;
 };
 
 struct ihs_osd {

@@ -84,7 +86,6 @@ struct ihs_fpga {
 #endif
 
 #ifdef CONFIG_IO64
-
 struct ihs_fpga_channel {
 	u16 status_int;
 	u16 config_int;

@@ -121,9 +122,9 @@ struct ihs_fpga {
 	u16 reserved_0[6];	/* 0x0008 */
 	struct ihs_gpio gpio;	/* 0x0014 */
 	u16 mpc3w_control;	/* 0x001a */
-	u16 reserved_1[19];	/* 0x001c */
-	u16 videocontrol;	/* 0x0042 */
-	u16 reserved_2[14];	/* 0x0044 */
+	u16 reserved_1[18];	/* 0x001c */
+	struct ihs_i2c i2c;	/* 0x0040 */
+	u16 reserved_2[10];	/* 0x004c */
 	u16 mc_int;		/* 0x0060 */
 	u16 mc_int_en;		/* 0x0062 */
 	u16 mc_status;		/* 0x0064 */

@@ -150,15 +151,13 @@ struct ihs_fpga {
 	u16 fpga_features;	/* 0x0006 */
 	u16 reserved_0[10];	/* 0x0008 */
 	u16 extended_interrupt; /* 0x001c */
-	u16 reserved_1[9];	/* 0x001e */
-	struct ihs_i2c i2c;	/* 0x0030 */
-	u16 reserved_2[16];	/* 0x0038 */
+	u16 reserved_1[29];	/* 0x001e */
 	u16 mpc3w_control;	/* 0x0058 */
-	u16 reserved_3[34];	/* 0x005a */
-	u16 videocontrol;	/* 0x009e */
-	u16 reserved_4[176];	/* 0x00a0 */
+	u16 reserved_2[3];	/* 0x005a */
+	struct ihs_i2c i2c;	/* 0x0060 */
+	u16 reserved_3[205];	/* 0x0066 */
 	struct ihs_osd osd;	/* 0x0200 */
-	u16 reserved_5[761];	/* 0x020e */
+	u16 reserved_4[761];	/* 0x020e */
 	u16 videomem[31736];	/* 0x0800 */
 };
 #endif
Signature · 1 line
1.8.3
#7 lore raw

PATCH 5/11 board: gdsys: IHS I2C master driver

Hello Dirk,

Am 30.04.2014 15:49, schrieb dirk.eibach at gdsys.cc:

Quoted 11 lines

From: Dirk Eibach<[email protected]>

IHS I2C master support was merely a hack in the osd driver. Now it is a proper u-boot I2C framework driver, supporting the v2.00 master features.

Signed-off-by Dirk Eibach<[email protected]>

---

  board/gdsys/common/Makefile    |   4 +-
  board/gdsys/common/ihs_i2c.c   | 196 +++++++++++++++++++++++++++++++++++++++++

Hmm... why you do not add this i2c driver to drivers/i2c ?

Quoted 12 lines
  board/gdsys/common/osd.c       |   1 -
  include/configs/dlvision-10g.h |  20 ++++-
  include/configs/iocon.h        |  22 +++--
  include/gdsys_fpga.h           |  25 +++---
  6 files changed, 245 insertions(+), 23 deletions(-)

create mode 100644 board/gdsys/common/ihs_i2c.c

board/gdsys/common/Makefile +0 -0
index 7f8b427..a5ee016 100644

board/gdsys/common/Makefile line 12

Quoted 7 lines
board/gdsys/common/Makefile +2 -2
@@ -8,6 +8,6 @@
  obj-$(CONFIG_SYS_FPGA_COMMON) += fpga.o
  obj-$(CONFIG_IO) += miiphybb.o
  obj-$(CONFIG_IO64) += miiphybb.o
-obj-$(CONFIG_IOCON) += osd.o mclink.o dp501.o
-obj-$(CONFIG_DLVISION_10G) += osd.o
+obj-$(CONFIG_IOCON) += osd.o mclink.o dp501.o ihs_i2c.o
+obj-$(CONFIG_DLVISION_10G) += osd.o ihs_i2c.o

and add a proper CONFIG define for this i2c driver.

Quoted 45 lines
board/gdsys/common/Makefile +0 -0
  obj-$(CONFIG_CONTROLCENTERD) += dp501.o
board/gdsys/common/ihs_i2c.c new file +38 -0
new file mode 100644
index 0000000..ba2f5dc
@@ -0,0 +1,196 @@
+/*
+ * (C) Copyright 2013
+ * Dirk Eibach,  Guntermann&  Drunck GmbH, eibach at gdsys.de
+ *
+ * SPDX-License-Identifier:	GPL-2.0+
+ */
+
+#include<common.h>
+#include<i2c.h>
+#include<gdsys_fpga.h>
+
+DECLARE_GLOBAL_DATA_PTR;
+
+enum {
+	I2CINT_ERROR_EV = 1<<  13,
+	I2CINT_TRANSMIT_EV = 1<<  14,
+	I2CINT_RECEIVE_EV = 1<<  15,
+};
+
+static int wait_for_int(bool read)
+{
+	u16 val;
+	unsigned int ctr = 0;
+
+	FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status,&val);
+	while (!(val&  (I2CINT_ERROR_EV
+	       | (read ? I2CINT_RECEIVE_EV : I2CINT_TRANSMIT_EV)))) {
+		udelay(10);
+		if (ctr++>  5000) {
+			printf("I2C timeout\n");
+			return 1;
+		}
+		FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status,&val);
+	}
+
+	return (val&  I2CINT_ERROR_EV) ? 1 : 0;
+}
+

board/gdsys/common/ihs_i2c.c line 48

Quoted 10 lines
board/gdsys/common/ihs_i2c.c new file +10 -0
+static int ihs_i2c_transfer(uchar chip, uchar *buffer, int len, bool read,
+			    bool is_last)
+{
+	u16 val;
+
+	FPGA_SET_REG(I2C_ADAP_HWNR, i2c.interrupt_status, I2CINT_ERROR_EV
+		     | I2CINT_RECEIVE_EV | I2CINT_TRANSMIT_EV);
+	FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status,&val);
+
+	if (!read&&  len) {

Nitpick: please:

if (!read && len) {

board/gdsys/common/ihs_i2c.c

Quoted 4 lines
board/gdsys/common/ihs_i2c.c new file +4 -0
+		val = buffer[0];
+
+		if (len>  1)
+			val |= buffer[1]<<  8;

here too:

val |= buffer[1] << 8;

Please check globally.

board/gdsys/common/ihs_i2c.c line 57

Quoted 5 lines
board/gdsys/common/ihs_i2c.c new file +5 -0
+		FPGA_SET_REG(I2C_ADAP_HWNR, i2c.write_mailbox_ext, val);
+	}
+
+	FPGA_SET_REG(I2C_ADAP_HWNR, i2c.write_mailbox,
+		     (read ? 0x8000 : 0x8400)

Could you use defines for this magic constants?

board/gdsys/common/ihs_i2c.c line 60

Quoted 3 lines
board/gdsys/common/ihs_i2c.c new file +3 -0
+		     | (chip<<  1)
+		     | ((len>  1) ? (1<<  11) : 0)
+		     | (is_last ? 0 : (1<<  13)));

and here too ...

Quoted 141 lines
board/gdsys/common/ihs_i2c.c new file +136 -0
+
+	if (wait_for_int(read))
+		return 1;
+
+	if (read) {
+		FPGA_GET_REG(I2C_ADAP_HWNR, i2c.read_mailbox_ext,&val);
+		buffer[0] = val&  0xff;
+		if (len>  1)
+			buffer[1] = val>>  8;
+	}
+
+	return 0;
+}
+
+static int ihs_i2c_address(uchar chip, uint addr, int alen, bool hold_bus)
+{
+	int shift = (alen-1) * 8;
+
+	while (alen) {
+		int transfer = MIN(alen, 2);
+		uchar buf[2];
+		bool is_last = alen<= transfer;
+
+		buf[0] = addr>>  shift;
+		if (alen>  1)
+			buf[1] = addr>>  (shift - 8);
+
+		if (ihs_i2c_transfer(chip, buf, transfer, false,
+				     hold_bus ? false : is_last))
+			return 1;
+
+		shift -= 16;
+		alen -= transfer;
+	}
+
+	return 0;
+}
+
+static int ihs_i2c_access(struct i2c_adapter *adap, uchar chip, uint addr,
+			  int alen, uchar *buffer, int len, bool read)
+{
+	if (len<= 0)
+		return 1;
+
+	if (ihs_i2c_address(chip, addr, alen, !read))
+		return 1;
+
+	while (len) {
+		int transfer = MIN(len, 2);
+
+		if (ihs_i2c_transfer(chip, buffer, transfer, read,
+				     len<= transfer))
+			return 1;
+
+		buffer += transfer;
+		addr += transfer;
+		len -= transfer;
+	}
+
+	return 0;
+}
+
+
+static void ihs_i2c_init(struct i2c_adapter *adap, int speed, int slaveaddr)
+{
+#ifdef CONFIG_SYS_I2C_INIT_BOARD
+	/*
+	 * Call board specific i2c bus reset routine before accessing the
+	 * environment, which might be in a chip on that bus. For details
+	 * about this problem see doc/I2C_Edge_Conditions.
+	 */
+	i2c_init_board();
+#endif
+}
+
+static int ihs_i2c_probe(struct i2c_adapter *adap, uchar chip)
+{
+	uchar buffer[2];
+
+	if (ihs_i2c_transfer(chip, buffer, 0, true, true))
+		return 1;
+
+	return 0;
+}
+
+static int ihs_i2c_read(struct i2c_adapter *adap, uchar chip, uint addr,
+			int alen, uchar *buffer, int len)
+{
+	return ihs_i2c_access(adap, chip, addr, alen, buffer, len, true);
+}
+
+static int ihs_i2c_write(struct i2c_adapter *adap, uchar chip, uint addr,
+			 int alen, uchar *buffer, int len)
+{
+	return ihs_i2c_access(adap, chip, addr, alen, buffer, len, false);
+}
+
+static unsigned int ihs_i2c_set_bus_speed(struct i2c_adapter *adap,
+					     unsigned int speed)
+{
+	if (speed != adap->speed)
+		return 1;
+	return speed;
+}
+
+/*
+ * Register IHS i2c adapters
+ */
+#ifdef CONFIG_SYS_I2C_IHS_CH0
+U_BOOT_I2C_ADAP_COMPLETE(ihs0, ihs_i2c_init, ihs_i2c_probe,
+			 ihs_i2c_read, ihs_i2c_write,
+			 ihs_i2c_set_bus_speed,
+			 CONFIG_SYS_I2C_IHS_SPEED_0,
+			 CONFIG_SYS_I2C_IHS_SLAVE_0, 0)
+#endif
+#ifdef CONFIG_SYS_I2C_IHS_CH1
+U_BOOT_I2C_ADAP_COMPLETE(ihs1, ihs_i2c_init, ihs_i2c_probe,
+			 ihs_i2c_read, ihs_i2c_write,
+			 ihs_i2c_set_bus_speed,
+			 CONFIG_SYS_I2C_IHS_SPEED_1,
+			 CONFIG_SYS_I2C_IHS_SLAVE_1, 1)
+#endif
+#ifdef CONFIG_SYS_I2C_IHS_CH2
+U_BOOT_I2C_ADAP_COMPLETE(ihs2, ihs_i2c_init, ihs_i2c_probe,
+			 ihs_i2c_read, ihs_i2c_write,
+			 ihs_i2c_set_bus_speed,
+			 CONFIG_SYS_I2C_IHS_SPEED_2,
+			 CONFIG_SYS_I2C_IHS_SLAVE_2, 2)
+#endif
+#ifdef CONFIG_SYS_I2C_IHS_CH3
+U_BOOT_I2C_ADAP_COMPLETE(ihs3, ihs_i2c_init, ihs_i2c_probe,
+			 ihs_i2c_read, ihs_i2c_write,
+			 ihs_i2c_set_bus_speed,
+			 CONFIG_SYS_I2C_IHS_SPEED_3,
+			 CONFIG_SYS_I2C_IHS_SLAVE_3, 3)
+#endif
board/gdsys/common/osd.c +0 -0
index 0290949..a839a4e 100644

board/gdsys/common/osd.c int osd_probe(unsigned screen) · line 345

Quoted 4 lines
board/gdsys/common/osd.c +0 -1
@@ -342,7 +342,6 @@ int osd_probe(unsigned screen)
  	i2c_reg_write(SIL1178_MASTER_I2C_ADDRESS, 0x08, 0x37);
  #endif
 
-	FPGA_SET_REG(screen, videocontrol, 0x0002);

What has this change to do with the i2c driver? Please move this to a seperate patch, thanks!

board/gdsys/common/osd.c int osd_probe(unsigned screen)

Quoted 3 lines
board/gdsys/common/osd.c +0 -0
  	FPGA_SET_REG(screen, osd.control, 0x0049);
 
  	FPGA_SET_REG(screen, osd.xy_size, ((32 - 1)<<  8) | (16 - 1));

Hmm... a lot of magic constants ...

include/configs/dlvision-10g.h line 103

Quoted 12 lines
include/configs/dlvision-10g.h +3 -0
index 7877897..d886740 100644
@@ -97,9 +97,22 @@
  /*
   * I2C stuff
   */
+#define CONFIG_SYS_I2C_PPC4XX
+#define CONFIG_SYS_I2C_PPC4XX_CH0
  #define CONFIG_SYS_I2C_PPC4XX_SPEED_0		100000
+#define CONFIG_SYS_I2C_PPC4XX_SLAVE_0		0x7F

Here again, this change has nothing todo with adding this new i2c driver ...

Please split this in another patch, thanks.

Quoted 16 lines
include/configs/dlvision-10g.h +10 -0
+
+#define CONFIG_SYS_I2C_IHS_CH0
+#define CONFIG_SYS_I2C_IHS_SPEED_0		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_0		0x7F
+#define CONFIG_SYS_I2C_IHS_CH1
+#define CONFIG_SYS_I2C_IHS_SPEED_1		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_1		0x7F
+
+#define CONFIG_SYS_SPD_BUS_NUM		2
 
  /* Temp sensor/hwmon/dtt */
+#define CONFIG_SYS_DTT_BUS_NUM	2
  #define CONFIG_DTT_LM63		1	/* National LM63	*/
  #define CONFIG_DTT_SENSORS	{ 0x4c, 0x4e, 0x18 } /* Sensor addresses */
  #define CONFIG_DTT_PWM_LOOKUPTABLE	\

include/configs/dlvision-10g.h line 126

Quoted 7 lines
include/configs/dlvision-10g.h +4 -0
@@ -107,6 +120,11 @@
  		  { 54, 27 }, { 56, 31 }, { 58, 36 }, { 60, 40 } }
  #define CONFIG_DTT_TACH_LIMIT	0xa10
 
+#define CONFIG_SYS_ICS8N3QV01
+#define CONFIG_SYS_ICS8N3QV01_I2C	{0, 1}
+#define CONFIG_SYS_SIL1178
+#define CONFIG_SYS_SIL1178_I2C		{0, 1}

and this ...

Quoted 116 lines
include/configs/dlvision-10g.h +1 -2
+
  /* EBC peripherals */
 
  #define CONFIG_SYS_FLASH_BASE		0xFC000000

@@ -306,9 +324,7 @@
  /*
   * OSD Setup
   */
-#define CONFIG_SYS_ICS8N3QV01
  #define CONFIG_SYS_MPC92469AC
-#define CONFIG_SYS_SIL1178
  #define CONFIG_SYS_OSD_SCREENS		CONFIG_SYS_FPGA_COUNT
 
  #endif	/* __CONFIG_H */
include/configs/iocon.h +17 -5
index 5636f38..7fa41ee 100644
@@ -101,10 +101,24 @@
  #define CONFIG_SYS_I2C_PPC4XX_SLAVE_0		0x7F
 
  #define CONFIG_SYS_I2C_SPEED		400000
+#define CONFIG_SYS_SPD_BUS_NUM		4
 
  #define CONFIG_PCA953X			/* NXP PCA9554 */
  #define CONFIG_PCA9698			/* NXP PCA9698 */
 
+#define CONFIG_SYS_I2C_IHS_CH0
+#define CONFIG_SYS_I2C_IHS_SPEED_0		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_0		0x7F
+#define CONFIG_SYS_I2C_IHS_CH1
+#define CONFIG_SYS_I2C_IHS_SPEED_1		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_1		0x7F
+#define CONFIG_SYS_I2C_IHS_CH2
+#define CONFIG_SYS_I2C_IHS_SPEED_2		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_2		0x7F
+#define CONFIG_SYS_I2C_IHS_CH3
+#define CONFIG_SYS_I2C_IHS_SPEED_3		50000
+#define CONFIG_SYS_I2C_IHS_SLAVE_3		0x7F
+
  /*
   * Software (bit-bang) I2C driver configuration
   */

@@ -121,9 +135,9 @@
  #define CONFIG_SYS_I2C_SOFT_SPEED_4		50000
  #define CONFIG_SYS_I2C_SOFT_SLAVE_4		0x7F
 
-#define CONFIG_SYS_ICS8N3QV01_I2C		{1, 2, 3, 4}
-#define CONFIG_SYS_CH7301_I2C			{1, 2, 3, 4}
-#define CONFIG_SYS_DP501_I2C			{1, 2, 3, 4}
+#define CONFIG_SYS_ICS8N3QV01_I2C		{5, 6, 7, 8}
+#define CONFIG_SYS_CH7301_I2C			{5, 6, 7, 8}
+#define CONFIG_SYS_DP501_I2C			{0, 1, 2, 3}
 
  #ifndef __ASSEMBLY__
  void fpga_gpio_set(unsigned int bus, int pin);

@@ -148,8 +162,6 @@ int fpga_gpio_get(unsigned int bus, int pin);
  			fpga_gpio_set(I2C_ADAP_HWNR, 0x0020); \
  		else \
  			fpga_gpio_clear(I2C_ADAP_HWNR, 0x0020); \
-		while (!!fpga_gpio_get(I2C_ADAP_HWNR, 0x0020) != !!bit) \
-			; \
  	} while (0)
  #define I2C_DELAY	udelay(25)	/* 1/4 I2C clock duration */
include/gdsys_fpga.h +8 -11
index 85ddbcb..276a01e 100644
@@ -43,10 +43,12 @@ struct ihs_gpio {
  };
 
  struct ihs_i2c {
-	u16 write_mailbox;
+	u16 interrupt_status;
+	u16 interrupt_enable;
  	u16 write_mailbox_ext;
-	u16 read_mailbox;
+	u16 write_mailbox;
  	u16 read_mailbox_ext;
+	u16 read_mailbox;
  };
 
  struct ihs_osd {

@@ -84,7 +86,6 @@ struct ihs_fpga {
  #endif
 
  #ifdef CONFIG_IO64
-
  struct ihs_fpga_channel {
  	u16 status_int;
  	u16 config_int;

@@ -121,9 +122,9 @@ struct ihs_fpga {
  	u16 reserved_0[6];	/* 0x0008 */
  	struct ihs_gpio gpio;	/* 0x0014 */
  	u16 mpc3w_control;	/* 0x001a */
-	u16 reserved_1[19];	/* 0x001c */
-	u16 videocontrol;	/* 0x0042 */
-	u16 reserved_2[14];	/* 0x0044 */
+	u16 reserved_1[18];	/* 0x001c */
+	struct ihs_i2c i2c;	/* 0x0040 */
+	u16 reserved_2[10];	/* 0x004c */
  	u16 mc_int;		/* 0x0060 */
  	u16 mc_int_en;		/* 0x0062 */
  	u16 mc_status;		/* 0x0064 */

@@ -150,15 +151,13 @@ struct ihs_fpga {
  	u16 fpga_features;	/* 0x0006 */
  	u16 reserved_0[10];	/* 0x0008 */
  	u16 extended_interrupt; /* 0x001c */
-	u16 reserved_1[9];	/* 0x001e */
-	struct ihs_i2c i2c;	/* 0x0030 */
-	u16 reserved_2[16];	/* 0x0038 */
+	u16 reserved_1[29];	/* 0x001e */
  	u16 mpc3w_control;	/* 0x0058 */
-	u16 reserved_3[34];	/* 0x005a */
-	u16 videocontrol;	/* 0x009e */

include/gdsys_fpga.h struct ihs_fpga { · line 163

Quoted 10 lines
include/gdsys_fpga.h +4 -2
-	u16 reserved_4[176];	/* 0x00a0 */
+	u16 reserved_2[3];	/* 0x005a */
+	struct ihs_i2c i2c;	/* 0x0060 */
+	u16 reserved_3[205];	/* 0x0066 */
  	struct ihs_osd osd;	/* 0x0200 */
-	u16 reserved_5[761];	/* 0x020e */
+	u16 reserved_4[761];	/* 0x020e */
  	u16 videomem[31736];	/* 0x0800 */
  };
  #endif

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
author
#8 lore raw

PATCH 6/11 board: gdsys: Increase iocon and dlv10g version string

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

Signed-off-by Dirk Eibach <[email protected]>

---

 include/configs/dlvision-10g.h | 2 +-
 include/configs/iocon.h        | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
include/configs/dlvision-10g.h +1 -1
index d886740..b866598 100644
@@ -17,7 +17,7 @@
  * Include common defines/options for all AMCC eval boards
  */
 #define CONFIG_HOSTNAME		dlvsion-10g
-#define CONFIG_IDENT_STRING	" dlvision-10g 0.05"
+#define CONFIG_IDENT_STRING	" dlvision-10g 0.06"
 #include "amcc-common.h"
 
 #define CONFIG_BOARD_EARLY_INIT_F
include/configs/iocon.h +1 -1
index 7fa41ee..d942e8f 100644
@@ -17,7 +17,7 @@
  * Include common defines/options for all AMCC eval boards
  */
 #define CONFIG_HOSTNAME		iocon
-#define CONFIG_IDENT_STRING	" iocon 0.05"
+#define CONFIG_IDENT_STRING	" iocon 0.06"
 #include "amcc-common.h"
 
 #define CONFIG_BOARD_EARLY_INIT_F
Signature · 1 line
1.8.3
author
#9 lore raw

PATCH 7/11 board: gdsys: Configure bridge on DP501 to support DDC only

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

The I2C bridge on DP501 supports EDID, MCCS and HDCP by default. Allow EDID only to avoid I2C address conflicts.

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/common/dp501.c       | 1 +
 include/configs/controlcenterd.h | 7 ++++---
 2 files changed, 5 insertions(+), 3 deletions(-)
board/gdsys/common/dp501.c +1 -0
index e00e589..7958bae 100644
@@ -54,6 +54,7 @@ static void dp501_link_training(u8 addr)
 void dp501_powerup(u8 addr)
 {
 	dp501_clrbits(addr, 0x0a, 0x30); /* power on encoder */
+	dp501_setbits(addr, 0x0a, 0x0e); /* block HDCP and MCCS on I2C bride*/
 	i2c_reg_write(addr, 0x27, 0x30); /* Hardware auto detect DVO timing */
 	dp501_setbits(addr, 0x72, 0x80); /* DPCD read enable */
 	dp501_setbits(addr, 0x30, 0x20); /* RS polynomial select */
include/configs/controlcenterd.h +4 -3
index 868813f..ec3145f 100644
@@ -199,9 +199,10 @@
 #define CONFIG_SYS_FSL_I2C2_SPEED	400000
 #define CONFIG_SYS_FSL_I2C2_SLAVE	0x7F
 #define CONFIG_SYS_FSL_I2C2_OFFSET	0x3100
-/* Probing DP501 I2C-Bridge will hang */
-#define CONFIG_SYS_I2C_NOPROBES		{ {0, 0x30}, {0, 0x37}, {0, 0x3a}, \
-					  {0, 0x3b}, {0, 0x50} }
+
+#ifndef CONFIG_TRAILBLAZER
+#define CONFIG_CMD_I2C
+#endif
 
 #define CONFIG_PCA9698			/* NXP PCA9698 */
Signature · 1 line
1.8.3
author
#10 lore raw

PATCH 8/11 board: gdsys: Make gdsys osd hardware detection more robust

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/405ep/iocon.c      |  17 ++-----
 board/gdsys/common/ihs_i2c.c   |   1 -
 board/gdsys/common/osd.c       | 113 +++++++++++++++++++++++++++--------------
 include/configs/dlvision-10g.h |   2 -
 include/configs/iocon.h        |   6 ---
 5 files changed, 79 insertions(+), 60 deletions(-)
board/gdsys/405ep/iocon.c +5 -12
index 079dfb2..6ae15e1 100644
@@ -15,7 +15,6 @@
 #include "405ep.h"
 #include <gdsys_fpga.h>
 
-#include "../common/dp501.h"
 #include "../common/osd.h"
 #include "../common/mclink.h"
 

@@ -99,8 +98,6 @@ enum {
 unsigned int mclink_fpgacount;
 struct ihs_fpga *fpga_ptr[] = CONFIG_SYS_FPGA_PTR;
 
-int dp501_i2c[] = CONFIG_SYS_DP501_I2C;
-
 static int setup_88e1518(const char *bus, unsigned char addr);
 
 int fpga_set_reg(u32 fpga, u16 *reg, off_t regoff, u16 data)

@@ -374,20 +371,16 @@ int last_stage_init(void)
 	u16 fpga_features;
 	int feature_carrier_speed = fpga_features & (1<<4);
 	bool ch0_rgmii2_present = false;
-	int old_bus = i2c_get_bus_num();
 
 	FPGA_GET_REG(0, fpga_features, &fpga_features);
 
-	/* Turn on Parade DP501 */
-	pca9698_direction_output(0x20, 9, 1);
-	udelay(500000);
-
-	i2c_set_bus_num(dp501_i2c[0]);
-	dp501_powerup(0x08);
-	i2c_set_bus_num(old_bus);
+	if (!legacy) {
+		/* Turn on Parade DP501 */
+		pca9698_direction_output(0x20, 9, 1);
+		udelay(500000);
 
-	if (!legacy)
 		ch0_rgmii2_present = !pca9698_get_value(0x20, 30);
+	}
 
 	print_fpga_info(0, ch0_rgmii2_present);
 	osd_probe(0);
board/gdsys/common/ihs_i2c.c +0 -1
index ba2f5dc..cdea155 100644
@@ -27,7 +27,6 @@ static int wait_for_int(bool read)
 	       | (read ? I2CINT_RECEIVE_EV : I2CINT_TRANSMIT_EV)))) {
 		udelay(10);
 		if (ctr++ > 5000) {
-			printf("I2C timeout\n");
 			return 1;
 		}
 		FPGA_GET_REG(I2C_ADAP_HWNR, i2c.interrupt_status, &val);
board/gdsys/common/osd.c +74 -39

168 lines not on this page · raw message

include/configs/dlvision-10g.h +0 -2
index b866598..77802d8 100644
@@ -120,9 +120,7 @@
 		  { 54, 27 }, { 56, 31 }, { 58, 36 }, { 60, 40 } }
 #define CONFIG_DTT_TACH_LIMIT	0xa10
 
-#define CONFIG_SYS_ICS8N3QV01
 #define CONFIG_SYS_ICS8N3QV01_I2C	{0, 1}
-#define CONFIG_SYS_SIL1178
 #define CONFIG_SYS_SIL1178_I2C		{0, 1}
 
 /* EBC peripherals */
include/configs/iocon.h +0 -6
index d942e8f..0aeb95a 100644
@@ -166,11 +166,6 @@ int fpga_gpio_get(unsigned int bus, int pin);
 #define I2C_DELAY	udelay(25)	/* 1/4 I2C clock duration */
 
 /*
- * OSD hardware
- */
-#define CONFIG_SYS_MPC92469AC
-
-/*
  * FLASH organization
  */
 #define CONFIG_SYS_FLASH_CFI		/* The flash is CFI compatible	*/

@@ -297,7 +292,6 @@ int fpga_gpio_get(unsigned int bus, int pin);
 /*
  * OSD Setup
  */
-#define CONFIG_SYS_ICS8N3QV01
 #define CONFIG_SYS_MPC92469AC
 #define CONFIG_SYS_OSD_SCREENS		1
 #define CONFIG_SYS_DP501_DIFFERENTIAL
Signature · 1 line
1.8.3
author
#11 lore raw

PATCH 9/11 board: gdsys: Enable scrambling on DP501

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

For proper displayport performance, scrambling has to be enabled, but is turned off on DP501 by default.

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/common/dp501.c | 1 +
 1 file changed, 1 insertion(+)
board/gdsys/common/dp501.c +1 -0
index 7958bae..7eb15ed 100644
@@ -61,6 +61,7 @@ void dp501_powerup(u8 addr)
 	i2c_reg_write(addr, 0x71, 0x20); /* Enable Aux burst write */
 	dp501_setbits(addr, 0x78, 0x30); /* Disable HPD2 IRQ */
 	dp501_clrbits(addr, 0x2f, 0x40); /* Link FIFO reset selection */
+	dp501_clrbits(addr, 0x60, 0x20); /* Enable scrambling */
 
 #ifdef CONFIG_SYS_DP501_VCAPCTRL0
 	i2c_reg_write(addr, 0x24, CONFIG_SYS_DP501_VCAPCTRL0);
Signature · 1 line
1.8.3
author
#12 lore raw

PATCH 10/11 board: iocon: Modify iocon hardware startup

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

To avoid peer "ChReceivePathStatus"-messages on iocon startup, initialize PHYs as soon as possible.

Signed-off-by Dirk Eibach <[email protected]>

---

 board/gdsys/405ep/iocon.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)
board/gdsys/405ep/iocon.c +4 -5
index 6ae15e1..1bac970 100644
@@ -377,14 +377,10 @@ int last_stage_init(void)
 	if (!legacy) {
 		/* Turn on Parade DP501 */
 		pca9698_direction_output(0x20, 9, 1);
-		udelay(500000);
 
 		ch0_rgmii2_present = !pca9698_get_value(0x20, 30);
 	}
 
-	print_fpga_info(0, ch0_rgmii2_present);
-	osd_probe(0);
-
 	/* wait for FPGA done */
 	for (k = 0; k < ARRAY_SIZE(mclink_controllers); ++k) {
 		unsigned int ctr = 0;

@@ -413,13 +409,16 @@ int last_stage_init(void)
 		}
 	}
 
-	/* wait for slave-PLLs to be up and running */
+	/* give slave-PLLs and Parade DP501 some time to be up and running */
 	udelay(500000);
 
 	mclink_fpgacount = CONFIG_SYS_MCLINK_MAX;
 	slaves = mclink_probe();
 	mclink_fpgacount = 0;
 
+	print_fpga_info(0, ch0_rgmii2_present);
+	osd_probe(0);
+
 	if (slaves <= 0)
 		return 0;
Signature · 1 line
1.8.3
author
#13 lore raw

PATCH 11/11 board: gdsys: Remove commands to reduce footprint

In reply to dirk.eibach at gdsys.cc

From: Dirk Eibach <[email protected]>

Commit "2842c1c fit: add sha256 support" badly increased memory footprint, so some of our boards did not build anymore. Since monitor base must not be changed I removed some commands to save memory.

Maybe making sha256 optional for fit would be an option for the future since it really has some beefy footprint.

Signed-off-by Dirk Eibach <[email protected]>

---

 include/configs/dlvision-10g.h | 7 ++++++-
 include/configs/io.h           | 7 ++++++-
 include/configs/iocon.h        | 8 ++++++--
 include/configs/neo.h          | 8 ++++++--
 4 files changed, 24 insertions(+), 6 deletions(-)
include/configs/dlvision-10g.h +6 -1
index 77802d8..d8a1c0c 100644
@@ -64,9 +64,14 @@
 /*
  * Commands additional to the ones defined in amcc-common.h
  */
-#define CONFIG_CMD_CACHE
 #define CONFIG_CMD_DTT
+#undef CONFIG_CMD_DHCP
+#undef CONFIG_CMD_DIAG
 #undef CONFIG_CMD_EEPROM
+#undef CONFIG_CMD_ELF
+#undef CONFIG_CMD_I2C
+#undef CONFIG_CMD_IRQ
+#undef CONFIG_CMD_NFS
 
 /*
  * SDRAM configuration (please see cpu/ppc/sdram.[ch])
include/configs/io.h +6 -1
index 9da6cc6..649c0fb 100644
@@ -64,9 +64,14 @@
 /*
  * Commands additional to the ones defined in amcc-common.h
  */
-#define CONFIG_CMD_CACHE
 #define CONFIG_CMD_DTT
+#undef CONFIG_CMD_DHCP
+#undef CONFIG_CMD_DIAG
 #undef CONFIG_CMD_EEPROM
+#undef CONFIG_CMD_ELF
+#undef CONFIG_CMD_I2C
+#undef CONFIG_CMD_IRQ
+#undef CONFIG_CMD_NFS
 
 /*
  * SDRAM configuration (please see cpu/ppc/sdram.[ch])
include/configs/iocon.h +6 -2
index 0aeb95a..e2a77b8 100644
@@ -61,9 +61,13 @@
 /*
  * Commands additional to the ones defined in amcc-common.h
  */
-#define CONFIG_CMD_CACHE
-#define CONFIG_CMD_FPGAD
+#undef CONFIG_CMD_DHCP
+#undef CONFIG_CMD_DIAG
 #undef CONFIG_CMD_EEPROM
+#undef CONFIG_CMD_ELF
+#undef CONFIG_CMD_I2C
+#undef CONFIG_CMD_IRQ
+#undef CONFIG_CMD_NFS
 
 /*
  * SDRAM configuration (please see cpu/ppc/sdram.[ch])
include/configs/neo.h +6 -2
index d549985..75dc862 100644
@@ -61,10 +61,14 @@
 /*
  * Commands additional to the ones defined in amcc-common.h
  */
-#define CONFIG_CMD_CACHE
-#define CONFIG_CMD_DATE
 #define CONFIG_CMD_DTT
+#undef CONFIG_CMD_DHCP
+#undef CONFIG_CMD_DIAG
 #undef CONFIG_CMD_EEPROM
+#undef CONFIG_CMD_ELF
+#undef CONFIG_CMD_I2C
+#undef CONFIG_CMD_IRQ
+#undef CONFIG_CMD_NFS
 
 /*
  * SDRAM configuration (please see cpu/ppc/sdram.[ch])
Signature · 1 line
1.8.3
#14 lore raw

PATCH 11/11 board: gdsys: Remove commands to reduce footprint

Hello Dirk,

Am 30.04.2014 15:50, schrieb dirk.eibach at gdsys.cc:

Quoted 9 lines

From: Dirk Eibach<[email protected]>

Commit "2842c1c fit: add sha256 support" badly increased memory footprint, so some of our boards did not build anymore. Since monitor base must not be changed I removed some commands to save memory.

Maybe making sha256 optional for fit would be an option for the future since it really has some beefy footprint.

Patches are welcome ;-)

bye,
Heiko

Signature · 2 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany