Cover Letter Detail
Show a cover letter.
GET /api/covers/812879/?format=api
{ "id": 812879, "url": "http://patchwork.ozlabs.org/api/covers/812879/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/cover/1505227539-17560-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": "<1505227539-17560-1-git-send-email-jjhiblot@ti.com>", "list_archive_url": null, "date": "2017-09-12T14:45:29", "name": "[U-Boot,v5,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/1505227539-17560-1-git-send-email-jjhiblot@ti.com/mbox/", "series": [ { "id": 2721, "url": "http://patchwork.ozlabs.org/api/series/2721/?format=api", "web_url": "http://patchwork.ozlabs.org/project/uboot/list/?series=2721", "date": "2017-09-12T14:45:29", "name": "spl: dm: Make it possible for the SPL to pick its own DTB from a FIT", "version": 5, "mbox": "http://patchwork.ozlabs.org/series/2721/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/812879/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=\"t3v5w+bK\";\n\tdkim-atps=neutral" ], "Received": [ "from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs6zg30X7z9s3T\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 00:45:51 +1000 (AEST)", "by lists.denx.de (Postfix, from userid 105)\n\tid 2F7C7C220BD; Tue, 12 Sep 2017 14:45:47 +0000 (UTC)", "from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 9C9B0C21E9C;\n\tTue, 12 Sep 2017 14:45:44 +0000 (UTC)", "by lists.denx.de (Postfix, from userid 105)\n\tid 443E8C21DE3; Tue, 12 Sep 2017 14:45:43 +0000 (UTC)", "from lelnx193.ext.ti.com (lelnx193.ext.ti.com [198.47.27.77])\n\tby lists.denx.de (Postfix) with ESMTPS id 8F920C21DC4\n\tfor <u-boot@lists.denx.de>; Tue, 12 Sep 2017 14:45:42 +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 v8CEjd3h025575; \n\tTue, 12 Sep 2017 09:45:39 -0500", "from DFLE106.ent.ti.com (dfle106.ent.ti.com [10.64.6.27])\n\tby dflxv15.itg.ti.com (8.14.3/8.13.8) with ESMTP id v8CEjdfj025725;\n\tTue, 12 Sep 2017 09:45:39 -0500", "from DFLE101.ent.ti.com (10.64.6.22) by DFLE106.ent.ti.com\n\t(10.64.6.27) with Microsoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.845.34;\n\tTue, 12 Sep 2017 09:45:38 -0500", "from dflp33.itg.ti.com (10.64.6.16) by DFLE101.ent.ti.com\n\t(10.64.6.22) 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; Tue, 12 Sep 2017 09:45:38 -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 v8CEjbAv023679;\n\tTue, 12 Sep 2017 09:45:38 -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=1505227539;\n\tbh=OYP5rmBkRW1GMRzj+jcfDRTEZvnvCBItb14MVOOG8GY=;\n\th=From:To:CC:Subject:Date;\n\tb=t3v5w+bKIg8PcmhJK7lN9WP1T/qdm+kxHOCwZuZzC/QBtUqC/X3Gus2hx0XhO6Dpr\n\tGD+zXN02t2DOZBVi0Z0Tr+ZiPGFgYMZoB7MxEKeU7lOCMj4T9nkVOqItJJ+fg2KTxA\n\tJwV16kPxN/7/pwyiOhcgM9iLW2AyL2j6Ku1UeNBM=", "From": "Jean-Jacques Hiblot <jjhiblot@ti.com>", "To": "<trini@konsulko.com>, <lokeshvutla@ti.com>, <sjg@chromium.org>,\n\t<fcooper@ti.com>", "Date": "Tue, 12 Sep 2017 16:45:29 +0200", "Message-ID": "<1505227539-17560-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 v5 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 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 and gunzip 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 | 17 +++++-\n lib/Makefile | 6 +--\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, 393 insertions(+), 59 deletions(-)\n create mode 100644 doc/README.multi-dtb-fit" }