Return-Path: From: Ilya Faenson To: Marcel Holtmann CC: Loic Poulain , "linux-bluetooth@vger.kernel.org" Subject: RE: [PATCH v2] Bluetooth: hci_intel: Add support for platform driver Date: Thu, 27 Aug 2015 18:35:07 +0000 Message-ID: References: <1440652911-39717-1-git-send-email-marcel@holtmann.org> <55DECF9E.9040302@intel.com> <95579268-3455-4C1E-96E6-89C6D93A05D3@holtmann.org> <8A9FC576-0B81-4AC0-8BA5-9042A29AB088@holtmann.org> In-Reply-To: <8A9FC576-0B81-4AC0-8BA5-9042A29AB088@holtmann.org> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Marcel, -----Original Message----- From: Marcel Holtmann [mailto:marcel@holtmann.org] Sent: Thursday, August 27, 2015 2:28 PM To: Ilya Faenson Cc: Loic Poulain; linux-bluetooth@vger.kernel.org Subject: Re: [PATCH v2] Bluetooth: hci_intel: Add support for platform driver Hi Ilya, >>> A platform device can be used to provide some specific resources in >>> order to manage the controller. In this first patch we retrieve the >>> reset gpio which is used to power on/off the controller. >>> >>> The main issue is to match the current tty with the correct pdev. >>> In case of ACPI, we can easily find the right tty/pdev pair because >>> they are both child of the same UART port. >>> >>> If controller is powered-on from the driver, we need to wait for a >>> HCI boot event before being able to send any command. >>> >>> Signed-off-by: Loic Poulain >>> Signed-off-by: Marcel Holtmann >>> --- >>> drivers/bluetooth/hci_intel.c | 203 +++++++++++++++++++++++++++++++++++++++--- >>> 1 file changed, 190 insertions(+), 13 deletions(-) >> >> ACK, It Works on my hardware. > > okay then, this patch has been applied to bluetooth-next now. > > IF: Just out of curiosity... How come you've accepted the Intel platform device implementation yourself while you've previously referred the similar Broadcom implementation to the Device Tree folks? these are ACPI devices where we have a clear tree of parent devices. I accepted also the Broadcom version for ACPI. Within ACPI the relationship between TTY and its GPIO etc. is expressed. That is the part that is missing within DT. And the DT folks should be signing off on that. In that case DT is defining the standard on how to express these relations. For ACPI based devices this is already expressed in shipping devices. IF: Appreciate the clarifications. Last time I looked, Loic's code has supported both ACPI and DT though. So just the ACPI part has been applied to the bluetooth-next? Regards Marcel