[{"id":1771510,"web_url":"http://patchwork.ozlabs.org/comment/1771510/","msgid":"","list_archive_url":null,"date":"2017-09-20T02:32:21","subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","submitter":{"id":65882,"url":"http://patchwork.ozlabs.org/api/people/65882/","name":"Masahiro Yamada","email":"yamada.masahiro@socionext.com"},"content":"Hi Simon,\n\n\n2017-09-17 6:23 GMT+09:00 Simon Glass :\n> U-Boot currently has fairly rudimentary logging features. A basic printf()\n> provides console output and debug() provides debug output which is\n> activated if DEBUG is defined in the file containing the debug()\n> statements.\n>\n> It would be useful to have a few more features:\n>\n> - control of debug output at runtime, so problems can potentially be\n> debugged without recompiling U-Boot\n> - control of which subsystems output debug information, so that (for\n> example) it is possible to enable debugging for MMC or SATA at runtime\n> - indication of severity with each message, so that the user can control\n> whether just errors are displayed, warnings, or all debug messages\n> - sending logging information to different destinations, such as console,\n> memory, linux, etc,\n>\n> At present U-Boot has a logbuffer feature which records output in a memory\n> buffer for later display or storage. This is useful but is not at present\n> enabled for any board.\n>\n> This series introduced a new logging system which supports:\n> - various log levels from panic to debug\n> - log categories including all uclasses and a few others\n> - log drivers to which all log records can be sent\n> - log filters which control which log records make it to which drivers\n>\n> Enabling logging with the default options does not add much to code size.\n> By default the maximum recorded log level is LOGL_INFO, meaning that debug\n> messages (and above) are discarded a build-time. Increasing this level\n> provides more run-time flexibility to enable/disable logging at the cost\n> of increased code size.\n>\n> This feature is by no means finished. The README provides a long list of\n> features and clean-ups that could be done. But hopefully this is a\n> starting point for improving this important area in U-Boot.\n>\n> The series is available at u-boot-dm/log-working\n\n\n\nAs you notice, this series has lots of conflicts with my clean-up works.\n\nThe lesson we leaned is we should prepare Linux-compatible APIs where possible,\ninstead of inventing similar ones for our own.\nOtherwise, people would start to sprinkle compat macros/headers everywhere.\n\nIn this sense, this series introduce similar, but different\ninterfaces.\n\nIf you want the log interface, could you implement it\nas a back-end of printk() (after my clean-ups) ?\nUsers can use printk(), or more preferably pr_()\nto call the log API.\n\n\nIf CONFIG_LOG is disabled, printk() falls back to printf(),\ni.e. the log is immediately printed to the console.\n\nIf CONFIG_LOG is enabled, printk() sends the log message\nto the log facility you are implementing.\n\n\nI am not sure how much demand exists for category/file filters.\nHaving both of them (or even one of them) might be over-implementation.\n\nI do not like the category filter because I do not want to see\n log(LOGC_BOARD, ...)","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=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=nifty.com header.i=@nifty.com\n\theader.b=\"m3fy+l+Y\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xxkM26sBXz9t2Q\n\tfor ;\n\tWed, 20 Sep 2017 12:33:33 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 3E125C21DAD; Wed, 20 Sep 2017 02:33:18 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 7862DC21C35;\n\tWed, 20 Sep 2017 02:33:15 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 47EBFC21C35; Wed, 20 Sep 2017 02:33:14 +0000 (UTC)","from conssluserg-06.nifty.com (conssluserg-06.nifty.com\n\t[210.131.2.91]) by lists.denx.de (Postfix) with ESMTPS id 30B00C21C34\n\tfor ; Wed, 20 Sep 2017 02:33:12 +0000 (UTC)","from mail-yw0-f181.google.com (mail-yw0-f181.google.com\n\t[209.85.161.181]) (authenticated)\n\tby conssluserg-06.nifty.com with ESMTP id v8K2X2n9012316\n\tfor ; Wed, 20 Sep 2017 11:33:03 +0900","by mail-yw0-f181.google.com with SMTP id v72so1012000ywa.3\n\tfor ; Tue, 19 Sep 2017 19:33:03 -0700 (PDT)","by 10.37.170.198 with HTTP; Tue, 19 Sep 2017 19:32:21 -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=KHOP_BIG_TO_CC,\n\tT_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Filter":"OpenDKIM Filter v2.10.3 conssluserg-06.nifty.com v8K2X2n9012316","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com;\n\ts=dec2015msa; t=1505874783;\n\tbh=FPUNu8MAbEdxgROEFy1H6rhrO9NqRKNkMyY5t0cP0/0=;\n\th=In-Reply-To:References:From:Date:Subject:To:Cc:From;\n\tb=m3fy+l+Y0WzrrWq4b/FiRalUfSU6TPn4BZz6+WjiixNIIoUrs0Kq463jPx3eGHlfD\n\thwYru3Xn6ZBJN5Vr0P25WE+I+Vjgm1f1KiYE38spp+DtTQY8HxgcX+Qvd4KM8sRJEq\n\tb4HFbMa5r/Bcr/EXiMHpkJhTe9K5bpW8+ryBZHVkOeUhfZYYL+E+lJIICgenOFvFo3\n\taiGGCkdUUkCeADp7f7tIAakY4UwhIgBnctSr5FbsvbwlK0LeSKbsCx8b+JLBf72FRR\n\tWoXkGoKVk33uT1VKtzZ3+oxqPRnv0mepb7yPKiFqrYs6ekuSSmFnz9fZfFwUtwnRa2\n\tcsSMccPDe7QgQ==","X-Nifty-SrcIP":"[209.85.161.181]","X-Gm-Message-State":"AHPjjUidQ5CRlhGGZj8oe8fi/p+UrZZRaRczoa9rzd+cj6hOncnXHrM2\n\tjLRk3K6uzW+3rqyclj8F3BZZDYqQCLkFLObAWuM=","X-Google-Smtp-Source":"AOwi7QBiQ76atuiezJQGo/IPMaOTTDj14DtKKvHcaiWOsF+2AI/17CHcQypZNSlxpxj0dMG32O/QRM7w2K19ChiLwAI=","X-Received":"by 10.37.80.213 with SMTP id e204mr2311536ybb.215.1505874782397; \n\tTue, 19 Sep 2017 19:33:02 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170916212331.170463-1-sjg@chromium.org>","References":"<20170916212331.170463-1-sjg@chromium.org>","From":"Masahiro Yamada ","Date":"Wed, 20 Sep 2017 11:32:21 +0900","X-Gmail-Original-Message-ID":"","Message-ID":"","To":"Simon Glass ","Cc":"Marek Vasut , Tom Rini ,\n\t=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , \n\tJan Kiszka , \"Andreas J. Reichel\"\n\t, Michal Simek ,\n\tRick Altherr , Jagan Teki ,\n\tU-Boot Mailing List , \n\tAndre Przywara , Wilson Lee , \n\t=?utf-8?q?Stefan_Br=C3=BCns?= ,\n\t\"xypron.glpk@gmx.de\" ,\n\tJagan Teki , Andy Yan ,\n\tMaxime Ripard\n\t, Stefan Roese ,\n\tChris Packham , Thomas Petazzoni\n\t, Franklin ","Subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","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":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1772175,"web_url":"http://patchwork.ozlabs.org/comment/1772175/","msgid":"<20170920195548.3FE3B120302@gemini.denx.de>","list_archive_url":null,"date":"2017-09-20T19:55:48","subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","submitter":{"id":387,"url":"http://patchwork.ozlabs.org/api/people/387/","name":"Wolfgang Denk","email":"wd@denx.de"},"content":"Dear Simon,\n\nsorry for jumping in so late...\n\nIn message <20170916212331.170463-1-sjg@chromium.org> you wrote:\n>\n> At present U-Boot has a logbuffer feature which records output in a memory\n> buffer for later display or storage. This is useful but is not at present\n> enabled for any board.\n\nBackground explanation: When this was implemented, the buffer\nhandling was fully compatible with the Linux kernel logbuffer.\nAnd U-Boot was able to reserve memory (at the top of the RAM) that\ncould be shared between U-Boot and Linux, so you could for example \nwrite U-Boot POST results into the log buffer, and read it using\nstandard syslog tools in Linux. Or keep the panic messages of the\nprevious crash and read it after reboot.\n\nIIRC this was in production use only on Power architectre systems,\nand it broke (and nover got fixed) when the Linux log buffer was\nreworked.\n\n> This series introduced a new logging system which supports:\n> - various log levels from panic to debug\n> - log categories including all uclasses and a few others\n> - log drivers to which all log records can be sent\n> - log filters which control which log records make it to which drivers\n\nYou don't mention it here, but would it be possible to keep\ncompatibility with the Linux logbuffer implementation in mind?\n\nSo we could re-implement this shared logbuffer thingy in case\nsomeone finds it useful?\n\nThanks!\n\nBest regards,\n\nWolfgang Denk","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 3xy9V01r8kz9sPk\n\tfor ;\n\tThu, 21 Sep 2017 05:56:08 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 05856C21E76; Wed, 20 Sep 2017 19:56:04 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 63EF4C21C5D;\n\tWed, 20 Sep 2017 19:56:02 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 75483C21C5D; Wed, 20 Sep 2017 19:56:00 +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 23646C21C35\n\tfor ; Wed, 20 Sep 2017 19:56:00 +0000 (UTC)","from frontend01.mail.m-online.net (unknown [192.168.8.182])\n\tby mail-out.m-online.net (Postfix) with ESMTP id 3xy9Tq6nj3z1rM6n\n\tfor ; Wed, 20 Sep 2017 21:55:59 +0200 (CEST)","from localhost (dynscan1.mnet-online.de [192.168.6.70])\n\tby mail.m-online.net (Postfix) with ESMTP id 3xy9Tq6fBfz1r0x3\n\tfor ; Wed, 20 Sep 2017 21:55:59 +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) with ESMTP id fZiZ5Vm_NvPb for ;\n\tWed, 20 Sep 2017 21:55:59 +0200 (CEST)","from janitor.denx.de (unknown [62.91.23.180])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.mnet-online.de (Postfix) with ESMTPSA\n\tfor ; Wed, 20 Sep 2017 21:55:59 +0200 (CEST)","by janitor.denx.de (Postfix, from userid 119)\n\tid CB9A447B; Wed, 20 Sep 2017 21:55:58 +0200 (CEST)","from gemini.denx.de (gemini.denx.de [10.4.0.2])\n\tby janitor.denx.de (Postfix) with ESMTPS id 9FA9722;\n\tWed, 20 Sep 2017 21:55:48 +0200 (CEST)","from gemini.denx.de (localhost [IPv6:::1])\n\tby gemini.denx.de (Postfix) with ESMTP id 3FE3B120302;\n\tWed, 20 Sep 2017 21:55: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=KHOP_BIG_TO_CC,\n\tRCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable\n\tautolearn_force=no version=3.4.0","X-Virus-Scanned":"amavisd-new at mnet-online.de","X-Auth-Info":"RHUjT3TsbwV1oOXrBsYZZ1HIjC26fXASMNFAuj66GqA=","To":"Simon Glass ","From":"Wolfgang Denk ","MIME-Version":"1.0","In-reply-to":"<20170916212331.170463-1-sjg@chromium.org>","References":"<20170916212331.170463-1-sjg@chromium.org>","Comments":"In-reply-to Simon Glass \n\tmessage dated \"Sat, 16 Sep 2017 15:23:13 -0600.\"","Date":"Wed, 20 Sep 2017 21:55:48 +0200","Message-Id":"<20170920195548.3FE3B120302@gemini.denx.de>","Cc":"Marek Vasut , Tom Rini ,\n\t=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , \n\tJan Kiszka , \"Andreas J. Reichel\"\n\t, Michal Simek ,\n\tRick Altherr , Jagan Teki ,\n\tU-Boot Mailing List , \n\tAndre Przywara , Wilson Lee , \n\t=?utf-8?q?Stefan_Br=C3=BCns?= ,\n\t\"xypron.glpk@gmx.de\" ,\n\tJagan Teki , Andy Yan ,\n\tMaxime Ripard\n\t, Stefan Roese ,\n\tChris Packham , Thomas Petazzoni\n\t, Franklin ","Subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","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":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1772187,"web_url":"http://patchwork.ozlabs.org/comment/1772187/","msgid":"<6d407465-6b41-b701-b3f0-798b8110ce8a@gmx.de>","list_archive_url":null,"date":"2017-09-20T20:20:08","subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","submitter":{"id":61270,"url":"http://patchwork.ozlabs.org/api/people/61270/","name":"Heinrich Schuchardt","email":"xypron.glpk@gmx.de"},"content":"On 09/20/2017 04:32 AM, Masahiro Yamada wrote:\n> Hi Simon,\n> \n> \n> 2017-09-17 6:23 GMT+09:00 Simon Glass :\n>> U-Boot currently has fairly rudimentary logging features. A basic printf()\n>> provides console output and debug() provides debug output which is\n>> activated if DEBUG is defined in the file containing the debug()\n>> statements.\n>>\n>> It would be useful to have a few more features:\n>>\n>> - control of debug output at runtime, so problems can potentially be\n>> debugged without recompiling U-Boot\n>> - control of which subsystems output debug information, so that (for\n>> example) it is possible to enable debugging for MMC or SATA at runtime\n>> - indication of severity with each message, so that the user can control\n>> whether just errors are displayed, warnings, or all debug messages\n>> - sending logging information to different destinations, such as console,\n>> memory, linux, etc,\n>>\n>> At present U-Boot has a logbuffer feature which records output in a memory\n>> buffer for later display or storage. This is useful but is not at present\n>> enabled for any board.\n>>\n>> This series introduced a new logging system which supports:\n>> - various log levels from panic to debug\n>> - log categories including all uclasses and a few others\n>> - log drivers to which all log records can be sent\n>> - log filters which control which log records make it to which drivers\n>>\n>> Enabling logging with the default options does not add much to code size.\n>> By default the maximum recorded log level is LOGL_INFO, meaning that debug\n>> messages (and above) are discarded a build-time. Increasing this level\n>> provides more run-time flexibility to enable/disable logging at the cost\n>> of increased code size.\n>>\n>> This feature is by no means finished. The README provides a long list of\n>> features and clean-ups that could be done. But hopefully this is a\n>> starting point for improving this important area in U-Boot.\n>>\n>> The series is available at u-boot-dm/log-working\n> \n> \n> \n> As you notice, this series has lots of conflicts with my clean-up works.\n> \n> The lesson we leaned is we should prepare Linux-compatible APIs where possible,\n> instead of inventing similar ones for our own.\n> Otherwise, people would start to sprinkle compat macros/headers everywhere.\n> \n> In this sense, this series introduce similar, but different\n> interfaces.\n> \n> If you want the log interface, could you implement it\n> as a back-end of printk() (after my clean-ups) ?\n> Users can use printk(), or more preferably pr_()\n> to call the log API.\n> \n> \n> If CONFIG_LOG is disabled, printk() falls back to printf(),\n> i.e. the log is immediately printed to the console.\n> \n> If CONFIG_LOG is enabled, printk() sends the log message\n> to the log facility you are implementing.\n> \n> \n> I am not sure how much demand exists for category/file filters.\n> Having both of them (or even one of them) might be over-implementation.\n> \n> I do not like the category filter because I do not want to see\n> log(LOGC_BOARD, ...)\n>\n\nFiltering by category is a good idea. I have been developing parts of\nthe EFI API implementation. There it showed up to be a necessity to\ndisable debug output for events (occuring dozens of times per\nmillisecond). Network services will fail if timeouts occur due to too\nmany log message passed over a slow serial console.\n\nAs fine grained control is currently missing I had to spray\n #undefine _DEBUG\n #define _DEBUG 0\n ...\n #define _DEBUG 1\nall over the code.\n\nOther codes (e.g. iPXE) can enable debugging per module with different\nverbosity levels, cf. http://ipxe.org/download#debug_builds .\n\nRegards\n\nHeinrich","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 3xyB2y0Dklz9s7h\n\tfor ;\n\tThu, 21 Sep 2017 06:21:12 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 88659C21ED9; Wed, 20 Sep 2017 20:21:08 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id AF1E3C21C5D;\n\tWed, 20 Sep 2017 20:21:05 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 48961C21C5D; Wed, 20 Sep 2017 20:21:04 +0000 (UTC)","from mout.gmx.net (mout.gmx.net [212.227.17.22])\n\tby lists.denx.de (Postfix) with ESMTPS id 0155EC21C35\n\tfor ; Wed, 20 Sep 2017 20:21:02 +0000 (UTC)","from [192.168.123.31] ([84.118.154.110]) by mail.gmx.com (mrgmx102\n\t[212.227.17.168]) with ESMTPSA (Nemesis) id\n\t0Mam2Q-1dek021Oaw-00KN7R; Wed, 20 Sep 2017 22:20:16 +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.7 required=5.0 tests=FREEMAIL_FROM,\n\tKHOP_BIG_TO_CC, RCVD_IN_DNSWL_LOW,\n\tRCVD_IN_MSPIKE_H2 autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","To":"Masahiro Yamada ,\n\tSimon Glass ","References":"<20170916212331.170463-1-sjg@chromium.org>\n\t","From":"Heinrich Schuchardt ","Message-ID":"<6d407465-6b41-b701-b3f0-798b8110ce8a@gmx.de>","Date":"Wed, 20 Sep 2017 22:20:08 +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":"","Content-Language":"en-US","X-Provags-ID":"V03:K0:Ngt+C/SM6CtNAYwcg80gCOXNAwSdOvS58jHTHprg6rgMg9jKRTI\n\t7v54hvdrgv8NwY4wC+9E5dRMWc7mqsIZ8BFxjBwo0/Uq1cSS7kvS/QTZu3Rrr/50+XbPNEI\n\t60t4clVGatnTZ7RV+1KxnRewP3CKxR0fw3ldPVbhYnP660OiMkAipFOoWD+U0NK3bxsP5yw\n\tmUZ0sw6tLCbg8Cn6C+OXw==","X-UI-Out-Filterresults":"notjunk:1; V01:K0:Z4fAryDxd8k=:ma7Eq/dVb6PXIptBi91VBS\n\tRE5whRkQWbrafKNFrr/1erEPkndQfrFt0a43oydB7IyituMcenvg4dBKTMrF+F/st7zCkTObs\n\tjMyOL8pB/Hrq3EzDb+91ziDI1biJXB8W/wR9QjQDSGN4X7wasEytwC9sgYZVlDQVtUtI+u940\n\t0BWuwjoxTv+huOZIipJLgUQMsCKonXD639El5RUHejH0u3G3Ka/MODYYo6PtYniTXIosCUnJY\n\t3hF5TU/VebNcPjsopQfBhugm3ijhOhXbkH1AY2ZQ4ac2nAmFjU93qumcd3Hl9e54Tnb6VJ9TV\n\t36EXGkFZc+lEYmY0duruPZxW1Rs93o4Yrke1WhdhGzRBtzA41Rr3LvW+3qp+NrRjRhI4OxMCL\n\tyTxGneFz+hipa6/JMrnVQQA5xPvSctbw7XByIZyFPMZv/HyX5YyOomhj2LpRRiXQuMfhNSo5z\n\twPqyEpHDM0hNtXAm8NO0R2276ajt2KBiqyUvHyt5d7f4xhWpM/rANI97nzS0+ftbWjjEmRMKc\n\tiT5ywRYm6W8ujzHxS0e189TTlxMP6RKJEOOyI+3EySRkCvAmxvax+33r6JY5H1LfMgZXO1OOh\n\tvBcQSvrFlMrIm81N7SVLmMKab3j5taOy18DKLAaPVknoNpdSaGdWy5ZrKxNMcxqLTcIh5Yi+H\n\tFCr5mV3ioefeFu9TfHq2FcdzI1bQ28JqbUtLWH5+ueKKRV5GFS9g8nResY+ntVaiJLE5qDZUr\n\tpEwFzPLlhE5dU6s0t1uoK6jqPeX3N9HGgR30rztw8O8Ee2iGLBe2PdQi+Io+eO3RMvxPVpxD6\n\tWBWhe3KuX/C2nRKgJx4b//ZkxIbdxxUTUxtn3Js/cpgP9D9QKE=","Cc":"Marek Vasut , Tom Rini ,\n\t=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , \n\tJan Kiszka , \"Andreas J. Reichel\"\n\t, Michal Simek ,\n\tRick Altherr , Jagan Teki ,\n\tU-Boot Mailing List , \n\tAndre Przywara , =?utf-8?q?Stefan_Br=C3=BCns?=\n\t, Wilson Lee ,\n\tJagan Teki , Andy Yan ,\n\tMaxime Ripard\n\t, Stefan Roese ,\n\tChris Packham , Thomas Petazzoni\n\t, Franklin ","Subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","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":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1772411,"web_url":"http://patchwork.ozlabs.org/comment/1772411/","msgid":"","list_archive_url":null,"date":"2017-09-21T04:58:59","subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Wolfgang,\n\nOn 20 September 2017 at 13:55, Wolfgang Denk wrote:\n> Dear Simon,\n>\n> sorry for jumping in so late...\n>\n> In message <20170916212331.170463-1-sjg@chromium.org> you wrote:\n>>\n>> At present U-Boot has a logbuffer feature which records output in a memory\n>> buffer for later display or storage. This is useful but is not at present\n>> enabled for any board.\n>\n> Background explanation: When this was implemented, the buffer\n> handling was fully compatible with the Linux kernel logbuffer.\n> And U-Boot was able to reserve memory (at the top of the RAM) that\n> could be shared between U-Boot and Linux, so you could for example\n> write U-Boot POST results into the log buffer, and read it using\n> standard syslog tools in Linux. Or keep the panic messages of the\n> previous crash and read it after reboot.\n>\n> IIRC this was in production use only on Power architectre systems,\n> and it broke (and nover got fixed) when the Linux log buffer was\n> reworked.\n\nOK I see, i did not realise the Linux connection.\n\n>\n>> This series introduced a new logging system which supports:\n>> - various log levels from panic to debug\n>> - log categories including all uclasses and a few others\n>> - log drivers to which all log records can be sent\n>> - log filters which control which log records make it to which drivers\n>\n> You don't mention it here, but would it be possible to keep\n> compatibility with the Linux logbuffer implementation in mind?\n>\n> So we could re-implement this shared logbuffer thingy in case\n> someone finds it useful?\n\nI think it would be fairly easy to do, yes. It just needs a driver\nwhich writes log records into memory at a given address. I certainly\ndon't like removing useful features, will take a look.\n\nRegards,\nSimon","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=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"YMMf4Xyw\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"m/HJq6hd\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyPbL35M2z9ryv\n\tfor ;\n\tThu, 21 Sep 2017 15:01:34 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 2ECFCC21EC7; Thu, 21 Sep 2017 05:00:16 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 87799C21ED3;\n\tThu, 21 Sep 2017 05:00:15 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 49824C21EE0; Thu, 21 Sep 2017 04:59:25 +0000 (UTC)","from mail-qk0-f175.google.com (mail-qk0-f175.google.com\n\t[209.85.220.175])\n\tby lists.denx.de (Postfix) with ESMTPS id 4A251C21EA5\n\tfor ; Thu, 21 Sep 2017 04:59:21 +0000 (UTC)","by mail-qk0-f175.google.com with SMTP id r141so4745826qke.2\n\tfor ; Wed, 20 Sep 2017 21:59:21 -0700 (PDT)","by 10.200.37.200 with HTTP; Wed, 20 Sep 2017 21:58:59 -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=KHOP_BIG_TO_CC,\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=google.com;\n\ts=20161025; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=Gw0P3kN1xrMQcarMd0Rb3sFwsIfhWAfLPR4FgKg2DXQ=;\n\tb=YMMf4XywtRcDmwN45wFtaheZ9ugPBGY4W3kYqJbo6QEUh8S3u/ukrSOqr46lfWhu76\n\tvSQ0xIq4i5BEoyHUkTuvIxCB2QuxPj05OX3jiYl3SrstmurcitdHw12g8pYYLQCG+UCL\n\tbYeI6rNsUPhTEhY59PvQH/uU3eqktlpulwA2mXqCqs2Y/Xeuiz2uFwu5suYQg8iBCfDM\n\tsjql1LIebGaL8w3VOYzEzaofx7751vkKmCllMVWjYNrq2bWS/ZH6WKoYWkkxOzgIOvI7\n\tuU5ItodSoASnQMllKOCoqipotcsR8e0KtQrLXCa7fGwlgBHYs7yPVZl0qeE/vg/hcZcX\n\tUDNw==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=Gw0P3kN1xrMQcarMd0Rb3sFwsIfhWAfLPR4FgKg2DXQ=;\n\tb=m/HJq6hdyMtd3Rn5ACx0w6O36PThEk7JCat5WwPXOoBHtBCNWXHN7MDRPUmWaiiWrr\n\t0WM/I+z9q1jM9dGGWqexLecabekL6oy6kYfBnRvWJap8ikdvWcZD1LJUEe5uc3YLOKri\n\tlrHIvTtjH5XPk/Ex3eaCwa5lUXiGAmY94P3c8="],"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:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=Gw0P3kN1xrMQcarMd0Rb3sFwsIfhWAfLPR4FgKg2DXQ=;\n\tb=MEW8dXKovylROVQP4iUcY7EeuoKfLtgybFrjTtEs2xJI1y0jjDlKEBD6HU+dulUjOM\n\tPrBq8qpFGBEC4ebvNM7RLufa4EKzuDhqdyOlyjrgNwbPifMqZBQ5Qma8CsKGCJM7OLN3\n\tFRDUFmZb4g1Pu7BlqqVLwIGuZ+Vnv8pIKn6GNNrnroewipjhvHT2AOkVOxZ8pM7shWiu\n\tB3+Ah6Xyw2usKd9TFQD/tr41II1Yzk5KkXrhBca3Fj5RLXmQ9QI/hYFdMiPlF0CggGuM\n\t5zMnO8wWsY0M7fBY62ydpGEeSkcSb59cFaVDXGZv0SzZ/HLk6iWjgxJCFSEf9SlEMqld\n\t7bNQ==","X-Gm-Message-State":"AHPjjUgbMQF3okicC1cZTI6gqQxry8DVGhZBpYPczCiUi0VLBeQZR5UN\n\tvvRqM7vaFfVNdiMoZqwY6WV2n4W+A+Rqw9BMhhXEWQ==","X-Google-Smtp-Source":"AOwi7QD0k5uV9kCk6gsztCg8A8m2mTCq/9PUch9OmYBH+5MO9ic/VsaHrgxkgGKTmkJFL66LKQxYcIs63mSrlqR0pMQ=","X-Received":"by 10.55.33.216 with SMTP id f85mr1424202qki.176.1505969959960; \n\tWed, 20 Sep 2017 21:59:19 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170920195548.3FE3B120302@gemini.denx.de>","References":"<20170916212331.170463-1-sjg@chromium.org>\n\t<20170920195548.3FE3B120302@gemini.denx.de>","From":"Simon Glass ","Date":"Wed, 20 Sep 2017 22:58:59 -0600","X-Google-Sender-Auth":"zGqi03M8znAnpvIC61SDH8Sj3Qc","Message-ID":"","To":"Wolfgang Denk ","Cc":"Marek Vasut , Tom Rini ,\n\t=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , \n\tJan Kiszka , \"Andreas J. Reichel\"\n\t, Michal Simek ,\n\tRick Altherr , Jagan Teki ,\n\tU-Boot Mailing List , \n\tAndre Przywara , Wilson Lee , \n\t=?utf-8?q?Stefan_Br=C3=BCns?= ,\n\t\"xypron.glpk@gmx.de\" ,\n\tJagan Teki , Andy Yan ,\n\tMaxime Ripard\n\t, Stefan Roese ,\n\tChris Packham , Thomas Petazzoni\n\t, Franklin ","Subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","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":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1772412,"web_url":"http://patchwork.ozlabs.org/comment/1772412/","msgid":"","list_archive_url":null,"date":"2017-09-21T04:58:53","subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","submitter":{"id":6170,"url":"http://patchwork.ozlabs.org/api/people/6170/","name":"Simon Glass","email":"sjg@chromium.org"},"content":"Hi Heinrich,\n\nOn 20 September 2017 at 14:20, Heinrich Schuchardt wrote:\n> On 09/20/2017 04:32 AM, Masahiro Yamada wrote:\n>> Hi Simon,\n>>\n>>\n>> 2017-09-17 6:23 GMT+09:00 Simon Glass :\n>>> U-Boot currently has fairly rudimentary logging features. A basic printf()\n>>> provides console output and debug() provides debug output which is\n>>> activated if DEBUG is defined in the file containing the debug()\n>>> statements.\n>>>\n>>> It would be useful to have a few more features:\n>>>\n>>> - control of debug output at runtime, so problems can potentially be\n>>> debugged without recompiling U-Boot\n>>> - control of which subsystems output debug information, so that (for\n>>> example) it is possible to enable debugging for MMC or SATA at runtime\n>>> - indication of severity with each message, so that the user can control\n>>> whether just errors are displayed, warnings, or all debug messages\n>>> - sending logging information to different destinations, such as console,\n>>> memory, linux, etc,\n>>>\n>>> At present U-Boot has a logbuffer feature which records output in a memory\n>>> buffer for later display or storage. This is useful but is not at present\n>>> enabled for any board.\n>>>\n>>> This series introduced a new logging system which supports:\n>>> - various log levels from panic to debug\n>>> - log categories including all uclasses and a few others\n>>> - log drivers to which all log records can be sent\n>>> - log filters which control which log records make it to which drivers\n>>>\n>>> Enabling logging with the default options does not add much to code size.\n>>> By default the maximum recorded log level is LOGL_INFO, meaning that debug\n>>> messages (and above) are discarded a build-time. Increasing this level\n>>> provides more run-time flexibility to enable/disable logging at the cost\n>>> of increased code size.\n>>>\n>>> This feature is by no means finished. The README provides a long list of\n>>> features and clean-ups that could be done. But hopefully this is a\n>>> starting point for improving this important area in U-Boot.\n>>>\n>>> The series is available at u-boot-dm/log-working\n>>\n>>\n>>\n>> As you notice, this series has lots of conflicts with my clean-up works.\n>>\n>> The lesson we leaned is we should prepare Linux-compatible APIs where possible,\n>> instead of inventing similar ones for our own.\n>> Otherwise, people would start to sprinkle compat macros/headers everywhere.\n>>\n>> In this sense, this series introduce similar, but different\n>> interfaces.\n>>\n>> If you want the log interface, could you implement it\n>> as a back-end of printk() (after my clean-ups) ?\n>> Users can use printk(), or more preferably pr_()\n>> to call the log API.\n>>\n>>\n>> If CONFIG_LOG is disabled, printk() falls back to printf(),\n>> i.e. the log is immediately printed to the console.\n>>\n>> If CONFIG_LOG is enabled, printk() sends the log message\n>> to the log facility you are implementing.\n>>\n>>\n>> I am not sure how much demand exists for category/file filters.\n>> Having both of them (or even one of them) might be over-implementation.\n>>\n>> I do not like the category filter because I do not want to see\n>> log(LOGC_BOARD, ...)\n>>\n>\n> Filtering by category is a good idea. I have been developing parts of\n> the EFI API implementation. There it showed up to be a necessity to\n> disable debug output for events (occuring dozens of times per\n> millisecond). Network services will fail if timeouts occur due to too\n> many log message passed over a slow serial console.\n>\n> As fine grained control is currently missing I had to spray\n> #undefine _DEBUG\n> #define _DEBUG 0\n> ...\n> #define _DEBUG 1\n> all over the code.\n>\n> Other codes (e.g. iPXE) can enable debugging per module with different\n> verbosity levels, cf. http://ipxe.org/download#debug_builds .\n\nActually I quite like those levels:\n\n- Enable basic debug messages\n- Enable verbose debug messages (e.g. one message per event)\n- Enable extra-verbose debug messages (e.g. one message per byte)\n- Enable I/O tracing (e.g. one message per readl()/writel() call)\n\nThis seems a lot better than having just a 'debug' level.\n\nRegards,\nSimon","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=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"Gd70H8Ll\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=chromium.org header.i=@chromium.org\n\theader.b=\"gKRGxf08\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyPbf1wxZz9sNr\n\tfor ;\n\tThu, 21 Sep 2017 15:01:50 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 08A77C21F33; Thu, 21 Sep 2017 05:00:57 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id CA91CC21EE8;\n\tThu, 21 Sep 2017 05:00:55 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 3A048C21EC7; Thu, 21 Sep 2017 04:59:18 +0000 (UTC)","from mail-qt0-f175.google.com (mail-qt0-f175.google.com\n\t[209.85.216.175])\n\tby lists.denx.de (Postfix) with ESMTPS id 2D12AC21E8A\n\tfor ; Thu, 21 Sep 2017 04:59:15 +0000 (UTC)","by mail-qt0-f175.google.com with SMTP id b1so4876074qtc.4\n\tfor ; Wed, 20 Sep 2017 21:59:15 -0700 (PDT)","by 10.200.37.200 with HTTP; Wed, 20 Sep 2017 21:58:53 -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=KHOP_BIG_TO_CC,\n\tRCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,\n\tT_DKIM_INVALID\n\tautolearn=unavailable autolearn_force=no version=3.4.0","DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=fR22bE2kre0ishq8a3b5V+vnGNHDZg28dreuBHh6A08=;\n\tb=Gd70H8LlvEmRtQEeksQn2Xq25Oy7mLgR0BHZvD/kyaHiB4BfHHW6ei7cRAOo1H1AgE\n\tYc2OmVRuIEq8v+QQZkLYgp8zFG0oyBllbN4mRrLSgYRfkqTIMf11kKCLiNei2GW6mVR4\n\tOr9H9BTIa1tHVTdpyctE7LNoHF7CgbwtwTO5z1UC++7kuESm7018QLo4YTiAOnGrfutY\n\t/EhM0tf6XgjaADQOWLIcP3eGLnHdJOtQVLoT/zDQDAp8KNdhIt/wetWaSsDc8AJoCJc7\n\t/AHUXwZkCOq3xRVhEm3lt4Yx2PaR0e0Kympm8pMq+p75QNH2vQ4A/dnKoafoo35vteHn\n\tW17w==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=fR22bE2kre0ishq8a3b5V+vnGNHDZg28dreuBHh6A08=;\n\tb=gKRGxf08i4kxmRes6Df6gHfCo655gTd/0kQSIhUbNhv8d6S5B5Yz08lB5KUyKmeFBM\n\tCjkPV7NWym8Yz1qtov0SirFBWbyTEpZRdcNvSs3N46NQt2o4uTIsnOYpA0aouBmNiqgt\n\tikJtKSTeMLB16zeV8SappQAbj+u6w+88KNr64="],"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:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=fR22bE2kre0ishq8a3b5V+vnGNHDZg28dreuBHh6A08=;\n\tb=J6OvwU8x5nxgzbbRsQ9YtBXdbpS56fWsGo2QXXNhDg38w8aoGD+2sW2Y5UqURP1M+X\n\t1vX4qq0r1bDPeXi6uTpyUc2U92OajnKEqf85P6czN1ekb+jNxXzsEtdf2+kH5XtUmvtC\n\tTYQ64qEcxM9RX2amm78S1wgC+8+5H8S++0PYDLY+N5RJuBoy262NYOt+H/8HkOLCMk4x\n\t0iH33y9hstvvPGVepnRbQ41S3bbxM591aOD/aQ+9sNSfHJZOSBVzVd2yUbPAHsVYNKqK\n\tBofGnQGQTqlbxJfvw7U8qNooAA+UhMO2wY6OfTOITaWmC4Cd+2KRHmpWQJvus+oeqy9q\n\tdtOA==","X-Gm-Message-State":"AHPjjUilmtpFPTS+C453zxhOMuVZ1eYPa1dVQj1Bf+pZZ0b4WSISUTjf\n\tUkuN34JrGW9dGsDJuoVW/v3g+22IGPBi5Oc5fYbNOA==","X-Google-Smtp-Source":"AOwi7QA5ZuM1IPB2W84E7sBCNdQnom4d/OkPkMjs9Gf8fGM2jbIqjKmgry5upDocw0kcDqugG0MKvgeRNbXRb93ejOE=","X-Received":"by 10.237.35.238 with SMTP id k43mr1455725qtc.214.1505969953768; \n\tWed, 20 Sep 2017 21:59:13 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<6d407465-6b41-b701-b3f0-798b8110ce8a@gmx.de>","References":"<20170916212331.170463-1-sjg@chromium.org>\n\t\n\t<6d407465-6b41-b701-b3f0-798b8110ce8a@gmx.de>","From":"Simon Glass ","Date":"Wed, 20 Sep 2017 22:58:53 -0600","X-Google-Sender-Auth":"mmq7zsS0dMKcDq8cSYglkidxAhI","Message-ID":"","To":"Heinrich Schuchardt ","Cc":"Marek Vasut , Tom Rini ,\n\t=?utf-8?q?=C5=81ukasz_Majewski?= ,\n\tAndy Shevchenko , U-Boot Mailing List\n\t, Jan Kiszka ,\n\t\"Andreas J. Reichel\" , \n\tMichal Simek ,\n\tRick Altherr , Jagan Teki ,\n\tAndre Przywara , =?utf-8?q?Stefan_Br=C3=BCns?=\n\t, Wilson Lee ,\n\tJagan Teki , Andy Yan ,\n\tMaxime Ripard\n\t, Stefan Roese ,\n\tChris Packham , Thomas Petazzoni\n\t, Franklin ","Subject":"Re: [U-Boot] [PATCH 00/13] log: Add a new logging feature","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":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]