Hi Johan, On Tue, Dec 27, 2011 at 3:45 PM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Hi Hemant, > > On Mon, Dec 26, 2011, Hemant Gupta wrote: >> This patch adds suppport for KeyboardDisplay IO capability in BlueZ. >> --- >> src/adapter.c | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/src/adapter.c b/src/adapter.c >> index 2d9e368..3bdf525 100644 >> --- a/src/adapter.c >> +++ b/src/adapter.c >> @@ -74,6 +74,7 @@ >> #define IO_CAPABILITY_DISPLAYYESNO 0x01 >> #define IO_CAPABILITY_KEYBOARDONLY 0x02 >> #define IO_CAPABILITY_NOINPUTNOOUTPUT 0x03 >> +#define IO_CAPABILITY_KEYBOARDDISPLAY 0x04 >> #define IO_CAPABILITY_INVALID 0xFF >> >> #define check_address(address) bachk(address) >> @@ -1634,6 +1635,8 @@ static uint8_t parse_io_capability(const char *capability) >> return IO_CAPABILITY_KEYBOARDONLY; >> if (g_str_equal(capability, "NoInputNoOutput")) >> return IO_CAPABILITY_NOINPUTNOOUTPUT; >> + if (g_str_equal(capability, "KeyboardDisplay")) >> + return IO_CAPABILITY_KEYBOARDDISPLAY; >> return IO_CAPABILITY_INVALID; >> } > > We will also need a patch to hciops to ensure that this is only used for > SMP whereas for SSP KEYBOARDDISPLAY needs to be mapped to DISPLAYYESNO. > Are you working on that? The same will also be needed for the mgmt code > in the kernel, but I can do that part unless someone beats me to it. > Thanks for pointing this out. At the moment I had not worked on that part but if no one else is working on it I will submit the patches for same as well soon (most probably early next week). > Johan -- Best Regards Hemant Gupta ST-Ericsson India -- 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