Missed one minor indent comment:

> +
> +#define PCI_CHIP_CHV_0 0x22B0
> +#define PCI_CHIP_CHV_1 0x22B1
> +#define PCI_CHIP_CHV_2 0x22B2
> +#define PCI_CHIP_CHV_3 0x22B3
> +#define IS_CHERRYVIEW(devid) (devid == PCI_CHIP_CHV_0 ||   \
> +   devid == PCI_CHIP_CHV_1 || \
> +   devid == PCI_CHIP_CHV_2 || \
> +   devid == PCI_CHIP_CHV_3)
> +
It's better to keep all the devid aligned to the same column.

Thanks,
Zhigang Gong.
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet

Reply via email to