Commit 6bea25af authored by James Hogan's avatar James Hogan Committed by Mauro Carvalho Chehab
Browse files

[media] rc-main: automatically refresh filter on protocol change 



When either of the normal or wakeup filter protocols are changed,
refresh the corresponding scancode filter, i.e. try and set the same
scancode filter with the new protocol. If that fails clear the filter
instead.

If no protocol was selected the filter is just cleared, and if no
s_filter callback exists the filter is left unmodified.

Similarly clear the filter mask when the filter is set if no protocol is
currently selected.

This simplifies driver code which no longer has to explicitly worry
about modifying the filter on a protocol change. This also allows the
change_wakeup_protocol callback to be omitted entirely if there is only
a single available wakeup protocol at a time, since selecting no
protocol will automatically clear the wakeup filter, disabling wakeup.

Signed-off-by: default avatarJames Hogan <james.hogan@imgtec.com>
Reviewed-by: default avatarAntti Seppälä <a.seppala@gmail.com>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent ab88c66d

drivers/media/rc/rc-main.c

+39 −2
Original line number Diff line number Diff line
@@ -918,11 +918,12 @@ static ssize_t store_protocols(struct device *device, 
	struct rc_filter_attribute *fattr = to_rc_filter_attr(mattr);

	bool enable, disable;

	const char *tmp;

	u64 type;

	u64 old_type, type;

	u64 mask;

	int rc, i, count = 0;

	ssize_t ret;

	int (*change_protocol)(struct rc_dev *dev, u64 *rc_type);

	struct rc_scancode_filter local_filter, *filter;



	/* Device is being removed */

	if (!dev)
@@ -935,7 +936,8 @@ static ssize_t store_protocols(struct device *device, 
		ret = -EINVAL;

		goto out;

	}

	type = dev->enabled_protocols[fattr->type];

	old_type = dev->enabled_protocols[fattr->type];

	type = old_type;



	while ((tmp = strsep((char **) &data, " \n")) != NULL) {

		if (!*tmp)
@@ -999,6 +1001,36 @@ static ssize_t store_protocols(struct device *device, 
	IR_dprintk(1, "Current protocol(s): 0x%llx\n",

		   (long long)type);



	/*

	 * If the protocol is changed the filter needs updating.

	 * Try setting the same filter with the new protocol (if any).

	 * Fall back to clearing the filter.

	 */

	filter = &dev->scancode_filters[fattr->type];

	if (old_type != type && filter->mask) {

		local_filter = *filter;

		if (!type) {

			/* no protocol => clear filter */

			ret = -1;

		} else if (!dev->s_filter) {

			/* generic filtering => accept any filter */

			ret = 0;

		} else {

			/* hardware filtering => try setting, otherwise clear */

			ret = dev->s_filter(dev, fattr->type, &local_filter);

		}

		if (ret < 0) {

			/* clear the filter */

			local_filter.data = 0;

			local_filter.mask = 0;

			if (dev->s_filter)

				dev->s_filter(dev, fattr->type, &local_filter);

		}



		/* commit the new filter */

		*filter = local_filter;

	}



	ret = len;



out:
@@ -1096,6 +1128,11 @@ static ssize_t store_filter(struct device *device, 
		local_filter.mask = val;

	else

		local_filter.data = val;

	if (!dev->enabled_protocols[fattr->type] && local_filter.mask) {

		/* refuse to set a filter unless a protocol is enabled */

		ret = -EINVAL;

		goto unlock;

	}

	if (dev->s_filter) {

		ret = dev->s_filter(dev, fattr->type, &local_filter);

		if (ret < 0)

CRA Git | Maintained and supported by SUSTech CRA and CCSE