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