On Mon, Sep 03, 2018 at 03:45:13AM -0400, Jyoti Yadav wrote:
> As DMC Package contain DMC FW for multiple steppings including default
> stepping. This patch will help to load FW for that particular stepping,
> if FW for that stepping is available, instead of loading default FW.
>
> Signed-off-by: Jyoti Yadav <jyoti.r.yadav@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/intel_csr.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
> index 1ec4f09..f6352ab 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -183,6 +183,11 @@ struct stepping_info {
> {'A', '0'}, {'A', '1'}, {'A', '2'},
> {'B', '0'}, {'B', '1'}, {'B', '2'}
> };
> +static const struct stepping_info icl_stepping_info[] = {
> + {'A', '0'}, {'A', '1'}, {'A', '2'},
Looks like the PCI revision ID for A1 and A2 is missing/typoed in BSpec,
but these are probably the correct values in any case. The current
firmware has an 'A','*' entry for all A steppings, so that works out
too.
> + {'B', '0'}, {'B', '2'},
> + {'C', '0'}
> +};
>
> static const struct stepping_info no_stepping_info = { '*', '*' };
>
> @@ -198,6 +203,9 @@ struct stepping_info {
> } else if (IS_BROXTON(dev_priv)) {
> size = ARRAY_SIZE(bxt_stepping_info);
> si = bxt_stepping_info;
> + } else if (IS_ICELAKE(dev_priv)) {
> + size = ARRAY_SIZE(icl_stepping_info);
> + si = icl_stepping_info;
While fixing the formatting asked by Rodrigo, could you also keep the
new->old platform order and so make this and icl_stepping_info be the
first in the list?
With those looks ok:
Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx>
> } else {
> size = 0;
> si = NULL;
> --
> 1.9.1
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
