Messages in this thread | ![/](/images/icornerl.gif) | | Subject | RE: [PATCH] clk: imx: Make the i.MX8MQ CCM clock driver CLK_IMX8MQ dependant | From | Stephen Boyd <> | Date | Thu, 13 Dec 2018 14:04:09 -0800 |
| |
Quoting Aisheng Dong (2018-12-13 08:51:18) > [...] > > > > --- a/drivers/clk/imx/Makefile > > > +++ b/drivers/clk/imx/Makefile > > > @@ -34,5 +34,6 @@ obj-$(CONFIG_SOC_IMX6SX) += clk-imx6sx.o > > > obj-$(CONFIG_SOC_IMX6UL) += clk-imx6ul.o > > > obj-$(CONFIG_SOC_IMX7D) += clk-imx7d.o > > > obj-$(CONFIG_SOC_IMX7ULP) += clk-imx7ulp.o > > > -obj-$(CONFIG_SOC_IMX8MQ) += clk-imx8mq.o > > > obj-$(CONFIG_SOC_VF610) += clk-vf610.o > > > + > > > +obj-$(CONFIG_CLK_IMX8MQ) += clk-imx8mq.o > > > > Nit: Do we want to keep CONFIG_ sorted? > > IMHO It might be okay to make MX8 (ARM64) a new group to start > To get a clear separation. (Slightly not sorted due to VF610) > Anyway, it leaves to Stephen to make the judge. >
This is fine to split the two if they're not related. Just have a comment indicating this.
| ![\](/images/icornerr.gif) |