Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20191018052405.3693555-1-bjorn.andersson@linaro.org> <739222E4-F173-42A9-8D67-1BD8FEE227EC@holtmann.org>
In-Reply-To: <739222E4-F173-42A9-8D67-1BD8FEE227EC@holtmann.org>
From:   Jeffrey Hugo <jeffrey.l.hugo@gmail.com>
Date:   Fri, 18 Oct 2019 08:52:43 -0600
Message-ID: <CAOCk7NpHVFBiWTh0Wk2fPAJDQxg8e36vGSP5mXRUUuOf8a57CA@mail.gmail.com>
Subject: Re: [PATCH 0/4] Bluetooth: hci_qca: Regulator usage cleanup
To:     Marcel Holtmann <marcel@holtmann.org>
Cc:     Bjorn Andersson <bjorn.andersson@linaro.org>,
        Johan Hedberg <johan.hedberg@gmail.com>,
        linux-bluetooth@vger.kernel.org,
        lkml <linux-kernel@vger.kernel.org>,
        MSM <linux-arm-msm@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

On Fri, Oct 18, 2019 at 1:59 AM Marcel Holtmann <marcel@holtmann.org> wrote:
>
> Hi Bjorn,
>
> > Clean up the regulator usage in hci_qca and in particular don't
> > regulator_set_voltage() for fixed voltages. It cleans up the driver, but more
> > important it makes bluetooth work on my Lenovo Yoga C630, where the regulator
> > for vddch0 is defined with a voltage range that doesn't overlap the values in
> > the driver.
> >
> > Bjorn Andersson (4):
> >  Bluetooth: hci_qca: Update regulator_set_load() usage
> >  Bluetooth: hci_qca: Don't vote for specific voltage
> >  Bluetooth: hci_qca: Use regulator bulk enable/disable
> >  Bluetooth: hci_qca: Split qca_power_setup()
> >
> > drivers/bluetooth/hci_qca.c | 135 +++++++++++++++---------------------
> > 1 file changed, 55 insertions(+), 80 deletions(-)
>
> all 4 patches have been applied to bluetooth-next tree.

I know this is already applied, but I just wanted to follow up and indicate:

Tested-by: Jeffrey Hugo <jeffrey.l.hugo@gmail.com>
Reviewed-by: Jeffrey Hugo <jeffrey.l.hugo@gmail.com>

Thanks for the cleanups Bjorn.