[PATCH 1/2] ramoops: allow to build without OFTREE support

[PATCH 1/2] ramoops: allow to build without OFTREE support

[Lucas Stach]

Ramoops may be useful even without oftree support, as kernels
booted without a DT may have other means to reserve the
ramoops memory.

Fixes:
In function `ramoops_probe':
undefined reference to `of_add_reserve_entry'

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
---
 fs/pstore/ram.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c
index 66b4975a3035..fdd92a60a57a 100644
--- a/fs/pstore/ram.c
+++ b/fs/pstore/ram.c
@@ -455,7 +455,8 @@ static int ramoops_probe(struct ramoops_platform_data *pdata)
 		  ramoops_ecc);
 	globalvar_add_simple("linux.bootargs.ramoops", kernelargs);
 
-	of_add_reserve_entry(cxt->phys_addr, cxt->phys_addr + mem_size);
+	if (IS_ENABLED(CONFIG_OFTREE))
+		of_add_reserve_entry(cxt->phys_addr, cxt->phys_addr + mem_size);
 
 	return 0;
 
-- 
2.1.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 2/2] ratp-console: select CRC16 and POLLER, depend on CONSOLE_FULL

[Lucas Stach]

RATP console selects RATP, which in turn selects CRC16, but those
chains don't work in Kconfig, so select CRC16 also from the
console option.

Also fix build errors by depending on CONSOLE_FULL and selecting
POLLER.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
---
 common/Kconfig | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/common/Kconfig b/common/Kconfig
index fc3ed8e57d73..3b87a3c8f8d2 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -623,6 +623,9 @@ config PBL_CONSOLE
 config CONSOLE_RATP
 	bool
 	select RATP
+	select CRC16
+	select POLLER
+	depends on CONSOLE_FULL
 	prompt "RATP console support"
 	help
 	  This option adds support for remote controlling barebox via serial
-- 
2.1.4


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 1/2] ramoops: allow to build without OFTREE support

[Sascha Hauer]

On Thu, Feb 11, 2016 at 10:33:55AM +0100, Lucas Stach wrote:
> Ramoops may be useful even without oftree support, as kernels
> booted without a DT may have other means to reserve the
> ramoops memory.
> 
> Fixes:
> In function `ramoops_probe':
> undefined reference to `of_add_reserve_entry'
> 
> Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> ---
>  fs/pstore/ram.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Applied, thanks

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox