powerpc/t1040qds: Remove Video - HDMI support
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
From: Wang Dongsheng <[email protected]>
We should remove this support.
First, there is not any documents to say we need to support this feature, and kernel also not support this feature.
Second, Kerneal not support this on T1040QDS, so if we open it in u-boot, FPGA will be changed and that will affect deep sleep feature.
board/freescale/t1040qds/Makefile +0 -1
@@ -10,4 +10,3 @@ obj-$(CONFIG_PCI) += pci.o obj-y += law.o obj-y += tlb.o obj-y += eth.o -obj-y += diu.o
board/freescale/t1040qds/diu.c deleted +0 -97
97 lines not on this page · raw message
board/freescale/t1040qds/t1040qds.h +0 -1
@@ -9,6 +9,5 @@void fdt_fixup_board_enet (void *blob);void pci_of_setup (void *blob,bd_t *bd); -int select_i2c_ch_pca9547 (u8 ch);#endif
board/freescale/t1040qds/t1040qds_qixis.h +0 -12
@@ -13,18 +13,6 @@#define BRDCFG4_EMISEL_MASK 0xE0#define BRDCFG4_EMISEL_SHIFT 5 -/* BRDCFG5[0:1] controls routing and use of I2C3 & I2C4 ports*/ -#define BRDCFG5_IMX_MASK 0xC0 -#define BRDCFG5_IMX_DIU 0x80 - -/* BRDCFG15[3] controls LCD Panel Powerdown*/ -#define BRDCFG15_LCDPD_MASK 0x10 -#define BRDCFG15_LCDPD_ENABLED 0x00 - -/* BRDCFG15[6:7] controls DIU MUX selction*/ -#define BRDCFG15_DIUSEL_MASK 0x03 -#define BRDCFG15_DIUSEL_HDMI 0x00 -/* SYSCLK */ #define QIXIS_SYSCLK_66 0x0#define QIXIS_SYSCLK_83 0x1
include/configs/T1040QDS.h +1 -27
@@ -375,7 +375,7 @@ unsigned long get_board_ddr_clk(void);#define CONFIG_SYS_INIT_SP_OFFSET CONFIG_SYS_GBL_DATA_OFFSET#define CONFIG_SYS_MONITOR_LEN (512 * 1024) -#define CONFIG_SYS_MALLOC_LEN (10 * 1024 * 1024) +#define CONFIG_SYS_MALLOC_LEN (4 * 1024 * 1024)/* Serial Port - controlled on board with jumper J8 * open - index 2
@@ -401,26 +401,6 @@ unsigned long get_board_ddr_clk(void);#define CONFIG_SYS_HUSH_PARSER #define CONFIG_SYS_PROMPT_HUSH_PS2 "> " -/* Video */ -#define CONFIG_FSL_DIU_FB -#ifdef CONFIG_FSL_DIU_FB -#define CONFIG_FSL_DIU_CH7301 -#define CONFIG_SYS_DIU_ADDR (CONFIG_SYS_CCSRBAR + 0x180000) -#define CONFIG_VIDEO -#define CONFIG_CMD_BMP -#define CONFIG_CFB_CONSOLE -#define CONFIG_VIDEO_SW_CURSOR -#define CONFIG_VGA_AS_SINGLE_DEVICE -#define CONFIG_VIDEO_LOGO -#define CONFIG_VIDEO_BMP_LOGO -#define CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS -/* -* With CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS, flash I/O is really slow, so -* disable empty flash sector detection, which is I/O-intensive. -*/ -#undef CONFIG_SYS_FLASH_EMPTY_INFO -#endif -/* pass open firmware flat tree */ #define CONFIG_OF_LIBFDT #define CONFIG_OF_BOARD_SETUP
@@ -446,11 +426,6 @@ unsigned long get_board_ddr_clk(void);/* I2C bus multiplexer */ #define I2C_MUX_CH_DEFAULT 0x8 -#define I2C_MUX_CH_DIU 0xC - -/* LDI/DVI Encoder for display */ -#define CONFIG_SYS_I2C_LDI_ADDR 0x38 -#define CONFIG_SYS_I2C_DVI_ADDR 0x75/* * RTC configuration
@@ -728,7 +703,6 @@ unsigned long get_board_ddr_clk(void);"bank_intlv=cs0_cs1;" \ "usb1:dr_mode=host,phy_type=" __stringify(__USB_PHY_TYPE) "\0"\"netdev=eth0\0" \ -"video-mode=fslfb:1024x768-32 at 60,monitor=dvi\0" \ "uboot=" __stringify(CONFIG_UBOOTPATH) "\0" \ "ubootaddr=" __stringify(CONFIG_SYS_TEXT_BASE) "\0" \"tftpflash=tftpboot $loadaddr $uboot && " \
Signature · 1 line
1.8.5
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
Hello Dongsheng,
We do have requirement to support this that's why code development was done. Also , what is the dependency of deep-sleep on this. Please elaborate And if something is broken, we should fix it. Instead of removing the feature.
Regards
Priyanka
Quoted 232 lines
-----Original Message-----
From: Dongsheng Wang [mailto:dongsheng.wang at freescale.com]
Sent: Friday, April 18, 2014 1:09 PM
To: Sun York-R58495
Cc: Jain Priyanka-B32167; Wood Scott-B07421; u-boot at lists.denx.de; Wang Dongsheng-B40534
Subject: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportFrom: Wang Dongsheng <[email protected]>
We should remove this support.
First, there is not any documents to say we need to support this feature, and kernel also not support this feature.
Second, Kerneal not support this on T1040QDS, so if we open it in u-boot, FPGA will be changed and that will affect deep sleep feature.Signed-off-by Wang Dongsheng <[email protected]>board/freescale/t1040qds/Makefile +0
-1@@ -10,4 +10,3 @@ obj-$(CONFIG_PCI) += pci.o obj-y += law.o obj-y += tlb.o obj-y += eth.o -obj-y += diu.o(quoted diff) +0
-0
b/board/freescale/t1040qds/diu.cdeleted file mode 100644 index 0214224..0000000board/freescale/t1040qds/diu.c deleted +0
-23@@ -1,97 +0,0 @@ -/* -* Copyright 2014 Freescale Semiconductor, Inc. -* Author: Priyanka Jain <[email protected]> -* -* SPDX-License-Identifier: GPL-2.0+ -*/ - -#include <common.h> -#include <command.h> -#include <linux/ctype.h> -#include <asm/io.h> -#include <stdio_dev.h> -#include <video_fb.h> -#include <fsl_diu_fb.h> -#include "../common/qixis.h" -#include "../common/diu_ch7301.h" -#include "t1040qds.h" -#include "t1040qds_qixis.h" - -/* -* DIU Area Descriptor -* -* Note that we need to byte-swap the value before it's written to the AD
- * register. So even though the registers don't look like they're in the same
- * bit positions as they are on the MPC8610, the same value is written to the
- * AD register on the MPC8610 and on the P1022.
- */board/freescale/t1040qds/diu.c deleted +0
-43-#define AD_BYTE_F 0x10000000 -#define AD_ALPHA_C_SHIFT 25 -#define AD_BLUE_C_SHIFT 23 -#define AD_GREEN_C_SHIFT 21 -#define AD_RED_C_SHIFT 19 -#define AD_PIXEL_S_SHIFT 16 -#define AD_COMP_3_SHIFT 12 -#define AD_COMP_2_SHIFT 8 -#define AD_COMP_1_SHIFT 4 -#define AD_COMP_0_SHIFT 0 - -void diu_set_pixel_clock(unsigned int pixclock) -{ -unsigned long speed_ccb, temp; -u32 pixval; -int ret =0 ; - speed_ccb =get_bus_freq (0 ); - temp =1000000000 / pixclock; - temp *=1000 ; - pixval = speed_ccb / temp; - -/* Program HDMI encoder */ -/* Switch channel to DIU */ -select_i2c_ch_pca9547 (I2C_MUX_CH_DIU ); - -/* Set dispaly encoder */ - ret =diu_set_dvi_encoder (temp); -if (ret) { -puts ("Failed to set DVI encoder\n" ); -return ; - } - -/* Switch channel to default */ -select_i2c_ch_pca9547 (I2C_MUX_CH_DEFAULT ); - -/* Program pixel clock */ -out_be32 ((unsigned *)CONFIG_SYS_FSL_SCFG_PIXCLK_ADDR , - ((pixval <<PXCK_BITS_START ) &PXCK_MASK )); -/* enable clock*/ -out_be32 ((unsigned *)CONFIG_SYS_FSL_SCFG_PIXCLK_ADDR ,PXCKEN_MASK | - ((pixval <<PXCK_BITS_START ) &PXCK_MASK )); -} - -int platform_diu_init(unsigned int xres, unsigned int yres, const char*port) -{
board/freescale/t1040qds/diu.c deleted +0
-25-u32 pixel_format; -u8 sw; - -/*Route I2C4 to DIU system as HSYNC/VSYNC*/ - sw =QIXIS_READ (brdcfg[5 ]); -QIXIS_WRITE (brdcfg[5 ], - ((sw & ~(BRDCFG5_IMX_MASK )) | (BRDCFG5_IMX_DIU ))); - -/*Configure Display ouput port as HDMI*/ - sw =QIXIS_READ (brdcfg[15 ]); -QIXIS_WRITE (brdcfg[15 ], - ((sw & ~(BRDCFG15_LCDPD_MASK |BRDCFG15_DIUSEL_MASK )) - | (BRDCFG15_LCDPD_ENABLED |BRDCFG15_DIUSEL_HDMI ))); - - pixel_format =cpu_to_le32 (AD_BYTE_F | (3 <<AD_ALPHA_C_SHIFT ) | - (0 <<AD_BLUE_C_SHIFT ) | (1 <<AD_GREEN_C_SHIFT ) | - (2 <<AD_RED_C_SHIFT ) | (8 <<AD_COMP_3_SHIFT ) | - (8 <<AD_COMP_2_SHIFT ) | (8 <<AD_COMP_1_SHIFT ) | - (8 <<AD_COMP_0_SHIFT ) | (3 <<AD_PIXEL_S_SHIFT )); - -printf ("DIU: Switching to monitor @ %ux%u\n" , xres, yres); - - -return fsl_diu_init (xres, yres, pixel_format,0 ); -}board/freescale/t1040qds/t1040qds.h +0
-0@@ -9,6 +9,5 @@void fdt_fixup_board_enet (void *blob); void pci_of_setup(void *blob, bd_t *bd); -int select_i2c_ch_pca9547(u8 ch);#endif board/freescale/t1040qds/t1040qds_qixis.h +0
-12@@ -13,18 +13,6 @@#define BRDCFG4_EMISEL_MASK 0xE0#define BRDCFG4_EMISEL_SHIFT 5 -/* BRDCFG5[0:1] controls routing and use of I2C3 & I2C4 ports*/ -#define BRDCFG5_IMX_MASK 0xC0 -#define BRDCFG5_IMX_DIU 0x80 - -/* BRDCFG15[3] controls LCD Panel Powerdown*/ -#define BRDCFG15_LCDPD_MASK 0x10 -#define BRDCFG15_LCDPD_ENABLED 0x00 - -/* BRDCFG15[6:7] controls DIU MUX selction*/ -#define BRDCFG15_DIUSEL_MASK 0x03 -#define BRDCFG15_DIUSEL_HDMI 0x00 -/* SYSCLK */ #define QIXIS_SYSCLK_66 0x0#define QIXIS_SYSCLK_83 0x1include/configs/T1040QDS.h +1
-1@@ -375,7 +375,7 @@ unsigned long get_board_ddr_clk(void);#define CONFIG_SYS_INIT_SP_OFFSET CONFIG_SYS_GBL_DATA_OFFSET#define CONFIG_SYS_MONITOR_LEN (512 * 1024) -#define CONFIG_SYS_MALLOC_LEN (10 * 1024 * 1024) +#define CONFIG_SYS_MALLOC_LEN (4 * 1024 * 1024)/* Serial Port - controlled on board with jumper J8 * open - index 2 @@ -401,26 +401,6 @@ unsigned long get_board_ddr_clk(void); #define
CONFIG_SYS_HUSH_PARSER#defineCONFIG_SYS_PROMPT_HUSH_PS2"> "include/configs/T1040QDS.h +0
-15-/* Video */ -#define CONFIG_FSL_DIU_FB -#ifdef CONFIG_FSL_DIU_FB -#define CONFIG_FSL_DIU_CH7301 -#define CONFIG_SYS_DIU_ADDR (CONFIG_SYS_CCSRBAR + 0x180000) -#define CONFIG_VIDEO -#define CONFIG_CMD_BMP -#define CONFIG_CFB_CONSOLE -#define CONFIG_VIDEO_SW_CURSOR -#define CONFIG_VGA_AS_SINGLE_DEVICE -#define CONFIG_VIDEO_LOGO -#define CONFIG_VIDEO_BMP_LOGO -#define CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS -/* -* With CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS, flash I/O is really slow, so
- * disable empty flash sector detection, which is I/O-intensive.
- */include/configs/T1040QDS.h +0
-10-#undef CONFIG_SYS_FLASH_EMPTY_INFO -#endif -/* pass open firmware flat tree */ #define CONFIG_OF_LIBFDT #define CONFIG_OF_BOARD_SETUP @@ -446,11 +426,6 @@ unsigned long get_board_ddr_clk(void);/* I2C bus multiplexer */ #define I2C_MUX_CH_DEFAULT 0x8 -#define I2C_MUX_CH_DIU 0xC - -/* LDI/DVI Encoder for display */ -#define CONFIG_SYS_I2C_LDI_ADDR 0x38 -#define CONFIG_SYS_I2C_DVI_ADDR 0x75/* * RTC configuration @@ -728,7 +703,6 @@ unsigned long get_board_ddr_clk(void);"bank_intlv=cs0_cs1;" \ "usb1:dr_mode=host,phy_type=" __stringify(__USB_PHY_TYPE) "\0"\"netdev=eth0\0" \ -"video-mode=fslfb:1024x768-32 at 60,monitor=dvi\0" \ "uboot=" __stringify(CONFIG_UBOOTPATH) "\0" \ "ubootaddr=" __stringify(CONFIG_SYS_TEXT_BASE) "\0" \"tftpflash=tftpboot $loadaddr $uboot && " \ --1.8.5
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
Quoted 9 lines
-----Original Message-----
From: Jain Priyanka-B32167
Sent: Friday, April 18, 2014 4:26 PM
To: Wang Dongsheng-B40534; Sun York-R58495
Cc: Wood Scott-B07421; u-boot at lists.denx.de; Wang Dongsheng-B40534
Subject: RE: [PATCH] powerpc/t1040qds: Remove Video - HDMI support
Quoted 6 lines
Hello Dongsheng,
We do have requirement to support this that's why code development was done. Also , what is the dependency of deep-sleep on this. Please elaborate And if something is broken, we should fix it. Instead of removing the feature.
If we have, why kernel doesn't support DIU on T1040QDS? That I must remove it reason.
Regards,
-Dongsheng
Quoted 236 lines
Regards
PriyankaQuoted 232 lines
-----Original Message-----
From: Dongsheng Wang [mailto:dongsheng.wang at freescale.com]
Sent: Friday, April 18, 2014 1:09 PM
To: Sun York-R58495
Cc: Jain Priyanka-B32167; Wood Scott-B07421; u-boot at lists.denx.de; Wang Dongsheng-B40534
Subject: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportFrom: Wang Dongsheng <[email protected]>
We should remove this support.
First, there is not any documents to say we need to support this feature, and kernel also not support this feature.
Second, Kerneal not support this on T1040QDS, so if we open it in u-boot, FPGA will be changed and that will affect deep sleep feature.Signed-off-by Wang Dongsheng <[email protected]>board/freescale/t1040qds/Makefile +0
-1@@ -10,4 +10,3 @@ obj-$(CONFIG_PCI) += pci.o obj-y += law.o obj-y += tlb.o obj-y += eth.o -obj-y += diu.o(quoted diff) +0
-0
b/board/freescale/t1040qds/diu.cdeleted file mode 100644 index 0214224..0000000board/freescale/t1040qds/diu.c deleted +0
-23@@ -1,97 +0,0 @@ -/* -* Copyright 2014 Freescale Semiconductor, Inc. -* Author: Priyanka Jain <[email protected]> -* -* SPDX-License-Identifier: GPL-2.0+ -*/ - -#include <common.h> -#include <command.h> -#include <linux/ctype.h> -#include <asm/io.h> -#include <stdio_dev.h> -#include <video_fb.h> -#include <fsl_diu_fb.h> -#include "../common/qixis.h" -#include "../common/diu_ch7301.h" -#include "t1040qds.h" -#include "t1040qds_qixis.h" - -/* -* DIU Area Descriptor -* -* Note that we need to byte-swap the value before it's written to the AD
- * register. So even though the registers don't look like they're in the same
- * bit positions as they are on the MPC8610, the same value is written to the
- * AD register on the MPC8610 and on the P1022.
- */board/freescale/t1040qds/diu.c deleted +0
-43-#define AD_BYTE_F 0x10000000 -#define AD_ALPHA_C_SHIFT 25 -#define AD_BLUE_C_SHIFT 23 -#define AD_GREEN_C_SHIFT 21 -#define AD_RED_C_SHIFT 19 -#define AD_PIXEL_S_SHIFT 16 -#define AD_COMP_3_SHIFT 12 -#define AD_COMP_2_SHIFT 8 -#define AD_COMP_1_SHIFT 4 -#define AD_COMP_0_SHIFT 0 - -void diu_set_pixel_clock(unsigned int pixclock) -{ -unsigned long speed_ccb, temp; -u32 pixval; -int ret =0 ; - speed_ccb =get_bus_freq (0 ); - temp =1000000000 / pixclock; - temp *=1000 ; - pixval = speed_ccb / temp; - -/* Program HDMI encoder */ -/* Switch channel to DIU */ -select_i2c_ch_pca9547 (I2C_MUX_CH_DIU ); - -/* Set dispaly encoder */ - ret =diu_set_dvi_encoder (temp); -if (ret) { -puts ("Failed to set DVI encoder\n" ); -return ; - } - -/* Switch channel to default */ -select_i2c_ch_pca9547 (I2C_MUX_CH_DEFAULT ); - -/* Program pixel clock */ -out_be32 ((unsigned *)CONFIG_SYS_FSL_SCFG_PIXCLK_ADDR , - ((pixval <<PXCK_BITS_START ) &PXCK_MASK )); -/* enable clock*/ -out_be32 ((unsigned *)CONFIG_SYS_FSL_SCFG_PIXCLK_ADDR ,PXCKEN_MASK | - ((pixval <<PXCK_BITS_START ) &PXCK_MASK )); -} - -int platform_diu_init(unsigned int xres, unsigned int yres, const char*port) -{
board/freescale/t1040qds/diu.c deleted +0
-25-u32 pixel_format; -u8 sw; - -/*Route I2C4 to DIU system as HSYNC/VSYNC*/ - sw =QIXIS_READ (brdcfg[5 ]); -QIXIS_WRITE (brdcfg[5 ], - ((sw & ~(BRDCFG5_IMX_MASK )) | (BRDCFG5_IMX_DIU ))); - -/*Configure Display ouput port as HDMI*/ - sw =QIXIS_READ (brdcfg[15 ]); -QIXIS_WRITE (brdcfg[15 ], - ((sw & ~(BRDCFG15_LCDPD_MASK |BRDCFG15_DIUSEL_MASK )) - | (BRDCFG15_LCDPD_ENABLED |BRDCFG15_DIUSEL_HDMI ))); - - pixel_format =cpu_to_le32 (AD_BYTE_F | (3 <<AD_ALPHA_C_SHIFT ) | - (0 <<AD_BLUE_C_SHIFT ) | (1 <<AD_GREEN_C_SHIFT ) | - (2 <<AD_RED_C_SHIFT ) | (8 <<AD_COMP_3_SHIFT ) | - (8 <<AD_COMP_2_SHIFT ) | (8 <<AD_COMP_1_SHIFT ) | - (8 <<AD_COMP_0_SHIFT ) | (3 <<AD_PIXEL_S_SHIFT )); - -printf ("DIU: Switching to monitor @ %ux%u\n" , xres, yres); - - -return fsl_diu_init (xres, yres, pixel_format,0 ); -}board/freescale/t1040qds/t1040qds.h +0
-0@@ -9,6 +9,5 @@void fdt_fixup_board_enet (void *blob); void pci_of_setup(void *blob, bd_t *bd); -int select_i2c_ch_pca9547(u8 ch);#endif board/freescale/t1040qds/t1040qds_qixis.h +0
-12@@ -13,18 +13,6 @@#define BRDCFG4_EMISEL_MASK 0xE0#define BRDCFG4_EMISEL_SHIFT 5 -/* BRDCFG5[0:1] controls routing and use of I2C3 & I2C4 ports*/ -#define BRDCFG5_IMX_MASK 0xC0 -#define BRDCFG5_IMX_DIU 0x80 - -/* BRDCFG15[3] controls LCD Panel Powerdown*/ -#define BRDCFG15_LCDPD_MASK 0x10 -#define BRDCFG15_LCDPD_ENABLED 0x00 - -/* BRDCFG15[6:7] controls DIU MUX selction*/ -#define BRDCFG15_DIUSEL_MASK 0x03 -#define BRDCFG15_DIUSEL_HDMI 0x00 -/* SYSCLK */ #define QIXIS_SYSCLK_66 0x0#define QIXIS_SYSCLK_83 0x1include/configs/T1040QDS.h +1
-1@@ -375,7 +375,7 @@ unsigned long get_board_ddr_clk(void);#define CONFIG_SYS_INIT_SP_OFFSET CONFIG_SYS_GBL_DATA_OFFSET#define CONFIG_SYS_MONITOR_LEN (512 * 1024) -#define CONFIG_SYS_MALLOC_LEN (10 * 1024 * 1024) +#define CONFIG_SYS_MALLOC_LEN (4 * 1024 * 1024)/* Serial Port - controlled on board with jumper J8 * open - index 2 @@ -401,26 +401,6 @@ unsigned long get_board_ddr_clk(void); #define
CONFIG_SYS_HUSH_PARSER#defineCONFIG_SYS_PROMPT_HUSH_PS2"> "include/configs/T1040QDS.h +0
-15-/* Video */ -#define CONFIG_FSL_DIU_FB -#ifdef CONFIG_FSL_DIU_FB -#define CONFIG_FSL_DIU_CH7301 -#define CONFIG_SYS_DIU_ADDR (CONFIG_SYS_CCSRBAR + 0x180000) -#define CONFIG_VIDEO -#define CONFIG_CMD_BMP -#define CONFIG_CFB_CONSOLE -#define CONFIG_VIDEO_SW_CURSOR -#define CONFIG_VGA_AS_SINGLE_DEVICE -#define CONFIG_VIDEO_LOGO -#define CONFIG_VIDEO_BMP_LOGO -#define CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS -/* -* With CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS, flash I/O is really slow, so
- * disable empty flash sector detection, which is I/O-intensive.
- */include/configs/T1040QDS.h +0
-10-#undef CONFIG_SYS_FLASH_EMPTY_INFO -#endif -/* pass open firmware flat tree */ #define CONFIG_OF_LIBFDT #define CONFIG_OF_BOARD_SETUP @@ -446,11 +426,6 @@ unsigned long get_board_ddr_clk(void);/* I2C bus multiplexer */ #define I2C_MUX_CH_DEFAULT 0x8 -#define I2C_MUX_CH_DIU 0xC - -/* LDI/DVI Encoder for display */ -#define CONFIG_SYS_I2C_LDI_ADDR 0x38 -#define CONFIG_SYS_I2C_DVI_ADDR 0x75/* * RTC configuration @@ -728,7 +703,6 @@ unsigned long get_board_ddr_clk(void);"bank_intlv=cs0_cs1;" \ "usb1:dr_mode=host,phy_type=" __stringify(__USB_PHY_TYPE) "\0"\"netdev=eth0\0" \ -"video-mode=fslfb:1024x768-32 at 60,monitor=dvi\0" \ "uboot=" __stringify(CONFIG_UBOOTPATH) "\0" \ "ubootaddr=" __stringify(CONFIG_SYS_TEXT_BASE) "\0" \"tftpflash=tftpboot $loadaddr $uboot && " \ --1.8.5
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
Quoted 9 lines · On 04/18/2014 02:32 AM, Wang Dongsheng-B40534 wrote:
Quoted 9 lines
-----Original Message-----
From: Jain Priyanka-B32167
Sent: Friday, April 18, 2014 4:26 PM
To: Wang Dongsheng-B40534; Sun York-R58495
Cc: Wood Scott-B07421; u-boot at lists.denx.de; Wang Dongsheng-B40534
Subject: RE: [PATCH] powerpc/t1040qds: Remove Video - HDMI support
Quoted 9 lines
Quoted 6 lines
Hello Dongsheng,
We do have requirement to support this that's why code development was done. Also , what is the dependency of deep-sleep on this. Please elaborate And if something is broken, we should fix it. Instead of removing the feature.
If we have, why kernel doesn't support DIU on T1040QDS? That I must remove it reason.
Guys,
Do we have an agreement if this feature should be kept or removed?
York
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
Hi York,
Sorry for late response. We don't need to remove this patch. Please keep this patch.
Regards,
Dongsheng
Quoted 34 lines
-----Original Message-----
From: Sun York-R58495
Sent: Thursday, August 07, 2014 1:17 AM
To: Wang Dongsheng-B40534; Jain Priyanka-B32167
Cc: Wood Scott-B07421; u-boot at lists.denx.de
Subject: Re: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportQuoted 9 lines · On 04/18/2014 02:32 AM, Wang Dongsheng-B40534 wrote:
Quoted 9 lines
-----Original Message-----
From: Jain Priyanka-B32167
Sent: Friday, April 18, 2014 4:26 PM
To: Wang Dongsheng-B40534; Sun York-R58495
Cc: Wood Scott-B07421; u-boot at lists.denx.de; Wang Dongsheng-B40534
Subject: RE: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportQuoted 8 lines
Quoted 6 lines
Hello Dongsheng,
We do have requirement to support this that's why code development was done. Also , what is the dependency of deep-sleep on this. Please elaborate And if something is broken, we should fix it. Instead of removing the feature.
If we have, why kernel doesn't support DIU on T1040QDS? That I must remove it
reason.
Quoted 1 line
Guys,
Do we have an agreement if this feature should be kept or removed?
York
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
In reply to York SunHi York,
Sorry, I'm wrong. Please drop the "powerpc/t1040qds: Remove Video - HDMI support" patch.
Cause of we need to keep u-boot support to QDS-DIU feature.
Regards,
Dongsheng
Quoted 51 lines
-----Original Message-----
From: Wang Dongsheng-B40534
Sent: Thursday, August 07, 2014 10:22 AM
To: Sun York-R58495; Jain Priyanka-B32167
Cc: Wood Scott-B07421; u-boot at lists.denx.de
Subject: RE: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportHi York,
Sorry for late response. We don't need to remove this patch. Please keep this patch.
Regards,
DongshengQuoted 7 lines
-----Original Message-----
From: Sun York-R58495
Sent: Thursday, August 07, 2014 1:17 AM
To: Wang Dongsheng-B40534; Jain Priyanka-B32167
Cc: Wood Scott-B07421; u-boot at lists.denx.de
Subject: Re: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportQuoted 15 lines
Quoted 14 lines · On 04/18/2014 02:32 AM, Wang Dongsheng-B40534 wrote:
Quoted 14 lines
-----Original Message-----
From: Jain Priyanka-B32167
Sent: Friday, April 18, 2014 4:26 PM
To: Wang Dongsheng-B40534; Sun York-R58495
Cc: Wood Scott-B07421; u-boot at lists.denx.de; Wang Dongsheng-B40534
Subject: RE: [PATCH] powerpc/t1040qds: Remove Video - HDMI supportHello Dongsheng,
We do have requirement to support this that's why code development was done. Also , what is the dependency of deep-sleep on this. Please elaborate And if something is broken, we should fix it. Instead of removing
the feature.
Quoted 13 lines
Quoted 4 lines
Quoted 1 line
If we have, why kernel doesn't support DIU on T1040QDS? That I must remove it
reason.
Quoted 1 line
Guys,
Do we have an agreement if this feature should be kept or removed?
York
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
In reply to Dongsheng.Wang at freescale.comQuoted 2 lines · Dongsheng.Wang at freescale.com
If we have, why kernel doesn't support DIU on T1040QDS? That I must remove it reason.
Because no one added that support yet. You should finish the job by fixing the kernel.
PATCH 1/1 powerpc/t1040qds: Remove Video - HDMI support
In reply to Dongsheng WangQuoted 10 lines · Dongsheng Wang
From: Wang Dongsheng <[email protected]>
We should remove this support.
First, there is not any documents to say we need to support this feature, and kernel also not support this feature.
Second, Kerneal not support this on T1040QDS, so if we open it in u-boot, FPGA will be changed and that will affect deep sleep feature.Signed-off-by Wang Dongsheng <[email protected]>
Hold on this. Let's sort it out before removing this feature. It was developed for a reason.
York