On 2016å¹´08æ13æ¥ 01:00, Sean Paul wrote:
> Since we can have multiple vops, use DRM_DEV_ERROR to
> make logs easier to process.
>
> Signed-off-by: Sean Paul <seanpaul at chromium.org>
looks good for me
Acked-by: Mark Yao <mark.yao at rockchip.com>
> ---
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 24 ++++++++++++++----------
> 1 file changed, 14 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> index 31744fe..ec8ad00 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c
> @@ -238,7 +238,7 @@ static enum vop_data_format vop_convert_format(uint32_t
> format)
> case DRM_FORMAT_NV24:
> return VOP_FMT_YUV444SP;
> default:
> - DRM_ERROR("unsupport format[%08x]\n", format);
> + DRM_ERROR("unsupported format[%08x]\n", format);
> return -EINVAL;
> }
> }
> @@ -315,7 +315,7 @@ static void scl_vop_cal_scl_fac(struct vop *vop, const
> struct vop_win_data *win,
> int vskiplines = 0;
>
> if (dst_w > 3840) {
> - DRM_ERROR("Maximum destination width (3840) exceeded\n");
> + DRM_DEV_ERROR(vop->dev, "Maximum dst width (3840) exceeded\n");
> return;
> }
>
> @@ -353,11 +353,11 @@ static void scl_vop_cal_scl_fac(struct vop *vop, const
> struct vop_win_data *win,
> VOP_SCL_SET_EXT(vop, win, lb_mode, lb_mode);
> if (lb_mode == LB_RGB_3840X2) {
> if (yrgb_ver_scl_mode != SCALE_NONE) {
> - DRM_ERROR("ERROR : not allow yrgb ver scale\n");
> + DRM_DEV_ERROR(vop->dev, "not allow yrgb ver scale\n");
> return;
> }
> if (cbcr_ver_scl_mode != SCALE_NONE) {
> - DRM_ERROR("ERROR : not allow cbcr ver scale\n");
> + DRM_DEV_ERROR(vop->dev, "not allow cbcr ver scale\n");
> return;
> }
> vsu_mode = SCALE_UP_BIL;
> @@ -970,7 +970,8 @@ static void vop_crtc_enable(struct drm_crtc *crtc)
> VOP_CTRL_SET(vop, mipi_en, 1);
> break;
> default:
> - DRM_ERROR("unsupport connector_type[%d]\n", s->output_type);
> + DRM_DEV_ERROR(vop->dev, "unsupported connector_type [%d]\n",
> + s->output_type);
> }
> VOP_CTRL_SET(vop, out_mode, s->output_mode);
>
> @@ -1154,7 +1155,8 @@ static irqreturn_t vop_isr(int irq, void *data)
>
> /* Unhandled irqs are spurious. */
> if (active_irqs)
> - DRM_ERROR("Unknown VOP IRQs: %#02x\n", active_irqs);
> + DRM_DEV_ERROR(vop->dev, "Unknown VOP IRQs: %#02x\n",
> + active_irqs);
>
> return ret;
> }
> @@ -1189,7 +1191,8 @@ static int vop_create_crtc(struct vop *vop)
> win_data->phy->nformats,
> win_data->type, NULL);
> if (ret) {
> - DRM_ERROR("failed to initialize plane\n");
> + DRM_DEV_ERROR(vop->dev, "failed to init plane %d\n",
> + ret);
> goto err_cleanup_planes;
> }
>
> @@ -1227,7 +1230,8 @@ static int vop_create_crtc(struct vop *vop)
> win_data->phy->nformats,
> win_data->type, NULL);
> if (ret) {
> - DRM_ERROR("failed to initialize overlay plane\n");
> + DRM_DEV_ERROR(vop->dev, "failed to init overlay %d\n",
> + ret);
> goto err_cleanup_crtc;
> }
> drm_plane_helper_add(&vop_win->base, &plane_helper_funcs);
> @@ -1235,8 +1239,8 @@ static int vop_create_crtc(struct vop *vop)
>
> port = of_get_child_by_name(dev->of_node, "port");
> if (!port) {
> - DRM_ERROR("no port node found in %s\n",
> - dev->of_node->full_name);
> + DRM_DEV_ERROR(vop->dev, "no port node found in %s\n",
> + dev->of_node->full_name);
> ret = -ENOENT;
> goto err_cleanup_crtc;
> }
--
ï¼ark Yao
