* [PATCH] i2c: stub: remove definition of DEBUG
@ 2021-01-14 2:44 trix
2021-01-17 12:02 ` Wolfram Sang
0 siblings, 1 reply; 2+ messages in thread
From: trix @ 2021-01-14 2:44 UTC (permalink / raw)
To: jdelvare, wsa; +Cc: linux-i2c, linux-kernel, Tom Rix
From: Tom Rix <trix@redhat.com>
Defining DEBUG should only be done in development.
So remove DEBUG.
Signed-off-by: Tom Rix <trix@redhat.com>
---
drivers/i2c/i2c-stub.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/i2c/i2c-stub.c b/drivers/i2c/i2c-stub.c
index 537a598e22db..d642cad219d9 100644
--- a/drivers/i2c/i2c-stub.c
+++ b/drivers/i2c/i2c-stub.c
@@ -7,7 +7,6 @@
*/
-#define DEBUG 1
#define pr_fmt(fmt) "i2c-stub: " fmt
#include <linux/errno.h>
--
2.27.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] i2c: stub: remove definition of DEBUG
2021-01-14 2:44 [PATCH] i2c: stub: remove definition of DEBUG trix
@ 2021-01-17 12:02 ` Wolfram Sang
0 siblings, 0 replies; 2+ messages in thread
From: Wolfram Sang @ 2021-01-17 12:02 UTC (permalink / raw)
To: trix; +Cc: jdelvare, linux-i2c, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 342 bytes --]
On Wed, Jan 13, 2021 at 06:44:56PM -0800, trix@redhat.com wrote:
> From: Tom Rix <trix@redhat.com>
>
> Defining DEBUG should only be done in development.
> So remove DEBUG.
>
> Signed-off-by: Tom Rix <trix@redhat.com>
Yes, we can still enable them easily with the Kconfg debug option for
I2C core. Applied to for-next, thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-01-17 12:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-14 2:44 [PATCH] i2c: stub: remove definition of DEBUG trix
2021-01-17 12:02 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).