Currently the driver will call WARN when a format isn't available,
resulting in a kernel backtrace in our logs. This makes it look much more
serious than it should be.

Replace the call to the WARN macro to a dev_warn, which will still allow us
to log what happened without making it too dramatic.

Signed-off-by: Maxime Ripard <maxime.rip...@bootlin.com>
---
 drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c 
b/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
index f10c3bc2a6c5..9a25aad8b6b1 100644
--- a/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
+++ b/drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c
@@ -299,7 +299,7 @@ static enum csi_output_fmt get_csi_output_format(struct 
sun6i_csi_dev *sdev,
                return buf_interlaced ? CSI_FRAME_PLANAR_YUV422 :
                                        CSI_FIELD_PLANAR_YUV422;
        default:
-               WARN(1, "Unsupported pixformat: 0x%x\n", pixformat);
+               dev_warn(sdev->dev, "Unsupported pixformat: 0x%x\n", pixformat);
                break;
        }
 
@@ -330,7 +330,8 @@ static enum csi_input_seq get_csi_input_seq(struct 
sun6i_csi_dev *sdev,
                case MEDIA_BUS_FMT_YVYU8_2X8:
                        return CSI_INPUT_SEQ_YVYU;
                default:
-                       WARN(1, "Unsupported mbus code: 0x%x\n", mbus_code);
+                       dev_warn(sdev->dev, "Unsupported mbus code: 0x%x\n",
+                                mbus_code);
                        break;
                }
                break;
@@ -351,12 +352,14 @@ static enum csi_input_seq get_csi_input_seq(struct 
sun6i_csi_dev *sdev,
                case MEDIA_BUS_FMT_YVYU8_2X8:
                        return CSI_INPUT_SEQ_YUYV;
                default:
-                       WARN(1, "Unsupported mbus code: 0x%x\n", mbus_code);
+                       dev_warn(sdev->dev, "Unsupported mbus code: 0x%x\n",
+                                mbus_code);
                        break;
                }
                break;
        default:
-               WARN(1, "Unsupported pixformat: 0x%x\n", pixformat);
+               dev_warn(sdev->dev, "Unsupported pixformat: 0x%x, defaulting to 
YUYV\n",
+                        pixformat);
                break;
        }
 
-- 
2.14.3

Reply via email to