Quoting Bryan O'Donoghue (2023-10-23 08:38:20) > diff --git a/drivers/clk/qcom/camcc-sc8280xp.c b/drivers/clk/qcom/camcc-sc8280xp.c > new file mode 100644 > index 000000000000..0631bd5fad57 > --- /dev/null > +++ b/drivers/clk/qcom/camcc-sc8280xp.c > @@ -0,0 +1,3048 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * Copyright (c) 2021, The Linux Foundation. All rights reserved. > + * Copyright (c) 2023, Linaro Ltd. > + */ > + > +#include <linux/clk-provider.h> > +#include <linux/err.h> Is this include used? > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/of_device.h> Should be mod_devicetable.h instead? > +#include <linux/of.h> Is this include used? > +#include <linux/pm_clock.h> > +#include <linux/pm_runtime.h> > +#include <linux/regmap.h> > +#include <linux/pm_runtime.h> This got included twice. > + > +#include <dt-bindings/clock/qcom,sc8280xp-camcc.h> > + > +#include "clk-alpha-pll.h"