Commit d633fd22 authored by James Clark's avatar James Clark Committed by Suzuki K Poulose
Browse files

coresight: Don't reject unrecognized ETMv3 format attributes 



config isn't the only field, there are also config1, config2, etc.
Rejecting unrecognized attributes is therefore inconsistent as it wasn't
done for all fields. It was only necessary when we were directly
programming attr->config into ETMCR and didn't hide the unsupported
fields, but now it's not needed so remove it.

Reviewed-by: default avatarLeo Yan <leo.yan@arm.com>
Reviewed-by: default avatarMike Leach <mike.leach@linaro.org>
Tested-by: default avatarLeo Yan <leo.yan@arm.com>
Signed-off-by: default avatarJames Clark <james.clark@linaro.org>
Signed-off-by: default avatarSuzuki K Poulose <suzuki.poulose@arm.com>
Link: https://lore.kernel.org/r/20251128-james-cs-syncfreq-v8-7-4d319764cc58@linaro.org
parent a1d19cd2

drivers/hwtracing/coresight/coresight-etm3x-core.c

+0 −7
Original line number Diff line number Diff line
@@ -333,13 +333,6 @@ static int etm_parse_event_config(struct etm_drvdata *drvdata, 
	if (config->mode)

		etm_config_trace_mode(config);



	/*

	 * At this time only cycle accurate, return stack  and timestamp

	 * options are available.

	 */

	if (attr->config & ~ETM3X_SUPPORTED_OPTIONS)

		return -EINVAL;



	config->ctrl = 0;



	if (ATTR_CFG_GET_FLD(attr, cycacc))