Re: [PATCH v3] drivers/hid: Check result of debugfs_create_dir() and debugfs_create_file()
Yes, sorry, the CONFIG_DEBUG_FS was 'n' in that case that time. Will resend 2015-09-04 19:17 GMT+06:00 Jiri Kosina: > On Thu, 20 Aug 2015, Alexander Kuleshov wrote: > >> The debugfs_create_dir() and debugfs_create_file() functions may return >> -errno >> if an error occurs. This patch adds a couple of checks of the result of the >> debufs_create_dir() and debugfs_create_file() functions execution in the >> hid_debug_register() and othre places. >> >> Changelog: >> >> v3: >> >> * do not check hid_debug before the call of the hid_debug_init() >> and hid_debug_exit() from v2 >> >> v2: >> >> * add check for the result of the debugfs_create_file() calls >> * call hid_debug_init() and hid_debug_exit() only if hid_debug >> * add check for the hid_debug_root in the hid_debug_register() >> >> Signed-off-by: Alexander Kuleshov > > Did you try to even compile-test your patch? > > -- > Jiri Kosina > SUSE Labs > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3] drivers/hid: Check result of debugfs_create_dir() and debugfs_create_file()
On Thu, 20 Aug 2015, Alexander Kuleshov wrote: > The debugfs_create_dir() and debugfs_create_file() functions may return -errno > if an error occurs. This patch adds a couple of checks of the result of the > debufs_create_dir() and debugfs_create_file() functions execution in the > hid_debug_register() and othre places. > > Changelog: > > v3: > > * do not check hid_debug before the call of the hid_debug_init() > and hid_debug_exit() from v2 > > v2: > > * add check for the result of the debugfs_create_file() calls > * call hid_debug_init() and hid_debug_exit() only if hid_debug > * add check for the hid_debug_root in the hid_debug_register() > > Signed-off-by: Alexander KuleshovDid you try to even compile-test your patch? -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html