All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <linux@roeck-us.net>
To: "Sa, Nuno" <Nuno.Sa@analog.com>
Cc: "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"mark.rutland@arm.com" <mark.rutland@arm.com>,
	"jdelvare@suse.com" <jdelvare@suse.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>
Subject: Re: [PATCH v2 1/2] hwmon: Support ADI Fan Control IP
Date: Wed, 5 Feb 2020 08:03:25 -0800	[thread overview]
Message-ID: <20200205160325.GA25403@roeck-us.net> (raw)
In-Reply-To: <7755456dfbaf6ae0a45baf7b1a3ed7eb9a817d4a.camel@analog.com>

On Wed, Feb 05, 2020 at 08:11:49AM +0000, Sa, Nuno wrote:
> Hi Guenter,
> 
> On Sun, 2019-10-20 at 09:12 -0700, Guenter Roeck wrote:
> > [External]
> > 
> > On Wed, Oct 09, 2019 at 12:28:05PM +0200, Nuno Sá wrote:
> > > The purpose of this IP Core is to control the fan used for the
> > > cooling of a
> > > Xilinx Zynq Ultrascale+ MPSoC without the need of any external
> > > temperature
> > > sensors. To achieve this, the IP core uses the PL SYSMONE4
> > > primitive to
> > > obtain the PL temperature and, based on those readings, it then
> > > outputs
> > > a PWM signal to control the fan rotation accordingly.
> > > 
> > > Signed-off-by: Nuno Sá <nuno.sa@analog.com>
> > 
> > Applied to hwmon-next.
> 
> Did you somehow decided not to merge this driver or did you forget
> about it? I was trying to find it mainline to sync our trees with the
> reviewed version and I could not see it there...
> 
Hmm. I have no idea what happened. I marked it as accepted,
and as mentioned above claimed that I added it to hwmon-next,
but it isn't there. Weird. Maybe I was waiting for a new version
after Rob's feedback for the bindings, but if so I should have
communicated that. My apologies. I queued it up for the next kernel
release, with the bindings change as requested by Rob
(adi,pulses-per-revolution -> pulses-per-revolution). It will show
up in hwmon-next (and linux-next) after the commit window closes.

Guenter

      reply	other threads:[~2020-02-05 16:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-09 10:28 [PATCH v2 1/2] hwmon: Support ADI Fan Control IP Nuno Sá
2019-10-09 10:28 ` [PATCH v2 2/2] dt-bindings: hwmon: Add AXI FAN Control documentation Nuno Sá
2019-10-11 16:59   ` Rob Herring
2019-10-20 16:12   ` Guenter Roeck
2019-10-20 16:12 ` [PATCH v2 1/2] hwmon: Support ADI Fan Control IP Guenter Roeck
2020-02-05  8:11   ` Sa, Nuno
2020-02-05 16:03     ` Guenter Roeck [this message]

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=20200205160325.GA25403@roeck-us.net \
    --to=linux@roeck-us.net \
    --cc=Nuno.Sa@analog.com \
    --cc=devicetree@vger.kernel.org \
    --cc=jdelvare@suse.com \
    --cc=linux-hwmon@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.