All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] common: console: Fix duplicated CONFIG in silent env callback
@ 2017-08-25  7:06 Wilson Lee
  2017-08-28 16:17 ` Joe Hershberger
  2017-08-29  2:56 ` [U-Boot] " Tom Rini
  0 siblings, 2 replies; 3+ messages in thread
From: Wilson Lee @ 2017-08-25  7:06 UTC (permalink / raw)
  To: u-boot

The silent environment callback function does not update the silent
flag during silent env set or unset. That is because of duplicated
CONFIG keyword at preprocessor condition in silent environment
callback function and cause silent env callback unable to work.

This patch is to remove the duplicated CONFIG keywork in silent
environment callback function.

Signed-off-by: Wilson Lee <wilson.lee@ni.com>
Cc: Keng Soon Cheah <keng.soon.cheah@ni.com>
Cc: Chen Yee Chew <chen.yee.chew@ni.com>
Cc: Joe Hershberger <joe.hershberger@ni.com>
---

 common/console.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/console.c b/common/console.c
index 3167921..f83528c 100644
--- a/common/console.c
+++ b/common/console.c
@@ -68,11 +68,11 @@ U_BOOT_ENV_CALLBACK(console, on_console);
 static int on_silent(const char *name, const char *value, enum env_op op,
 	int flags)
 {
-#if !CONFIG_IS_ENABLED(CONFIG_SILENT_CONSOLE_UPDATE_ON_SET)
+#if !CONFIG_IS_ENABLED(SILENT_CONSOLE_UPDATE_ON_SET)
 	if (flags & H_INTERACTIVE)
 		return 0;
 #endif
-#if !CONFIG_IS_ENABLED(CONFIG_SILENT_CONSOLE_UPDATE_ON_RELOC)
+#if !CONFIG_IS_ENABLED(SILENT_CONSOLE_UPDATE_ON_RELOC)
 	if ((flags & H_INTERACTIVE) == 0)
 		return 0;
 #endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] common: console: Fix duplicated CONFIG in silent env callback
  2017-08-25  7:06 [U-Boot] [PATCH] common: console: Fix duplicated CONFIG in silent env callback Wilson Lee
@ 2017-08-28 16:17 ` Joe Hershberger
  2017-08-29  2:56 ` [U-Boot] " Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Joe Hershberger @ 2017-08-28 16:17 UTC (permalink / raw)
  To: u-boot

On Fri, Aug 25, 2017 at 2:06 AM, Wilson Lee <wilson.lee@ni.com> wrote:
> The silent environment callback function does not update the silent
> flag during silent env set or unset. That is because of duplicated
> CONFIG keyword at preprocessor condition in silent environment
> callback function and cause silent env callback unable to work.
>
> This patch is to remove the duplicated CONFIG keywork in silent
> environment callback function.
>
> Signed-off-by: Wilson Lee <wilson.lee@ni.com>
> Cc: Keng Soon Cheah <keng.soon.cheah@ni.com>
> Cc: Chen Yee Chew <chen.yee.chew@ni.com>
> Cc: Joe Hershberger <joe.hershberger@ni.com>

Reviewed-by: Joe Hershberger <joe.hershberger@ni.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] common: console: Fix duplicated CONFIG in silent env callback
  2017-08-25  7:06 [U-Boot] [PATCH] common: console: Fix duplicated CONFIG in silent env callback Wilson Lee
  2017-08-28 16:17 ` Joe Hershberger
@ 2017-08-29  2:56 ` Tom Rini
  1 sibling, 0 replies; 3+ messages in thread
From: Tom Rini @ 2017-08-29  2:56 UTC (permalink / raw)
  To: u-boot

On Fri, Aug 25, 2017 at 12:06:29AM -0700, Wilson Lee wrote:

> The silent environment callback function does not update the silent
> flag during silent env set or unset. That is because of duplicated
> CONFIG keyword at preprocessor condition in silent environment
> callback function and cause silent env callback unable to work.
> 
> This patch is to remove the duplicated CONFIG keywork in silent
> environment callback function.
> 
> Signed-off-by: Wilson Lee <wilson.lee@ni.com>
> Cc: Keng Soon Cheah <keng.soon.cheah@ni.com>
> Cc: Chen Yee Chew <chen.yee.chew@ni.com>
> Cc: Joe Hershberger <joe.hershberger@ni.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170828/a1427a62/attachment.sig>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-08-29  2:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-25  7:06 [U-Boot] [PATCH] common: console: Fix duplicated CONFIG in silent env callback Wilson Lee
2017-08-28 16:17 ` Joe Hershberger
2017-08-29  2:56 ` [U-Boot] " Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.