On 02-12-21, 05:57, Hongxing Zhu wrote: > > > +static struct platform_driver imx8_pcie_phy_driver = { > > > + .probe = imx8_pcie_phy_probe, > > > + .driver = { > > > + .name = "imx8-pcie-phy", > > > + .of_match_table = imx8_pcie_phy_of_match, > > > + } > > > +}; > > > +module_platform_driver(imx8_pcie_phy_driver); > > > + > > > +MODULE_DESCRIPTION("FSL IMX8 PCIE PHY driver"); > > > +MODULE_LICENSE("GPL"); > > > > This does not match the SPDX tag you have given > [Richard Zhu] Should the "GPL v2" correct one? > If yes, I would change it this way later. That would match spdx tag -- ~Vinod