ACK: [PATCH 17/26] bios: hdaaudio: remove redundant fw parameter from hda_audio_check_pins

Alex Hung alex.hung at canonical.com
Mon Oct 15 05:44:25 UTC 2012


On 10/15/2012 04:32 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Function hda_audio_check_pins does not require the fwts_framework fw
> parameter, so remove it.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/bios/hdaaudio/hdaaudio.c |   12 ++++--------
>   1 file changed, 4 insertions(+), 8 deletions(-)
>
> diff --git a/src/bios/hdaaudio/hdaaudio.c b/src/bios/hdaaudio/hdaaudio.c
> index 455a35a..764e18e 100644
> --- a/src/bios/hdaaudio/hdaaudio.c
> +++ b/src/bios/hdaaudio/hdaaudio.c
> @@ -26,8 +26,7 @@ typedef struct {
>   	uint32_t	setting;
>   } hda_audio_pin_setting;
>
> -static int hda_audio_read_pins(fwts_framework *fw, const char *path,
> -	const char *file, fwts_list *settings)
> +static int hda_audio_read_pins(const char *path, const char *file, fwts_list *settings)
>   {
>   	FILE 		*fp;
>   	uint16_t	pin;
> @@ -108,12 +107,9 @@ static int hda_audio_check_pins(fwts_framework *fw, const char *path)
>   	hda_audio_dev_info(fw, "Subsystem ID", path, "subsystem_id");
>   	hda_audio_dev_info(fw, "Revision ID", path, "revision_id");
>
> -	(void)hda_audio_read_pins(fw, path, "init_pin_configs",
> -		&init_pin_configs);
> -	(void)hda_audio_read_pins(fw, path, "driver_pin_configs",
> -		&driver_pin_configs);
> -	(void)hda_audio_read_pins(fw, path, "user_pin_configs",
> -		&user_pin_configs);
> +	(void)hda_audio_read_pins(path, "init_pin_configs", &init_pin_configs);
> +	(void)hda_audio_read_pins(path, "driver_pin_configs", &driver_pin_configs);
> +	(void)hda_audio_read_pins(path, "user_pin_configs", &user_pin_configs);
>
>   	if (fwts_list_len(&init_pin_configs) > 0)
>   		hda_audio_dump_pins(fw, "BIOS pin configurations",
>

Acked-by: Alex Hung <alex.hung at canonical.com>

I also found that the below code seems incorrect. Is it a typo in the 
&driver_pin_configs when it should be &user_pin_configs?

if (fwts_list_len(&user_pin_configs) > 0) {
     hda_audio_dump_pins(fw, "User defined pin configurations",
            &driver_pin_configs);
     warn++;
}




More information about the fwts-devel mailing list