netdev - Re: [PATCH] ssb: Add sysfs attributes to ssb devices

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1291069172.14734.0.camel@maggie>
Date:	Mon, 29 Nov 2010 23:19:32 +0100
From:	Michael Büsch <mb@...sch.de>
To:	Hauke Mehrtens <hauke@...ke-m.de>
Cc:	davem@...emloft.net, netdev@...r.kernel.org,
	Bernhard Loos <bernhardloos@...glemail.com>
Subject: Re: [PATCH] ssb: Add sysfs attributes to ssb devices

On Sun, 2010-11-28 at 12:22 +0100, Hauke Mehrtens wrote: 
> Make it possible to read out the attributes, till now only show on
> dmesg, through sysfs.
> 
> This patch was some time in OpenWrt.
> 
> Signed-off-by: Bernhard Loos <bernhardloos@...glemail.com>
> Signed-off-by: Hauke Mehrtens <hauke@...ke-m.de>

Acked-by: Michael Buesch <mb@...sch.de>

> ---
>  drivers/ssb/main.c |   30 ++++++++++++++++++++++++++++++
>  1 files changed, 30 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/ssb/main.c b/drivers/ssb/main.c
> index c68b3dc..3918d2c 100644
> --- a/drivers/ssb/main.c
> +++ b/drivers/ssb/main.c
> @@ -383,6 +383,35 @@ static int ssb_device_uevent(struct device *dev, struct kobj_uevent_env *env)
>  			     ssb_dev->id.revision);
>  }
>  
> +#define ssb_config_attr(attrib, field, format_string) \
> +static ssize_t \
> +attrib##_show(struct device *dev, struct device_attribute *attr, char *buf) \
> +{ \
> +	return sprintf(buf, format_string, dev_to_ssb_dev(dev)->field); \
> +}
> +
> +ssb_config_attr(core_num, core_index, "%u\n")
> +ssb_config_attr(coreid, id.coreid, "0x%04x\n")
> +ssb_config_attr(vendor, id.vendor, "0x%04x\n")
> +ssb_config_attr(revision, id.revision, "%u\n")
> +ssb_config_attr(irq, irq, "%u\n")
> +static ssize_t
> +name_show(struct device *dev, struct device_attribute *attr, char *buf)
> +{
> +	return sprintf(buf, "%s\n",
> +		       ssb_core_name(dev_to_ssb_dev(dev)->id.coreid));
> +}
> +
> +static struct device_attribute ssb_device_attrs[] = {
> +	__ATTR_RO(name),
> +	__ATTR_RO(core_num),
> +	__ATTR_RO(coreid),
> +	__ATTR_RO(vendor),
> +	__ATTR_RO(revision),
> +	__ATTR_RO(irq),
> +	__ATTR_NULL,
> +};
> +
>  static struct bus_type ssb_bustype = {
>  	.name		= "ssb",
>  	.match		= ssb_bus_match,
> @@ -392,6 +421,7 @@ static struct bus_type ssb_bustype = {
>  	.suspend	= ssb_device_suspend,
>  	.resume		= ssb_device_resume,
>  	.uevent		= ssb_device_uevent,
> +	.dev_attrs	= ssb_device_attrs,
>  };
>  
>  static void ssb_buses_lock(void)


-- 
Greetings Michael.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html