From cc6296194d41aa1480287a7173137437630a167c Mon Sep 17 00:00:00 2001
From: Arman Uguray <armansito@chromium.org>
Date: Fri, 27 Feb 2015 20:42:51 -0800
Subject: [PATCH] TODO: Added item about PF_ALG error

Added GATT related TODO item to gracefully handle signed writes and
bt_att initialization if the CONFIG_CRYPTO_USER_API modules that are
necessary for bt_crypto are not enabled in the system.
---
 TODO | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/TODO b/TODO
index 279c0cf80..65e19ee73 100644
--- a/TODO
+++ b/TODO
@@ -189,6 +189,22 @@ ATT/GATT (new shared stack)
   Priority: Medium
   Complexity: C1
 
+- The recently added support for ATT signed writes requires the following kernel
+  modules to be enabled:
+
+     CONFIG_CRYPTO_USER_API
+     CONFIG_CRYPTO_USER_API_HASH
+     CONFIG_CRYPTO_USER_API_SKCIPHER
+
+  Currently, if these are not enabled, bt_att_new silently returns NULL. We
+  should handle this more gracefully by not supporting signed writes if we can't
+  initialize bt_crypto while succeeding bt_att initialization regardless.
+
+  This behavior should be documented in the README.
+
+  Priority: High
+  Complexity: C1
+
 
 ATT/GATT (old/outdated)
 =======================
-- 
2.47.3