On Mon, Apr 18, 2016 at 06:09:28PM +0300, Joonas Lahtinen wrote:
> @@ -251,7 +245,7 @@ static int swsci(struct drm_device *dev, u32 function, 
> u32 parm, u32 *parm_out)
>       struct drm_i915_private *dev_priv = dev->dev_private;
>       struct opregion_swsci *swsci = dev_priv->opregion.swsci;
>       u32 main_function, sub_function, scic;
> -     u16 pci_swsci;
> +     u16 swsci_reg;

This threw me. In other places _reg means the register offset and _val
means the value we want to poke into the register.

Happy with everything else.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to