[prev in list] [next in list] [prev in thread] [next in thread] 

List:       fwts-devel
Subject:    ACK: [PATCH] lib: print reference names in fwts_acpi_object_dump
From:       ivanhu <ivan.hu () canonical ! com>
Date:       2017-10-25 6:38:21
Message-ID: b62a28e2-45eb-dc7a-2393-f33fe5ad4b0b () canonical ! com
[Download RAW message or body]



On 10/18/2017 05:21 AM, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>   src/lib/src/fwts_acpi_object_eval.c | 14 +++++++++++++-
>   1 file changed, 13 insertions(+), 1 deletion(-)
> 
> diff --git a/src/lib/src/fwts_acpi_object_eval.c b/src/lib/src/fwts_acpi_object_eval.c
> index 3ac66d2..337b311 100644
> --- a/src/lib/src/fwts_acpi_object_eval.c
> +++ b/src/lib/src/fwts_acpi_object_eval.c
> @@ -352,6 +352,12 @@ static void fwts_acpi_object_dump_recursive(
>   {
>   	uint32_t i;
>   	char index_buf[5];
> +	ACPI_BUFFER buffer;
> +	ACPI_STATUS status;
> +	char full_name[128];
> +
> +	buffer.Length = sizeof(full_name);
> +	buffer.Pointer = full_name;
>   
>   	if (index > -1)
>   		snprintf(index_buf, sizeof(index_buf), "%2.2d: ", index);
> @@ -379,8 +385,14 @@ static void fwts_acpi_object_dump_recursive(
>   			fwts_acpi_object_dump_recursive(fw, element, depth + 1, i);
>   		}
>   		break;
> +	case ACPI_TYPE_LOCAL_REFERENCE:
> +		status = AcpiGetName(obj->Reference.Handle, ACPI_FULL_PATHNAME, &buffer);
> +		if (ACPI_SUCCESS(status))
> +			fwts_log_info_verbatim(fw, "%*s%sReference:  %s", depth * 2, "",
> +				index_buf, full_name);
> +		break;
>   	default:
> -		fwts_log_info_verbatim(fw, "%*s%sUnknown type %d\n", depth * 2, "",
> +		fwts_log_info_verbatim(fw, "%*s%sUnknown type 0x%2.2x\n", depth * 2, "",
>   			index_buf, obj->Type);
>   		break;
>   	}
> 

Acked-by: Ivan Hu <ivan.hu@canonical.com>

-- 
fwts-devel mailing list
fwts-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/fwts-devel

[prev in list] [next in list] [prev in thread] [next in thread]