Hi Zheng, On Wed, Jan 6, 2021 at 2:23 PM Zheng Yongjun <zhengyongjun3@xxxxxxxxxx> wrote: > Use resource_size rather than a verbose computation on > the end and start fields. > > Signed-off-by: Zheng Yongjun <zhengyongjun3@xxxxxxxxxx> Thanks for your patch! But the one-line summary and patch description don't seem to match with the actual change? > --- a/drivers/video/fbdev/pxa3xx-gcu.c > +++ b/drivers/video/fbdev/pxa3xx-gcu.c > @@ -594,8 +594,8 @@ static int pxa3xx_gcu_probe(struct platform_device *pdev) > * container_of(). This isn't really necessary as we have a fixed minor > * number anyway, but this is to avoid statics. */ > > - priv->misc_dev.minor = PXA3XX_GCU_MINOR, > - priv->misc_dev.name = DRV_NAME, > + priv->misc_dev.minor = PXA3XX_GCU_MINOR; > + priv->misc_dev.name = DRV_NAME; > priv->misc_dev.fops = &pxa3xx_gcu_miscdev_fops; > > /* handle IO resources */ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel