fbdev: Convert to using %pOFn instead of device_node.name

In preparation to remove the node name pointer from struct device_node,
convert printf users to use the %pOFn format specifier.

Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
This commit is contained in:
Rob Herring 2018-10-08 12:57:36 +02:00 committed by Bartlomiej Zolnierkiewicz
parent 60e5e48dba
commit 5c63e407aa
8 changed files with 16 additions and 14 deletions

View File

@ -355,9 +355,7 @@ static int cg14_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg)
static void cg14_init_fix(struct fb_info *info, int linebytes, static void cg14_init_fix(struct fb_info *info, int linebytes,
struct device_node *dp) struct device_node *dp)
{ {
const char *name = dp->name; snprintf(info->fix.id, sizeof(info->fix.id), "%pOFn", dp);
strlcpy(info->fix.id, name, sizeof(info->fix.id));
info->fix.type = FB_TYPE_PACKED_PIXELS; info->fix.type = FB_TYPE_PACKED_PIXELS;
info->fix.visual = FB_VISUAL_PSEUDOCOLOR; info->fix.visual = FB_VISUAL_PSEUDOCOLOR;

View File

@ -246,7 +246,7 @@ static int cg3_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg)
static void cg3_init_fix(struct fb_info *info, int linebytes, static void cg3_init_fix(struct fb_info *info, int linebytes,
struct device_node *dp) struct device_node *dp)
{ {
strlcpy(info->fix.id, dp->name, sizeof(info->fix.id)); snprintf(info->fix.id, sizeof(info->fix.id), "%pOFn", dp);
info->fix.type = FB_TYPE_PACKED_PIXELS; info->fix.type = FB_TYPE_PACKED_PIXELS;
info->fix.visual = FB_VISUAL_PSEUDOCOLOR; info->fix.visual = FB_VISUAL_PSEUDOCOLOR;

View File

@ -1480,8 +1480,8 @@ int of_get_fb_videomode(struct device_node *np, struct fb_videomode *fb,
if (ret) if (ret)
return ret; return ret;
pr_debug("%pOF: got %dx%d display mode from %s\n", pr_debug("%pOF: got %dx%d display mode\n",
np, vm.hactive, vm.vactive, np->name); np, vm.hactive, vm.vactive);
dump_fb_videomode(fb); dump_fb_videomode(fb);
return 0; return 0;

View File

@ -1473,7 +1473,7 @@ static int imsttfb_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
dp = pci_device_to_OF_node(pdev); dp = pci_device_to_OF_node(pdev);
if(dp) if(dp)
printk(KERN_INFO "%s: OF name %s\n",__func__, dp->name); printk(KERN_INFO "%s: OF name %pOFn\n",__func__, dp);
else if (IS_ENABLED(CONFIG_OF)) else if (IS_ENABLED(CONFIG_OF))
printk(KERN_ERR "imsttfb: no OF node for pci device\n"); printk(KERN_ERR "imsttfb: no OF node for pci device\n");

View File

@ -434,7 +434,7 @@ static int leo_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg)
static void static void
leo_init_fix(struct fb_info *info, struct device_node *dp) leo_init_fix(struct fb_info *info, struct device_node *dp)
{ {
strlcpy(info->fix.id, dp->name, sizeof(info->fix.id)); snprintf(info->fix.id, sizeof(info->fix.id), "%pOFn", dp);
info->fix.type = FB_TYPE_PACKED_PIXELS; info->fix.type = FB_TYPE_PACKED_PIXELS;
info->fix.visual = FB_VISUAL_TRUECOLOR; info->fix.visual = FB_VISUAL_TRUECOLOR;

View File

@ -419,9 +419,13 @@ static void __init offb_init_fb(const char *name,
var = &info->var; var = &info->var;
info->par = par; info->par = par;
strcpy(fix->id, "OFfb "); if (name) {
strncat(fix->id, name, sizeof(fix->id) - sizeof("OFfb ")); strcpy(fix->id, "OFfb ");
fix->id[sizeof(fix->id) - 1] = '\0'; strncat(fix->id, name, sizeof(fix->id) - sizeof("OFfb "));
fix->id[sizeof(fix->id) - 1] = '\0';
} else
snprintf(fix->id, sizeof(fix->id), "OFfb %pOFn", dp);
var->xres = var->xres_virtual = width; var->xres = var->xres_virtual = width;
var->yres = var->yres_virtual = height; var->yres = var->yres_virtual = height;
@ -644,7 +648,7 @@ static void __init offb_init_nodriver(struct device_node *dp, int no_real_node)
/* kludge for valkyrie */ /* kludge for valkyrie */
if (strcmp(dp->name, "valkyrie") == 0) if (strcmp(dp->name, "valkyrie") == 0)
address += 0x1000; address += 0x1000;
offb_init_fb(no_real_node ? "bootx" : dp->name, offb_init_fb(no_real_node ? "bootx" : NULL,
width, height, depth, pitch, address, width, height, depth, pitch, address,
foreign_endian, no_real_node ? NULL : dp); foreign_endian, no_real_node ? NULL : dp);
} }

View File

@ -239,7 +239,7 @@ static int p9100_ioctl(struct fb_info *info, unsigned int cmd,
static void p9100_init_fix(struct fb_info *info, int linebytes, struct device_node *dp) static void p9100_init_fix(struct fb_info *info, int linebytes, struct device_node *dp)
{ {
strlcpy(info->fix.id, dp->name, sizeof(info->fix.id)); snprintf(info->fix.id, sizeof(info->fix.id), "%pOFn", dp);
info->fix.type = FB_TYPE_PACKED_PIXELS; info->fix.type = FB_TYPE_PACKED_PIXELS;
info->fix.visual = FB_VISUAL_PSEUDOCOLOR; info->fix.visual = FB_VISUAL_PSEUDOCOLOR;

View File

@ -170,7 +170,7 @@ struct display_timings *of_get_display_timings(const struct device_node *np)
goto entryfail; goto entryfail;
} }
pr_debug("%pOF: using %s as default timing\n", np, entry->name); pr_debug("%pOF: using %pOFn as default timing\n", np, entry);
native_mode = entry; native_mode = entry;