On Tue, 2014-04-01 at 02:28 -0500, Jin Zhengxiong-R64188 wrote: > > > + if (!diu_ops.set_pixel_clock) { > > > + data->pixelclk_reg = of_iomap(np, 1); > > > + if (!data->pixelclk_reg) { > > > + dev_err(&pdev->dev, "Cannot map pixelclk registers, > > please \ > > > + provide the diu_ops for pixclk setting > > instead.\n"); > > > > The error message should be in one line if possible, or it will hard to > > grep. > > If cannot, should split it into two or more lines, like: > > + dev_err(&pdev->dev, "Cannot map pixelclk registers,\n" > > + "please provide the diu_ops for pixclk setting > > instead.\n"); > Thanks, This has been fixed in the update version, please help to review it > at: > http://patchwork.ozlabs.org/patch/335225/ > I forgot to add the V2 information in the subject in the update patch so this > may confuse the reviewer, sorry for that.
It is not fixed in that patch (or did you link the wrong version?). You should never use \ to continue a line in C, other than in macros. Further, it is not permitted to wrap kernel output strings. This is an exception to the 80-column rule. Checkpatch should have told you this. -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev