Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <32ad354611a35a55547377005ce3a70f6edd037c.camel@fi.rohmeurope.com>
Subject: Re: [PATCH] regulator: bd70528: Convert to use
 regulator_set_ramp_delay_regmap
From:   Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Reply-To: matti.vaittinen@fi.rohmeurope.com
To:     Axel Lin <axel.lin@ingics.com>, Mark Brown <broonie@kernel.org>
Cc:     Liam Girdwood <lgirdwood@gmail.com>, linux-kernel@vger.kernel.org
In-Reply-To: <20210522124250.2121076-1-axel.lin@ingics.com>
References: <20210522124250.2121076-1-axel.lin@ingics.com>
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
Date:   Mon, 24 May 2021 08:21:09 +0300
User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) 
Content-Transfer-Encoding: 7bit
Precedence: bulk


On Sat, 2021-05-22 at 20:42 +0800, Axel Lin wrote:
> Use regulator_set_ramp_delay_regmap instead of open-coded.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---

As a side-note - yesterday I sent an email asking the fate/status of
BD70528. It may be we can drop whole BD70528 support and that's why I
have more or less neglected the updates (for example the binding docs
are still .txt) - but let's wait for a reply for a little while.
Dropping BD70528 would simplify the RTC driver used by BD71828 and
BD71815.

I won't in any ways object applying this patch though - it looks good
to me.

Reviewed-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>