Cover Letter Detail
Show a cover letter.
GET /api/covers/814216/?format=api
{ "id": 814216, "url": "http://patchwork.ozlabs.org/api/covers/814216/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/cover/1505473053-32520-1-git-send-email-jjhiblot@ti.com/", "project": { "id": 18, "url": "http://patchwork.ozlabs.org/api/projects/18/?format=api", "name": "U-Boot", "link_name": "uboot", "list_id": "u-boot.lists.denx.de", "list_email": "u-boot@lists.denx.de", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<1505473053-32520-1-git-send-email-jjhiblot@ti.com>", "list_archive_url": null, "date": "2017-09-15T10:57:23", "name": "[U-Boot,v6,00/10] spl: dm: Make it possible for the SPL to pick its own DTB from a FIT", "submitter": { "id": 70508, "url": "http://patchwork.ozlabs.org/api/people/70508/?format=api", "name": "Jean-Jacques Hiblot", "email": "jjhiblot@ti.com" }, "mbox": "http://patchwork.ozlabs.org/project/uboot/cover/1505473053-32520-1-git-send-email-jjhiblot@ti.com/mbox/", "series": [ { "id": 3280, "url": "http://patchwork.ozlabs.org/api/series/3280/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/list/?series=3280", "date": "2017-09-15T10:57:23", "name": "spl: dm: Make it possible for the SPL to pick its own DTB from a FIT", "version": 6, "mbox": "http://patchwork.ozlabs.org/series/3280/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/814216/comments/", "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\" (1024-bit key;\n\tunprotected) header.d=ti.com header.i=@ti.com header.b=\"CptO86U1\";\n\tdkim-atps=neutral" ], "Received": [ "from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xtsnJ4nQKz9s7c\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 15 Sep 2017 20:57:56 +1000 (AEST)", "by lists.denx.de (Postfix, from userid 105)\n\tid 3AA48C21FA6; Fri, 15 Sep 2017 10:57:52 +0000 (UTC)", "from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id C3F78C21EED;\n\tFri, 15 Sep 2017 10:57:49 +0000 (UTC)", "by lists.denx.de (Postfix, from userid 105)\n\tid EEF4EC21C93; Fri, 15 Sep 2017 10:57:47 +0000 (UTC)", "from lelnx193.ext.ti.com (lelnx193.ext.ti.com [198.47.27.77])\n\tby lists.denx.de (Postfix) with ESMTPS id 5313FC21EEC\n\tfor <u-boot@lists.denx.de>; Fri, 15 Sep 2017 10:57:47 +0000 (UTC)", "from dflxv15.itg.ti.com ([128.247.5.124])\n\tby lelnx193.ext.ti.com (8.15.1/8.15.1) with ESMTP id v8FAviaI006033; \n\tFri, 15 Sep 2017 05:57:44 -0500", "from DFLE104.ent.ti.com (dfle104.ent.ti.com [10.64.6.25])\n\tby dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8FAviwp002803;\n\tFri, 15 Sep 2017 05:57:44 -0500", "from DFLE113.ent.ti.com (10.64.6.34) by DFLE104.ent.ti.com\n\t(10.64.6.25) with Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.845.34;\n\tFri, 15 Sep 2017 05:57:44 -0500", "from dflp33.itg.ti.com (10.64.6.16) by DFLE113.ent.ti.com\n\t(10.64.6.34) with Microsoft SMTP Server (version=TLS1_0,\n\tcipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.845.34 via Frontend\n\tTransport; Fri, 15 Sep 2017 05:57:44 -0500", "from localhost (ileax41-snat.itg.ti.com [10.172.224.153])\n\tby dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8FAvhCw020763;\n\tFri, 15 Sep 2017 05:57:44 -0500" ], "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\tT_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.0", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ti.com;\n\ts=ti-com-17Q1; t=1505473064;\n\tbh=7YIVdjF+uD6ATrKlFWTZz3/lkSw0zjto5CiElUzoTvs=;\n\th=From:To:CC:Subject:Date;\n\tb=CptO86U1igp5YnrOY2sxBnVzAWrSBDF18SO7k+Q+YsoZAQrySKHB5AWs+bd2/Ei09\n\tWfwlZAWUfDqlMAI0hD08jGrkxDI/ftjkTR3hb+SXLJm+IKD5OEFYeFmOHv34uaprP/\n\teKL9rDU+Nh/LRu+96ytON8eORMQ4PcbtTCX69EGc=", "From": "Jean-Jacques Hiblot <jjhiblot@ti.com>", "To": "<trini@konsulko.com>, <lokeshvutla@ti.com>, <sjg@chromium.org>,\n\t<fcooper@ti.com>", "Date": "Fri, 15 Sep 2017 12:57:23 +0200", "Message-ID": "<1505473053-32520-1-git-send-email-jjhiblot@ti.com>", "X-Mailer": "git-send-email 1.9.1", "MIME-Version": "1.0", "X-EXCLAIMER-MD-CONFIG": "e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180", "Cc": "u-boot@lists.denx.de", "Subject": "[U-Boot] [PATCH v6 00/10] spl: dm: Make it possible for the SPL to\n\tpick its own DTB from a FIT", "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>" }, "content": "Following the RFC, here is the series implementing the mechanism in a cleaner\nway. The idea is that the SPL may take advantage of selecting its DTB from a\npool of available DTBs. To do that several DTBs are embedded in a compressed\nFIT image appended at the end of the SPL. The patch is done in such way that\nthis mechanism can easily be extended to other ways of loading the DTBs.\n\nHere are some numbers:\n +----------+------------+-----------+------------+\n | size | size delta | SPL boot | boot time |\n | (bytes) | (bytes) | time (s) | delta (s) |\n+---------------------------+----------+------------+-----------+------------+\n| 1 DTB | | | | |\n+---------------------------+----------+------------+-----------+------------+\n| reference | 125305 | 0 | 1.389 | 0 |\n| LZO (dynamic allocation) | 125391 | 86 | 1.381 | -0.008 |\n+---------------------------+----------+------------+-----------+------------+\n| 4 DTBs (DRA7, DRA71, | | | | |\n| DRA72, DRA72 revC) | | | | |\n+---------------------------+----------+------------+-----------+------------+\n| LZO (dynamic allocation) | 125991 | 686 | 1.39 | 0.001 |\n| LZO (user defined area) | 125927 | 622 | 1.403 | 0.014 |\n| GZIP (user defined area) | 133880 | 8575 | 1.421 | 0.032 |\n| No compression (in place) | 137472 | 12167 | 1.412 | 0.023 |\n+---------------------------+----------+------------+-----------+------------+\n\nchanges since v5:\n - rebased on top of latest u-boot/master: updated patch #7 as support for\n SPL_GZIP has been recently added to u-boot/master.\n - Fixed the wrong ordering of includes in lib/fdtdec.c\n - changed Kconfig messages to start with an an upper-case\n \n\nchanges since v4:\n - Updated the documentation. Hopefully it's more intelligible.\n - renamed SPL_MULTI_DTB_FIT_GZ in SPL_MULTI_DTB_FIT_GZIP\n - Added a patch to sort the included headers in lib/fdtdec.c\n - reduced the level of #ifdeffing of the patch\n\nchanges since v3:\n - Added reviewed-by tags\n - Fixed nits and spelling mistakes reported by Simon G.\n - updated the help in Kconfig to take in account commit f1896c (\"spl: make SPL\n and normal u-boot stage use independent SYS_MALLOC_F_LEN\") \n\nchanges since v2:\n - rebased on u-boot/master\n - improved help in Kconfig for the compression options in SPL\n - improved commit message for patch #1\n - Added some comments to describe locate_dtb_in_fit() and fit_find_config_node()\n - Added a description of the mecanism in a README\n\nchanges since v1:\n - improved help in Kconfig for the SPL_MULTI_DTB_FIT option\n - Added \"reviewed-by\" tags\n\nchanges since RFC:\n - split the RFC patch in several patches.\n - leverage work from Cooper Jr., Franklin (CONFIG_FIT_EMBED) as a big part\n of the code is similar. Rename CONFIG_FIT_EMBED as CONFIG_MULTI_DTB_FIT\n to prevent confusion with CONFIG_OF_EMBED.\n - use the default configuration if no match is provided by\n board_fit_config_name_match(). note that this is generic not just for this\n feature.\n - provide an option to not use dynamic memory allocation.\n - added a patch related to board detection on omap that's required to take\n advantage of this feature.\n \nJean-Jacques Hiblot (10):\n dts: renamed FIT_EMBED to MULTI_DTB_FIT and moved it to the dts\n Kconfig\n fit: use 'const' for the input of fdt_offset() and locate_dtb_in_fit()\n fit: fixed bug in locate_dtb_in_fit()\n fit: If no matching config is found in fit_find_config_node(), use the\n default one\n lzo: add a function to check the validity of the header\n gzip: add a function to parse the header\n lib: allow building lzo for the SPL\n fdtdec: sort include files\n spl: dm: Make it possible for the SPL to pick its own DTB from a FIT\n omap: detect board before spl_early_init()\n\n Makefile | 2 +-\n arch/arm/mach-omap2/hwinit-common.c | 4 +-\n board/ti/ks2_evm/board_k2e.c | 2 +-\n board/ti/ks2_evm/board_k2g.c | 2 +-\n board/ti/ks2_evm/board_k2hk.c | 2 +-\n board/ti/ks2_evm/board_k2l.c | 2 +-\n common/Kconfig | 16 ------\n common/Makefile | 2 +-\n common/boot_fit.c | 6 +--\n common/common_fit.c | 20 +++++++\n configs/k2e_evm_defconfig | 2 +-\n configs/k2g_evm_defconfig | 2 +-\n configs/k2hk_evm_defconfig | 2 +-\n configs/k2l_evm_defconfig | 2 +-\n doc/README.multi-dtb-fit | 65 +++++++++++++++++++++++\n dts/Kconfig | 103 +++++++++++++++++++++++++++++++++++-\n include/boot_fit.h | 9 +++-\n include/common.h | 1 +\n include/image.h | 13 +++++\n include/linux/lzo.h | 3 ++\n lib/Kconfig | 5 ++\n lib/Makefile | 3 +-\n lib/fdtdec.c | 98 +++++++++++++++++++++++++++++-----\n lib/gunzip.c | 15 ++++--\n lib/lzo/lzo1x_decompress.c | 21 ++++++--\n scripts/Makefile.spl | 35 +++++++++++-\n 26 files changed, 381 insertions(+), 56 deletions(-)\n create mode 100644 doc/README.multi-dtb-fit" }