[{"id":1764291,"web_url":"http://patchwork.ozlabs.org/comment/1764291/","msgid":"<2856b1b4-49a0-82ed-b7c5-dfb43f8bb5e2@nelint.com>","list_archive_url":null,"date":"2017-09-06T17:56:45","subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","submitter":{"id":67231,"url":"http://patchwork.ozlabs.org/api/people/67231/","name":"Eric Nelson","email":"eric@nelint.com"},"content":"Thanks Fabio,\n\nOn 09/06/2017 09:49 AM, Fabio Estevam wrote:\n> From: Fabio Estevam <fabio.estevam@nxp.com>\n> \n> MX6Q/QP IPU operates at 264MHz and MX6DL IPU at 198MHz.\n> \n> When running a SPL target, which supports multiple MX6 variants we cannot\n> properly setup the IPU clock frequency via CONFIG_IPUV3_CLK option as\n> such decision is done in build-time currently.\n> \n> Remove the CONFIG_IPUV3_CLK option and let the IPU clock frequency be\n> configured in run-time on mx6.\n> \n> Reported-by: Eric Nelson <eric@nelint.com>\n> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n> Changes since v1:\n> - Improve the mx6 detection logic (Troy)\n> \n>   drivers/video/ipu_common.c            | 14 +++++++++++++-\n>   include/configs/advantech_dms-ba16.h  |  1 -\n>   include/configs/apalis_imx6.h         |  1 -\n>   include/configs/aristainetos-common.h |  1 -\n>   include/configs/cgtqmx6eval.h         |  4 ----\n>   include/configs/cm_fx6.h              |  1 -\n>   include/configs/colibri_imx6.h        |  1 -\n>   include/configs/embestmx6boards.h     |  1 -\n>   include/configs/ge_bx50v3.h           |  1 -\n>   include/configs/gw_ventana.h          |  1 -\n>   include/configs/imx6-engicam.h        |  1 -\n>   include/configs/m53evk.h              |  1 -\n>   include/configs/mx51evk.h             |  1 -\n>   include/configs/mx53cx9020.h          |  1 -\n>   include/configs/mx53loco.h            |  1 -\n>   include/configs/mx6cuboxi.h           |  1 -\n>   include/configs/mx6sabre_common.h     |  5 -----\n>   include/configs/nitrogen6x.h          |  1 -\n>   include/configs/novena.h              |  1 -\n>   include/configs/tbs2910.h             |  1 -\n>   include/configs/wandboard.h           |  1 -\n>   scripts/config_whitelist.txt          |  1 -\n>   22 files changed, 13 insertions(+), 29 deletions(-)\n> \n> diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c\n> index f8d4488..a9584b8 100644\n> --- a/drivers/video/ipu_common.c\n> +++ b/drivers/video/ipu_common.c\n> @@ -19,6 +19,7 @@\n>   #include <linux/errno.h>\n>   #include <asm/arch/imx-regs.h>\n>   #include <asm/arch/crm_regs.h>\n> +#include <asm/arch/sys_proto.h>\n>   #include <div64.h>\n>   #include \"ipu.h\"\n>   #include \"ipu_regs.h\"\n> @@ -81,6 +82,11 @@ struct ipu_ch_param {\n>   \n>   #define IPU_SW_RST_TOUT_USEC\t(10000)\n>   \n> +#define IPUV3_CLK_MX51\t\t133000000\n> +#define IPUV3_CLK_MX53\t\t200000000\n> +#define IPUV3_CLK_MX6Q\t\t264000000\n> +#define IPUV3_CLK_MX6DL\t\t198000000\n> +\n>   void clk_enable(struct clk *clk)\n>   {\n>   \tif (clk) {\n> @@ -196,7 +202,6 @@ static void clk_ipu_disable(struct clk *clk)\n>   \n>   static struct clk ipu_clk = {\n>   \t.name = \"ipu_clk\",\n> -\t.rate = CONFIG_IPUV3_CLK,\n>   #if defined(CONFIG_MX51) || defined(CONFIG_MX53)\n>   \t.enable_reg = (u32 *)(CCM_BASE_ADDR +\n>   \t\toffsetof(struct mxc_ccm_reg, CCGR5)),\n> @@ -476,6 +481,13 @@ int ipu_probe(void)\n>   \tg_pixel_clk[1] = &pixel_clk[1];\n>   \n>   \tg_ipu_clk = &ipu_clk;\n> +#if defined(CONFIG_MX51)\n> +\tg_ipu_clk->rate = IPUV3_CLK_MX51;\n> +#elif defined(CONFIG_MX53)\n> +\tg_ipu_clk->rate = IPUV3_CLK_MX53;\n> +#else\n> +\tg_ipu_clk->rate = is_mx6sdl() ? IPUV3_CLK_MX6DL : IPUV3_CLK_MX6Q;\n> +#endif\n>   \tdebug(\"ipu_clk = %u\\n\", clk_get_rate(g_ipu_clk));\n>   \tg_ldb_clk = &ldb_clk;\n>   \tdebug(\"ldb_clk = %u\\n\", clk_get_rate(g_ldb_clk));\n> diff --git a/include/configs/advantech_dms-ba16.h b/include/configs/advantech_dms-ba16.h\n> index 6329bf6..6e380d0 100644\n> --- a/include/configs/advantech_dms-ba16.h\n> +++ b/include/configs/advantech_dms-ba16.h\n> @@ -260,7 +260,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK                260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   #endif\n> diff --git a/include/configs/apalis_imx6.h b/include/configs/apalis_imx6.h\n> index 16af141..f10ce6d 100644\n> --- a/include/configs/apalis_imx6.h\n> +++ b/include/configs/apalis_imx6.h\n> @@ -124,7 +124,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>   #define CONFIG_CONSOLE_MUX\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n> diff --git a/include/configs/aristainetos-common.h b/include/configs/aristainetos-common.h\n> index 1c28fcf..3afc7a6 100644\n> --- a/include/configs/aristainetos-common.h\n> +++ b/include/configs/aristainetos-common.h\n> @@ -217,7 +217,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 198000000\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   \n>   #define CONFIG_PWM_IMX\n> diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h\n> index 4996a89..6a6c063 100644\n> --- a/include/configs/cgtqmx6eval.h\n> +++ b/include/configs/cgtqmx6eval.h\n> @@ -87,10 +87,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#ifdef CONFIG_MX6DL\n> -#define CONFIG_IPUV3_CLK 198000000\n> -#else\n> -#define CONFIG_IPUV3_CLK 264000000\n>   #endif\n>   #define CONFIG_IMX_HDMI\n>   \n> diff --git a/include/configs/cm_fx6.h b/include/configs/cm_fx6.h\n> index 4f45be1..5d4b670 100644\n> --- a/include/configs/cm_fx6.h\n> +++ b/include/configs/cm_fx6.h\n> @@ -240,7 +240,6 @@\n>   \n>   /* Display */\n>   #define CONFIG_VIDEO_IPUV3\n> -#define CONFIG_IPUV3_CLK          260000000\n>   #define CONFIG_IMX_HDMI\n>   \n>   #define CONFIG_SPLASH_SCREEN\n> diff --git a/include/configs/colibri_imx6.h b/include/configs/colibri_imx6.h\n> index fca72f4..cb8a0e2 100644\n> --- a/include/configs/colibri_imx6.h\n> +++ b/include/configs/colibri_imx6.h\n> @@ -110,7 +110,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>   #define CONFIG_CONSOLE_MUX\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n> diff --git a/include/configs/embestmx6boards.h b/include/configs/embestmx6boards.h\n> index d1dec80..1400d66 100644\n> --- a/include/configs/embestmx6boards.h\n> +++ b/include/configs/embestmx6boards.h\n> @@ -113,7 +113,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   \n> diff --git a/include/configs/ge_bx50v3.h b/include/configs/ge_bx50v3.h\n> index d090cdd..9906241 100644\n> --- a/include/configs/ge_bx50v3.h\n> +++ b/include/configs/ge_bx50v3.h\n> @@ -290,7 +290,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   #endif\n> diff --git a/include/configs/gw_ventana.h b/include/configs/gw_ventana.h\n> index aeacd46..b7a1e10 100644\n> --- a/include/configs/gw_ventana.h\n> +++ b/include/configs/gw_ventana.h\n> @@ -160,7 +160,6 @@\n>   /* Framebuffer and LCD */\n>   #define CONFIG_VIDEO_IPUV3\n>   #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK          260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   #define CONFIG_VIDEO_BMP_LOGO\n> diff --git a/include/configs/imx6-engicam.h b/include/configs/imx6-engicam.h\n> index a1b7036..c34dc30 100644\n> --- a/include/configs/imx6-engicam.h\n> +++ b/include/configs/imx6-engicam.h\n> @@ -199,7 +199,6 @@\n>   \n>   /* Framebuffer */\n>   #ifdef CONFIG_VIDEO_IPUV3\n> -# define CONFIG_IPUV3_CLK\t\t260000000\n>   # define CONFIG_IMX_VIDEO_SKIP\n>   \n>   # define CONFIG_SPLASH_SCREEN\n> diff --git a/include/configs/m53evk.h b/include/configs/m53evk.h\n> index 4dc6e16..a0d74a0 100644\n> --- a/include/configs/m53evk.h\n> +++ b/include/configs/m53evk.h\n> @@ -179,7 +179,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_SYS_VIDEO_LOGO_MAX_SIZE\t(2 << 20)\n> -#define CONFIG_IPUV3_CLK\t\t200000000\n>   #endif\n>   \n>   /*\n> diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h\n> index 5daa50d..6677686 100644\n> --- a/include/configs/mx51evk.h\n> +++ b/include/configs/mx51evk.h\n> @@ -87,7 +87,6 @@\n>   #define CONFIG_SPLASH_SCREEN\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t133000000\n>   \n>   /* allow to overwrite serial and ethaddr */\n>   #define CONFIG_ENV_OVERWRITE\n> diff --git a/include/configs/mx53cx9020.h b/include/configs/mx53cx9020.h\n> index 4137592..04c3d09 100644\n> --- a/include/configs/mx53cx9020.h\n> +++ b/include/configs/mx53cx9020.h\n> @@ -187,6 +187,5 @@\n>   #define CONFIG_SPLASH_SCREEN\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t200000000\n>   \n>   #endif /* __CONFIG_H */\n> diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h\n> index 3828b21..317ffab 100644\n> --- a/include/configs/mx53loco.h\n> +++ b/include/configs/mx53loco.h\n> @@ -204,6 +204,5 @@\n>   #define CONFIG_SPLASH_SCREEN\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t200000000\n>   \n>   #endif\t\t\t\t/* __CONFIG_H */\n> diff --git a/include/configs/mx6cuboxi.h b/include/configs/mx6cuboxi.h\n> index 5930f59..c9eec52 100644\n> --- a/include/configs/mx6cuboxi.h\n> +++ b/include/configs/mx6cuboxi.h\n> @@ -41,7 +41,6 @@\n>   \n>   /* Framebuffer */\n>   #define CONFIG_VIDEO_IPUV3\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>   #define CONFIG_VIDEO_BMP_RLE8\n>   #define CONFIG_SPLASH_SCREEN\n>   #define CONFIG_SPLASH_SCREEN_ALIGN\n> diff --git a/include/configs/mx6sabre_common.h b/include/configs/mx6sabre_common.h\n> index b849eea..cb700e5 100644\n> --- a/include/configs/mx6sabre_common.h\n> +++ b/include/configs/mx6sabre_common.h\n> @@ -205,11 +205,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#ifdef CONFIG_MX6DL\n> -#define CONFIG_IPUV3_CLK 198000000\n> -#else\n> -#define CONFIG_IPUV3_CLK 264000000\n> -#endif\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   \n> diff --git a/include/configs/nitrogen6x.h b/include/configs/nitrogen6x.h\n> index bc17b51..f710a9c 100644\n> --- a/include/configs/nitrogen6x.h\n> +++ b/include/configs/nitrogen6x.h\n> @@ -85,7 +85,6 @@\n>   #define CONFIG_VIDEO_BMP_GZIP\n>   #define CONFIG_SYS_VIDEO_LOGO_MAX_SIZE (6 * 1024 * 1024)\n>   #define CONFIG_BMP_16BPP\n> -#define CONFIG_IPUV3_CLK 260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   \n> diff --git a/include/configs/novena.h b/include/configs/novena.h\n> index 7c5445d..d661aff 100644\n> --- a/include/configs/novena.h\n> +++ b/include/configs/novena.h\n> @@ -145,7 +145,6 @@\n>   #define CONFIG_SPLASH_SCREEN\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   #endif\n> diff --git a/include/configs/tbs2910.h b/include/configs/tbs2910.h\n> index c3fc8a3..e70ebbf 100644\n> --- a/include/configs/tbs2910.h\n> +++ b/include/configs/tbs2910.h\n> @@ -64,7 +64,6 @@\n>   /* Framebuffer */\n>   #ifdef CONFIG_VIDEO\n>   #define CONFIG_VIDEO_IPUV3\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>   #define CONFIG_VIDEO_BMP_RLE8\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n> diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h\n> index d9237d7..3ba4c29 100644\n> --- a/include/configs/wandboard.h\n> +++ b/include/configs/wandboard.h\n> @@ -71,7 +71,6 @@\n>   #define CONFIG_BMP_16BPP\n>   #define CONFIG_VIDEO_LOGO\n>   #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 260000000\n>   #define CONFIG_IMX_HDMI\n>   #define CONFIG_IMX_VIDEO_SKIP\n>   #endif\n> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt\n> index 0a751e4..83c3d83 100644\n> --- a/scripts/config_whitelist.txt\n> +++ b/scripts/config_whitelist.txt\n> @@ -1127,7 +1127,6 @@ CONFIG_IPAM390_GPIO_BOOTMODE\n>   CONFIG_IPAM390_GPIO_LED_GREEN\n>   CONFIG_IPAM390_GPIO_LED_RED\n>   CONFIG_IPROC\n> -CONFIG_IPUV3_CLK\n>   CONFIG_IP_DEFRAG\n>   CONFIG_IRAM_BASE\n>   CONFIG_IRAM_END\n> \n\nReviewed-by: Eric Nelson <eric@nelint.com>","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"key not found in DNS\" (0-bit key;\n\tsecure) header.d=nelint.com header.i=@nelint.com header.b=\"ZGhzYY7X\";\n\tdkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xnWW06j2Yz9sBW\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu,  7 Sep 2017 03:57:00 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 7F3FDC21F0E; Wed,  6 Sep 2017 17:56:57 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id EC32AC21DB5;\n\tWed,  6 Sep 2017 17:56:53 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid A4A34C21DB5; Wed,  6 Sep 2017 17:56:52 +0000 (UTC)","from mail-pg0-f44.google.com (mail-pg0-f44.google.com\n\t[74.125.83.44])\n\tby lists.denx.de (Postfix) with ESMTPS id A2A4EC21D88\n\tfor <u-boot@lists.denx.de>; Wed,  6 Sep 2017 17:56:51 +0000 (UTC)","by mail-pg0-f44.google.com with SMTP id m9so16543999pgd.3\n\tfor <u-boot@lists.denx.de>; Wed, 06 Sep 2017 10:56:51 -0700 (PDT)","from ?IPv6:2600:8800:1300:1c00:a094:db6e:de0e:2110?\n\t([2600:8800:1300:1c00:a094:db6e:de0e:2110])\n\tby smtp.gmail.com with ESMTPSA id\n\ti2sm391886pgq.81.2017.09.06.10.56.46\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 06 Sep 2017 10:56:49 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H2,\n\tT_DKIM_INVALID autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=nelint.com; s=google;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=ZLSh85A6TxS0W5raKpdyKL5Obp1BN9v4+QxxDod2m9s=;\n\tb=ZGhzYY7Xmrm8+mjejPYDto//99OtmEV87w4ZROFLbmBV6mTXug/rjrnUeyCWZ5bkj2\n\t4uoSEOfGlSAa397tVrDqi3MEeEhDWDjvqUs3QMq33qyxLlpFHvZ00yzbp1pbnCBuoDSd\n\tfR2IRESI2ZS0oRjqOZAJ9ZCKfFh3scnB79r+E=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=ZLSh85A6TxS0W5raKpdyKL5Obp1BN9v4+QxxDod2m9s=;\n\tb=rLrYeViesR+upuiLjiEXUJgI0KSDp2qPJivcefXYzvv1EPOiVEdHCO4HW/BLcu7idp\n\tuJP+dLbiRHiBObllndbGnW8QB39+HDfaZRPeAY2bjf7EZQ3wn+LzXw5dCfdo8b4xPuDu\n\tqDKsta91Raw8JaKbNpWSuYwDCL80XjBYfsb+sB0SzriIJqvGWBUCE6r8qbsMnYo4QhYB\n\t7TqVB2XyOwvpR5vV39sKepQkFUXwhvHoXZ74/Af2WCOZteh7ufLfLQD2JTli44gGQE1o\n\t99p30kEOBE4F+EFP31attDHP3HhjFa3r/0/QEo+ii5ZMqecqSCPGT/R6+eNtMJ4wmr+C\n\tGIyA==","X-Gm-Message-State":"AHPjjUjJKI8enOFWrt7aQRELTmC45voGNZkvt72AWbC1ractHBjHLwpR\n\tEbicD4lxDmZ55+KK","X-Google-Smtp-Source":"ADKCNb7rfknsDRkvHFwi0krb40UCihZhTaW0E4zq1EB4Awhvvw0c44QPc6BfQoe1K8fPa9L9JrYFJQ==","X-Received":"by 10.84.131.79 with SMTP id 73mr9253583pld.281.1504720609716;\n\tWed, 06 Sep 2017 10:56:49 -0700 (PDT)","To":"Fabio Estevam <festevam@gmail.com>, agust@denx.de","References":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","From":"Eric Nelson <eric@nelint.com>","Message-ID":"<2856b1b4-49a0-82ed-b7c5-dfb43f8bb5e2@nelint.com>","Date":"Wed, 6 Sep 2017 10:56:45 -0700","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","Content-Language":"en-US","Cc":"u-boot@lists.denx.de, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1764596,"web_url":"http://patchwork.ozlabs.org/comment/1764596/","msgid":"<1a4910d2-4bf2-fb36-88fe-245d77e87f06@denx.de>","list_archive_url":null,"date":"2017-09-07T08:54:07","subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","submitter":{"id":5771,"url":"http://patchwork.ozlabs.org/api/people/5771/","name":"Stefano Babic","email":"sbabic@denx.de"},"content":"Hi Fabio,\n\nthanks for this.\n\nOn 06/09/2017 18:49, Fabio Estevam wrote:\n> From: Fabio Estevam <fabio.estevam@nxp.com>\n> \n> MX6Q/QP IPU operates at 264MHz and MX6DL IPU at 198MHz.\n> \n> When running a SPL target, which supports multiple MX6 variants we cannot\n> properly setup the IPU clock frequency via CONFIG_IPUV3_CLK option as\n> such decision is done in build-time currently.\n> \n> Remove the CONFIG_IPUV3_CLK option and let the IPU clock frequency be\n> configured in run-time on mx6.\n> \n> Reported-by: Eric Nelson <eric@nelint.com>\n> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n> Changes since v1:\n> - Improve the mx6 detection logic (Troy)\n> \n>  drivers/video/ipu_common.c            | 14 +++++++++++++-\n>  include/configs/advantech_dms-ba16.h  |  1 -\n>  include/configs/apalis_imx6.h         |  1 -\n>  include/configs/aristainetos-common.h |  1 -\n>  include/configs/cgtqmx6eval.h         |  4 ----\n>  include/configs/cm_fx6.h              |  1 -\n>  include/configs/colibri_imx6.h        |  1 -\n>  include/configs/embestmx6boards.h     |  1 -\n>  include/configs/ge_bx50v3.h           |  1 -\n>  include/configs/gw_ventana.h          |  1 -\n>  include/configs/imx6-engicam.h        |  1 -\n>  include/configs/m53evk.h              |  1 -\n>  include/configs/mx51evk.h             |  1 -\n>  include/configs/mx53cx9020.h          |  1 -\n>  include/configs/mx53loco.h            |  1 -\n>  include/configs/mx6cuboxi.h           |  1 -\n>  include/configs/mx6sabre_common.h     |  5 -----\n>  include/configs/nitrogen6x.h          |  1 -\n>  include/configs/novena.h              |  1 -\n>  include/configs/tbs2910.h             |  1 -\n>  include/configs/wandboard.h           |  1 -\n>  scripts/config_whitelist.txt          |  1 -\n>  22 files changed, 13 insertions(+), 29 deletions(-)\n> \n> diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c\n> index f8d4488..a9584b8 100644\n> --- a/drivers/video/ipu_common.c\n> +++ b/drivers/video/ipu_common.c\n> @@ -19,6 +19,7 @@\n>  #include <linux/errno.h>\n>  #include <asm/arch/imx-regs.h>\n>  #include <asm/arch/crm_regs.h>\n> +#include <asm/arch/sys_proto.h>\n>  #include <div64.h>\n>  #include \"ipu.h\"\n>  #include \"ipu_regs.h\"\n> @@ -81,6 +82,11 @@ struct ipu_ch_param {\n>  \n>  #define IPU_SW_RST_TOUT_USEC\t(10000)\n>  \n> +#define IPUV3_CLK_MX51\t\t133000000\n> +#define IPUV3_CLK_MX53\t\t200000000\n> +#define IPUV3_CLK_MX6Q\t\t264000000\n> +#define IPUV3_CLK_MX6DL\t\t198000000\n> +\n>  void clk_enable(struct clk *clk)\n>  {\n>  \tif (clk) {\n> @@ -196,7 +202,6 @@ static void clk_ipu_disable(struct clk *clk)\n>  \n>  static struct clk ipu_clk = {\n>  \t.name = \"ipu_clk\",\n> -\t.rate = CONFIG_IPUV3_CLK,\n>  #if defined(CONFIG_MX51) || defined(CONFIG_MX53)\n>  \t.enable_reg = (u32 *)(CCM_BASE_ADDR +\n>  \t\toffsetof(struct mxc_ccm_reg, CCGR5)),\n> @@ -476,6 +481,13 @@ int ipu_probe(void)\n>  \tg_pixel_clk[1] = &pixel_clk[1];\n>  \n>  \tg_ipu_clk = &ipu_clk;\n> +#if defined(CONFIG_MX51)\n> +\tg_ipu_clk->rate = IPUV3_CLK_MX51;\n> +#elif defined(CONFIG_MX53)\n> +\tg_ipu_clk->rate = IPUV3_CLK_MX53;\n> +#else\n> +\tg_ipu_clk->rate = is_mx6sdl() ? IPUV3_CLK_MX6DL : IPUV3_CLK_MX6Q;\n> +#endif\n>  \tdebug(\"ipu_clk = %u\\n\", clk_get_rate(g_ipu_clk));\n>  \tg_ldb_clk = &ldb_clk;\n>  \tdebug(\"ldb_clk = %u\\n\", clk_get_rate(g_ldb_clk));\n> diff --git a/include/configs/advantech_dms-ba16.h b/include/configs/advantech_dms-ba16.h\n> index 6329bf6..6e380d0 100644\n> --- a/include/configs/advantech_dms-ba16.h\n> +++ b/include/configs/advantech_dms-ba16.h\n> @@ -260,7 +260,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK                260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  #endif\n> diff --git a/include/configs/apalis_imx6.h b/include/configs/apalis_imx6.h\n> index 16af141..f10ce6d 100644\n> --- a/include/configs/apalis_imx6.h\n> +++ b/include/configs/apalis_imx6.h\n> @@ -124,7 +124,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>  #define CONFIG_CONSOLE_MUX\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n> diff --git a/include/configs/aristainetos-common.h b/include/configs/aristainetos-common.h\n> index 1c28fcf..3afc7a6 100644\n> --- a/include/configs/aristainetos-common.h\n> +++ b/include/configs/aristainetos-common.h\n> @@ -217,7 +217,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 198000000\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  \n>  #define CONFIG_PWM_IMX\n> diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h\n> index 4996a89..6a6c063 100644\n> --- a/include/configs/cgtqmx6eval.h\n> +++ b/include/configs/cgtqmx6eval.h\n> @@ -87,10 +87,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#ifdef CONFIG_MX6DL\n> -#define CONFIG_IPUV3_CLK 198000000\n> -#else\n> -#define CONFIG_IPUV3_CLK 264000000\n>  #endif\n>  #define CONFIG_IMX_HDMI\n>  \n> diff --git a/include/configs/cm_fx6.h b/include/configs/cm_fx6.h\n> index 4f45be1..5d4b670 100644\n> --- a/include/configs/cm_fx6.h\n> +++ b/include/configs/cm_fx6.h\n> @@ -240,7 +240,6 @@\n>  \n>  /* Display */\n>  #define CONFIG_VIDEO_IPUV3\n> -#define CONFIG_IPUV3_CLK          260000000\n>  #define CONFIG_IMX_HDMI\n>  \n>  #define CONFIG_SPLASH_SCREEN\n> diff --git a/include/configs/colibri_imx6.h b/include/configs/colibri_imx6.h\n> index fca72f4..cb8a0e2 100644\n> --- a/include/configs/colibri_imx6.h\n> +++ b/include/configs/colibri_imx6.h\n> @@ -110,7 +110,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>  #define CONFIG_CONSOLE_MUX\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n> diff --git a/include/configs/embestmx6boards.h b/include/configs/embestmx6boards.h\n> index d1dec80..1400d66 100644\n> --- a/include/configs/embestmx6boards.h\n> +++ b/include/configs/embestmx6boards.h\n> @@ -113,7 +113,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  \n> diff --git a/include/configs/ge_bx50v3.h b/include/configs/ge_bx50v3.h\n> index d090cdd..9906241 100644\n> --- a/include/configs/ge_bx50v3.h\n> +++ b/include/configs/ge_bx50v3.h\n> @@ -290,7 +290,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  #endif\n> diff --git a/include/configs/gw_ventana.h b/include/configs/gw_ventana.h\n> index aeacd46..b7a1e10 100644\n> --- a/include/configs/gw_ventana.h\n> +++ b/include/configs/gw_ventana.h\n> @@ -160,7 +160,6 @@\n>  /* Framebuffer and LCD */\n>  #define CONFIG_VIDEO_IPUV3\n>  #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK          260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  #define CONFIG_VIDEO_BMP_LOGO\n> diff --git a/include/configs/imx6-engicam.h b/include/configs/imx6-engicam.h\n> index a1b7036..c34dc30 100644\n> --- a/include/configs/imx6-engicam.h\n> +++ b/include/configs/imx6-engicam.h\n> @@ -199,7 +199,6 @@\n>  \n>  /* Framebuffer */\n>  #ifdef CONFIG_VIDEO_IPUV3\n> -# define CONFIG_IPUV3_CLK\t\t260000000\n>  # define CONFIG_IMX_VIDEO_SKIP\n>  \n>  # define CONFIG_SPLASH_SCREEN\n> diff --git a/include/configs/m53evk.h b/include/configs/m53evk.h\n> index 4dc6e16..a0d74a0 100644\n> --- a/include/configs/m53evk.h\n> +++ b/include/configs/m53evk.h\n> @@ -179,7 +179,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_SYS_VIDEO_LOGO_MAX_SIZE\t(2 << 20)\n> -#define CONFIG_IPUV3_CLK\t\t200000000\n>  #endif\n>  \n>  /*\n> diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h\n> index 5daa50d..6677686 100644\n> --- a/include/configs/mx51evk.h\n> +++ b/include/configs/mx51evk.h\n> @@ -87,7 +87,6 @@\n>  #define CONFIG_SPLASH_SCREEN\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t133000000\n>  \n>  /* allow to overwrite serial and ethaddr */\n>  #define CONFIG_ENV_OVERWRITE\n> diff --git a/include/configs/mx53cx9020.h b/include/configs/mx53cx9020.h\n> index 4137592..04c3d09 100644\n> --- a/include/configs/mx53cx9020.h\n> +++ b/include/configs/mx53cx9020.h\n> @@ -187,6 +187,5 @@\n>  #define CONFIG_SPLASH_SCREEN\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t200000000\n>  \n>  #endif /* __CONFIG_H */\n> diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h\n> index 3828b21..317ffab 100644\n> --- a/include/configs/mx53loco.h\n> +++ b/include/configs/mx53loco.h\n> @@ -204,6 +204,5 @@\n>  #define CONFIG_SPLASH_SCREEN\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t200000000\n>  \n>  #endif\t\t\t\t/* __CONFIG_H */\n> diff --git a/include/configs/mx6cuboxi.h b/include/configs/mx6cuboxi.h\n> index 5930f59..c9eec52 100644\n> --- a/include/configs/mx6cuboxi.h\n> +++ b/include/configs/mx6cuboxi.h\n> @@ -41,7 +41,6 @@\n>  \n>  /* Framebuffer */\n>  #define CONFIG_VIDEO_IPUV3\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>  #define CONFIG_VIDEO_BMP_RLE8\n>  #define CONFIG_SPLASH_SCREEN\n>  #define CONFIG_SPLASH_SCREEN_ALIGN\n> diff --git a/include/configs/mx6sabre_common.h b/include/configs/mx6sabre_common.h\n> index b849eea..cb700e5 100644\n> --- a/include/configs/mx6sabre_common.h\n> +++ b/include/configs/mx6sabre_common.h\n> @@ -205,11 +205,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#ifdef CONFIG_MX6DL\n> -#define CONFIG_IPUV3_CLK 198000000\n> -#else\n> -#define CONFIG_IPUV3_CLK 264000000\n> -#endif\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  \n> diff --git a/include/configs/nitrogen6x.h b/include/configs/nitrogen6x.h\n> index bc17b51..f710a9c 100644\n> --- a/include/configs/nitrogen6x.h\n> +++ b/include/configs/nitrogen6x.h\n> @@ -85,7 +85,6 @@\n>  #define CONFIG_VIDEO_BMP_GZIP\n>  #define CONFIG_SYS_VIDEO_LOGO_MAX_SIZE (6 * 1024 * 1024)\n>  #define CONFIG_BMP_16BPP\n> -#define CONFIG_IPUV3_CLK 260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  \n> diff --git a/include/configs/novena.h b/include/configs/novena.h\n> index 7c5445d..d661aff 100644\n> --- a/include/configs/novena.h\n> +++ b/include/configs/novena.h\n> @@ -145,7 +145,6 @@\n>  #define CONFIG_SPLASH_SCREEN\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  #endif\n> diff --git a/include/configs/tbs2910.h b/include/configs/tbs2910.h\n> index c3fc8a3..e70ebbf 100644\n> --- a/include/configs/tbs2910.h\n> +++ b/include/configs/tbs2910.h\n> @@ -64,7 +64,6 @@\n>  /* Framebuffer */\n>  #ifdef CONFIG_VIDEO\n>  #define CONFIG_VIDEO_IPUV3\n> -#define CONFIG_IPUV3_CLK\t\t260000000\n>  #define CONFIG_VIDEO_BMP_RLE8\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n> diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h\n> index d9237d7..3ba4c29 100644\n> --- a/include/configs/wandboard.h\n> +++ b/include/configs/wandboard.h\n> @@ -71,7 +71,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#define CONFIG_IPUV3_CLK 260000000\n>  #define CONFIG_IMX_HDMI\n>  #define CONFIG_IMX_VIDEO_SKIP\n>  #endif\n> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt\n> index 0a751e4..83c3d83 100644\n> --- a/scripts/config_whitelist.txt\n> +++ b/scripts/config_whitelist.txt\n> @@ -1127,7 +1127,6 @@ CONFIG_IPAM390_GPIO_BOOTMODE\n>  CONFIG_IPAM390_GPIO_LED_GREEN\n>  CONFIG_IPAM390_GPIO_LED_RED\n>  CONFIG_IPROC\n> -CONFIG_IPUV3_CLK\n>  CONFIG_IP_DEFRAG\n>  CONFIG_IRAM_BASE\n>  CONFIG_IRAM_END\n> \n\nReviewed-by: Stefano Babic <sbabic@denx.de>\n\nBest regards,\nStefano Babic","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xnvQT0RXgz9s8J\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu,  7 Sep 2017 18:54:25 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid B75A6C21E6C; Thu,  7 Sep 2017 08:54:21 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 54D8CC21D7B;\n\tThu,  7 Sep 2017 08:54:15 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 44E26C21C72; Thu,  7 Sep 2017 08:54:14 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id AAA9DC21D7B\n\tfor <u-boot@lists.denx.de>; Thu,  7 Sep 2017 08:54:13 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xnvQF37ZJz1qql2;\n\tThu,  7 Sep 2017 10:54:13 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xnvQF1fkTz3hjDM;\n\tThu,  7 Sep 2017 10:54:13 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id O_YfolIl4jRN; Thu,  7 Sep 2017 10:54:11 +0200 (CEST)","from babic.homelinux.org (host-88-217-136-221.customer.m-online.net\n\t[88.217.136.221])\n\t(using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPS;\n\tThu,  7 Sep 2017 10:54:11 +0200 (CEST)","from localhost (mail.babic.homelinux.org [127.0.0.1])\n\tby babic.homelinux.org (Postfix) with ESMTP id D9C2645405F9;\n\tThu,  7 Sep 2017 10:54:10 +0200 (CEST)","from babic.homelinux.org ([127.0.0.1])\n\tby localhost (mail.babic.homelinux.org [127.0.0.1]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id ZXyYhXzzQ8oF; Thu,  7 Sep 2017 10:54:08 +0200 (CEST)","from [192.168.178.132] (papero.fritz.box [192.168.178.132])\n\tby babic.homelinux.org (Postfix) with ESMTP id CA61E4540555;\n\tThu,  7 Sep 2017 10:54:07 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":["amavisd-new at mnet-online.de","Debian amavisd-new at babic.homelinux.org"],"To":"Fabio Estevam <festevam@gmail.com>, agust@denx.de","References":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","From":"Stefano Babic <sbabic@denx.de>","Message-ID":"<1a4910d2-4bf2-fb36-88fe-245d77e87f06@denx.de>","Date":"Thu, 7 Sep 2017 10:54:07 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","Content-Language":"de-DE","Cc":"u-boot@lists.denx.de, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1766138,"web_url":"http://patchwork.ozlabs.org/comment/1766138/","msgid":"<20170911105659.78d7f83b@crub>","list_archive_url":null,"date":"2017-09-11T08:56:59","subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","submitter":{"id":872,"url":"http://patchwork.ozlabs.org/api/people/872/","name":"Anatolij Gustschin","email":"agust@denx.de"},"content":"Hi Fabio,\n\nOn Wed,  6 Sep 2017 13:49:31 -0300\nFabio Estevam festevam@gmail.com wrote:\n\n> From: Fabio Estevam <fabio.estevam@nxp.com>\n> \n> MX6Q/QP IPU operates at 264MHz and MX6DL IPU at 198MHz.\n> \n> When running a SPL target, which supports multiple MX6 variants we cannot\n> properly setup the IPU clock frequency via CONFIG_IPUV3_CLK option as\n> such decision is done in build-time currently.\n> \n> Remove the CONFIG_IPUV3_CLK option and let the IPU clock frequency be\n> configured in run-time on mx6.\n> \n> Reported-by: Eric Nelson <eric@nelint.com>\n> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n> Changes since v1:\n> - Improve the mx6 detection logic (Troy)\n\nThanks for the patch! Is this a bug fix and should it go into v2017.09\nrelease?\n\n--\nAnatolij","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrMHn2QWcz9s7G\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 11 Sep 2017 18:57:09 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 185F6C21E4E; Mon, 11 Sep 2017 08:57:05 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A9375C21D57;\n\tMon, 11 Sep 2017 08:57:03 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 9ABC0C21D57; Mon, 11 Sep 2017 08:57:02 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id 4B54CC21CB6\n\tfor <u-boot@lists.denx.de>; Mon, 11 Sep 2017 08:57:02 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xrMHd6Kmlz1qsDC;\n\tMon, 11 Sep 2017 10:57:01 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xrMHd4Yfwz3hqXG;\n\tMon, 11 Sep 2017 10:57:01 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id r0Qo_BQmsVOt; Mon, 11 Sep 2017 10:57:00 +0200 (CEST)","from crub (p4FCB54C4.dip0.t-ipconnect.de [79.203.84.196])\n\t(using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 11 Sep 2017 10:57:00 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"w/LvN+ELlgt4FgytZiphE6G/MKPkqZFRWbqCE6ZSN+E=","Date":"Mon, 11 Sep 2017 10:56:59 +0200","From":"Anatolij Gustschin <agust@denx.de>","To":"Fabio Estevam <festevam@gmail.com>, Fabio Estevam <fabio.estevam@nxp.com>","Message-ID":"<20170911105659.78d7f83b@crub>","In-Reply-To":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","References":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","X-Mailer":"Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu)","MIME-Version":"1.0","Cc":"u-boot@lists.denx.de","Subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1766224,"web_url":"http://patchwork.ozlabs.org/comment/1766224/","msgid":"<CAOMZO5Du+M2f_3=vGAy+TojnTk-tD6s6+i8j7ZvmrcVQQzSZEA@mail.gmail.com>","list_archive_url":null,"date":"2017-09-11T11:04:48","subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","submitter":{"id":6978,"url":"http://patchwork.ozlabs.org/api/people/6978/","name":"Fabio Estevam","email":"festevam@gmail.com"},"content":"Hi Anatolij,\n\nOn Mon, Sep 11, 2017 at 5:56 AM, Anatolij Gustschin <agust@denx.de> wrote:\n\n> Thanks for the patch! Is this a bug fix and should it go into v2017.09\n> release?\n\nI think it can wait until 2017.11.\n\nThanks!","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"umvfdowq\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrQ7G6Kb9z9s0g\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 11 Sep 2017 21:04:58 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 9CB96C21EFE; Mon, 11 Sep 2017 11:04:54 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 25BB0C21E01;\n\tMon, 11 Sep 2017 11:04:52 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid D56F2C21C72; Mon, 11 Sep 2017 11:04:50 +0000 (UTC)","from mail-io0-f176.google.com (mail-io0-f176.google.com\n\t[209.85.223.176])\n\tby lists.denx.de (Postfix) with ESMTPS id 5DD45C21DA9\n\tfor <u-boot@lists.denx.de>; Mon, 11 Sep 2017 11:04:50 +0000 (UTC)","by mail-io0-f176.google.com with SMTP id g32so6030853ioj.2\n\tfor <u-boot@lists.denx.de>; Mon, 11 Sep 2017 04:04:50 -0700 (PDT)","by 10.157.17.176 with HTTP; Mon, 11 Sep 2017 04:04:48 -0700 (PDT)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=FREEMAIL_FROM,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,\n\tT_DKIM_INVALID autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=IZ7WaXBIzHCpWSjKD283RCG/mloLCePLzpELLhNrZjM=;\n\tb=umvfdowqNbXSIY8zL0iRDsJ/xpxZBgCSVke4lefFdJq8uIqk8WcY6OZMzTpNl1EYRN\n\tmWjyjSMpNmbj3yiel+3tGD9sYn9QQCWa6MS7GOY6U6bOK/H0JHqneYScCvcg9oBKJZIy\n\td8OHJCXmP4nNSnumSkDgRqHdj6wkFjGhPzfYNQB5EEoGRphgEId3IemfvXnH3Rkz12Hi\n\tHrY1j080D50IXw/IhqSV5mNMWZ5Kyb15LBSvxKPCoYVxsu13c3PK1HsDc9wKqsw69JFR\n\tyHm+ppde7yTGMiXLKqSlF1EyrYkORmouvqQD5F8Octl2YvmHbxfatvmWeztqCzdddfK7\n\tebTA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=IZ7WaXBIzHCpWSjKD283RCG/mloLCePLzpELLhNrZjM=;\n\tb=nM5RKaUrELrhfhBfnHZEcC1ydS6BKLOPkpCvnN1y89iHK1mhtGC+llt/jkwTqZEYwG\n\t8pS4PYUG6u9Gfm81rNvxBqf7P3N97mPjcbHgEySo3foWrQOcpP4CMLYSVCKcK+n3G2Pr\n\tFTJzMeC1Rgw+aSQJqopfh6QoRRwSJ5uBSJMYINctvfV+VoYtQljvntz4BnGbPBhQbCh6\n\tEBfDQd2UpVFkyh3wzGI8xBBZZg63PvNGRYzTtWGLNdN6VroPMCNbwcsWNP0Bte7fmvZT\n\t3CTYTIBnFHp9Ptq0I5xCbyN8sce7ajHvSqntWCxj9l8aQygR8Le3kyBYkpqscAoQ3DkO\n\tPdhQ==","X-Gm-Message-State":"AHPjjUjG/mMOZ0QJidP9hIjOJHrpovz1AUZmbxCLhF5STF4NyrWxnDj+\n\t3DIhBDA9nJbCe+Ajc2qM8rJ1EIIPUw==","X-Google-Smtp-Source":"AOwi7QBbdjSfEGeFiQFI81g2nRrNawjjcLxMdPzRW9LMkJXZxo2rcKUoDUKiXBQTQUzQA8oc39/PqBRDEgnNcVpofM8=","X-Received":"by 10.202.50.131 with SMTP id y125mr7108489oiy.248.1505127888694;\n\tMon, 11 Sep 2017 04:04:48 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170911105659.78d7f83b@crub>","References":"<1504716571-3208-1-git-send-email-festevam@gmail.com>\n\t<20170911105659.78d7f83b@crub>","From":"Fabio Estevam <festevam@gmail.com>","Date":"Mon, 11 Sep 2017 08:04:48 -0300","Message-ID":"<CAOMZO5Du+M2f_3=vGAy+TojnTk-tD6s6+i8j7ZvmrcVQQzSZEA@mail.gmail.com>","To":"Anatolij Gustschin <agust@denx.de>","Cc":"U-Boot-Denx <u-boot@lists.denx.de>, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1766229,"web_url":"http://patchwork.ozlabs.org/comment/1766229/","msgid":"<20170911131359.3ed1b60b@crub>","list_archive_url":null,"date":"2017-09-11T11:13:59","subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","submitter":{"id":872,"url":"http://patchwork.ozlabs.org/api/people/872/","name":"Anatolij Gustschin","email":"agust@denx.de"},"content":"Hi Fabio,\n\nOn Wed,  6 Sep 2017 13:49:31 -0300\nFabio Estevam festevam@gmail.com wrote:\n\n> From: Fabio Estevam <fabio.estevam@nxp.com>\n> \n> MX6Q/QP IPU operates at 264MHz and MX6DL IPU at 198MHz.\n> \n> When running a SPL target, which supports multiple MX6 variants we cannot\n> properly setup the IPU clock frequency via CONFIG_IPUV3_CLK option as\n> such decision is done in build-time currently.\n> \n> Remove the CONFIG_IPUV3_CLK option and let the IPU clock frequency be\n> configured in run-time on mx6.\n> \n> Reported-by: Eric Nelson <eric@nelint.com>\n> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>\n> ---\n> Changes since v1:\n> - Improve the mx6 detection logic (Troy)\n> \n>  drivers/video/ipu_common.c            | 14 +++++++++++++-\n>  include/configs/advantech_dms-ba16.h  |  1 -\n>  include/configs/apalis_imx6.h         |  1 -\n>  include/configs/aristainetos-common.h |  1 -\n>  include/configs/cgtqmx6eval.h         |  4 ----\n>  include/configs/cm_fx6.h              |  1 -\n>  include/configs/colibri_imx6.h        |  1 -\n>  include/configs/embestmx6boards.h     |  1 -\n>  include/configs/ge_bx50v3.h           |  1 -\n>  include/configs/gw_ventana.h          |  1 -\n>  include/configs/imx6-engicam.h        |  1 -\n>  include/configs/m53evk.h              |  1 -\n>  include/configs/mx51evk.h             |  1 -\n>  include/configs/mx53cx9020.h          |  1 -\n>  include/configs/mx53loco.h            |  1 -\n>  include/configs/mx6cuboxi.h           |  1 -\n>  include/configs/mx6sabre_common.h     |  5 -----\n>  include/configs/nitrogen6x.h          |  1 -\n>  include/configs/novena.h              |  1 -\n>  include/configs/tbs2910.h             |  1 -\n>  include/configs/wandboard.h           |  1 -\n>  scripts/config_whitelist.txt          |  1 -\n>  22 files changed, 13 insertions(+), 29 deletions(-)\n\nApplied to u-boot-video/master after fixing cgtqmx6eval build. Thanks!\n\n...\n> diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h\n> index 4996a89..6a6c063 100644\n> --- a/include/configs/cgtqmx6eval.h\n> +++ b/include/configs/cgtqmx6eval.h\n> @@ -87,10 +87,6 @@\n>  #define CONFIG_BMP_16BPP\n>  #define CONFIG_VIDEO_LOGO\n>  #define CONFIG_VIDEO_BMP_LOGO\n> -#ifdef CONFIG_MX6DL\n> -#define CONFIG_IPUV3_CLK 198000000\n> -#else\n> -#define CONFIG_IPUV3_CLK 264000000\n>  #endif\n\ndropped this #endif.\n\nThanks,\n\nAnatolij","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrQKw0X23z9s83\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 11 Sep 2017 21:14:12 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid C88BDC21F01; Mon, 11 Sep 2017 11:14:09 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id A5B2DC21C72;\n\tMon, 11 Sep 2017 11:14:06 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 60264C21C72; Mon, 11 Sep 2017 11:14:05 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9])\n\tby lists.denx.de (Postfix) with ESMTPS id F09A5C21C3F\n\tfor <u-boot@lists.denx.de>; Mon, 11 Sep 2017 11:14:04 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xrQKm42CQz1qtwt;\n\tMon, 11 Sep 2017 13:14:04 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xrQKl0t76z3jgXw;\n\tMon, 11 Sep 2017 13:14:03 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id EgINHKzgup2Q; Mon, 11 Sep 2017 13:14:01 +0200 (CEST)","from crub (p4FCB54C4.dip0.t-ipconnect.de [79.203.84.196])\n\t(using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 11 Sep 2017 13:14:01 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"JYCBlQUAfCsKdydMDo2Lj3MS80gLjIVgaP2kxwY96Pc=","Date":"Mon, 11 Sep 2017 13:13:59 +0200","From":"Anatolij Gustschin <agust@denx.de>","To":"Fabio Estevam <festevam@gmail.com>","Message-ID":"<20170911131359.3ed1b60b@crub>","In-Reply-To":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","References":"<1504716571-3208-1-git-send-email-festevam@gmail.com>","X-Mailer":"Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu)","MIME-Version":"1.0","Cc":"u-boot@lists.denx.de, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}},{"id":1766230,"web_url":"http://patchwork.ozlabs.org/comment/1766230/","msgid":"<20170911131447.319290d8@crub>","list_archive_url":null,"date":"2017-09-11T11:14:47","subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","submitter":{"id":872,"url":"http://patchwork.ozlabs.org/api/people/872/","name":"Anatolij Gustschin","email":"agust@denx.de"},"content":"Hi Fabio,\n\nOn Mon, 11 Sep 2017 08:04:48 -0300\nFabio Estevam festevam@gmail.com wrote:\n...\n> I think it can wait until 2017.11.\n\nOK, thanks!\n\n--\nAnatolij","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrQLs4xv8z9s7G\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon, 11 Sep 2017 21:15:01 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 68C12C21DFE; Mon, 11 Sep 2017 11:14:56 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 149CFC21DFE;\n\tMon, 11 Sep 2017 11:14:55 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 26E81C21E44; Mon, 11 Sep 2017 11:14:53 +0000 (UTC)","from mail-out.m-online.net (mail-out.m-online.net [212.18.0.10])\n\tby lists.denx.de (Postfix) with ESMTPS id D25C6C21DFE\n\tfor <u-boot@lists.denx.de>; Mon, 11 Sep 2017 11:14:52 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xrQLh3Jz9z1qsCK;\n\tMon, 11 Sep 2017 13:14:52 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xrQLh0cJYz3jgXx;\n\tMon, 11 Sep 2017 13:14:52 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id TwRk4SwIKO3N; Mon, 11 Sep 2017 13:14:48 +0200 (CEST)","from crub (p4FCB54C4.dip0.t-ipconnect.de [79.203.84.196])\n\t(using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tMon, 11 Sep 2017 13:14:48 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"Wzz+NB9u9yahbvhCNxxhofljiYQ2Rhbcnuf6lhAVjUo=","Date":"Mon, 11 Sep 2017 13:14:47 +0200","From":"Anatolij Gustschin <agust@denx.de>","To":"Fabio Estevam <festevam@gmail.com>","Message-ID":"<20170911131447.319290d8@crub>","In-Reply-To":"<CAOMZO5Du+M2f_3=vGAy+TojnTk-tD6s6+i8j7ZvmrcVQQzSZEA@mail.gmail.com>","References":"<1504716571-3208-1-git-send-email-festevam@gmail.com>\n\t<20170911105659.78d7f83b@crub>\n\t<CAOMZO5Du+M2f_3=vGAy+TojnTk-tD6s6+i8j7ZvmrcVQQzSZEA@mail.gmail.com>","X-Mailer":"Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu)","MIME-Version":"1.0","Cc":"U-Boot-Denx <u-boot@lists.denx.de>, Fabio Estevam <fabio.estevam@nxp.com>","Subject":"Re: [U-Boot] [PATCH] ipu_common: Let the MX6 IPU clock be\n\tcalculated in run-time","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}}]