chengkai15 commented on PR #20686:
URL: https://github.com/apache/nuttx/pull/20686#issuecomment-6489802107

   hi anchao
   
   Thanks, good to see we are aligned on the direction.
   
   On open point 2, I think the difference is mainly about the starting point,
   not the end state. We agree the stack should end up reviewed, built and
   tested as part of the NuttX repositories. Step 3 of the plan already allows
   moving the userspace part into nuttx-apps.git.
   
   Two things I would like to understand better:
   
   1. By "NuttX-style public headers" and "native integration", do you mean
      reformatting the Zephyr-derived code to nxstyle, or only the NuttX glue
      (Kconfig, build, driver binding, public API headers)? We are fine with the
      latter. The former makes picking up upstream fixes much harder, which is
      our main concern.
   2. NimBLE is already integrated this way (download at a pinned revision).
      Could you share which specific dependency or maintenance issues you are
      worried about? That would help us address them in the design.
   
   Since this is a project-level decision, I would like to hear from the PMC
   and everyone who has reviewed PR #20686 -- Xiang Xiao, Alan Carvalho de
   Assis, Alin Jerpelea, raiden00pl, Tomasz CEDRO (cederom), LingaoM and
   casaroli -- on whether to start with a pinned external repository or with
   in-tree code. We will follow the community's decision.
   
   Best regards,
   Kai Cheng
   
   
   ________________________________
   From: archer ***@***.***>
   Sent: Thursday, October 29, 2026 12:19 PM
   To: apache/nuttx
   Cc: 成凯; Mention
   Subject: [External Mail]Re: [apache/nuttx] net/bt: Add native Bluetooth host 
stack (PR #20686)
   
   
   [外部邮件] ***@***.***进行反馈
   
   [https://avatars.githubusercontent.com/u/758493?s=20&v=4]anchao left a 
comment 
(apache/nuttx#20686)<https://github.com/apache/nuttx/pull/20686#issuecomment-6487811997>
   
   @chengkai15<https://github.com/chengkai15>
   
   Thanks for the clarification. I think we are mostly aligned on the long-term 
direction, especially about converging to one Zephyr-based Bluetooth host and 
moving toward a BlueZ/Linux-style split for protected/kernel builds.
   
   For open point 2, I still prefer not to start from an external repository 
model in the main NuttX integration path.
   
   My suggestion is that the code should first be adapted into a NuttX-native 
integration style, and then submitted directly to either nuttx.git or 
nuttx-apps.git, depending on the agreed layering. Fetching a large external 
repository at build time will also bring dependency and maintenance issues with 
other third-party code, and I do not think that is the best long-term 
integration model for an official NuttX Bluetooth stack.
   
   In my view, if the stack is going to become part of the NuttX Bluetooth 
architecture, it should be reviewed, built, tested, and maintained as part of 
the NuttX repositories, not as an external component that is only pinned by 
revision.
   
   Maybe you can use the current PR as a reference for the NuttX-native 
integration structure: top-level CONFIG_ZBLUE_ENABLED, NuttX build/Kconfig 
integration, NuttX-style public headers, and direct integration with the NuttX 
Bluetooth driver path. The implementation details can certainly be changed, but 
I think this kind of native integration is a better direction than relying on 
an external fetched repository.
   
   If the openvela version is adjusted in this direction and accepted by the 
community as the common upstream solution, I am willing to drop my current PR 
and help with that effort. My goal is not to keep my own patch, but to help 
NuttX get a complete and maintainable Bluetooth stack.
   
   ―
   Reply to this email directly, view it on 
GitHub<https://github.com/apache/nuttx/pull/20686?email_source=notifications&email_token=ABDILOITAS5PRPRBASVZXOD5WLAN7A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTMNBYG44DCMJZHE32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-6487811997>,
 or 
unsubscribe<https://github.com/notifications/unsubscribe-auth/ABDILONP3W2NPDI7EX4JEGT5WLAN7AVCNFSNUABFKJSXA33TNF2G64TZHMZDEOBRGAZTENZTHNEXG43VMU5TKOBWG4ZTIMBYG4Z2C5QC>.
   You are receiving this because you were mentioned.Message ID: ***@***.***>
   
   
#/******本邮件及其附件含有小米公司的保密信息,仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用(包括但不限于全部或部分地泄露、复制、或散发)本邮件中的信息。如果您错收了本邮件,请您立即电话或邮件通知发件人并删除本邮件!
 This e-mail and its attachments contain confidential information from XIAOMI, 
which is intended only for the person or entity whose address is listed above. 
Any use of the information contained herein in any way (including, but not 
limited to, total or partial disclosure, reproduction, or dissemination) by 
persons other than the intended recipient(s) is prohibited. If you receive this 
e-mail in error, please notify the sender by phone or email immediately and 
delete it!******/#
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to