diff --git a/src/driver-etekcity.c b/src/driver-etekcity.c index e9af6fbd..012ecd58 100644 --- a/src/driver-etekcity.c +++ b/src/driver-etekcity.c @@ -284,12 +284,6 @@ etekcity_set_current_profile(struct ratbag_device *device, unsigned int index) return ret == sizeof(buf) ? 0 : ret; } -static int -etekcity_set_default_profile(struct ratbag_device *device, unsigned int index) -{ - return -ENOTSUP; -} - static int etekcity_set_config_profile(struct ratbag_device *device, uint8_t profile, uint8_t type) { @@ -714,7 +708,6 @@ struct ratbag_driver etekcity_driver = { .read_profile = etekcity_read_profile, .write_profile = etekcity_write_profile, .set_active_profile = etekcity_set_current_profile, - .set_default_profile = etekcity_set_default_profile, .read_button = etekcity_read_button, .write_button = etekcity_write_button, .write_resolution_dpi = etekcity_write_resolution_dpi, diff --git a/src/driver-hidpp10.c b/src/driver-hidpp10.c index 5fa93dc8..53f73e42 100644 --- a/src/driver-hidpp10.c +++ b/src/driver-hidpp10.c @@ -326,12 +326,6 @@ hidpp10drv_set_current_profile(struct ratbag_device *device, unsigned int index) return hidpp10_set_current_profile(hidpp10, index); } -static int -hidpp10drv_set_default_profile(struct ratbag_device *device, unsigned int index) -{ - return -ENOTSUP; -} - static void hidpp10drv_read_profile(struct ratbag_profile *profile, unsigned int index) { @@ -574,7 +568,6 @@ hidpp10drv_probe(struct ratbag_device *device) ratbag_device_init_profiles(device, 1, 1, 3); profile = ratbag_device_get_profile(device, 0); profile->is_active = true; - profile->is_default = true; ratbag_profile_unref(profile); } @@ -612,7 +605,6 @@ struct ratbag_driver hidpp10_driver = { .read_profile = hidpp10drv_read_profile, .write_profile = hidpp10drv_write_profile, .set_active_profile = hidpp10drv_set_current_profile, - .set_default_profile = hidpp10drv_set_default_profile, .read_button = hidpp10drv_read_button, .write_button = hidpp10drv_write_button, .write_resolution_dpi = hidpp10drv_write_resolution_dpi, diff --git a/src/driver-hidpp20.c b/src/driver-hidpp20.c index 1ac3ca4f..774ce1be 100644 --- a/src/driver-hidpp20.c +++ b/src/driver-hidpp20.c @@ -369,12 +369,6 @@ hidpp20drv_set_current_profile(struct ratbag_device *device, unsigned int index) return hidpp20_onboard_profiles_set_current_profile(drv_data->dev, index); } -static int -hidpp20drv_set_default_profile(struct ratbag_device *device, unsigned int index) -{ - return -ENOTSUP; -} - static int hidpp20drv_read_resolution_dpi_2201(struct ratbag_device *device) { @@ -854,7 +848,6 @@ struct ratbag_driver hidpp20_driver = { .read_profile = hidpp20drv_read_profile, .write_profile = hidpp20drv_write_profile, .set_active_profile = hidpp20drv_set_current_profile, - .set_default_profile = hidpp20drv_set_default_profile, .read_button = hidpp20drv_read_button, .write_button = hidpp20drv_write_button, .write_resolution_dpi = hidpp20drv_write_resolution_dpi, diff --git a/src/driver-test.c b/src/driver-test.c index 7447e8a5..9cfb3708 100644 --- a/src/driver-test.c +++ b/src/driver-test.c @@ -52,7 +52,6 @@ test_read_profile(struct ratbag_profile *profile, unsigned int index) } profile->is_active = p->active; - profile->is_default = p->dflt; } static int diff --git a/src/libratbag-private.h b/src/libratbag-private.h index c6e5b9ca..a72d7077 100644 --- a/src/libratbag-private.h +++ b/src/libratbag-private.h @@ -155,14 +155,6 @@ struct ratbag_driver { */ int (*set_active_profile)(struct ratbag_device *device, unsigned int index); - /** - * Called to mark a previously writen profile as the default. - * - * There should be no need to write the profile here, a - * .write_profile() call is issued before calling this. - */ - int (*set_default_profile)(struct ratbag_device *device, unsigned int index); - /** * For the given button, fill in the struct ratbag_button * with the available information. @@ -226,7 +218,6 @@ struct ratbag_profile { } resolution; bool is_active; /**< profile is the currently active one */ - bool is_default; /**< profile is the default one */ }; #define BUTTON_ACTION_NONE \ diff --git a/src/libratbag.c b/src/libratbag.c index bfd7068b..96aef4f0 100644 --- a/src/libratbag.c +++ b/src/libratbag.c @@ -209,7 +209,6 @@ ratbag_sanity_check_device(struct ratbag_device *device) _cleanup_profile_ struct ratbag_profile *profile = NULL; _cleanup_resolution_ struct ratbag_resolution *res = NULL; bool has_active = false; - bool has_default = false; unsigned int nres, nprofiles; bool rc = false; unsigned int i; @@ -231,17 +230,6 @@ ratbag_sanity_check_device(struct ratbag_device *device) if (!profile) goto out; - /* Allow max 1 default profile */ - if (profile->is_default) { - if (has_default) { - log_bug_libratbag(ratbag, - "%s: multiple default profiles\n", - device->name); - goto out; - } - has_default = true; - } - /* Allow max 1 active profile */ if (profile->is_active) { if (has_active) { @@ -685,14 +673,6 @@ ratbag_profile_is_active(struct ratbag_profile *profile) return profile->is_active; } -LIBRATBAG_EXPORT int -ratbag_profile_is_default(struct ratbag_profile *profile) -{ - /* FIXME: unclear if any device actually supports this function */ - - return profile->is_default; -} - LIBRATBAG_EXPORT unsigned int ratbag_device_get_num_profiles(struct ratbag_device *device) { @@ -749,35 +729,6 @@ ratbag_profile_set_active(struct ratbag_profile *profile) return rc; } -LIBRATBAG_EXPORT int -ratbag_profile_set_default(struct ratbag_profile *profile) -{ - struct ratbag_device *device = profile->device; - struct ratbag_profile *p; - int rc; - - /* FIXME: unclear if any device actually supports this function */ - - assert(device->driver->write_profile); - rc = device->driver->write_profile(profile); - if (rc) - return rc; - - if (ratbag_device_has_capability(device, RATBAG_DEVICE_CAP_SWITCHABLE_PROFILE)) { - assert(device->driver->set_default_profile); - rc = device->driver->set_default_profile(device, profile->index); - } - - if (rc) - return rc; - - list_for_each(p, &device->profiles, link) { - p->is_default = false; - } - profile->is_default = true; - return rc; -} - LIBRATBAG_EXPORT int ratbag_profile_get_num_resolutions(struct ratbag_profile *profile) { diff --git a/src/libratbag.h b/src/libratbag.h index c2917577..6b65f279 100644 --- a/src/libratbag.h +++ b/src/libratbag.h @@ -601,18 +601,6 @@ ratbag_profile_set_active(struct ratbag_profile *profile); int ratbag_profile_is_default(struct ratbag_profile *profile); -/** - * @ingroup profile - * - * Make the given profile the default profile - * - * @param profile The profile to make the default profile. - * - * @return 0 on success or nonzero otherwise. - */ -int -ratbag_profile_set_default(struct ratbag_profile *profile); - /** * @ingroup profile * diff --git a/src/libratbag.sym b/src/libratbag.sym index d8b59ae1..13f99589 100644 --- a/src/libratbag.sym +++ b/src/libratbag.sym @@ -45,11 +45,9 @@ global: ratbag_profile_get_resolution; ratbag_profile_get_user_data; ratbag_profile_is_active; - ratbag_profile_is_default; ratbag_profile_ref; ratbag_profile_set_user_data; ratbag_profile_set_active; - ratbag_profile_set_default; ratbag_profile_unref; ratbag_resolution_get_dpi; ratbag_resolution_get_dpi_x; diff --git a/test/test-device.c b/test/test-device.c index 776dc15e..f7828376 100644 --- a/test/test-device.c +++ b/test/test-device.c @@ -195,7 +195,7 @@ START_TEST(device_profiles) struct ratbag_profile *p; int nprofiles; int i; - bool is_active, is_default; + bool is_active; int device_freed_count = 0; struct ratbag_test_device td = sane_device; @@ -214,8 +214,6 @@ START_TEST(device_profiles) is_active = ratbag_profile_is_active(p); ck_assert_int_eq(is_active, (i == 0)); - is_default = ratbag_profile_is_default(p); - ck_assert_int_eq(is_default, (i == 1)); ratbag_profile_unref(p); } ratbag_device_unref(d); diff --git a/tools/ratbag-command.c b/tools/ratbag-command.c index a9540317..5c850d68 100644 --- a/tools/ratbag-command.c +++ b/tools/ratbag-command.c @@ -351,9 +351,8 @@ ratbag_cmd_info(const struct ratbag_cmd *cmd, if (!profile) continue; - printf(" Profile %d%s%s\n", i, - ratbag_profile_is_active(profile) ? " (active)" : "", - ratbag_profile_is_default(profile) ? " (default)" : ""); + printf(" Profile %d%s\n", i, + ratbag_profile_is_active(profile) ? " (active)" : ""); printf(" Resolutions:\n"); for (j = 0; j < ratbag_profile_get_num_resolutions(profile); j++) { struct ratbag_resolution *res;