Hi Paul, Thank you for your review. > -----Original Message----- > From: Paul Bolle [mailto:pebolle@xxxxxxxxxx] > Sent: Saturday, July 11, 2015 9:16 PM > To: Wang Jianwei-B52261 > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux- > arm-kernel@xxxxxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > airlied@xxxxxxxx; daniel.vetter@xxxxxxxx; mark.yao@xxxxxxxxxxxxxx; Wood > Scott-B07421; thierry.reding@xxxxxxxxx; Wang Huan-B18965; Xiubo Li > Subject: Re: [PATCH v7 1/4] drm/layerscape: Add Freescale DCU DRM driver > > A question and a nit follow. > > On vr, 2015-07-10 at 19:17 +0800, Jianwei Wang wrote: > > --- /dev/null > > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c > > > +MODULE_ALIAS("platform:fsl-dcu-drm"); > > Question: this appears to be only useful if there's a corresponding struct > platform_device. That is, a platform_device with a "fsl-dcu-drm" > .name. It will fire off a "MODALIAS=platform:fsl-dcu-drm" uevent when it's > created. > > I couldn't find this corresponding platform_device. Does it exist? Or is > this alias needed for some other reason? > > > --- /dev/null > > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h > > > +#define DRIVER_NAME "fsl-dcu-drm" > > Nit: I don't think DRIVER_NAME is actually used anywhere. > Yes, you are right. Both are useless, I'll remove them. I copied source code from other DRM driver at first, and forgot to remove them. Thanks for your kindly remind. BR. Jianwei > Thanks, > > > Paul Bolle _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
