[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <45c7cf75876c5822f8c51f05375af507.sboyd@kernel.org>
Date: Fri, 30 Aug 2024 15:16:18 -0700
From: Stephen Boyd <sboyd@...nel.org>
To: Ryan Chen <ryan_chen@...eedtech.com>, andrew@...econstruct.com.au, conor+dt@...nel.org, devicetree@...r.kernel.org, joel@....id.au, krzk+dt@...nel.org, linux-arm-kernel@...ts.infradead.org, linux-aspeed@...ts.ozlabs.org, linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org, mturquette@...libre.com, p.zabel@...gutronix.de, robh@...nel.org
Subject: RE: [PATCH v2 3/3] clk: aspeed: add AST2700 clk driver
Quoting Ryan Chen (2024-08-30 00:50:21)
> > Subject: RE: [PATCH v2 3/3] clk: aspeed: add AST2700 clk driver
> >
> > Quoting Ryan Chen (2024-08-29 00:09:12)
> > > > Subject: Re: [PATCH v2 3/3] clk: aspeed: add AST2700 clk driver
> > > >
> > > > Quoting Ryan Chen (2024-08-27 23:27:40)
> > > > > a/drivers/clk/clk-ast2700.c b/drivers/clk/clk-ast2700.c new file
> > > > > mode
> > > > > 100644 index 000000000000..7e0466e73980
> > > > > --- /dev/null
> > > > > +++ b/drivers/clk/clk-ast2700.c
> > > > > @@ -0,0 +1,1198 @@
> > > > > +// SPDX-License-Identifier: GPL-2.0
> > > > > +/*
> > > > > + * Copyright (c) 2024 ASPEED Technology Inc.
> > > > > + * Author: Ryan Chen <ryan_chen@...eedtech.com> */
> > > > > +
> > > > > +#include <linux/bits.h>
> > > > > +#include <linux/clk-provider.h>
> > > > [...]
> > > > > +
> > > > > +struct ast2700_reset {
> > > > > + void __iomem *base;
> > > > > + struct ast2700_reset_signal const *signal;
> > > > > + struct reset_controller_dev rcdev; };
> > > >
> > > > Please move the reset controller to the drivers/reset directory by
> > > > means of using an auxiliary device. There are some existing examples
> > > > in there if you grep for auxiliary_device in drivers/reset to help guide.
> > >
> > > Do you mean to have another reset controller driver?
> > > If yes, I may need syscon for remap two drivers.
> >
> > Yes. A syscon is not necessary to do that.
> Thanks for point out auxiliary device.
> Since our SoC reset and clock share the same register region.
> Your recommend will use clk auxiliary_device_add for reset driver am I right?
Yes. Either the clk or the reset driver can be a platform driver and the
other an auxiliary driver.
Powered by blists - more mailing lists