[{"id":1760705,"web_url":"http://patchwork.ozlabs.org/comment/1760705/","msgid":"<59A7B9CE.8060902@denx.de>","list_archive_url":null,"date":"2017-08-31T07:25:02","subject":"Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read","submitter":{"id":99,"url":"http://patchwork.ozlabs.org/api/people/99/","name":"Heiko Schocher","email":"hs@denx.de"},"content":"Hello Holger,\n\nAm 30.08.2017 um 18:38 schrieb Holger Dengler:\n> After a successful read of a UBI volume, the variable filesize is set\n> to the number of read bytes. To boot linux with a raw initramfs/initrd,\n> you must specify the exact size of the initramfs/initrd image in boot\n> command. If the initramfs/inird is read from UBI volume, $filesize is\n> required to access the number of read bytes.\n>\n> Example:\n> ubi read ${loadaddr} kernelvol\n> ubi read ${fdtaddr} dtbvol\n> ubi read ${initrd_addr} initrdvol\n> bootz ${loadaddr} ${initrd_addr}:${filesize} ${fdt_addr}\n>\n> Signed-off-by: Holger Dengler \n> ---\n> cmd/ubi.c | 5 +++++\n> 1 file changed, 5 insertions(+)\n\nThanks!\n\nReviewed-by: Heiko Schocher \n\nAs new release is near, I pickup this patch when the next merge window\nopens.\n\nbye,\nHeiko","headers":{"Return-Path":"","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xjYn50cBDz9sNc\n\tfor ;\n\tThu, 31 Aug 2017 17:25:27 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid E9E8AC21D58; Thu, 31 Aug 2017 07:25:18 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 8EE10C21C59;\n\tThu, 31 Aug 2017 07:25:16 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 07CFBC21C51; Thu, 31 Aug 2017 07:25: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 07B44C21C59\n\tfor ; Thu, 31 Aug 2017 07:25:14 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xjYmm6gKnz1rCxL;\n\tThu, 31 Aug 2017 09:25:12 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xjYmm4cXvz3hhTL;\n\tThu, 31 Aug 2017 09:25:12 +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 CkfenSs0wK69; Thu, 31 Aug 2017 09:25:08 +0200 (CEST)","from localhost.localdomain (87.97.1.128.pool.invitel.hu\n\t[87.97.1.128])\n\t(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tThu, 31 Aug 2017 09:25:05 +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":"tA/GhtcvrDEvwSJ8DeUBdeKisd8DTom4g3MLHBn40Oc=","To":"Holger Dengler ","References":"<20170830163852.27454-1-dengler@linutronix.de>","From":"Heiko Schocher ","Message-ID":"<59A7B9CE.8060902@denx.de>","Date":"Thu, 31 Aug 2017 09:25:02 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101\n\tThunderbird/38.1.0","MIME-Version":"1.0","In-Reply-To":"<20170830163852.27454-1-dengler@linutronix.de>","Cc":"u-boot@lists.denx.de, Kyungmin Park ","Subject":"Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Reply-To":"hs@denx.de","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1766720,"web_url":"http://patchwork.ozlabs.org/comment/1766720/","msgid":"<59B78E5F.10302@denx.de>","list_archive_url":null,"date":"2017-09-12T07:35:59","subject":"Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read","submitter":{"id":99,"url":"http://patchwork.ozlabs.org/api/people/99/","name":"Heiko Schocher","email":"hs@denx.de"},"content":"Hello Holger,\n\nAm 31.08.2017 um 09:25 schrieb Heiko Schocher:\n> Hello Holger,\n>\n> Am 30.08.2017 um 18:38 schrieb Holger Dengler:\n>> After a successful read of a UBI volume, the variable filesize is set\n>> to the number of read bytes. To boot linux with a raw initramfs/initrd,\n>> you must specify the exact size of the initramfs/initrd image in boot\n>> command. If the initramfs/inird is read from UBI volume, $filesize is\n>> required to access the number of read bytes.\n>>\n>> Example:\n>> ubi read ${loadaddr} kernelvol\n>> ubi read ${fdtaddr} dtbvol\n>> ubi read ${initrd_addr} initrdvol\n>> bootz ${loadaddr} ${initrd_addr}:${filesize} ${fdt_addr}\n>>\n>> Signed-off-by: Holger Dengler \n>> ---\n>> cmd/ubi.c | 5 +++++\n>> 1 file changed, 5 insertions(+)\n>\n> Thanks!\n>\n> Reviewed-by: Heiko Schocher \n>\n> As new release is near, I pickup this patch when the next merge window\n> opens.\n\nJust tried a travis build, and it breaks a lot of boards, see\n\nhttps://travis-ci.org/hsdenx/u-boot-ubi/builds/274457374\n\nDo you have time to look into?\n\nbye,\nHeiko","headers":{"Return-Path":"","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xrxRy0YWpz9s7B\n\tfor ;\n\tTue, 12 Sep 2017 17:36:14 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid CE34FC21F0B; Tue, 12 Sep 2017 07:36:10 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id BA4AFC21CB3;\n\tTue, 12 Sep 2017 07:36:07 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid A7F4EC21CB3; Tue, 12 Sep 2017 07:36: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 56915C21C57\n\tfor ; Tue, 12 Sep 2017 07:36:05 +0000 (UTC)","from frontend03.mail.m-online.net (unknown [192.168.6.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xrxRm5sqbz1qwQn;\n\tTue, 12 Sep 2017 09:36:04 +0200 (CEST)","from localhost (dynscan3.mnet-online.de [192.168.6.84])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xrxRm5QQlz1qsQ5;\n\tTue, 12 Sep 2017 09:36:04 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan3.mail.m-online.net [192.168.6.84]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id 8v8FssMzo4KC; Tue, 12 Sep 2017 09:36:00 +0200 (CEST)","from localhost.localdomain (91.82.144.174.pool.invitel.hu\n\t[91.82.144.174])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tTue, 12 Sep 2017 09:36: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":"wELwXwWSF7pi7m+iohOgnVbt1PcYD0xR7S0sRqYDces=","To":"Holger Dengler ","References":"<20170830163852.27454-1-dengler@linutronix.de>\n\t<59A7B9CE.8060902@denx.de>","From":"Heiko Schocher ","Message-ID":"<59B78E5F.10302@denx.de>","Date":"Tue, 12 Sep 2017 09:35:59 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101\n\tThunderbird/38.1.0","MIME-Version":"1.0","In-Reply-To":"<59A7B9CE.8060902@denx.de>","Cc":"u-boot@lists.denx.de, Kyungmin Park ","Subject":"Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Reply-To":"hs@denx.de","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1766747,"web_url":"http://patchwork.ozlabs.org/comment/1766747/","msgid":"<59B79926.6030401@denx.de>","list_archive_url":null,"date":"2017-09-12T08:21:58","subject":"Re: [U-Boot] [PATCH v1] cmd: set filesize variable in ubi read","submitter":{"id":99,"url":"http://patchwork.ozlabs.org/api/people/99/","name":"Heiko Schocher","email":"hs@denx.de"},"content":"Hello Holger,\n\nthanks for the fast fix! Hm... some nitpick:\n\nThis is v2 not v1 ... seems patchwork has a problem, as your patch\ndoes not popup at:\nhttp://patchwork.ozlabs.org/project/uboot/list/?page=1\n\n(or patchwork does work slowly today ...)\n\nAm 12.09.2017 um 09:58 schrieb Holger Dengler:\n> After a successful read of a UBI volume, the variable filesize is set\n> to the number of read bytes. To boot linux with a raw initramfs/initrd,\n> you must specify the exact size of the initramfs/initrd image in boot\n> command. If the initramfs/inird is read from UBI volume, $filesize is\n> required to access the number of read bytes.\n>\n> Example:\n> ubi read ${loadaddr} kernelvol\n> ubi read ${fdtaddr} dtbvol\n> ubi read ${initrd_addr} initrdvol\n> bootz ${loadaddr} ${initrd_addr}:${filesize} ${fdt_addr}\n>\n> Signed-off-by: Holger Dengler \n> ---\n\nadd here please a summary, what has changed since v1. You may\nuse patman ... see u-boot:tools/patman/README\n\nThanks!\n\nNevertheless, I started a new travis build with this patch, see:\nhttps://travis-ci.org/hsdenx/u-boot-ubi/builds/274502903\n\nbye,\nHeiko","headers":{"Return-Path":"","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xryTJ3j1Xz9s7F\n\tfor ;\n\tTue, 12 Sep 2017 18:22:28 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 44CBAC21F3F; Tue, 12 Sep 2017 08:22:06 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5CB01C21C6D;\n\tTue, 12 Sep 2017 08:22:04 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 89EDEC21D55; Tue, 12 Sep 2017 08:22:03 +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 6FEC1C21C57\n\tfor ; Tue, 12 Sep 2017 08:22:02 +0000 (UTC)","from frontend03.mail.m-online.net (unknown [192.168.6.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xrySn4rXjz1qwq0;\n\tTue, 12 Sep 2017 10:22:01 +0200 (CEST)","from localhost (dynscan3.mnet-online.de [192.168.6.84])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xrySn4QBRz1qsPb;\n\tTue, 12 Sep 2017 10:22:01 +0200 (CEST)","from mail.mnet-online.de ([192.168.8.182])\n\tby localhost (dynscan3.mail.m-online.net [192.168.6.84]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id Hkx1pOyPhrSW; Tue, 12 Sep 2017 10:21:59 +0200 (CEST)","from localhost.localdomain (91.82.144.174.pool.invitel.hu\n\t[91.82.144.174])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA;\n\tTue, 12 Sep 2017 10:21:59 +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":"v2Utslo63S65unkuTHglqemlcQ6ovr5GSjymQovM7FI=","To":"Holger Dengler ","References":"<20170830163852.27454-1-dengler@linutronix.de>\n\t<20170912075825.5849-1-dengler@linutronix.de>","From":"Heiko Schocher ","Message-ID":"<59B79926.6030401@denx.de>","Date":"Tue, 12 Sep 2017 10:21:58 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101\n\tThunderbird/38.1.0","MIME-Version":"1.0","In-Reply-To":"<20170912075825.5849-1-dengler@linutronix.de>","Cc":"u-boot@lists.denx.de, Kyungmin Park ","Subject":"Re: [U-Boot] [PATCH v1] cmd: set filesize variable in ubi read","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Reply-To":"hs@denx.de","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1766860,"web_url":"http://patchwork.ozlabs.org/comment/1766860/","msgid":"","list_archive_url":null,"date":"2017-09-12T07:45:34","subject":"Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read","submitter":{"id":72011,"url":"http://patchwork.ozlabs.org/api/people/72011/","name":"Holger Dengler","email":"dengler@linutronix.de"},"content":"Hi Heiko,\n\nsorry, my fault. I missed, that the function has been renamed. I'll resend the patch.\n\nOn 09/12/2017 09:35 AM, Heiko Schocher wrote:\n> Hello Holger,\n> \n> Am 31.08.2017 um 09:25 schrieb Heiko Schocher:\n>> Hello Holger,\n>>\n>> Am 30.08.2017 um 18:38 schrieb Holger Dengler:\n>>> After a successful read of a UBI volume, the variable filesize is set\n>>> to the number of read bytes. To boot linux with a raw initramfs/initrd,\n>>> you must specify the exact size of the initramfs/initrd image in boot\n>>> command. If the initramfs/inird is read from UBI volume, $filesize is\n>>> required to access the number of read bytes.\n>>>\n>>> Example:\n>>>    ubi read ${loadaddr} kernelvol\n>>>    ubi read ${fdtaddr} dtbvol\n>>>    ubi read ${initrd_addr} initrdvol\n>>>    bootz ${loadaddr} ${initrd_addr}:${filesize} ${fdt_addr}\n>>>\n>>> Signed-off-by: Holger Dengler \n>>> ---\n>>>   cmd/ubi.c | 5 +++++\n>>>   1 file changed, 5 insertions(+)\n>>\n>> Thanks!\n>>\n>> Reviewed-by: Heiko Schocher \n>>\n>> As new release is near, I pickup this patch when the next merge window\n>> opens.\n> \n> Just tried a travis build, and it breaks a lot of boards, see\n> \n> https://travis-ci.org/hsdenx/u-boot-ubi/builds/274457374\n> \n> Do you have time to look into?\n> \n> bye,\n> Heiko","headers":{"Return-Path":"","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=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs1WB2Py0z9s7g\n\tfor ;\n\tTue, 12 Sep 2017 20:39:18 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 18785C21E32; Tue, 12 Sep 2017 10:38:36 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id F31E5C21F46;\n\tTue, 12 Sep 2017 10:37:10 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 1B822C21D56; Tue, 12 Sep 2017 07:45:37 +0000 (UTC)","from Galois.linutronix.de (Galois.linutronix.de [146.0.238.70])\n\tby lists.denx.de (Postfix) with ESMTPS id C94FAC21C57\n\tfor ; Tue, 12 Sep 2017 07:45:36 +0000 (UTC)","from p54bc52b0.dip0.t-ipconnect.de ([84.188.82.176]\n\thelo=[10.10.2.128]) by Galois.linutronix.de with esmtpsa\n\t(TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.80) (envelope-from )\n\tid 1drfrO-0002r2-T5; Tue, 12 Sep 2017 09:43:55 +0200"],"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=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","To":"hs@denx.de","References":"<20170830163852.27454-1-dengler@linutronix.de>\n\t<59A7B9CE.8060902@denx.de> <59B78E5F.10302@denx.de>","From":"Holger Dengler ","Organization":"linutronix GmbH","Message-ID":"","Date":"Tue, 12 Sep 2017 09:45:34 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<59B78E5F.10302@denx.de>","X-Linutronix-Spam-Score":"-1.0","X-Linutronix-Spam-Level":"-","X-Linutronix-Spam-Status":"No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,\n\tSHORTCIRCUIT=-0.0001","X-Mailman-Approved-At":"Tue, 12 Sep 2017 10:37:09 +0000","X-Content-Filtered-By":"Mailman/MimeDel 2.1.18","Cc":"u-boot@lists.denx.de, Kyungmin Park ","Subject":"Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============2755789121534063219==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]