Hi Ilya,
IF: Sorry for the delay in responding. I am not actively working on the similar DT driver at the moment as we have hit a DT maintainer roadblock for its development. I do not read this bulletin board much therefore. Great seeing a first take on the Broadcom BT UART ACPI driver! Looks like a good start to me. A few comments:
Most of Bluetooth UART chips have some extra configuration (PCM, I2S,
LPM...)
with specific vendor commands. Since it looks hard to include this
information
into DT and much more difficult into ACPI, I suggest to implement a
common way
to that with a generic btconf file. This file could be retrieved via
request_firmware
and filled with a list of HCI commands for vendor config. This
configuration should
happen after firmware download. If no conf file found, you can apply a
default
configuration or just ignore it.
It's just a proposal, I don't know if it does completely match your
requirements.
Regards,
Loic
--
Intel Open Source Technology Center
http://oss.intel.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
