[U-Boot] [PATCH] x86: tsc: Add support for APL soc

Bin Meng bmeng.cn at gmail.com
Mon Nov 19 02:23:52 UTC 2018


Hi Bernhard,

On Thu, Oct 25, 2018 at 6:51 PM Bernhard Messerklinger
<bernhard.messerklinger at br-automation.com> wrote:
>

Missing commit message

APL means ApolloLake? Could you please spell it out?

> Signed-off-by: Bernhard Messerklinger <bernhard.messerklinger at br-automation.com>
> ---
>
>  drivers/timer/tsc_timer.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/timer/tsc_timer.c b/drivers/timer/tsc_timer.c
> index da7c812908..b2a982812a 100644
> --- a/drivers/timer/tsc_timer.c
> +++ b/drivers/timer/tsc_timer.c
> @@ -64,6 +64,8 @@ static struct freq_desc freq_desc_tables[] = {
>                         80000, 93300, 90000, 88900, 87500 } },
>         /* Ivybridge */
>         { 6, 0x3a, 2, { 0, 0, 0, 0, 0, 0, 0, 0, 0 } },
> +       /* Intel Atom processor E3900 series */
> +       { 6, 0x5c, 2, { 0, 0, 0, 0, 0, 0, 0, 0, 0 } },

Please avoid hardcoding TSC freq this way. Isn't calibrating from MSR
not working for ApolloLake?

>  };
>
>  static int match_cpu(u8 family, u8 model)
> --

Regards,
Bin


More information about the U-Boot mailing list