gpu: Convert to using %pOF instead of full_name
Subsystems (21)
PATCH 1/1 gpu: Convert to using %pOF instead of full_name
Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.
--- drivers/gpu/drm/armada/armada_crtc.c | 3 +-- drivers/gpu/drm/armada/armada_drv.c | 4 ++-- drivers/gpu/drm/drm_mipi_dsi.c | 6 +++--- drivers/gpu/drm/drm_modes.c | 4 ++-- drivers/gpu/drm/drm_of.c | 4 ++-- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 3 +-- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 3 +-- drivers/gpu/drm/mediatek/mtk_disp_color.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_dpi.c | 6 +++--- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 7 +++---- drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 6 ++---- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 16 ++++++++-------- drivers/gpu/drm/mediatek/mtk_dsi.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_hdmi.c | 8 ++++---- drivers/gpu/drm/meson/meson_drv.c | 5 ++--- drivers/gpu/drm/panel/panel-lvds.c | 16 ++++++++-------- drivers/gpu/drm/rcar-du/rcar_du_encoder.c | 4 ++-- drivers/gpu/drm/rcar-du/rcar_du_kms.c | 16 ++++++++-------- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 4 ++-- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 4 ++-- drivers/gpu/drm/stm/ltdc.c | 3 +-- drivers/gpu/drm/sun4i/sun4i_drv.c | 9 ++++----- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 4 ++-- drivers/gpu/ipu-v3/ipu-common.c | 4 ++-- 26 files changed, 73 insertions(+), 82 deletions(-)
drivers/gpu/drm/armada/armada_crtc.c +1 -2
@@ -1329,8 +1329,7 @@ armada_lcd_bind(struct device *dev, struct device *master, void *data) port =of_get_child_by_name (parent,"port" );of_node_put (np);if (!port) { -dev_err (dev,"no port node found in %s\n" , - parent->full_name); +dev_err (dev,"no port node found in %pOF\n" , parent);return -ENXIO ; }
drivers/gpu/drm/armada/armada_drv.c +2 -2
@@ -232,8 +232,8 @@ static void armada_add_endpoints(struct device *dev,of_node_put (remote);continue ; }else if (!of_device_is_available (remote->parent)) { -dev_warn (dev,"parent device of %s is not available\n" , - remote->full_name); +dev_warn (dev,"parent device of %pOF is not available\n" , + remote);of_node_put (remote);continue ; }
drivers/gpu/drm/drm_mipi_dsi.c +3 -3
@@ -165,14 +165,14 @@ of_mipi_dsi_device_add(struct mipi_dsi_host *host, struct device_node *node)u32 reg;if (of_modalias_node (node, info.type,sizeof (info.type)) <0 ) { -dev_err (dev,"modalias failure on %s\n" , node->full_name); +dev_err (dev,"modalias failure on %pOF\n" , node);return ERR_PTR (-EINVAL ); } ret =of_property_read_u32 (node,"reg" , ®);if (ret) { -dev_err (dev,"device node %s has no valid reg property: %d\n" , - node->full_name, ret); +dev_err (dev,"device node %pOF has no valid reg property: %d\n" , + node, ret);return ERR_PTR (-EINVAL ); }
drivers/gpu/drm/drm_modes.c +2 -2
@@ -709,8 +709,8 @@ int of_get_drm_display_mode(struct device_node *np,if (bus_flags)drm_bus_flags_from_videomode (&vm, bus_flags); -pr_debug ("%s: got %dx%d display mode from %s\n" , -of_node_full_name (np), vm.hactive, vm.vactive, np->name); +pr_debug ("%pOF: got %dx%d display mode from %s\n" , + np, vm.hactive, vm.vactive, np->name);drm_mode_debug_printmodeline (dmode);return 0 ;
drivers/gpu/drm/drm_of.c +2 -2
@@ -160,8 +160,8 @@ int drm_of_component_probe(struct device *dev,of_node_put (remote);continue ; }else if (!of_device_is_available (remote->parent)) { -dev_warn (dev,"parent device of %s is not available\n" , - remote->full_name); +dev_warn (dev,"parent device of %pOF is not available\n" , + remote);of_node_put (remote);continue ; }
drivers/gpu/drm/exynos/exynos_drm_dsi.c +1 -2
@@ -1618,8 +1618,7 @@ static int exynos_dsi_of_read_u32(const struct device_node *np,int ret =of_property_read_u32 (np, propname, out_value);if (ret <0 ) -pr_err ("%s: failed to get '%s' property\n" , np->full_name, - propname); +pr_err ("%pOF: failed to get '%s' property\n" , np, propname);return ret; }
drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c +1 -2
@@ -583,8 +583,7 @@ static int ade_crtc_init(struct drm_device *dev, struct drm_crtc *crtc,*/ port =of_get_child_by_name (dev->dev->of_node,"port" );if (!port) { -DRM_ERROR ("no port node found in %s\n" , - dev->dev->of_node->full_name); +DRM_ERROR ("no port node found in %pOF\n" , dev->dev->of_node);return -EINVAL ; }of_node_put (port);
drivers/gpu/drm/mediatek/mtk_disp_color.c +2 -2
@@ -84,8 +84,8 @@ static int mtk_disp_color_bind(struct device *dev, struct device *master, ret =mtk_ddp_comp_register (drm_dev, &priv->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }
drivers/gpu/drm/mediatek/mtk_disp_ovl.c +2 -2
@@ -235,8 +235,8 @@ static int mtk_disp_ovl_bind(struct device *dev, struct device *master, ret =mtk_ddp_comp_register (drm_dev, &priv->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }
drivers/gpu/drm/mediatek/mtk_disp_rdma.c +2 -2
@@ -155,8 +155,8 @@ static int mtk_disp_rdma_bind(struct device *dev, struct device *master, ret =mtk_ddp_comp_register (drm_dev, &priv->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }
drivers/gpu/drm/mediatek/mtk_dpi.c +3 -3
@@ -605,8 +605,8 @@ static int mtk_dpi_bind(struct device *dev, struct device *master, void *data) ret =mtk_ddp_comp_register (drm_dev, &dpi->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }
@@ -710,7 +710,7 @@ static int mtk_dpi_probe(struct platform_device *pdev)if (!bridge_node)return -ENODEV ; -dev_info (dev,"Found bridge node: %s\n" , bridge_node->full_name); +dev_info (dev,"Found bridge node: %pOF\n" , bridge_node); dpi->bridge =of_drm_find_bridge (bridge_node);of_node_put (bridge_node);
drivers/gpu/drm/mediatek/mtk_drm_crtc.c +3 -4
@@ -577,8 +577,7 @@ int mtk_drm_crtc_create(struct drm_device *drm_dev, node = priv->comp_node[comp_id]; comp = priv->ddp_comp[comp_id];if (!comp) { -dev_err (dev,"Component %s not initialized\n" , - node->full_name); +dev_err (dev,"Component %pOF not initialized\n" , node); ret = -ENODEV ; goto unprepare; }
@@ -586,8 +585,8 @@ int mtk_drm_crtc_create(struct drm_device *drm_dev, ret =clk_prepare (comp->clk);if (ret) {dev_err (dev, -"Failed to prepare clock for component %s: %d\n" , - node->full_name, ret); +"Failed to prepare clock for component %pOF: %d\n" , + node, ret); goto unprepare; }
drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c +2 -4
@@ -295,15 +295,13 @@ int mtk_ddp_comp_init(struct device *dev, struct device_node *node, larb_node =of_parse_phandle (node,"mediatek,larb" ,0 );if (!larb_node) {dev_err (dev, -"Missing mediadek,larb phandle in %s node\n" , - node->full_name); +"Missing mediadek,larb phandle in %pOF node\n" , node);return -EINVAL ; } larb_pdev =of_find_device_by_node (larb_node);if (!larb_pdev) { -dev_warn (dev,"Waiting for larb device %s\n" , - larb_node->full_name); +dev_warn (dev,"Waiting for larb device %pOF\n" , larb_node);of_node_put (larb_node);return -EPROBE_DEFER ; }
drivers/gpu/drm/mediatek/mtk_drm_drv.c +8 -8
@@ -187,8 +187,8 @@ static int mtk_drm_kms_init(struct drm_device *drm) pdev =of_find_device_by_node (private->mutex_node);if (!pdev) { -dev_err (drm->dev,"Waiting for disp-mutex device %s\n" , - private->mutex_node->full_name); +dev_err (drm->dev,"Waiting for disp-mutex device %pOF\n" , + private->mutex_node);of_node_put (private->mutex_node);return -EPROBE_DEFER ; }
@@ -417,8 +417,8 @@ static int mtk_drm_probe(struct platform_device *pdev)continue ;if (!of_device_is_available (node)) { -dev_dbg (dev,"Skipping disabled component %s\n" , - node->full_name); +dev_dbg (dev,"Skipping disabled component %pOF\n" , + node);continue ; }
@@ -431,8 +431,8 @@ static int mtk_drm_probe(struct platform_device *pdev) comp_id =mtk_ddp_comp_get_id (node, comp_type);if (comp_id <0 ) { -dev_warn (dev,"Skipping unknown component %s\n" , - node->full_name); +dev_warn (dev,"Skipping unknown component %pOF\n" , + node);continue ; }
@@ -448,8 +448,8 @@ static int mtk_drm_probe(struct platform_device *pdev) comp_type == MTK_DISP_RDMA || comp_type == MTK_DSI || comp_type == MTK_DPI) { - dev_info(dev, "Adding component match for %s\n", - node->full_name); + dev_info(dev, "Adding component match for %pOF\n", + node); drm_of_component_match_add(dev, &match, compare_of, node); } else {
drivers/gpu/drm/mediatek/mtk_dsi.c +2 -2
@@ -1048,8 +1048,8 @@ static int mtk_dsi_bind(struct device *dev, struct device *master, void *data) ret =mtk_ddp_comp_register (drm, &dsi->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }
drivers/gpu/drm/mediatek/mtk_hdmi.c +4 -4
@@ -1456,8 +1456,8 @@ static int mtk_hdmi_dt_parse_pdata(struct mtk_hdmi *hdmi, cec_pdev =of_find_device_by_node (cec_np);if (!cec_pdev) { -dev_err (hdmi->dev,"Waiting for CEC device %s\n" , - cec_np->full_name); +dev_err (hdmi->dev,"Waiting for CEC device %pOF\n" , + cec_np);return -EPROBE_DEFER ; } hdmi->cec_dev = &cec_pdev->dev;
@@ -1501,8 +1501,8 @@ static int mtk_hdmi_dt_parse_pdata(struct mtk_hdmi *hdmi, i2c_np =of_parse_phandle (remote,"ddc-i2c-bus" ,0 );if (!i2c_np) { -dev_err (dev,"Failed to find ddc-i2c-bus node in %s\n" , - remote->full_name); +dev_err (dev,"Failed to find ddc-i2c-bus node in %pOF\n" , + remote);of_node_put (remote);return -EINVAL ; }
drivers/gpu/drm/meson/meson_drv.c +2 -3
@@ -303,9 +303,8 @@ static const struct component_master_ops meson_drv_master_ops = {static int compare_of (struct device *dev,void *data) { -DRM_DEBUG_DRIVER ("Comparing of node %s with %s\n" , -of_node_full_name (dev->of_node), -of_node_full_name (data)); +DRM_DEBUG_DRIVER ("Comparing of node %pOF with %pOF\n" , + dev->of_node, data);return dev->of_node == data; }
drivers/gpu/drm/panel/panel-lvds.c +8 -8
@@ -143,14 +143,14 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) ret =of_property_read_u32 (np,"width-mm" , &lvds->width);if (ret <0 ) { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"width-mm" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"width-mm" );return -ENODEV ; } ret =of_property_read_u32 (np,"height-mm" , &lvds->height);if (ret <0 ) { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"height-mm" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"height-mm" );return -ENODEV ; }
@@ -158,8 +158,8 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) ret =of_property_read_string (np,"data-mapping" , &mapping);if (ret <0 ) { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"data-mapping" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"data-mapping" );return -ENODEV ; }
@@ -170,8 +170,8 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) }else if (!strcmp (mapping,"vesa-24" )) { lvds->bus_format =MEDIA_BUS_FMT_RGB888_1X7X4_SPWG ; }else { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"data-mapping" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"data-mapping" );return -EINVAL ; }
drivers/gpu/drm/rcar-du/rcar_du_encoder.c +2 -2
@@ -186,8 +186,8 @@ int rcar_du_encoder_init(struct rcar_du_device *rcdu, }if (enc_node) { -dev_dbg (rcdu->dev,"initializing encoder %s for output %u\n" , -of_node_full_name (enc_node), output); +dev_dbg (rcdu->dev,"initializing encoder %pOF for output %u\n" , + enc_node, output);/* Locate the DRM bridge from the encoder DT node. */ bridge =of_drm_find_bridge (enc_node);
drivers/gpu/drm/rcar-du/rcar_du_kms.c +8 -8
@@ -297,15 +297,15 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,*/ entity =of_graph_get_remote_port_parent (ep->local_node);if (!entity) { -dev_dbg (rcdu->dev,"unconnected endpoint %s, skipping\n" , - ep->local_node->full_name); +dev_dbg (rcdu->dev,"unconnected endpoint %pOF, skipping\n" , + ep->local_node);return -ENODEV ; }if (!of_device_is_available (entity)) {dev_dbg (rcdu->dev, -"connected entity %s is disabled, skipping\n" , - entity->full_name); +"connected entity %pOF is disabled, skipping\n" , + entity);return -ENODEV ; }
@@ -325,8 +325,8 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,if (!connector) {dev_warn (rcdu->dev, -"no connector for encoder %s, skipping\n" , - encoder->full_name); +"no connector for encoder %pOF, skipping\n" , + encoder);of_node_put (entity_ep_node);of_node_put (encoder);return -ENODEV ;
@@ -348,8 +348,8 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu, ret =rcar_du_encoder_init (rcdu, output, encoder, connector);if (ret && ret != -EPROBE_DEFER )dev_warn (rcdu->dev, -"failed to initialize encoder %s on output %u (%d), skipping\n" , -of_node_full_name (encoder), output, ret); +"failed to initialize encoder %pOF on output %u (%d), skipping\n" , + encoder, output, ret);of_node_put (encoder);of_node_put (connector);
drivers/gpu/drm/rockchip/rockchip_drm_drv.c +2 -2
@@ -370,8 +370,8 @@ static int rockchip_drm_platform_of_probe(struct device *dev) iommu =of_parse_phandle (port->parent,"iommus" ,0 );if (!iommu || !of_device_is_available (iommu->parent)) { -dev_dbg (dev,"no iommu attached for %s, using non-iommu buffers\n" , - port->parent->full_name); +dev_dbg (dev,"no iommu attached for %pOF, using non-iommu buffers\n" , + port->parent);/* * if there is a crtc not support iommu, force set all * crtc use non-iommu buffer.
drivers/gpu/drm/rockchip/rockchip_drm_vop.c +2 -2
@@ -1340,8 +1340,8 @@ static int vop_create_crtc(struct vop *vop) port =of_get_child_by_name (dev->of_node,"port" );if (!port) { -DRM_DEV_ERROR (vop->dev,"no port node found in %s\n" , - dev->of_node->full_name); +DRM_DEV_ERROR (vop->dev,"no port node found in %pOF\n" , + dev->of_node); ret = -ENOENT ; goto err_cleanup_crtc; }
drivers/gpu/drm/stm/ltdc.c +1 -2
@@ -995,8 +995,7 @@ static struct drm_panel *ltdc_get_panel(struct drm_device *ddev) panel =of_drm_find_panel (port);of_node_put (port);if (panel) { -DRM_DEBUG_DRIVER ("remote panel %s\n" , - port->full_name); +DRM_DEBUG_DRIVER ("remote panel %pOF\n" , port); }else {DRM_DEBUG_DRIVER ("panel missing\n" );of_node_put (entity);
drivers/gpu/drm/sun4i/sun4i_drv.c +4 -5
@@ -187,9 +187,9 @@ static bool sun4i_drv_node_is_tcon(struct device_node *node)static int compare_of (struct device *dev,void *data) { -DRM_DEBUG_DRIVER ("Comparing of node %s with %s\n" , -of_node_full_name (dev->of_node), -of_node_full_name (data)); +DRM_DEBUG_DRIVER ("Comparing of node %pOF with %pOF\n" , + dev->of_node, + data);return dev->of_node == data; }
@@ -219,8 +219,7 @@ static int sun4i_drv_add_endpoints(struct device *dev,if (!sun4i_drv_node_is_frontend (node)) {/* Add current component */ -DRM_DEBUG_DRIVER ("Adding component %s\n" , -of_node_full_name (node)); +DRM_DEBUG_DRIVER ("Adding component %pOF\n" , node);drm_of_component_match_add (dev, match, compare_of, node); count++; }
drivers/gpu/drm/tilcdc/tilcdc_crtc.c +2 -2
@@ -1038,8 +1038,8 @@ int tilcdc_crtc_create(struct drm_device *dev)if (priv->is_componentized) { crtc->port =of_graph_get_port_by_id (dev->dev->of_node,0 );if (!crtc->port) {/* This should never happen */ -dev_err (dev->dev,"Port node not found in %s\n" , - dev->dev->of_node->full_name); +dev_err (dev->dev,"Port node not found in %pOF\n" , + dev->dev->of_node); ret = -EINVAL ; goto fail; }
drivers/gpu/ipu-v3/ipu-common.c +2 -2
@@ -1217,8 +1217,8 @@ static int ipu_add_client_devices(struct ipu_soc *ipu, unsigned long ipu_base) of_node =of_graph_get_port_by_id (dev->of_node, i);if (!of_node) {dev_info (dev, -"no port@%d node in %s, not using %s%d\n" , - i, dev->of_node->full_name, +"no port@%d node in %pOF, not using %s%d\n" , + i, dev->of_node, (i /2 ) ?"DI" :"CSI" , i %2 );continue ; }
--
2.11.0
Quoted 59 lines · Rob Herring
Now that we have a custom printf format specifier, convert users of
full_nameto use %pOF instead. This is preparation to remove storing of the full path string for each node.Signed-off-by Rob Herring <[email protected]>Cc Russell King <[email protected]>Cc David Airlie <[email protected]>Cc Daniel Vetter <[email protected]>Cc Jani Nikula <[email protected]>Cc Sean Paul <[email protected]>Cc Inki Dae <[email protected]>Cc Joonyoung Shim <[email protected]>Cc Seung-Woo Kim <[email protected]>Cc Kyungmin Park <[email protected]>Cc Kukjin Kim <[email protected]>Cc Krzysztof Kozlowski <[email protected]>Cc Javier Martinez Canillas <[email protected]>Cc Xinliang Liu <[email protected]>Cc Rongrong Zou <[email protected]>Cc Xinwei Kong <[email protected]>Cc Chen Feng <[email protected]>Cc CK Hu <[email protected]>Cc Philipp Zabel <[email protected]>Cc Matthias Brugger <[email protected]>Cc Neil Armstrong <[email protected]>Cc Carlo Caione <[email protected]>Cc Kevin Hilman <[email protected]>Cc Thierry Reding <[email protected]>Cc Laurent Pinchart <[email protected]>Cc Mark Yao <[email protected]>Cc Heiko Stuebner <[email protected]>Cc Maxime Ripard <[email protected]>Cc Chen-Yu Tsai <[email protected]>Cc Jyri Sarha <[email protected]>Cc Tomi Valkeinen <[email protected]>Cc dri-devel at lists.freedesktop.orgCc linux-arm-kernel at lists.infradead.orgCc linux-samsung-soc at vger.kernel.orgCc linux-mediatek at lists.infradead.orgCc linux-amlogic at lists.infradead.orgCc linux-renesas-soc at vger.kernel.orgCc linux-rockchip at lists.infradead.org--- drivers/gpu/drm/armada/armada_crtc.c | 3 +-- drivers/gpu/drm/armada/armada_drv.c | 4 ++-- drivers/gpu/drm/drm_mipi_dsi.c | 6 +++--- drivers/gpu/drm/drm_modes.c | 4 ++-- drivers/gpu/drm/drm_of.c | 4 ++-- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 3 +-- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 3 +-- drivers/gpu/drm/mediatek/mtk_disp_color.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_dpi.c | 6 +++--- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 7 +++---- drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 6 ++---- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 16 ++++++++-------- drivers/gpu/drm/mediatek/mtk_dsi.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_hdmi.c | 8 ++++----
Quoted 8 lines
drivers/gpu/drm/meson/meson_drv.c | 5 ++--- drivers/gpu/drm/panel/panel-lvds.c | 16 ++++++++-------- drivers/gpu/drm/rcar-du/rcar_du_encoder.c | 4 ++-- drivers/gpu/drm/rcar-du/rcar_du_kms.c | 16 ++++++++-------- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 4 ++-- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 4 ++-- drivers/gpu/drm/stm/ltdc.c | 3 +-- drivers/gpu/drm/sun4i/sun4i_drv.c | 9 ++++-----
For sun4i-drm:
Thanks!
Maxime
Signature · 10 lines
Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-amlogic/attachments/20170719/025aa192/attachment.sig>
Quoted 42 lines · Rob Herring
Now that we have a custom printf format specifier, convert users of
full_nameto use %pOF instead. This is preparation to remove storing of the full path string for each node.Signed-off-by Rob Herring <[email protected]>Cc Russell King <[email protected]>Cc David Airlie <[email protected]>Cc Daniel Vetter <[email protected]>Cc Jani Nikula <[email protected]>Cc Sean Paul <[email protected]>Cc Inki Dae <[email protected]>Cc Joonyoung Shim <[email protected]>Cc Seung-Woo Kim <[email protected]>Cc Kyungmin Park <[email protected]>Cc Kukjin Kim <[email protected]>Cc Krzysztof Kozlowski <[email protected]>Cc Javier Martinez Canillas <[email protected]>Cc Xinliang Liu <[email protected]>Cc Rongrong Zou <[email protected]>Cc Xinwei Kong <[email protected]>Cc Chen Feng <[email protected]>Cc CK Hu <[email protected]>Cc Philipp Zabel <[email protected]>Cc Matthias Brugger <[email protected]>Cc Neil Armstrong <[email protected]>Cc Carlo Caione <[email protected]>Cc Kevin Hilman <[email protected]>Cc Thierry Reding <[email protected]>Cc Laurent Pinchart <[email protected]>Cc Mark Yao <[email protected]>Cc Heiko Stuebner <[email protected]>Cc Maxime Ripard <[email protected]>Cc Chen-Yu Tsai <[email protected]>Cc Jyri Sarha <[email protected]>Cc Tomi Valkeinen <[email protected]>
Quoted 1 line
---
regards
Philipp
Hi Rob,
Thank you for the patch.
Quoted 43 lines · On Tuesday 18 Jul 2017 16:43:04 Rob Herring wrote:
Now that we have a custom printf format specifier, convert users of
full_nameto use %pOF instead. This is preparation to remove storing of the full path string for each node.Signed-off-by Rob Herring <[email protected]>Cc Russell King <[email protected]>Cc David Airlie <[email protected]>Cc Daniel Vetter <[email protected]>Cc Jani Nikula <[email protected]>Cc Sean Paul <[email protected]>Cc Inki Dae <[email protected]>Cc Joonyoung Shim <[email protected]>Cc Seung-Woo Kim <[email protected]>Cc Kyungmin Park <[email protected]>Cc Kukjin Kim <[email protected]>Cc Krzysztof Kozlowski <[email protected]>Cc Javier Martinez Canillas <[email protected]>Cc Xinliang Liu <[email protected]>Cc Rongrong Zou <[email protected]>Cc Xinwei Kong <[email protected]>Cc Chen Feng <[email protected]>Cc CK Hu <[email protected]>Cc Philipp Zabel <[email protected]>Cc Matthias Brugger <[email protected]>Cc Neil Armstrong <[email protected]>Cc Carlo Caione <[email protected]>Cc Kevin Hilman <[email protected]>Cc Thierry Reding <[email protected]>Cc Laurent Pinchart <[email protected]>Cc Mark Yao <[email protected]>Cc Heiko Stuebner <[email protected]>Cc Maxime Ripard <[email protected]>Cc Chen-Yu Tsai <[email protected]>Cc Jyri Sarha <[email protected]>Cc Tomi Valkeinen <[email protected]>---
[snip]
Quoted 3 lines
drivers/gpu/drm/drm_mipi_dsi.c | 6 +++--- drivers/gpu/drm/drm_modes.c | 4 ++-- drivers/gpu/drm/drm_of.c | 4 ++--
[snip]
Quoted 3 lines
drivers/gpu/drm/panel/panel-lvds.c | 16 ++++++++-------- drivers/gpu/drm/rcar-du/rcar_du_encoder.c | 4 ++-- drivers/gpu/drm/rcar-du/rcar_du_kms.c | 16 ++++++++--------
For all the above,
Signature · 8 lines
Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
Sean Paul
AppliedQuoted 5 lines · Rob Herring
Now that we have a custom printf format specifier, convert users of
full_nameto use %pOF instead. This is preparation to remove storing of the full path string for each node.Signed-off-by Rob Herring <[email protected]>
Hi Rob,
Thanks for sending this patch, and thanks to Laurent, Philipp, and Maxime for reviews/acks.
I've applied it to drm-misc-next.
Sean
Quoted 606 lines
Cc Russell King <[email protected]>Cc David Airlie <[email protected]>Cc Daniel Vetter <[email protected]>Cc Jani Nikula <[email protected]>Cc Sean Paul <[email protected]>Cc Inki Dae <[email protected]>Cc Joonyoung Shim <[email protected]>Cc Seung-Woo Kim <[email protected]>Cc Kyungmin Park <[email protected]>Cc Kukjin Kim <[email protected]>Cc Krzysztof Kozlowski <[email protected]>Cc Javier Martinez Canillas <[email protected]>Cc Xinliang Liu <[email protected]>Cc Rongrong Zou <[email protected]>Cc Xinwei Kong <[email protected]>Cc Chen Feng <[email protected]>Cc CK Hu <[email protected]>Cc Philipp Zabel <[email protected]>Cc Matthias Brugger <[email protected]>Cc Neil Armstrong <[email protected]>Cc Carlo Caione <[email protected]>Cc Kevin Hilman <[email protected]>Cc Thierry Reding <[email protected]>Cc Laurent Pinchart <[email protected]>Cc Mark Yao <[email protected]>Cc Heiko Stuebner <[email protected]>Cc Maxime Ripard <[email protected]>Cc Chen-Yu Tsai <[email protected]>Cc Jyri Sarha <[email protected]>Cc Tomi Valkeinen <[email protected]>--- drivers/gpu/drm/armada/armada_crtc.c | 3 +-- drivers/gpu/drm/armada/armada_drv.c | 4 ++-- drivers/gpu/drm/drm_mipi_dsi.c | 6 +++--- drivers/gpu/drm/drm_modes.c | 4 ++-- drivers/gpu/drm/drm_of.c | 4 ++-- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 3 +-- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 3 +-- drivers/gpu/drm/mediatek/mtk_disp_color.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_dpi.c | 6 +++--- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 7 +++---- drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 6 ++---- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 16 ++++++++-------- drivers/gpu/drm/mediatek/mtk_dsi.c | 4 ++-- drivers/gpu/drm/mediatek/mtk_hdmi.c | 8 ++++---- drivers/gpu/drm/meson/meson_drv.c | 5 ++--- drivers/gpu/drm/panel/panel-lvds.c | 16 ++++++++-------- drivers/gpu/drm/rcar-du/rcar_du_encoder.c | 4 ++-- drivers/gpu/drm/rcar-du/rcar_du_kms.c | 16 ++++++++-------- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 4 ++-- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 4 ++-- drivers/gpu/drm/stm/ltdc.c | 3 +-- drivers/gpu/drm/sun4i/sun4i_drv.c | 9 ++++----- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 4 ++-- drivers/gpu/ipu-v3/ipu-common.c | 4 ++-- 26 files changed, 73 insertions(+), 82 deletions(-)drivers/gpu/drm/armada/armada_crtc.c +1
-2@@ -1329,8 +1329,7 @@ armada_lcd_bind(struct device *dev, struct device *master, void *data) port =of_get_child_by_name (parent,"port" );of_node_put (np);if (!port) { -dev_err (dev,"no port node found in %s\n" , - parent->full_name); +dev_err (dev,"no port node found in %pOF\n" , parent);return -ENXIO ; }drivers/gpu/drm/armada/armada_drv.c +2
-2@@ -232,8 +232,8 @@ static void armada_add_endpoints(struct device *dev,of_node_put (remote);continue ; }else if (!of_device_is_available (remote->parent)) { -dev_warn (dev,"parent device of %s is not available\n" , - remote->full_name); +dev_warn (dev,"parent device of %pOF is not available\n" , + remote);of_node_put (remote);continue ; }drivers/gpu/drm/drm_mipi_dsi.c +3
-3@@ -165,14 +165,14 @@ of_mipi_dsi_device_add(struct mipi_dsi_host *host, struct device_node *node)u32 reg;if (of_modalias_node (node, info.type,sizeof (info.type)) <0 ) { -dev_err (dev,"modalias failure on %s\n" , node->full_name); +dev_err (dev,"modalias failure on %pOF\n" , node);return ERR_PTR (-EINVAL ); } ret =of_property_read_u32 (node,"reg" , ®);if (ret) { -dev_err (dev,"device node %s has no valid reg property: %d\n" , - node->full_name, ret); +dev_err (dev,"device node %pOF has no valid reg property: %d\n" , + node, ret);return ERR_PTR (-EINVAL ); }drivers/gpu/drm/drm_modes.c +2
-2@@ -709,8 +709,8 @@ int of_get_drm_display_mode(struct device_node *np,if (bus_flags)drm_bus_flags_from_videomode (&vm, bus_flags); -pr_debug ("%s: got %dx%d display mode from %s\n" , -of_node_full_name (np), vm.hactive, vm.vactive, np->name); +pr_debug ("%pOF: got %dx%d display mode from %s\n" , + np, vm.hactive, vm.vactive, np->name);drm_mode_debug_printmodeline (dmode);return 0 ;drivers/gpu/drm/drm_of.c +2
-2@@ -160,8 +160,8 @@ int drm_of_component_probe(struct device *dev,of_node_put (remote);continue ; }else if (!of_device_is_available (remote->parent)) { -dev_warn (dev,"parent device of %s is not available\n" , - remote->full_name); +dev_warn (dev,"parent device of %pOF is not available\n" , + remote);of_node_put (remote);continue ; }drivers/gpu/drm/exynos/exynos_drm_dsi.c +1
-2@@ -1618,8 +1618,7 @@ static int exynos_dsi_of_read_u32(const struct device_node *np,int ret =of_property_read_u32 (np, propname, out_value);if (ret <0 ) -pr_err ("%s: failed to get '%s' property\n" , np->full_name, - propname); +pr_err ("%pOF: failed to get '%s' property\n" , np, propname);return ret; }drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c +1
-2@@ -583,8 +583,7 @@ static int ade_crtc_init(struct drm_device *dev, struct drm_crtc *crtc,*/ port =of_get_child_by_name (dev->dev->of_node,"port" );if (!port) { -DRM_ERROR ("no port node found in %s\n" , - dev->dev->of_node->full_name); +DRM_ERROR ("no port node found in %pOF\n" , dev->dev->of_node);return -EINVAL ; }of_node_put (port);drivers/gpu/drm/mediatek/mtk_disp_color.c +2
-2@@ -84,8 +84,8 @@ static int mtk_disp_color_bind(struct device *dev, struct device *master, ret =mtk_ddp_comp_register (drm_dev, &priv->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }drivers/gpu/drm/mediatek/mtk_disp_ovl.c +2
-2@@ -235,8 +235,8 @@ static int mtk_disp_ovl_bind(struct device *dev, struct device *master, ret =mtk_ddp_comp_register (drm_dev, &priv->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }drivers/gpu/drm/mediatek/mtk_disp_rdma.c +2
-2@@ -155,8 +155,8 @@ static int mtk_disp_rdma_bind(struct device *dev, struct device *master, ret =mtk_ddp_comp_register (drm_dev, &priv->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }drivers/gpu/drm/mediatek/mtk_dpi.c +3
-3@@ -605,8 +605,8 @@ static int mtk_dpi_bind(struct device *dev, struct device *master, void *data) ret =mtk_ddp_comp_register (drm_dev, &dpi->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }@@ -710,7 +710,7 @@ static int mtk_dpi_probe(struct platform_device *pdev)if (!bridge_node)return -ENODEV ; -dev_info (dev,"Found bridge node: %s\n" , bridge_node->full_name); +dev_info (dev,"Found bridge node: %pOF\n" , bridge_node); dpi->bridge =of_drm_find_bridge (bridge_node);of_node_put (bridge_node);drivers/gpu/drm/mediatek/mtk_drm_crtc.c +3
-4@@ -577,8 +577,7 @@ int mtk_drm_crtc_create(struct drm_device *drm_dev, node = priv->comp_node[comp_id]; comp = priv->ddp_comp[comp_id];if (!comp) { -dev_err (dev,"Component %s not initialized\n" , - node->full_name); +dev_err (dev,"Component %pOF not initialized\n" , node); ret = -ENODEV ; goto unprepare; }@@ -586,8 +585,8 @@ int mtk_drm_crtc_create(struct drm_device *drm_dev, ret =clk_prepare (comp->clk);if (ret) {dev_err (dev, -"Failed to prepare clock for component %s: %d\n" , - node->full_name, ret); +"Failed to prepare clock for component %pOF: %d\n" , + node, ret); goto unprepare; }drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c +2
-4@@ -295,15 +295,13 @@ int mtk_ddp_comp_init(struct device *dev, struct device_node *node, larb_node =of_parse_phandle (node,"mediatek,larb" ,0 );if (!larb_node) {dev_err (dev, -"Missing mediadek,larb phandle in %s node\n" , - node->full_name); +"Missing mediadek,larb phandle in %pOF node\n" , node);return -EINVAL ; } larb_pdev =of_find_device_by_node (larb_node);if (!larb_pdev) { -dev_warn (dev,"Waiting for larb device %s\n" , - larb_node->full_name); +dev_warn (dev,"Waiting for larb device %pOF\n" , larb_node);of_node_put (larb_node);return -EPROBE_DEFER ; }drivers/gpu/drm/mediatek/mtk_drm_drv.c +8
-8@@ -187,8 +187,8 @@ static int mtk_drm_kms_init(struct drm_device *drm) pdev =of_find_device_by_node (private->mutex_node);if (!pdev) { -dev_err (drm->dev,"Waiting for disp-mutex device %s\n" , - private->mutex_node->full_name); +dev_err (drm->dev,"Waiting for disp-mutex device %pOF\n" , + private->mutex_node);of_node_put (private->mutex_node);return -EPROBE_DEFER ; }@@ -417,8 +417,8 @@ static int mtk_drm_probe(struct platform_device *pdev)continue ;if (!of_device_is_available (node)) { -dev_dbg (dev,"Skipping disabled component %s\n" , - node->full_name); +dev_dbg (dev,"Skipping disabled component %pOF\n" , + node);continue ; }@@ -431,8 +431,8 @@ static int mtk_drm_probe(struct platform_device *pdev) comp_id =mtk_ddp_comp_get_id (node, comp_type);if (comp_id <0 ) { -dev_warn (dev,"Skipping unknown component %s\n" , - node->full_name); +dev_warn (dev,"Skipping unknown component %pOF\n" , + node);continue ; }@@ -448,8 +448,8 @@ static int mtk_drm_probe(struct platform_device *pdev) comp_type == MTK_DISP_RDMA || comp_type == MTK_DSI || comp_type == MTK_DPI) { - dev_info(dev, "Adding component match for %s\n", - node->full_name); + dev_info(dev, "Adding component match for %pOF\n", + node); drm_of_component_match_add(dev, &match, compare_of, node); } else {drivers/gpu/drm/mediatek/mtk_dsi.c +2
-2@@ -1048,8 +1048,8 @@ static int mtk_dsi_bind(struct device *dev, struct device *master, void *data) ret =mtk_ddp_comp_register (drm, &dsi->ddp_comp);if (ret <0 ) { -dev_err (dev,"Failed to register component %s: %d\n" , - dev->of_node->full_name, ret); +dev_err (dev,"Failed to register component %pOF: %d\n" , + dev->of_node, ret);return ret; }drivers/gpu/drm/mediatek/mtk_hdmi.c +4
-4@@ -1456,8 +1456,8 @@ static int mtk_hdmi_dt_parse_pdata(struct mtk_hdmi *hdmi, cec_pdev =of_find_device_by_node (cec_np);if (!cec_pdev) { -dev_err (hdmi->dev,"Waiting for CEC device %s\n" , - cec_np->full_name); +dev_err (hdmi->dev,"Waiting for CEC device %pOF\n" , + cec_np);return -EPROBE_DEFER ; } hdmi->cec_dev = &cec_pdev->dev;@@ -1501,8 +1501,8 @@ static int mtk_hdmi_dt_parse_pdata(struct mtk_hdmi *hdmi, i2c_np =of_parse_phandle (remote,"ddc-i2c-bus" ,0 );if (!i2c_np) { -dev_err (dev,"Failed to find ddc-i2c-bus node in %s\n" , - remote->full_name); +dev_err (dev,"Failed to find ddc-i2c-bus node in %pOF\n" , + remote);of_node_put (remote);return -EINVAL ; }drivers/gpu/drm/meson/meson_drv.c +2
-3@@ -303,9 +303,8 @@ static const struct component_master_ops meson_drv_master_ops = {static int compare_of (struct device *dev,void *data) { -DRM_DEBUG_DRIVER ("Comparing of node %s with %s\n" , -of_node_full_name (dev->of_node), -of_node_full_name (data)); +DRM_DEBUG_DRIVER ("Comparing of node %pOF with %pOF\n" , + dev->of_node, data);return dev->of_node == data; }drivers/gpu/drm/panel/panel-lvds.c +8
-8@@ -143,14 +143,14 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) ret =of_property_read_u32 (np,"width-mm" , &lvds->width);if (ret <0 ) { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"width-mm" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"width-mm" );return -ENODEV ; } ret =of_property_read_u32 (np,"height-mm" , &lvds->height);if (ret <0 ) { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"height-mm" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"height-mm" );return -ENODEV ; }@@ -158,8 +158,8 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) ret =of_property_read_string (np,"data-mapping" , &mapping);if (ret <0 ) { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"data-mapping" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"data-mapping" );return -ENODEV ; }@@ -170,8 +170,8 @@ static int panel_lvds_parse_dt(struct panel_lvds *lvds) }else if (!strcmp (mapping,"vesa-24" )) { lvds->bus_format =MEDIA_BUS_FMT_RGB888_1X7X4_SPWG ; }else { -dev_err (lvds->dev,"%s: invalid or missing %s DT property\n" , -of_node_full_name (np),"data-mapping" ); +dev_err (lvds->dev,"%pOF: invalid or missing %s DT property\n" , + np,"data-mapping" );return -EINVAL ; }drivers/gpu/drm/rcar-du/rcar_du_encoder.c +2
-2@@ -186,8 +186,8 @@ int rcar_du_encoder_init(struct rcar_du_device *rcdu, }if (enc_node) { -dev_dbg (rcdu->dev,"initializing encoder %s for output %u\n" , -of_node_full_name (enc_node), output); +dev_dbg (rcdu->dev,"initializing encoder %pOF for output %u\n" , + enc_node, output);/* Locate the DRM bridge from the encoder DT node. */ bridge =of_drm_find_bridge (enc_node);drivers/gpu/drm/rcar-du/rcar_du_kms.c +8
-8@@ -297,15 +297,15 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,*/ entity =of_graph_get_remote_port_parent (ep->local_node);if (!entity) { -dev_dbg (rcdu->dev,"unconnected endpoint %s, skipping\n" , - ep->local_node->full_name); +dev_dbg (rcdu->dev,"unconnected endpoint %pOF, skipping\n" , + ep->local_node);return -ENODEV ; }if (!of_device_is_available (entity)) {dev_dbg (rcdu->dev, -"connected entity %s is disabled, skipping\n" , - entity->full_name); +"connected entity %pOF is disabled, skipping\n" , + entity);return -ENODEV ; }@@ -325,8 +325,8 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu,if (!connector) {dev_warn (rcdu->dev, -"no connector for encoder %s, skipping\n" , - encoder->full_name); +"no connector for encoder %pOF, skipping\n" , + encoder);of_node_put (entity_ep_node);of_node_put (encoder);return -ENODEV ;@@ -348,8 +348,8 @@ static int rcar_du_encoders_init_one(struct rcar_du_device *rcdu, ret =rcar_du_encoder_init (rcdu, output, encoder, connector);if (ret && ret != -EPROBE_DEFER )dev_warn (rcdu->dev, -"failed to initialize encoder %s on output %u (%d), skipping\n" , -of_node_full_name (encoder), output, ret); +"failed to initialize encoder %pOF on output %u (%d), skipping\n" , + encoder, output, ret);of_node_put (encoder);of_node_put (connector);drivers/gpu/drm/rockchip/rockchip_drm_drv.c +2
-2@@ -370,8 +370,8 @@ static int rockchip_drm_platform_of_probe(struct device *dev) iommu =of_parse_phandle (port->parent,"iommus" ,0 );if (!iommu || !of_device_is_available (iommu->parent)) { -dev_dbg (dev,"no iommu attached for %s, using non-iommu buffers\n" , - port->parent->full_name); +dev_dbg (dev,"no iommu attached for %pOF, using non-iommu buffers\n" , + port->parent);/* * if there is a crtc not support iommu, force set all * crtc use non-iommu buffer. drivers/gpu/drm/rockchip/rockchip_drm_vop.c +2
-2@@ -1340,8 +1340,8 @@ static int vop_create_crtc(struct vop *vop) port =of_get_child_by_name (dev->of_node,"port" );if (!port) { -DRM_DEV_ERROR (vop->dev,"no port node found in %s\n" , - dev->of_node->full_name); +DRM_DEV_ERROR (vop->dev,"no port node found in %pOF\n" , + dev->of_node); ret = -ENOENT ; goto err_cleanup_crtc; }drivers/gpu/drm/stm/ltdc.c +1
-2@@ -995,8 +995,7 @@ static struct drm_panel *ltdc_get_panel(struct drm_device *ddev) panel =of_drm_find_panel (port);of_node_put (port);if (panel) { -DRM_DEBUG_DRIVER ("remote panel %s\n" , - port->full_name); +DRM_DEBUG_DRIVER ("remote panel %pOF\n" , port); }else {DRM_DEBUG_DRIVER ("panel missing\n" );of_node_put (entity);drivers/gpu/drm/sun4i/sun4i_drv.c +4
-5@@ -187,9 +187,9 @@ static bool sun4i_drv_node_is_tcon(struct device_node *node)static int compare_of (struct device *dev,void *data) { -DRM_DEBUG_DRIVER ("Comparing of node %s with %s\n" , -of_node_full_name (dev->of_node), -of_node_full_name (data)); +DRM_DEBUG_DRIVER ("Comparing of node %pOF with %pOF\n" , + dev->of_node, + data);return dev->of_node == data; }@@ -219,8 +219,7 @@ static int sun4i_drv_add_endpoints(struct device *dev,if (!sun4i_drv_node_is_frontend (node)) {/* Add current component */ -DRM_DEBUG_DRIVER ("Adding component %s\n" , -of_node_full_name (node)); +DRM_DEBUG_DRIVER ("Adding component %pOF\n" , node);drm_of_component_match_add (dev, match, compare_of, node); count++; }drivers/gpu/drm/tilcdc/tilcdc_crtc.c +2
-2@@ -1038,8 +1038,8 @@ int tilcdc_crtc_create(struct drm_device *dev)if (priv->is_componentized) { crtc->port =of_graph_get_port_by_id (dev->dev->of_node,0 );if (!crtc->port) {/* This should never happen */ -dev_err (dev->dev,"Port node not found in %s\n" , - dev->dev->of_node->full_name); +dev_err (dev->dev,"Port node not found in %pOF\n" , + dev->dev->of_node); ret = -EINVAL ; goto fail; }drivers/gpu/ipu-v3/ipu-common.c +2
-3@@ -1217,8 +1217,8 @@ static int ipu_add_client_devices(struct ipu_soc *ipu, unsigned long ipu_base) of_node =of_graph_get_port_by_id (dev->of_node, i);if (!of_node) {dev_info (dev, -"no port@%d node in %s, not using %s%d\n" , - i, dev->of_node->full_name, +"no port@%d node in %pOF, not using %s%d\n" , + i, dev->of_node, (i /2 ) ?"DI" :"CSI" , i %2 );continue ; } --2.11.0
Signature · 5 lines
Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel