Re: [PATCH] coresight: trbe: Hide enable_sink sysfs file

On Thu, May 07, 2026 at 11:53:45AM +0100, James Clark wrote:

[...]

+static umode_t coresight_attr_is_visible(struct kobject *kobj,

• 			 struct attribute *attr, int n)
  

{ struct device *dev = kobj_to_dev(kobj);

• struct coresight_device *csdev = to_coresight_device(dev);

if (attr == &dev_attr_label.attr) { if (fwnode_property_present(dev_fwnode(dev), "label")) return attr->mode; else return 0;

• } else if (attr == &dev_attr_enable_sink.attr ||

•    attr == &dev_attr_enable_source.attr) {
  

• if (csdev->no_sysfs_mode)
  

• 	return 0;
  

• else
  

• 	return attr->mode;
  

I'd prefer no_sysfs_mode to work as a general flag rather than being limited to sink/source devices only. Otherwise, LGTM.

Thanks, Leo