From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 05 Nov 2014 07:57:27 +0100 Subject: [U-Boot] [PATCH 1/2] ubi: enable error reporting in initialization In-Reply-To: <1415117278-1255-1-git-send-email-andrew.ruder@elecsyscorp.com> References: <1415117278-1255-1-git-send-email-andrew.ruder@elecsyscorp.com> Message-ID: <5459CA57.9090009@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Andrew, Am 04.11.2014 17:07, schrieb Andrew Ruder: > The UBI layer will disable much of its error reporting when it is > compiled into the linux kernel to avoid stopping boot. We want this > error reporting in U-Boot since we don't initialize the UBI layer until > it is used and want the error reporting. > > We force this by telling the UBI layer we are building as a module. > > Signed-off-by: Andrew Ruder > Cc: Wolfgang Denk > Cc: Heiko Schocher > Cc: Kyungmin Park > --- > include/ubi_uboot.h | 2 ++ > 1 file changed, 2 insertions(+) I posted such a question [1] on the ML ... The problem is in generally enabling this feature in the size impact ... This should be discussed if we want this for all boards ... > diff --git a/include/ubi_uboot.h b/include/ubi_uboot.h > index 1fd15f4..6ff0e23 100644 > --- a/include/ubi_uboot.h > +++ b/include/ubi_uboot.h > @@ -51,6 +51,8 @@ > > #undef CONFIG_MTD_UBI_BLOCK > > +#define CONFIG_MTD_UBI_MODULE > + > #if !defined(CONFIG_MTD_UBI_BEB_LIMIT) > #define CONFIG_MTD_UBI_BEB_LIMIT 20 > #endif Ok, didn;t tried this with enabling "CONFIG_MTD_UBI_MODULE" ... But the name CONFIG_MTD_UBI_MODULE is not perfect, if we want just to enable error messages ... bye, Heiko [1] http://patchwork.ozlabs.org/patch/405259/ -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany