Mirrors/u-boot
2
0
Fork 0
mirror of https://github.com/AsahiLinux/u-boot synced 2024-11-10 15:14:43 +00:00
Code Issues Projects Releases Packages Wiki Activity

dm: replace dm_dbg() with pr_debug()

Browse source 
As we discussed before in ML, dm_dbg() causes undefined reference
error if #define DEBUG is added to users, but not drivers/core/util.c

We do not need this macro because we can use pr_debug() instead, and
it is pretty easy to enable it for the DM core by using ccflags-y.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
This commit is contained in:
Masahiro Yamada 2017-09-29 12:31:20 +09:00 committed by Tom Rini
parent 6990e91f09
commit ceb9190969
7 changed files with 19 additions and 29 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
drivers/core/Kconfig
View file

@ -45,6 +45,12 @@ config DM_WARN
This will cause dm_warn() to be compiled out - it will do nothing
when called.
config DM_DEBUG
bool "Enable debug messages in driver model core"
depends on DM
help
Say Y here if you want to compile in debug messages in DM core.
config DM_DEVICE_REMOVE
bool "Support device removal"
depends on DM

2
drivers/core/Makefile
View file

@ -16,3 +16,5 @@ ifndef CONFIG_DM_DEV_READ_INLINE
obj-$(CONFIG_OF_CONTROL) += read.o
endif
obj-$(CONFIG_OF_CONTROL) += of_extra.o ofnode.o read_extra.o
ccflags-$(CONFIG_DM_DEBUG) += -DDEBUG

2
drivers/core/device.c
View file

@ -161,7 +161,7 @@ static int device_bind_common(struct udevice *parent, const struct driver *drv,
}
if (parent)
dm_dbg("Bound device %s to %s\n", dev->name, parent->name);
pr_debug("Bound device %s to %s\n", dev->name, parent->name);
if (devp)
*devp = dev;

13
drivers/core/lists.c
View file

@ -139,12 +139,13 @@ int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp)
if (devp)
*devp = NULL;
name = ofnode_get_name(node);
dm_dbg("bind node %s\n", name);
pr_debug("bind node %s\n", name);
compat_list = ofnode_get_property(node, "compatible", &compat_length);
if (!compat_list) {
if (compat_length == -FDT_ERR_NOTFOUND) {
dm_dbg("Device '%s' has no compatible string\n", name);
pr_debug("Device '%s' has no compatible string\n",
name);
return 0;
}
@ -159,7 +160,7 @@ int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp)
*/
for (i = 0; i < compat_length; i += strlen(compat) + 1) {
compat = compat_list + i;
dm_dbg(" - attempt to match compatible string '%s'\n",
pr_debug(" - attempt to match compatible string '%s'\n",
compat);
for (entry = driver; entry != driver + n_ents; entry++) {
@ -171,11 +172,11 @@ int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp)
if (entry == driver + n_ents)
continue;
dm_dbg(" - found match at '%s'\n", entry->name);
pr_debug(" - found match at '%s'\n", entry->name);
ret = device_bind_with_driver_data(parent, entry, name,
id->data, node, &dev);
if (ret == -ENODEV) {
dm_dbg("Driver '%s' refuses to bind\n", entry->name);
pr_debug("Driver '%s' refuses to bind\n", entry->name);
continue;
}
if (ret) {
@ -191,7 +192,7 @@ int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp)
}
if (!found && !result && ret != -ENODEV)
dm_dbg("No match for node '%s'\n", name);
pr_debug("No match for node '%s'\n", name);
return result;
}

4
drivers/core/root.c
View file

@ -227,7 +227,7 @@ static int dm_scan_fdt_live(struct udevice *parent,
!of_find_property(np, "u-boot,dm-pre-reloc", NULL))
continue;
if (!of_device_is_available(np)) {
dm_dbg(" - ignoring disabled device\n");
pr_debug(" - ignoring disabled device\n");
continue;
}
err = lists_bind_fdt(parent, np_to_ofnode(np), NULL);
@ -270,7 +270,7 @@ static int dm_scan_fdt_node(struct udevice *parent, const void *blob,
!dm_fdt_pre_reloc(blob, offset))
continue;
if (!fdtdec_get_is_enabled(blob, offset)) {
dm_dbg(" - ignoring disabled device\n");
pr_debug(" - ignoring disabled device\n");
continue;
}
err = lists_bind_fdt(parent, offset_to_ofnode(offset), NULL);

11
drivers/core/util.c
View file

@ -20,17 +20,6 @@ void dm_warn(const char *fmt, ...)
}
#endif
#ifdef DEBUG
void dm_dbg(const char *fmt, ...)
{
va_list args;
va_start(args, fmt);
vprintf(fmt, args);
va_end(args);
}
#endif
int list_count_items(struct list_head *head)
{
struct list_head *node;

8
include/dm/util.h
View file

@ -15,14 +15,6 @@ static inline void dm_warn(const char *fmt, ...)
}
#endif
#ifdef DEBUG
void dm_dbg(const char *fmt, ...);
#else
static inline void dm_dbg(const char *fmt, ...)
{
}
#endif
struct list_head;
/**