Resent, due to wrong sender address. Also cut out unneeded quote lines. Hi
Does this really belong into meta-freescale? It looks like it is needed because some NXP boards use these chips. However they are not specific to NXP SoCs. And see inline comments below, I start them with 'MAX:': Regards Max Krummenacher Am Montag, den 30.07.2018, 14:44 -0500 schrieb Tom Hochstein: > Signed-off-by: Tom Hochstein <[email protected]> --- recipes- > connectivity/bluez5/bluez5_5.%.bbappend | 14 + ...th-Add-bluetooth-support-for-QCA6174- > chip.patch | 2404 ++++++++++++++++++++ ...tach-set-flag-to-enable-HCI-reset-on-init.patch | 32 + ...nstead-of-strlcpy-with-strncpy-to-avoid-r.patch | 35 + 4 files changed, 2485 insertions(+) c > reate mode 100644 recipes-connectivity/bluez5/bluez5_5.%.bbappend create mode 100644 recipes- > connectivity/bluez5/files/0001-bluetooth-Add-bluetooth-support-for- QCA6174-chip.patch create mode > 100644 > recipes-connectivity/bluez5/files/0002-hciattach-set-flag-to-enable-HCI-reset- on-init.patch create mode 100644 recipes-connectivity/bluez5/files/0003-hciattach-instead-of-strlcpy-with- strnc > py-to-avoid-r.patch diff --git a/recipes-connectivity/bluez5/bluez5_5.%.bbappend b/recipes- connectivity/bluez5/bluez5_5.%.bbappend new file mode 100644 index 0000000..00eb248 --- /dev/null +++ b/recipes-connectivity/bluez5/bluez5_5.%.bbappend @@ -0,0 +1,14 @@ +FILESEXTRAPATHS_prepend := "${THISDIR}/files:" + +# Do not start the service during system boot up +INITSCRIPT_PARAMS_${PN} = "stop 20 0 1 6 ." Max: That one would merit some text in the commit log. This changes behavior for all users of bluez5. But it looks that the line gets removed in the follow up patch so probably it can be deleted for good? > + +# Add patch for module bcm43xx +# Add patches for QCA modules with Qca6174 and Qca9377-3 chips +SR > C_URI += " \ + file://0001-bluetooth-Add-bluetooth-support-for-QCA6174-chip.patch \ + > file://0002-hciattach-set-flag-to-enable-HCI-reset-on-init.patch \ + file://0003 > -hciattach-instead-of-strlcpy-with-strncpy-to-avoid-r.patch \ + file://0004-Add-support- > for-Tufello-1.1-SOC.patch \ + file://0005-bluetooth-Add-support-for-multi-baud- > rate.patch \ Max: Patch 0004 and 0005 are only added with a follow up patch. IMHO one could squash the QCAxxxx and Tufello patches, if that is not desired then only add 0004 and 0005 to SRC_URI in the patch which actually does also add the corresponding files. > +" diff --git a/recipes-connectivity/bluez5/files/0001-bluetooth-Add-bluetooth-support-for-QCA6174- chip.patch b/recipes-connectivity/bluez5/files/0001-bluetooth-Add-bluetooth-support-for-QCA6174- ch > ip.patch new file mode 100644 index 0000000..5ae3c98 --- /dev/null +++ b/recipes- > connectivity/bluez5/files/0001-bluetooth-Add-bluetooth-support-for-QCA6174- chip.patch @@ -0,0 > +1,2404 @@ +From d509b84f237c11874087d7ea527e5ba2f460ed2c Mon Sep 17 00:00:00 2001 +From: Fugang > Duan <[email protected]> +Date: Tue, 29 Aug 2017 10:12:57 +0800 +Subject: [PATCH 1/5] bluetooth : > Add bluetooth support for QCA6174 chip. + +Register the QCA6174 initialization routine with > hciattach for +downloading firmware patches to the bluetooth controller. +Add optional support 'f' > to control installation of line +discipline driver. Invoke hciattach from command line and +download > the firmware patches: + hciattach /dev/ttyHS0 qca 3000000 -t120 flow -f0 + +cherry-pick and > merged from: + https://source.codeaurora.org/quic/la/platform/external/bluetooth/bluez + branch: LNX.LE.5.3 + commit: commit bb96f3b759e0b99db70014302ca12929fb42f554 Max: I think a proper Upstream-Status: (with upstream meaning bluez.org) is missing. > + +Change-Id: I87f2927d7096904071a02d73d3afef0dc34db414 +Signed-off-by: Rupesh Tatiya <rtatiya@codea > urora.org> +Signed-off-by: Fugang Duan <[email protected]> +--- + Makefile.tools | 3 +- + > tools/hciattach.c | 26 +- + tools/hciattach.h | 2 + + tools/hciattach_rome.c | 1864 > ++++++++++++++++++++++++++++++++++++++++++++++++ + tools/hciattach_rome.h | 371 ++++++++++ + 5 > files changed, 2264 insertions(+), 2 deletions(-) + diff --git a/recipes-connectivity/bluez5/files/0002-hciattach-set-flag-to-enable-HCI-reset-on- init.patch b/recipes-connectivity/bluez5/files/0002-hciattach-set-flag-to-enable-HCI-reset-on- init.patch new file mode 100644 index 0000000..a9ce17d --- /dev/null +++ b/recipes-connectivity/bluez5/files/0002-hciattach-set-flag-to-enable-HCI-reset-on-init.patch @@ -0,0 +1,32 @@ +From 3648d3c33b1dd0e11707328d87101d8044d23302 Mon Sep 17 00:00:00 2001 +From: Fugang Duan <[email protected]> +Date: Tue, 29 Aug 2017 10:21:09 +0800 +Subject: [PATCH 2/5] hciattach: set flag to enable HCI reset on init + +On some qca devices, correct setup of BT+WLAN co-existance requires +HCI reset being set at the start of power on sequence. Sending HCI +resets has no side effect. + Max: Upstream-Status is missing. > +Change-Id: I71cb367d10d4d19d82b41af6a4a0b8b2f770f691 +Signed-off-by: Rupesh Tatiya <rtatiya@codeau > rora.org> +--- > connectivity/bluez5/files/0003-hciattach-instead-of-strlcpy-with-strncpy-to-avoid- r.patch @@ -0,0 > +1,35 @@ +From 5877685b10560c40f1c4af375e30adf2a631fefc Mon Sep 17 00:00:00 2001 +From: Fugang Duan > <[email protected]> +Date: Tue, 29 Aug 2017 14:54:44 +0800 +Subject: [PATCH 3/5] hciattach: > instead of strlcpy with strncpy to avoid rome + build error + +Instead of strlcpy with strncpy to > avoid rome build error. + +igned-off-by: Fugang Duan <[email protected]> Max: Typo, missing 'S' Upstream-Status is missing. > +--- + tools/hciattach_rome.c | 4 ++-- + 1 file changed, 2 insertions(+), 2 deletions(-) + +diff --git > a/tools/hciattach_rome.c b/tools/hciattach_rome.c +index 242a49f..c3e9fde 100644 -- _______________________________________________ meta-freescale mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-freescale
