Hi Simon, [...] > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) The driver is only based on GPL v2.0, should this be the same, or is there a reason for dual-license? [...] > +title: DesignWare based PCIe RC controller on Rockchip SoCs [...] What does the "RC" here stands for? Would it be "Rockchip"? If so, then perhaps dropping it would be fine to do, as rest of the title mentions "Rockchip SoCs"." What do you think? Krzysztof