[PATCH v2 4/4] fbdev: imxfb: remove unneeded lable
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Mon Jul 10 12:10:35 PDT 2023
On Mon, Jul 10, 2023 at 09:20:01PM +0800, Yangtao Li wrote:
> These lables are redundant and don't do anything, let's remove it.
s/lable/label/ here an in the subject line.
Are you using this driver, or did you just stumble over it while looking
for some janitorial todo? If the former, I suggest you look into the drm
driver (imx-lcdc) instead.
Having said that, I'm not sure this cleanup is really valuable. While
a single jump target position has several names,
ret = imxfb_of_read_mode(&pdev->dev, display_np, fbi->mode);
of_node_put(display_np);
if (ret)
goto failed_of_parse;
is more obvious correct than
ret = imxfb_of_read_mode(&pdev->dev, display_np, fbi->mode);
of_node_put(display_np);
if (ret)
goto failed_init;
. *shrug*
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20230710/6d36e396/attachment.sig>
More information about the linux-arm-kernel
mailing list