Re: [PATCH v8 17/23] drm/mediatek: Support MT8188 Padding in display driver

From: Shawn Sung (宋孝謙)
Date: Tue Oct 17 2023 - 07:17:48 EST


Hi Angelo,

On Tue, 2023-10-17 at 11:44 +0200, AngeloGioacchino Del Regno wrote:
> +static const struct of_device_id mtk_padding_driver_dt_match[] = {
> > + { .compatible = "mediatek,mt8188-padding" },
> > + { /* sentinel */ }
> > +};
> > +MODULE_DEVICE_TABLE(of, mtk_padding_driver_dt_match);
> > +
> > +struct platform_driver mtk_padding_driver = {
> > + .probe = mtk_padding_probe,
> > + .remove = mtk_padding_remove,
> > + .driver = {
> > + .name = "mediatek-padding",
>
> Please let's be consistent with the driver names, this should be
> "mediatek-disp-padding".
>
> After changing that,
>
> Reviewed-by: AngeloGioacchino Del Regno <
> angelogioacchino.delregno@xxxxxxxxxxxxx>
>

Will change in the next version.

Thanks,
Shawn