[PATCH v12 2/3] soc/sophgo: add top sysctrl layout file for CV18XX/SG200X

Inochi Amaoto inochiama at outlook.com
Wed Aug 28 18:38:54 PDT 2024


On Wed, Aug 28, 2024 at 11:24:24PM GMT, Vinod Koul wrote:
> On 27-08-24, 14:49, Inochi Amaoto wrote:
> > The "top" system controller of CV18XX/SG200X exposes control
> > register access for various devices. Add soc header file to
> > describe it.
> 
> I dont think I am full onboard this idea, 

Feel free to share your idea. I just added this file for
convenience to access the offset of the syscon device. 
In fact, I am not sure whether it is better to use reg
offset. Using reg adds some unncessary complexity, but
can avoid use this offset file. If you prefer this way,
it is OK for me to change.

> but still need someone to ack it

I am not sure there will be someone to ack it. If this patch
is kept.

> > 
> > Signed-off-by: Inochi Amaoto <inochiama at outlook.com>
> > ---
> >  include/soc/sophgo/cv1800-sysctl.h | 30 ++++++++++++++++++++++++++++++
> 
> is soc/sophgo/ right path? why not include/soc/sifive/... (sorry dont
> know much about this here...)
> 

CV1800 is a SoC from Sophgo, it is not from SiFive.

Regards,
Inochi



More information about the linux-riscv mailing list