13c14b734SPratik R. Sampat // SPDX-License-Identifier: GPL-2.0-or-later
23c14b734SPratik R. Sampat /*
33c14b734SPratik R. Sampat  * Platform energy and frequency attributes driver
43c14b734SPratik R. Sampat  *
53c14b734SPratik R. Sampat  * This driver creates a sys file at /sys/firmware/papr/ which encapsulates a
63c14b734SPratik R. Sampat  * directory structure containing files in keyword - value pairs that specify
73c14b734SPratik R. Sampat  * energy and frequency configuration of the system.
83c14b734SPratik R. Sampat  *
93c14b734SPratik R. Sampat  * The format of exposing the sysfs information is as follows:
103c14b734SPratik R. Sampat  * /sys/firmware/papr/energy_scale_info/
113c14b734SPratik R. Sampat  *  |-- <id>/
123c14b734SPratik R. Sampat  *    |-- desc
133c14b734SPratik R. Sampat  *    |-- value
143c14b734SPratik R. Sampat  *    |-- value_desc (if exists)
153c14b734SPratik R. Sampat  *  |-- <id>/
163c14b734SPratik R. Sampat  *    |-- desc
173c14b734SPratik R. Sampat  *    |-- value
183c14b734SPratik R. Sampat  *    |-- value_desc (if exists)
193c14b734SPratik R. Sampat  *
203c14b734SPratik R. Sampat  * Copyright 2022 IBM Corp.
213c14b734SPratik R. Sampat  */
223c14b734SPratik R. Sampat 
233c14b734SPratik R. Sampat #include <asm/hvcall.h>
243c14b734SPratik R. Sampat #include <asm/machdep.h>
2546d60bdbSChristophe Leroy #include <asm/firmware.h>
263c14b734SPratik R. Sampat 
273c14b734SPratik R. Sampat #include "pseries.h"
283c14b734SPratik R. Sampat 
293c14b734SPratik R. Sampat /*
303c14b734SPratik R. Sampat  * Flag attributes to fetch either all or one attribute from the HCALL
313c14b734SPratik R. Sampat  * flag = BE(0) => fetch all attributes with firstAttributeId = 0
323c14b734SPratik R. Sampat  * flag = BE(1) => fetch a single attribute with firstAttributeId = id
333c14b734SPratik R. Sampat  */
343c14b734SPratik R. Sampat #define ESI_FLAGS_ALL		0
353c14b734SPratik R. Sampat #define ESI_FLAGS_SINGLE	(1ull << 63)
363c14b734SPratik R. Sampat 
373c14b734SPratik R. Sampat #define KOBJ_MAX_ATTRS		3
383c14b734SPratik R. Sampat 
393c14b734SPratik R. Sampat #define ESI_HDR_SIZE		sizeof(struct h_energy_scale_info_hdr)
403c14b734SPratik R. Sampat #define ESI_ATTR_SIZE		sizeof(struct energy_scale_attribute)
413c14b734SPratik R. Sampat #define CURR_MAX_ESI_ATTRS	8
423c14b734SPratik R. Sampat 
433c14b734SPratik R. Sampat struct energy_scale_attribute {
443c14b734SPratik R. Sampat 	__be64 id;
453c14b734SPratik R. Sampat 	__be64 val;
463c14b734SPratik R. Sampat 	u8 desc[64];
473c14b734SPratik R. Sampat 	u8 value_desc[64];
483c14b734SPratik R. Sampat } __packed;
493c14b734SPratik R. Sampat 
503c14b734SPratik R. Sampat struct h_energy_scale_info_hdr {
513c14b734SPratik R. Sampat 	__be64 num_attrs;
523c14b734SPratik R. Sampat 	__be64 array_offset;
533c14b734SPratik R. Sampat 	u8 data_header_version;
543c14b734SPratik R. Sampat } __packed;
553c14b734SPratik R. Sampat 
563c14b734SPratik R. Sampat struct papr_attr {
573c14b734SPratik R. Sampat 	u64 id;
583c14b734SPratik R. Sampat 	struct kobj_attribute kobj_attr;
593c14b734SPratik R. Sampat };
603c14b734SPratik R. Sampat 
613c14b734SPratik R. Sampat struct papr_group {
623c14b734SPratik R. Sampat 	struct attribute_group pg;
633c14b734SPratik R. Sampat 	struct papr_attr pgattrs[KOBJ_MAX_ATTRS];
643c14b734SPratik R. Sampat };
653c14b734SPratik R. Sampat 
663c14b734SPratik R. Sampat static struct papr_group *papr_groups;
673c14b734SPratik R. Sampat /* /sys/firmware/papr */
683c14b734SPratik R. Sampat static struct kobject *papr_kobj;
693c14b734SPratik R. Sampat /* /sys/firmware/papr/energy_scale_info */
703c14b734SPratik R. Sampat static struct kobject *esi_kobj;
713c14b734SPratik R. Sampat 
723c14b734SPratik R. Sampat /*
733c14b734SPratik R. Sampat  * Energy modes can change dynamically hence making a new hcall each time the
743c14b734SPratik R. Sampat  * information needs to be retrieved
753c14b734SPratik R. Sampat  */
papr_get_attr(u64 id,struct energy_scale_attribute * esi)763c14b734SPratik R. Sampat static int papr_get_attr(u64 id, struct energy_scale_attribute *esi)
773c14b734SPratik R. Sampat {
783c14b734SPratik R. Sampat 	int esi_buf_size = ESI_HDR_SIZE + (CURR_MAX_ESI_ATTRS * ESI_ATTR_SIZE);
793c14b734SPratik R. Sampat 	int ret, max_esi_attrs = CURR_MAX_ESI_ATTRS;
803c14b734SPratik R. Sampat 	struct energy_scale_attribute *curr_esi;
813c14b734SPratik R. Sampat 	struct h_energy_scale_info_hdr *hdr;
823c14b734SPratik R. Sampat 	char *buf;
833c14b734SPratik R. Sampat 
843c14b734SPratik R. Sampat 	buf = kmalloc(esi_buf_size, GFP_KERNEL);
853c14b734SPratik R. Sampat 	if (buf == NULL)
863c14b734SPratik R. Sampat 		return -ENOMEM;
873c14b734SPratik R. Sampat 
883c14b734SPratik R. Sampat retry:
893c14b734SPratik R. Sampat 	ret = plpar_hcall_norets(H_GET_ENERGY_SCALE_INFO, ESI_FLAGS_SINGLE,
903c14b734SPratik R. Sampat 				 id, virt_to_phys(buf),
913c14b734SPratik R. Sampat 				 esi_buf_size);
923c14b734SPratik R. Sampat 
933c14b734SPratik R. Sampat 	/*
943c14b734SPratik R. Sampat 	 * If the hcall fails with not enough memory for either the
953c14b734SPratik R. Sampat 	 * header or data, attempt to allocate more
963c14b734SPratik R. Sampat 	 */
973c14b734SPratik R. Sampat 	if (ret == H_PARTIAL || ret == H_P4) {
983c14b734SPratik R. Sampat 		char *temp_buf;
993c14b734SPratik R. Sampat 
1003c14b734SPratik R. Sampat 		max_esi_attrs += 4;
1013c14b734SPratik R. Sampat 		esi_buf_size = ESI_HDR_SIZE + (CURR_MAX_ESI_ATTRS * max_esi_attrs);
1023c14b734SPratik R. Sampat 
1033c14b734SPratik R. Sampat 		temp_buf = krealloc(buf, esi_buf_size, GFP_KERNEL);
104*1699fb91SQiheng Lin 		if (temp_buf) {
1053c14b734SPratik R. Sampat 			buf = temp_buf;
106*1699fb91SQiheng Lin 		} else {
107*1699fb91SQiheng Lin 			ret = -ENOMEM;
108*1699fb91SQiheng Lin 			goto out_buf;
109*1699fb91SQiheng Lin 		}
1103c14b734SPratik R. Sampat 
1113c14b734SPratik R. Sampat 		goto retry;
1123c14b734SPratik R. Sampat 	}
1133c14b734SPratik R. Sampat 
1143c14b734SPratik R. Sampat 	if (ret != H_SUCCESS) {
1153c14b734SPratik R. Sampat 		pr_warn("hcall failed: H_GET_ENERGY_SCALE_INFO");
1163c14b734SPratik R. Sampat 		ret = -EIO;
1173c14b734SPratik R. Sampat 		goto out_buf;
1183c14b734SPratik R. Sampat 	}
1193c14b734SPratik R. Sampat 
1203c14b734SPratik R. Sampat 	hdr = (struct h_energy_scale_info_hdr *) buf;
1213c14b734SPratik R. Sampat 	curr_esi = (struct energy_scale_attribute *)
1223c14b734SPratik R. Sampat 		(buf + be64_to_cpu(hdr->array_offset));
1233c14b734SPratik R. Sampat 
1243c14b734SPratik R. Sampat 	if (esi_buf_size <
1253c14b734SPratik R. Sampat 	    be64_to_cpu(hdr->array_offset) + (be64_to_cpu(hdr->num_attrs)
1263c14b734SPratik R. Sampat 	    * sizeof(struct energy_scale_attribute))) {
1273c14b734SPratik R. Sampat 		ret = -EIO;
1283c14b734SPratik R. Sampat 		goto out_buf;
1293c14b734SPratik R. Sampat 	}
1303c14b734SPratik R. Sampat 
1313c14b734SPratik R. Sampat 	*esi = *curr_esi;
1323c14b734SPratik R. Sampat 
1333c14b734SPratik R. Sampat out_buf:
1343c14b734SPratik R. Sampat 	kfree(buf);
1353c14b734SPratik R. Sampat 
1363c14b734SPratik R. Sampat 	return ret;
1373c14b734SPratik R. Sampat }
1383c14b734SPratik R. Sampat 
1393c14b734SPratik R. Sampat /*
1403c14b734SPratik R. Sampat  * Extract and export the description of the energy scale attributes
1413c14b734SPratik R. Sampat  */
desc_show(struct kobject * kobj,struct kobj_attribute * kobj_attr,char * buf)1423c14b734SPratik R. Sampat static ssize_t desc_show(struct kobject *kobj,
1433c14b734SPratik R. Sampat 			  struct kobj_attribute *kobj_attr,
1443c14b734SPratik R. Sampat 			  char *buf)
1453c14b734SPratik R. Sampat {
1463c14b734SPratik R. Sampat 	struct papr_attr *pattr = container_of(kobj_attr, struct papr_attr,
1473c14b734SPratik R. Sampat 					       kobj_attr);
1483c14b734SPratik R. Sampat 	struct energy_scale_attribute esi;
1493c14b734SPratik R. Sampat 	int ret;
1503c14b734SPratik R. Sampat 
1513c14b734SPratik R. Sampat 	ret = papr_get_attr(pattr->id, &esi);
1523c14b734SPratik R. Sampat 	if (ret)
1533c14b734SPratik R. Sampat 		return ret;
1543c14b734SPratik R. Sampat 
1553c14b734SPratik R. Sampat 	return sysfs_emit(buf, "%s\n", esi.desc);
1563c14b734SPratik R. Sampat }
1573c14b734SPratik R. Sampat 
1583c14b734SPratik R. Sampat /*
1593c14b734SPratik R. Sampat  * Extract and export the numeric value of the energy scale attributes
1603c14b734SPratik R. Sampat  */
val_show(struct kobject * kobj,struct kobj_attribute * kobj_attr,char * buf)1613c14b734SPratik R. Sampat static ssize_t val_show(struct kobject *kobj,
1623c14b734SPratik R. Sampat 			 struct kobj_attribute *kobj_attr,
1633c14b734SPratik R. Sampat 			 char *buf)
1643c14b734SPratik R. Sampat {
1653c14b734SPratik R. Sampat 	struct papr_attr *pattr = container_of(kobj_attr, struct papr_attr,
1663c14b734SPratik R. Sampat 					       kobj_attr);
1673c14b734SPratik R. Sampat 	struct energy_scale_attribute esi;
1683c14b734SPratik R. Sampat 	int ret;
1693c14b734SPratik R. Sampat 
1703c14b734SPratik R. Sampat 	ret = papr_get_attr(pattr->id, &esi);
1713c14b734SPratik R. Sampat 	if (ret)
1723c14b734SPratik R. Sampat 		return ret;
1733c14b734SPratik R. Sampat 
1743c14b734SPratik R. Sampat 	return sysfs_emit(buf, "%llu\n", be64_to_cpu(esi.val));
1753c14b734SPratik R. Sampat }
1763c14b734SPratik R. Sampat 
1773c14b734SPratik R. Sampat /*
1783c14b734SPratik R. Sampat  * Extract and export the value description in string format of the energy
1793c14b734SPratik R. Sampat  * scale attributes
1803c14b734SPratik R. Sampat  */
val_desc_show(struct kobject * kobj,struct kobj_attribute * kobj_attr,char * buf)1813c14b734SPratik R. Sampat static ssize_t val_desc_show(struct kobject *kobj,
1823c14b734SPratik R. Sampat 			      struct kobj_attribute *kobj_attr,
1833c14b734SPratik R. Sampat 			      char *buf)
1843c14b734SPratik R. Sampat {
1853c14b734SPratik R. Sampat 	struct papr_attr *pattr = container_of(kobj_attr, struct papr_attr,
1863c14b734SPratik R. Sampat 					       kobj_attr);
1873c14b734SPratik R. Sampat 	struct energy_scale_attribute esi;
1883c14b734SPratik R. Sampat 	int ret;
1893c14b734SPratik R. Sampat 
1903c14b734SPratik R. Sampat 	ret = papr_get_attr(pattr->id, &esi);
1913c14b734SPratik R. Sampat 	if (ret)
1923c14b734SPratik R. Sampat 		return ret;
1933c14b734SPratik R. Sampat 
1943c14b734SPratik R. Sampat 	return sysfs_emit(buf, "%s\n", esi.value_desc);
1953c14b734SPratik R. Sampat }
1963c14b734SPratik R. Sampat 
1973c14b734SPratik R. Sampat static struct papr_ops_info {
1983c14b734SPratik R. Sampat 	const char *attr_name;
1993c14b734SPratik R. Sampat 	ssize_t (*show)(struct kobject *kobj, struct kobj_attribute *kobj_attr,
2003c14b734SPratik R. Sampat 			char *buf);
2013c14b734SPratik R. Sampat } ops_info[KOBJ_MAX_ATTRS] = {
2023c14b734SPratik R. Sampat 	{ "desc", desc_show },
2033c14b734SPratik R. Sampat 	{ "value", val_show },
2043c14b734SPratik R. Sampat 	{ "value_desc", val_desc_show },
2053c14b734SPratik R. Sampat };
2063c14b734SPratik R. Sampat 
add_attr(u64 id,int index,struct papr_attr * attr)2073c14b734SPratik R. Sampat static void add_attr(u64 id, int index, struct papr_attr *attr)
2083c14b734SPratik R. Sampat {
2093c14b734SPratik R. Sampat 	attr->id = id;
2103c14b734SPratik R. Sampat 	sysfs_attr_init(&attr->kobj_attr.attr);
2113c14b734SPratik R. Sampat 	attr->kobj_attr.attr.name = ops_info[index].attr_name;
2123c14b734SPratik R. Sampat 	attr->kobj_attr.attr.mode = 0444;
2133c14b734SPratik R. Sampat 	attr->kobj_attr.show = ops_info[index].show;
2143c14b734SPratik R. Sampat }
2153c14b734SPratik R. Sampat 
add_attr_group(u64 id,struct papr_group * pg,bool show_val_desc)2163c14b734SPratik R. Sampat static int add_attr_group(u64 id, struct papr_group *pg, bool show_val_desc)
2173c14b734SPratik R. Sampat {
2183c14b734SPratik R. Sampat 	int i;
2193c14b734SPratik R. Sampat 
2203c14b734SPratik R. Sampat 	for (i = 0; i < KOBJ_MAX_ATTRS; i++) {
2213c14b734SPratik R. Sampat 		if (!strcmp(ops_info[i].attr_name, "value_desc") &&
2223c14b734SPratik R. Sampat 		    !show_val_desc) {
2233c14b734SPratik R. Sampat 			continue;
2243c14b734SPratik R. Sampat 		}
2253c14b734SPratik R. Sampat 		add_attr(id, i, &pg->pgattrs[i]);
2263c14b734SPratik R. Sampat 		pg->pg.attrs[i] = &pg->pgattrs[i].kobj_attr.attr;
2273c14b734SPratik R. Sampat 	}
2283c14b734SPratik R. Sampat 
2293c14b734SPratik R. Sampat 	return sysfs_create_group(esi_kobj, &pg->pg);
2303c14b734SPratik R. Sampat }
2313c14b734SPratik R. Sampat 
2323c14b734SPratik R. Sampat 
papr_init(void)2333c14b734SPratik R. Sampat static int __init papr_init(void)
2343c14b734SPratik R. Sampat {
2353c14b734SPratik R. Sampat 	int esi_buf_size = ESI_HDR_SIZE + (CURR_MAX_ESI_ATTRS * ESI_ATTR_SIZE);
2363c14b734SPratik R. Sampat 	int ret, idx, i, max_esi_attrs = CURR_MAX_ESI_ATTRS;
2373c14b734SPratik R. Sampat 	struct h_energy_scale_info_hdr *esi_hdr;
2383c14b734SPratik R. Sampat 	struct energy_scale_attribute *esi_attrs;
2393c14b734SPratik R. Sampat 	uint64_t num_attrs;
2403c14b734SPratik R. Sampat 	char *esi_buf;
2413c14b734SPratik R. Sampat 
2423c14b734SPratik R. Sampat 	if (!firmware_has_feature(FW_FEATURE_LPAR) ||
2433c14b734SPratik R. Sampat 	    !firmware_has_feature(FW_FEATURE_ENERGY_SCALE_INFO)) {
2443c14b734SPratik R. Sampat 		return -ENXIO;
2453c14b734SPratik R. Sampat 	}
2463c14b734SPratik R. Sampat 
2473c14b734SPratik R. Sampat 	esi_buf = kmalloc(esi_buf_size, GFP_KERNEL);
2483c14b734SPratik R. Sampat 	if (esi_buf == NULL)
2493c14b734SPratik R. Sampat 		return -ENOMEM;
2503c14b734SPratik R. Sampat 	/*
2513c14b734SPratik R. Sampat 	 * hcall(
2523c14b734SPratik R. Sampat 	 * uint64 H_GET_ENERGY_SCALE_INFO,  // Get energy scale info
2533c14b734SPratik R. Sampat 	 * uint64 flags,            // Per the flag request
2543c14b734SPratik R. Sampat 	 * uint64 firstAttributeId, // The attribute id
2553c14b734SPratik R. Sampat 	 * uint64 bufferAddress,    // Guest physical address of the output buffer
2563c14b734SPratik R. Sampat 	 * uint64 bufferSize);      // The size in bytes of the output buffer
2573c14b734SPratik R. Sampat 	 */
2583c14b734SPratik R. Sampat retry:
2593c14b734SPratik R. Sampat 
2603c14b734SPratik R. Sampat 	ret = plpar_hcall_norets(H_GET_ENERGY_SCALE_INFO, ESI_FLAGS_ALL, 0,
2613c14b734SPratik R. Sampat 				 virt_to_phys(esi_buf), esi_buf_size);
2623c14b734SPratik R. Sampat 
2633c14b734SPratik R. Sampat 	/*
2643c14b734SPratik R. Sampat 	 * If the hcall fails with not enough memory for either the
2653c14b734SPratik R. Sampat 	 * header or data, attempt to allocate more
2663c14b734SPratik R. Sampat 	 */
2673c14b734SPratik R. Sampat 	if (ret == H_PARTIAL || ret == H_P4) {
2683c14b734SPratik R. Sampat 		char *temp_esi_buf;
2693c14b734SPratik R. Sampat 
2703c14b734SPratik R. Sampat 		max_esi_attrs += 4;
2713c14b734SPratik R. Sampat 		esi_buf_size = ESI_HDR_SIZE + (CURR_MAX_ESI_ATTRS * max_esi_attrs);
2723c14b734SPratik R. Sampat 
2733c14b734SPratik R. Sampat 		temp_esi_buf = krealloc(esi_buf, esi_buf_size, GFP_KERNEL);
2743c14b734SPratik R. Sampat 		if (temp_esi_buf)
2753c14b734SPratik R. Sampat 			esi_buf = temp_esi_buf;
2763c14b734SPratik R. Sampat 		else
2773c14b734SPratik R. Sampat 			return -ENOMEM;
2783c14b734SPratik R. Sampat 
2793c14b734SPratik R. Sampat 		goto retry;
2803c14b734SPratik R. Sampat 	}
2813c14b734SPratik R. Sampat 
2823c14b734SPratik R. Sampat 	if (ret != H_SUCCESS) {
2833c14b734SPratik R. Sampat 		pr_warn("hcall failed: H_GET_ENERGY_SCALE_INFO, ret: %d\n", ret);
2843c14b734SPratik R. Sampat 		goto out_free_esi_buf;
2853c14b734SPratik R. Sampat 	}
2863c14b734SPratik R. Sampat 
2873c14b734SPratik R. Sampat 	esi_hdr = (struct h_energy_scale_info_hdr *) esi_buf;
2883c14b734SPratik R. Sampat 	num_attrs = be64_to_cpu(esi_hdr->num_attrs);
2893c14b734SPratik R. Sampat 	esi_attrs = (struct energy_scale_attribute *)
2903c14b734SPratik R. Sampat 		    (esi_buf + be64_to_cpu(esi_hdr->array_offset));
2913c14b734SPratik R. Sampat 
2923c14b734SPratik R. Sampat 	if (esi_buf_size <
2933c14b734SPratik R. Sampat 	    be64_to_cpu(esi_hdr->array_offset) +
2943c14b734SPratik R. Sampat 	    (num_attrs * sizeof(struct energy_scale_attribute))) {
2953c14b734SPratik R. Sampat 		goto out_free_esi_buf;
2963c14b734SPratik R. Sampat 	}
2973c14b734SPratik R. Sampat 
2983c14b734SPratik R. Sampat 	papr_groups = kcalloc(num_attrs, sizeof(*papr_groups), GFP_KERNEL);
2993c14b734SPratik R. Sampat 	if (!papr_groups)
3003c14b734SPratik R. Sampat 		goto out_free_esi_buf;
3013c14b734SPratik R. Sampat 
3023c14b734SPratik R. Sampat 	papr_kobj = kobject_create_and_add("papr", firmware_kobj);
3033c14b734SPratik R. Sampat 	if (!papr_kobj) {
3043c14b734SPratik R. Sampat 		pr_warn("kobject_create_and_add papr failed\n");
3053c14b734SPratik R. Sampat 		goto out_papr_groups;
3063c14b734SPratik R. Sampat 	}
3073c14b734SPratik R. Sampat 
3083c14b734SPratik R. Sampat 	esi_kobj = kobject_create_and_add("energy_scale_info", papr_kobj);
3093c14b734SPratik R. Sampat 	if (!esi_kobj) {
3103c14b734SPratik R. Sampat 		pr_warn("kobject_create_and_add energy_scale_info failed\n");
3113c14b734SPratik R. Sampat 		goto out_kobj;
3123c14b734SPratik R. Sampat 	}
3133c14b734SPratik R. Sampat 
3143c14b734SPratik R. Sampat 	/* Allocate the groups before registering */
3153c14b734SPratik R. Sampat 	for (idx = 0; idx < num_attrs; idx++) {
3163c14b734SPratik R. Sampat 		papr_groups[idx].pg.attrs = kcalloc(KOBJ_MAX_ATTRS + 1,
3173c14b734SPratik R. Sampat 					    sizeof(*papr_groups[idx].pg.attrs),
3183c14b734SPratik R. Sampat 					    GFP_KERNEL);
3193c14b734SPratik R. Sampat 		if (!papr_groups[idx].pg.attrs)
3203c14b734SPratik R. Sampat 			goto out_pgattrs;
3213c14b734SPratik R. Sampat 
3223c14b734SPratik R. Sampat 		papr_groups[idx].pg.name = kasprintf(GFP_KERNEL, "%lld",
3233c14b734SPratik R. Sampat 					     be64_to_cpu(esi_attrs[idx].id));
3243c14b734SPratik R. Sampat 		if (papr_groups[idx].pg.name == NULL)
3253c14b734SPratik R. Sampat 			goto out_pgattrs;
3263c14b734SPratik R. Sampat 	}
3273c14b734SPratik R. Sampat 
3283c14b734SPratik R. Sampat 	for (idx = 0; idx < num_attrs; idx++) {
3293c14b734SPratik R. Sampat 		bool show_val_desc = true;
3303c14b734SPratik R. Sampat 
3313c14b734SPratik R. Sampat 		/* Do not add the value desc attr if it does not exist */
3323c14b734SPratik R. Sampat 		if (strnlen(esi_attrs[idx].value_desc,
3333c14b734SPratik R. Sampat 			    sizeof(esi_attrs[idx].value_desc)) == 0)
3343c14b734SPratik R. Sampat 			show_val_desc = false;
3353c14b734SPratik R. Sampat 
3363c14b734SPratik R. Sampat 		if (add_attr_group(be64_to_cpu(esi_attrs[idx].id),
3373c14b734SPratik R. Sampat 				   &papr_groups[idx],
3383c14b734SPratik R. Sampat 				   show_val_desc)) {
3393c14b734SPratik R. Sampat 			pr_warn("Failed to create papr attribute group %s\n",
3403c14b734SPratik R. Sampat 				papr_groups[idx].pg.name);
3413c14b734SPratik R. Sampat 			idx = num_attrs;
3423c14b734SPratik R. Sampat 			goto out_pgattrs;
3433c14b734SPratik R. Sampat 		}
3443c14b734SPratik R. Sampat 	}
3453c14b734SPratik R. Sampat 
3463c14b734SPratik R. Sampat 	kfree(esi_buf);
3473c14b734SPratik R. Sampat 	return 0;
3483c14b734SPratik R. Sampat out_pgattrs:
3493c14b734SPratik R. Sampat 	for (i = 0; i < idx ; i++) {
3503c14b734SPratik R. Sampat 		kfree(papr_groups[i].pg.attrs);
3513c14b734SPratik R. Sampat 		kfree(papr_groups[i].pg.name);
3523c14b734SPratik R. Sampat 	}
3533c14b734SPratik R. Sampat 	kobject_put(esi_kobj);
3543c14b734SPratik R. Sampat out_kobj:
3553c14b734SPratik R. Sampat 	kobject_put(papr_kobj);
3563c14b734SPratik R. Sampat out_papr_groups:
3573c14b734SPratik R. Sampat 	kfree(papr_groups);
3583c14b734SPratik R. Sampat out_free_esi_buf:
3593c14b734SPratik R. Sampat 	kfree(esi_buf);
3603c14b734SPratik R. Sampat 
3613c14b734SPratik R. Sampat 	return -ENOMEM;
3623c14b734SPratik R. Sampat }
3633c14b734SPratik R. Sampat 
3643c14b734SPratik R. Sampat machine_device_initcall(pseries, papr_init);
365