From: "Andrew Jeffery" <andrew@aj.id.au>
To: "Ryan Chen" <ryan_chen@aspeedtech.com>,
"Troy Lee" <troy_lee@aspeedtech.com>,
"openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
"Joel Stanley" <joel@jms.id.au>,
"Philipp Zabel" <p.zabel@pengutronix.de>,
"open list" <linux-kernel@vger.kernel.org>,
"moderated list:ARM/ASPEED MACHINE SUPPORT"
<linux-arm-kernel@lists.infradead.org>,
"moderated list:ARM/ASPEED MACHINE SUPPORT"
<linux-aspeed@lists.ozlabs.org>
Cc: "Chia-Wei, Wang" <chiawei_wang@aspeedtech.com>,
"Troy Lee" <leetroy@gmail.com>,
"kbuild test robot" <lkp@intel.com>
Subject: Re: [PATCH v3 4/4] hwmon: Support Aspeed AST2600 PWM/Fan tachometer
Date: Wed, 20 Jan 2021 17:07:56 +1030 [thread overview]
Message-ID: <31ea8453-bbb1-4e68-8ee4-f7e524e38b88@beta.fastmail.com> (raw)
In-Reply-To: <HK0PR06MB338019E403823F0BF9962FE3F2A20@HK0PR06MB3380.apcprd06.prod.outlook.com>
On Wed, 20 Jan 2021, at 15:53, Ryan Chen wrote:
> > -----Original Message-----
> > From: Andrew Jeffery <andrew@aj.id.au>
> > Sent: Wednesday, January 20, 2021 1:16 PM
> > To: Troy Lee <troy_lee@aspeedtech.com>; openbmc@lists.ozlabs.org; Joel
> > Stanley <joel@jms.id.au>; Philipp Zabel <p.zabel@pengutronix.de>; open list
> > <linux-kernel@vger.kernel.org>; moderated list:ARM/ASPEED MACHINE
> > SUPPORT <linux-arm-kernel@lists.infradead.org>; moderated
> > list:ARM/ASPEED MACHINE SUPPORT <linux-aspeed@lists.ozlabs.org>
> > Cc: Ryan Chen <ryan_chen@aspeedtech.com>; ChiaWei Wang
> > <chiawei_wang@aspeedtech.com>; Troy Lee <leetroy@gmail.com>; kbuild
> > test robot <lkp@intel.com>
> > Subject: Re: [PATCH v3 4/4] hwmon: Support Aspeed AST2600 PWM/Fan
> > tachometer
> >
> > Hi Troy,
> >
> > On Mon, 18 Jan 2021, at 17:20, Troy Lee wrote:
> > > Add Aspeed AST2600 PWM/Fan tacho driver. AST2600 has 16 PWM channel
> > > and
> > > 16 FAN tacho channel.
> > >
> > > Changes since v2:
> > > - declare local function as static function
> > >
> > > Changes since v1:
> > > - fixed review comments
> > > - fixed double-looped calculation of div_h and div_l
> > > - moving configuration to device tree
> > > - register hwmon driver with devm_hwmon_device_register_with_info()
> > >
> > > Signed-off-by: Troy Lee <troy_lee@aspeedtech.com>
> > > Reported-by: kernel test robot <lkp@intel.com>
> > > ---
> > > drivers/hwmon/Kconfig | 10 +
> > > drivers/hwmon/Makefile | 1 +
> > > drivers/hwmon/aspeed2600-pwm-tacho.c | 756
> > > +++++++++++++++++++++++++++
> > > 3 files changed, 767 insertions(+)
> > > create mode 100644 drivers/hwmon/aspeed2600-pwm-tacho.c
> >
> > ...
> >
> > > diff --git a/drivers/hwmon/Makefile b/drivers/hwmon/Makefile index
> > > 09a86c5e1d29..1a415d493ffc 100644
> > > --- a/drivers/hwmon/Makefile
> > > +++ b/drivers/hwmon/Makefile
> > > @@ -52,6 +52,7 @@ obj-$(CONFIG_SENSORS_ARM_SCPI) +=
> > scpi-hwmon.o
> > > obj-$(CONFIG_SENSORS_AS370) += as370-hwmon.o
> > > obj-$(CONFIG_SENSORS_ASC7621) += asc7621.o
> > > obj-$(CONFIG_SENSORS_ASPEED) += aspeed-pwm-tacho.o
> > > +obj-$(CONFIG_SENSORS_ASPEED2600_PWM_TACHO) +=
> > aspeed2600-pwm-tacho.o
> >
> > Why does this have to be a separate implementation from the existing ASPEED
> > PWM/Tacho driver? Is there really nothing in common?
> >
> Hello Andrew,
> The register set is fully re-arrange. And it is new design at AST2600,
> can't be compatible with older PWM.
>
Ah, okay. I hadn't looked. Thanks Ryan.
Andrew
next prev parent reply other threads:[~2021-01-20 6:43 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-18 6:50 [PATCH v3 0/4] hwmon: aspeed2600-pwm-tacho: Add driver support Troy Lee
2021-01-18 6:50 ` [PATCH v3 1/4] dt-bindings: hwmon: Add Aspeed AST2600 PWM/Fan Troy Lee
2021-01-22 4:26 ` Andrew Jeffery
2021-01-18 6:50 ` [PATCH v3 2/4] hwmon: Add Aspeed AST2600 support Troy Lee
2021-01-18 6:50 ` [PATCH v3 3/4] ARM: dts: aspeed: Add Aspeed AST2600 PWM/Fan node in devicetree Troy Lee
2021-01-18 6:50 ` [PATCH v3 4/4] hwmon: Support Aspeed AST2600 PWM/Fan tachometer Troy Lee
2021-01-20 5:16 ` Andrew Jeffery
2021-01-20 5:23 ` Ryan Chen
2021-01-20 6:37 ` Andrew Jeffery [this message]
2021-01-22 4:38 ` Andrew Jeffery
2021-01-26 10:05 ` Troy Lee
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=31ea8453-bbb1-4e68-8ee4-f7e524e38b88@beta.fastmail.com \
--to=andrew@aj.id.au \
--cc=chiawei_wang@aspeedtech.com \
--cc=joel@jms.id.au \
--cc=leetroy@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-aspeed@lists.ozlabs.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
--cc=openbmc@lists.ozlabs.org \
--cc=p.zabel@pengutronix.de \
--cc=ryan_chen@aspeedtech.com \
--cc=troy_lee@aspeedtech.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).