Re: [PATCH v6 00/16] Bluetooth: Multi-advertising infrastructure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Florian,

On Wed, May 27, 2015, Florian Grandel wrote:
> >Are multiple instances suppose to work with this patch set?
> 
> No. There are a few remaining TODOs (documented in the code) for actual
> multi-advertising. I mention these in the commit messages where applicable.
> 
> The idea was to do the more complex changes in a functionally neutral
> refactoring first as a safe starting point for functional changes (see my
> conv. with Arman on the list).
> 
> Of course I'm prepared to resolve those remaining TODOs in a second patch
> set. I just think it is a good idea if I go full cycle once to avoid those
> beginner's errors during the next iteration.
> 
> If you think that there are no more fundamental flaws in that first patch
> set then I can start with the second one now. What do you think?

I was also under the impression that this set implemented the full
multi-advertising feature (rather than being refactoring in preparation
for it). I think a kernel release should contain the whole thing or
nothing at all, which makes me now a bit doubtful of whether this can
make it to 4.2.

How soon can you send the follow-up set? It will essentially validate
the work of your first set, so it'd be good to see and test it as soon
as possible.

Johan
--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux