[U-Boot] [PATCH 1/3] x86: fsp: Consolidate Azalia header file

Bin Meng bmeng.cn at gmail.com
Thu Oct 19 03:36:46 UTC 2017


On Fri, Oct 13, 2017 at 7:14 PM, Stefan Roese <sr at denx.de> wrote:
> On 13.10.2017 10:30, Bin Meng wrote:
>>
>> So far there are two copies of Azalia struct defines with one in
>> baytrail and the other one in braswell. This consolidates these
>> two into one, put it in the common place, and remove the prefix
>> pch_ to these structs to make their names more generic.
>>
>> This also corrects reset_wait_timer from us to ms.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>   arch/x86/cpu/baytrail/fsp_configs.c              | 13 ++++----
>>   arch/x86/include/asm/arch-baytrail/fsp/azalia.h  | 39
>> ------------------------
>>   arch/x86/include/asm/arch-braswell/fsp/fsp_vpd.h | 26 ----------------
>>   arch/x86/include/asm/fsp/fsp_azalia.h            | 39
>> ++++++++++++++++++++++++
>>   arch/x86/include/asm/fsp/fsp_support.h           |  1 +
>>   5 files changed, 46 insertions(+), 72 deletions(-)
>>   delete mode 100644 arch/x86/include/asm/arch-baytrail/fsp/azalia.h
>>   create mode 100644 arch/x86/include/asm/fsp/fsp_azalia.h
>>
>> diff --git a/arch/x86/cpu/baytrail/fsp_configs.c
>> b/arch/x86/cpu/baytrail/fsp_configs.c
>> index 45f9bf9..926f26b 100644
>> --- a/arch/x86/cpu/baytrail/fsp_configs.c
>> +++ b/arch/x86/cpu/baytrail/fsp_configs.c
>> @@ -8,13 +8,12 @@
>>     #include <common.h>
>>   #include <fdtdec.h>
>> -#include <asm/arch/fsp/azalia.h>
>>   #include <asm/fsp/fsp_support.h>
>>     DECLARE_GLOBAL_DATA_PTR;
>>     /* ALC262 Verb Table - 10EC0262 */
>> -static const uint32_t verb_table_data13[] = {
>> +static const u32 verb_table_data13[] = {
>>         /* Pin Complex (NID 0x11) */
>>         0x01171cf0,
>>         0x01171d11,
>> @@ -94,7 +93,7 @@ static const uint32_t verb_table_data13[] = {
>>    * Codec Address: CAd value (0/1/2)
>>    * Codec Vendor: 0x10EC0262
>>    */
>> -static const struct pch_azalia_verb_table azalia_verb_table[] = {
>> +static const struct azalia_verb_table azalia_verb_table[] = {
>>         {
>>                 {
>>                         0x10ec0262,
>> @@ -108,16 +107,16 @@ static const struct pch_azalia_verb_table
>> azalia_verb_table[] = {
>>         }
>>   };
>>   -const struct pch_azalia_config azalia_config = {
>> +const struct azalia_config azalia_config = {
>>         .pme_enable = 1,
>>         .docking_supported = 1,
>>         .docking_attached = 0,
>>         .hdmi_codec_enable = 1,
>>         .azalia_v_ci_enable = 1,
>>         .rsvdbits = 0,
>> -       .azalia_verb_table_num = 1,
>> -       .azalia_verb_table = azalia_verb_table,
>> -       .reset_wait_timer_us = 300
>> +       .verb_table_num = 1,
>> +       .verb_table = azalia_verb_table,
>> +       .reset_wait_timer_ms = 300
>>   };
>>     /**
>> diff --git a/arch/x86/include/asm/arch-baytrail/fsp/azalia.h
>> b/arch/x86/include/asm/arch-baytrail/fsp/azalia.h
>> deleted file mode 100644
>> index d96a20f..0000000
>> --- a/arch/x86/include/asm/arch-baytrail/fsp/azalia.h
>> +++ /dev/null
>> @@ -1,39 +0,0 @@
>> -/*
>> - * Copyright (C) 2013, Intel Corporation
>> - * Copyright (C) 2015 Google, Inc
>> - *
>> - * SPDX-License-Identifier:    Intel
>> - */
>> -
>> -#ifndef _FSP_AZALIA_H_
>> -#define _FSP_AZALIA_H_
>> -
>> -struct __packed pch_azalia_verb_table_header {
>> -       uint32_t vendor_device_id;
>> -       uint16_t sub_system_id;
>> -       uint8_t revision_id;            /* 0xff applies to all steppings
>> */
>> -       uint8_t front_panel_support;
>> -       uint16_t number_of_rear_jacks;
>> -       uint16_t number_of_front_jacks;
>> -};
>> -
>> -struct __packed pch_azalia_verb_table {
>> -       struct pch_azalia_verb_table_header verb_table_header;
>> -       const uint32_t *verb_table_data;
>> -};
>> -
>> -struct __packed pch_azalia_config {
>> -       uint8_t pme_enable:1;
>> -       uint8_t docking_supported:1;
>> -       uint8_t docking_attached:1;
>> -       uint8_t hdmi_codec_enable:1;
>> -       uint8_t azalia_v_ci_enable:1;
>> -       uint8_t rsvdbits:3;
>> -       /* number of verb tables provided by platform */
>> -       uint8_t azalia_verb_table_num;
>> -       const struct pch_azalia_verb_table *azalia_verb_table;
>> -       /* delay timer after azalia reset */
>> -       uint16_t reset_wait_timer_us;
>> -};
>> -
>> -#endif
>> diff --git a/arch/x86/include/asm/arch-braswell/fsp/fsp_vpd.h
>> b/arch/x86/include/asm/arch-braswell/fsp/fsp_vpd.h
>> index ecb01fa..99c4c0a 100644
>> --- a/arch/x86/include/asm/arch-braswell/fsp/fsp_vpd.h
>> +++ b/arch/x86/include/asm/arch-braswell/fsp/fsp_vpd.h
>> @@ -29,32 +29,6 @@ struct __packed memory_upd {
>>         u8 reserved[189];                       /* Offset 0x0043 */
>>   };
>>   -struct __packed azalia_verb_table_header {
>> -       u32 vendor_device_id;
>> -       u16 sub_system_id;
>> -       u8 revision_id;
>> -       u8 front_panel_support;
>> -       u16 number_of_rear_jacks;
>> -       u16 number_of_front_jacks;
>> -};
>> -
>> -struct __packed azalia_verb_table {
>> -       struct azalia_verb_table_header header;
>> -       u32 *data;
>> -};
>> -
>> -struct __packed azalia_config {
>> -       u8 pme_enable:1;
>> -       u8 docking_supported:1;
>> -       u8 docking_attached:1;
>> -       u8 hdmi_codec_enable:1;
>> -       u8 azalia_v_ci_enable:1;
>> -       u8 reserved:3;
>> -       u8 verb_table_num;
>> -       struct azalia_verb_table *verb_table;
>> -       u16 reset_wait_timer_ms;
>> -};
>> -
>>   struct gpio_family {
>>         u32 confg;
>>         u32 confg_changes;
>> diff --git a/arch/x86/include/asm/fsp/fsp_azalia.h
>> b/arch/x86/include/asm/fsp/fsp_azalia.h
>> new file mode 100644
>> index 0000000..a1467bf
>> --- /dev/null
>> +++ b/arch/x86/include/asm/fsp/fsp_azalia.h
>> @@ -0,0 +1,39 @@
>> +/*
>> + * Copyright (C) 2013, Intel Corporation
>> + * Copyright (C) 2015, Google, Inc
>> + *
>> + * SPDX-License-Identifier:    Intel
>> + */
>> +
>> +#ifndef _FSP_AZALIA_H_
>> +#define _FSP_AZALIA_H_
>> +
>> +struct __packed azalia_verb_table_header {
>> +       u32 vendor_device_id;
>> +       u16 sub_system_id;
>> +       u8 revision_id;         /* 0xff applies to all steppings */
>> +       u8 front_panel_support;
>> +       u16 number_of_rear_jacks;
>> +       u16 number_of_front_jacks;
>> +};
>> +
>> +struct __packed azalia_verb_table {
>> +       struct azalia_verb_table_header header;
>> +       const u32 *data;
>> +};
>> +
>> +struct __packed azalia_config {
>> +       u8 pme_enable:1;
>> +       u8 docking_supported:1;
>> +       u8 docking_attached:1;
>> +       u8 hdmi_codec_enable:1;
>> +       u8 azalia_v_ci_enable:1;
>> +       u8 rsvdbits:3;
>> +       /* number of verb tables provided by platform */
>> +       u8 verb_table_num;
>> +       const struct azalia_verb_table *verb_table;
>> +       /* delay timer after azalia reset */
>> +       u16 reset_wait_timer_ms;
>> +};
>> +
>> +#endif
>> diff --git a/arch/x86/include/asm/fsp/fsp_support.h
>> b/arch/x86/include/asm/fsp/fsp_support.h
>> index 97a50b0..df3add0 100644
>> --- a/arch/x86/include/asm/fsp/fsp_support.h
>> +++ b/arch/x86/include/asm/fsp/fsp_support.h
>> @@ -15,6 +15,7 @@
>>   #include "fsp_hob.h"
>>   #include "fsp_infoheader.h"
>>   #include "fsp_bootmode.h"
>> +#include "fsp_azalia.h"
>>   #include <asm/arch/fsp/fsp_vpd.h>
>>   #include <asm/arch/fsp/fsp_configs.h>
>>
>
>
> Reviewed-by: Stefan Roese <sr at denx.de>
>

applied to u-boot-x86, thanks!


More information about the U-Boot mailing list