Re: Re: [PATCH v7 2/2] clock: eswin: Add eic7700 clock driver

["Xuyang Dong" <dongxuyang@eswincomputing.com>]

> > diff --git a/drivers/clk/eswin/Kconfig b/drivers/clk/eswin/Kconfig
> > new file mode 100644
> > index 000000000000..9bc9008d3156
> > --- /dev/null
> > +++ b/drivers/clk/eswin/Kconfig
> > @@ -0,0 +1,15 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +
> > +config COMMON_CLK_ESWIN
> > +	bool
> > +
> > +config COMMON_CLK_EIC7700
> > +	bool "EIC7700 Clock Driver"
> We can't build this driver as module?
> If no, see here: https://lore.kernel.org/all/Z%2FlN7qrISpbK6MQY@goko-w.tailce6c49.ts.net/
 
OK

> > +	depends on ARCH_ESWIN || COMPILE_TEST
> > +	select COMMON_CLK_ESWIN
> > +	default ARCH_ESWIN
> > +	help
> > +	  This driver provides support for clock controller on ESWIN EIC7700
> > +	  SoC. The clock controller can generates and supplies clock to various
> > +	  peripherals within the SoC.
> > +	  Say yes here to support the clock controller on the EIC7700 SoC.
> [...]
> > diff --git a/drivers/clk/eswin/clk-eic7700.c b/drivers/clk/eswin/clk-eic7700.c
> Can you give us a clock-tree img or pdf?
> 
>                         - Troy

Hi Troy,

The link [1] provides the official documentation for the EIC7700. 
Section 3.2 covers the clock subsystem, where you can find detailed 
information about the clock-tree.

[1] https://www.sifive.com/document-file/eic7700x-datasheet

Regards,
Xuyang Dong