Commit 290b799c authored by Dmitry Torokhov's avatar Dmitry Torokhov

Input: psmouse - use IS_ENABLED instead of homegrown code

Instead of having various protocols provide <protocol>_supported()
functions, let's use IS_ENABLED() macro that works well in "if" statements.
Acked-by: default avatarHans de Goede <hdegoede@redhat.com>
Reviewed-by: default avatarBenjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent 97ae2b5c
...@@ -710,8 +710,3 @@ int cypress_init(struct psmouse *psmouse) ...@@ -710,8 +710,3 @@ int cypress_init(struct psmouse *psmouse)
return -1; return -1;
} }
bool cypress_supported(void)
{
return true;
}
...@@ -172,7 +172,6 @@ struct cytp_data { ...@@ -172,7 +172,6 @@ struct cytp_data {
#ifdef CONFIG_MOUSE_PS2_CYPRESS #ifdef CONFIG_MOUSE_PS2_CYPRESS
int cypress_detect(struct psmouse *psmouse, bool set_properties); int cypress_detect(struct psmouse *psmouse, bool set_properties);
int cypress_init(struct psmouse *psmouse); int cypress_init(struct psmouse *psmouse);
bool cypress_supported(void);
#else #else
inline int cypress_detect(struct psmouse *psmouse, bool set_properties) inline int cypress_detect(struct psmouse *psmouse, bool set_properties)
{ {
...@@ -182,10 +181,6 @@ inline int cypress_init(struct psmouse *psmouse) ...@@ -182,10 +181,6 @@ inline int cypress_init(struct psmouse *psmouse)
{ {
return -ENOSYS; return -ENOSYS;
} }
inline bool cypress_supported(void)
{
return 0;
}
#endif /* CONFIG_MOUSE_PS2_CYPRESS */ #endif /* CONFIG_MOUSE_PS2_CYPRESS */
#endif /* _CYPRESS_PS2_H */ #endif /* _CYPRESS_PS2_H */
...@@ -424,11 +424,6 @@ int focaltech_init(struct psmouse *psmouse) ...@@ -424,11 +424,6 @@ int focaltech_init(struct psmouse *psmouse)
return error; return error;
} }
bool focaltech_supported(void)
{
return true;
}
#else /* CONFIG_MOUSE_PS2_FOCALTECH */ #else /* CONFIG_MOUSE_PS2_FOCALTECH */
int focaltech_init(struct psmouse *psmouse) int focaltech_init(struct psmouse *psmouse)
...@@ -438,9 +433,4 @@ int focaltech_init(struct psmouse *psmouse) ...@@ -438,9 +433,4 @@ int focaltech_init(struct psmouse *psmouse)
return 0; return 0;
} }
bool focaltech_supported(void)
{
return false;
}
#endif /* CONFIG_MOUSE_PS2_FOCALTECH */ #endif /* CONFIG_MOUSE_PS2_FOCALTECH */
...@@ -19,6 +19,5 @@ ...@@ -19,6 +19,5 @@
int focaltech_detect(struct psmouse *psmouse, bool set_properties); int focaltech_detect(struct psmouse *psmouse, bool set_properties);
int focaltech_init(struct psmouse *psmouse); int focaltech_init(struct psmouse *psmouse);
bool focaltech_supported(void);
#endif #endif
...@@ -727,7 +727,7 @@ static int psmouse_extensions(struct psmouse *psmouse, ...@@ -727,7 +727,7 @@ static int psmouse_extensions(struct psmouse *psmouse,
if (psmouse_do_detect(focaltech_detect, psmouse, set_properties) == 0) { if (psmouse_do_detect(focaltech_detect, psmouse, set_properties) == 0) {
if (max_proto > PSMOUSE_IMEX) { if (max_proto > PSMOUSE_IMEX) {
if (!set_properties || focaltech_init(psmouse) == 0) { if (!set_properties || focaltech_init(psmouse) == 0) {
if (focaltech_supported()) if (IS_ENABLED(CONFIG_MOUSE_PS2_FOCALTECH))
return PSMOUSE_FOCALTECH; return PSMOUSE_FOCALTECH;
/* /*
* Note that we need to also restrict * Note that we need to also restrict
...@@ -776,7 +776,7 @@ static int psmouse_extensions(struct psmouse *psmouse, ...@@ -776,7 +776,7 @@ static int psmouse_extensions(struct psmouse *psmouse,
* Try activating protocol, but check if support is enabled first, since * Try activating protocol, but check if support is enabled first, since
* we try detecting Synaptics even when protocol is disabled. * we try detecting Synaptics even when protocol is disabled.
*/ */
if (synaptics_supported() && if (IS_ENABLED(CONFIG_MOUSE_PS2_SYNAPTICS) &&
(!set_properties || synaptics_init(psmouse) == 0)) { (!set_properties || synaptics_init(psmouse) == 0)) {
return PSMOUSE_SYNAPTICS; return PSMOUSE_SYNAPTICS;
} }
...@@ -801,7 +801,7 @@ static int psmouse_extensions(struct psmouse *psmouse, ...@@ -801,7 +801,7 @@ static int psmouse_extensions(struct psmouse *psmouse,
*/ */
if (max_proto > PSMOUSE_IMEX && if (max_proto > PSMOUSE_IMEX &&
cypress_detect(psmouse, set_properties) == 0) { cypress_detect(psmouse, set_properties) == 0) {
if (cypress_supported()) { if (IS_ENABLED(CONFIG_MOUSE_PS2_CYPRESS)) {
if (cypress_init(psmouse) == 0) if (cypress_init(psmouse) == 0)
return PSMOUSE_CYPRESS; return PSMOUSE_CYPRESS;
......
...@@ -1454,11 +1454,6 @@ int synaptics_init_relative(struct psmouse *psmouse) ...@@ -1454,11 +1454,6 @@ int synaptics_init_relative(struct psmouse *psmouse)
return __synaptics_init(psmouse, false); return __synaptics_init(psmouse, false);
} }
bool synaptics_supported(void)
{
return true;
}
#else /* CONFIG_MOUSE_PS2_SYNAPTICS */ #else /* CONFIG_MOUSE_PS2_SYNAPTICS */
void __init synaptics_module_init(void) void __init synaptics_module_init(void)
...@@ -1470,9 +1465,4 @@ int synaptics_init(struct psmouse *psmouse) ...@@ -1470,9 +1465,4 @@ int synaptics_init(struct psmouse *psmouse)
return -ENOSYS; return -ENOSYS;
} }
bool synaptics_supported(void)
{
return false;
}
#endif /* CONFIG_MOUSE_PS2_SYNAPTICS */ #endif /* CONFIG_MOUSE_PS2_SYNAPTICS */
...@@ -175,6 +175,5 @@ int synaptics_detect(struct psmouse *psmouse, bool set_properties); ...@@ -175,6 +175,5 @@ int synaptics_detect(struct psmouse *psmouse, bool set_properties);
int synaptics_init(struct psmouse *psmouse); int synaptics_init(struct psmouse *psmouse);
int synaptics_init_relative(struct psmouse *psmouse); int synaptics_init_relative(struct psmouse *psmouse);
void synaptics_reset(struct psmouse *psmouse); void synaptics_reset(struct psmouse *psmouse);
bool synaptics_supported(void);
#endif /* _SYNAPTICS_H */ #endif /* _SYNAPTICS_H */
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment