On Thu, Sep 05, 2024 at 08:57:42PM +0530, Manivannan Sadhasivam wrote: > On Wed, Sep 04, 2024 at 11:39:09AM +0200, Johan Hovold wrote: > > > diff --git a/drivers/pci/controller/dwc/pcie-qcom-common.h b/drivers/pci/controller/dwc/pcie-qcom-common.h > > > new file mode 100644 > > > index 000000000000..259e04b7bdf9 > > > --- /dev/null > > > +++ b/drivers/pci/controller/dwc/pcie-qcom-common.h > > > @@ -0,0 +1,8 @@ > > > +/* SPDX-License-Identifier: GPL-2.0 */ > > > +/* > > > + * Copyright (c) 2024 Qualcomm Innovation Center, Inc. All rights reserved. > > > + */ > > > + > > > +#include "pcie-designware.h" > > > > You only need a forward declaration: > > > > struct dw_pcie; > > > > > + > > > +void qcom_pcie_common_set_16gt_eq_settings(struct dw_pcie *pci); > > > > Compile guard still missing. Sorry, I meant to say *include* guard here. > Perhaps we can just get rid of the Kconfig entry and build it by default for > both RC and EP drivers? I don't see a value in building it as a separate module. > And we may also move more common code in the future. It is already built by default for both drivers. I'm not sure what you're suggesting here. Johan