[U-Boot] [PATCH V3 02/12] video:mipidsim:fdt: Add DT support for mipi dsim driver
Piotr Wilczek
p.wilczek at samsung.com
Fri Feb 28 08:48:26 CET 2014
Hi Ajay,
On 02/27/2014 03:59 PM, Ajay kumar wrote:
> Piotr,
>
> DT bindings should usually be free of "_" (underscore)
> Please use hyphen: "-"
I will use hypen.
> On Tue, Feb 25, 2014 at 11:33 PM, Piotr Wilczek <p.wilczek at samsung.com>wrote:
>
>> This patch enables parsing mipi data from device tree.
>> Non device tree case is still supported.
>>
>> Signed-off-by: Piotr Wilczek <p.wilczek at samsung.com>
>> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
>> Cc: Minkyu Kang <mk7.kang at samsung.com>
>> ---
>> Changes for v3:
>> - none
>>
>> Changes for v2:
>> - removed panel specific init function 's6e8ax0_init' to the board file
>>
>> arch/arm/include/asm/arch-exynos/mipi_dsim.h | 5 ++
>> drivers/video/exynos_mipi_dsi.c | 96
>> ++++++++++++++++++++++++++++
>> include/fdtdec.h | 1 +
>> lib/fdtdec.c | 1 +
>> 4 files changed, 103 insertions(+)
>>
>> diff --git a/arch/arm/include/asm/arch-exynos/mipi_dsim.h
>> b/arch/arm/include/asm/arch-exynos/mipi_dsim.h
>> index 40aca71..50e5c25 100644
>> --- a/arch/arm/include/asm/arch-exynos/mipi_dsim.h
>> +++ b/arch/arm/include/asm/arch-exynos/mipi_dsim.h
>> @@ -12,6 +12,7 @@
>>
>> #include <linux/list.h>
>> #include <linux/fb.h>
>> +#include <lcd.h>
>>
>> #define PANEL_NAME_SIZE (32)
>>
>> @@ -368,8 +369,12 @@ int exynos_mipi_dsi_register_lcd_device(struct
>> mipi_dsim_lcd_device
>> *lcd_dev);
>>
>> void exynos_set_dsim_platform_data(struct exynos_platform_mipi_dsim *pd);
>> +void exynos_init_dsim_platform_data(vidinfo_t *vid);
>>
>> /* panel driver init based on mipi dsi interface */
>> void s6e8ax0_init(void);
>>
>> +#ifdef CONFIG_OF_CONTROL
>> +extern int mipi_power(void);
>> +#endif
>> #endif /* _DSIM_H */
>> diff --git a/drivers/video/exynos_mipi_dsi.c
>> b/drivers/video/exynos_mipi_dsi.c
>> index 8bb8fea..6c10f11 100644
>> --- a/drivers/video/exynos_mipi_dsi.c
>> +++ b/drivers/video/exynos_mipi_dsi.c
>> @@ -9,6 +9,8 @@
>>
>> #include <common.h>
>> #include <malloc.h>
>> +#include <fdtdec.h>
>> +#include <libfdt.h>
>> #include <linux/err.h>
>> #include <asm/arch/dsim.h>
>> #include <asm/arch/mipi_dsim.h>
>> @@ -22,7 +24,14 @@
>> #define master_to_driver(a) (a->dsim_lcd_drv)
>> #define master_to_device(a) (a->dsim_lcd_dev)
>>
>> +DECLARE_GLOBAL_DATA_PTR;
>> +
>> static struct exynos_platform_mipi_dsim *dsim_pd;
>> +#ifdef CONFIG_OF_CONTROL
>> +static struct mipi_dsim_config dsim_config_dt;
>> +static struct exynos_platform_mipi_dsim dsim_platform_data_dt;
>> +static struct mipi_dsim_lcd_device mipi_lcd_device_dt;
>> +#endif
>>
>> struct mipi_dsim_ddi {
>> int bus_id;
>> @@ -238,3 +247,90 @@ void exynos_set_dsim_platform_data(struct
>> exynos_platform_mipi_dsim *pd)
>>
>> dsim_pd = pd;
>> }
>> +
>> +#ifdef CONFIG_OF_CONTROL
>> +int exynos_dsim_config_parse_dt(const void *blob)
>> +{
>> + int node;
>> +
>> + node = fdtdec_next_compatible(blob, 0,
>> COMPAT_SAMSUNG_EXYNOS_MIPI_DSI);
>> + if (node <= 0) {
>> + printf("exynos_mipi_dsi: Can't get device node for mipi
>> dsi\n");
>> + return -ENODEV;
>> + }
>> +
>> + dsim_config_dt.e_interface = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-e_interface", 0);
>> +
>> + dsim_config_dt.e_virtual_ch = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-e_virtual_ch", 0);
>> +
>> + dsim_config_dt.e_pixel_format = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-e_pixel_format", 0);
>> +
>> + dsim_config_dt.e_burst_mode = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-e_burst_mode", 0);
>> +
>> + dsim_config_dt.e_no_data_lane = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-e_no_data_lane", 0);
>> +
>> + dsim_config_dt.e_byte_clk = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-e_byte_clk", 0);
>> +
>> + dsim_config_dt.hfp = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-hfp", 0);
>> +
>> + dsim_config_dt.p = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-p", 0);
>> + dsim_config_dt.m = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-m", 0);
>> + dsim_config_dt.s = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-s", 0);
>> +
>> + dsim_config_dt.pll_stable_time = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-pll_stable_time", 0);
>> +
>> + dsim_config_dt.esc_clk = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-esc_clk", 0);
>> +
>> + dsim_config_dt.stop_holding_cnt = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-stop_holding_cnt", 0);
>> +
>> + dsim_config_dt.bta_timeout = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-bta_timeout", 0);
>> +
>> + dsim_config_dt.rx_timeout = fdtdec_get_int(blob, node,
>> + "samsung,dsim-config-rx_timeout", 0);
>> +
>> + mipi_lcd_device_dt.name = fdtdec_get_config_string(blob,
>> + "samsung,dsim-device-name");
>> +
>> + mipi_lcd_device_dt.id = fdtdec_get_int(blob, node,
>> + "samsung,dsim-device-id", 0);
>> +
>> + mipi_lcd_device_dt.bus_id = fdtdec_get_int(blob, node,
>> + "samsung,dsim-device-bus_id", 0);
>> +
>> + mipi_lcd_device_dt.reverse_panel = fdtdec_get_int(blob, node,
>> + "samsung,dsim-device-reverse_panel", 0);
>> +
>> + return 0;
>> +}
>> +
>> +void exynos_init_dsim_platform_data(vidinfo_t *vid)
>> +{
>> + if (exynos_dsim_config_parse_dt(gd->fdt_blob))
>> + debug("Can't get proper dsim config.\n");
>> +
>> + strcpy(dsim_platform_data_dt.lcd_panel_name,
>> mipi_lcd_device_dt.name);
>> + dsim_platform_data_dt.dsim_config = &dsim_config_dt;
>> + dsim_platform_data_dt.mipi_power = mipi_power;
>> + dsim_platform_data_dt.phy_enable = set_mipi_phy_ctrl;
>> + dsim_platform_data_dt.lcd_panel_info = (void *)vid;
>> +
>> + mipi_lcd_device_dt.platform_data = (void *)&dsim_platform_data_dt;
>> + exynos_mipi_dsi_register_lcd_device(&mipi_lcd_device_dt);
>> +
>> + dsim_pd = &dsim_platform_data_dt;
>> +}
>> +#endif
>> diff --git a/include/fdtdec.h b/include/fdtdec.h
>> index 19bab79..bd84c83 100644
>> --- a/include/fdtdec.h
>> +++ b/include/fdtdec.h
>> @@ -79,6 +79,7 @@ enum fdt_compat_id {
>> COMPAT_SAMSUNG_EXYNOS5_USB3_PHY,/* Exynos phy controller for
>> usb3.0 */
>> COMPAT_SAMSUNG_EXYNOS_TMU, /* Exynos TMU */
>> COMPAT_SAMSUNG_EXYNOS_FIMD, /* Exynos Display controller */
>> + COMPAT_SAMSUNG_EXYNOS_MIPI_DSI, /* Exynos mipi dsi */
>> COMPAT_SAMSUNG_EXYNOS5_DP, /* Exynos Display port controller
>> */
>> COMPAT_SAMSUNG_EXYNOS5_DWMMC, /* Exynos5 DWMMC controller */
>> COMPAT_SAMSUNG_EXYNOS_SERIAL, /* Exynos UART */
>> diff --git a/lib/fdtdec.c b/lib/fdtdec.c
>> index 1fecab3..c97fad3 100644
>> --- a/lib/fdtdec.c
>> +++ b/lib/fdtdec.c
>> @@ -52,6 +52,7 @@ static const char * const compat_names[COMPAT_COUNT] = {
>> COMPAT(SAMSUNG_EXYNOS5_USB3_PHY, "samsung,exynos5250-usb3-phy"),
>> COMPAT(SAMSUNG_EXYNOS_TMU, "samsung,exynos-tmu"),
>> COMPAT(SAMSUNG_EXYNOS_FIMD, "samsung,exynos-fimd"),
>> + COMPAT(SAMSUNG_EXYNOS_MIPI_DSI, "samsung,exynos-mipi-dsi"),
>> COMPAT(SAMSUNG_EXYNOS5_DP, "samsung,exynos5-dp"),
>> COMPAT(SAMSUNG_EXYNOS5_DWMMC, "samsung,exynos5250-dwmmc"),
>> COMPAT(SAMSUNG_EXYNOS_SERIAL, "samsung,exynos4210-uart"),
>> --
>> 1.8.3.2
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
>>
>
> Regards,
> Ajay Kumar
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
Best regards,
Piotr Wilczek
More information about the U-Boot
mailing list