Content-Type: text/plain;
        charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
Subject: Re: [Patch v1] btusb: Add shutdown routine for BTUSB_INTEL_NEW
 devices
From:   Marcel Holtmann <marcel@holtmann.org>
In-Reply-To: <1548764688-5429-1-git-send-email-raghuram.hegde@intel.com>
Date:   Tue, 29 Jan 2019 16:17:26 +0100
Cc:     linux-bluetooth@vger.kernel.org,
        Chethan T N <chethan.tumkur.narayan@intel.com>
Content-Transfer-Encoding: 7bit
Message-Id: <C15CC489-9116-4FAB-868E-17DE935345AC@holtmann.org>
References: <1548764688-5429-1-git-send-email-raghuram.hegde@intel.com>
To:     Raghuram Hegde <raghuram.hegde@intel.com>
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Raghuram,

> Observation:
> If BT operations (Eg:BREDR inquiry/LE scan) were triggered
> through the stack, followed by BT turn off through
> 'hciconfig hci0 down', the controller would still be active
> and consume power.
> Also, there is a possibility that a race condition/
> synchronization issue might arise on the subsequent BT turn
> on, as the controller might try to push the
> events that were queued up before processing the HCI Reset
> command.
> 
> btusb_shutdown_intel_new routine shall reset the controller
> and stop all BT operation.
> Advantages:
> 	1. Power save on the platform
> 	2. Host and controller will be in Sync.
> 
> Signed-off-by: Raghuram Hegde <raghuram.hegde@intel.com>
> Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com>
> ---
> drivers/bluetooth/btusb.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)

patch has been applied to bluetooth-next tree.

Regards

Marcel