On Fri, 29 Sep 2017, Joonas Lahtinen wrote:
> On Wed, 2017-09-27 at 12:32 -0700, Rodrigo Vivi wrote:
>> On Wed, Sep 27, 2017 at 12:03 AM, Jani Nikula
>> wrote:
>> > On Tue, 26 Sep 2017, Rodrigo Vivi wrote:
>>
On Wed, 2017-09-27 at 12:32 -0700, Rodrigo Vivi wrote:
> On Wed, Sep 27, 2017 at 12:03 AM, Jani Nikula
> wrote:
> > On Tue, 26 Sep 2017, Rodrigo Vivi wrote:
> > > On Tue, Sep 26, 2017 at 09:21:43PM +, Paulo Zanoni wrote:
> > > > Em Ter,
On Wed, Sep 27, 2017 at 12:03 AM, Jani Nikula
wrote:
> On Tue, 26 Sep 2017, Rodrigo Vivi wrote:
>> On Tue, Sep 26, 2017 at 09:21:43PM +, Paulo Zanoni wrote:
>>> Em Ter, 2017-09-26 às 14:13 -0700, Rodrigo Vivi escreveu:
>>> > Let's stop
On Tue, 26 Sep 2017, Rodrigo Vivi wrote:
> On Tue, Sep 26, 2017 at 09:21:43PM +, Paulo Zanoni wrote:
>> Em Ter, 2017-09-26 às 14:13 -0700, Rodrigo Vivi escreveu:
>> > Let's stop this usage before it spreads so much.
>> >
>> > 1. This check is not part of usual
On Tue, Sep 26, 2017 at 09:21:43PM +, Paulo Zanoni wrote:
> Em Ter, 2017-09-26 às 14:13 -0700, Rodrigo Vivi escreveu:
> > Let's stop this usage before it spreads so much.
> >
> > 1. This check is not part of usual searches happening when adding
> > new platform.
> > 2. There is already a
On Tue, Sep 26, 2017 at 02:13:46PM -0700, Rodrigo Vivi wrote:
> Let's stop this usage before it spreads so much.
>
> 1. This check is not part of usual searches happening when adding
> new platform.
> 2. There is already a duplication here with INTEL_INFO(dev_priv)->gen
> and INTEL_GEN(dev_priv).
Em Ter, 2017-09-26 às 14:13 -0700, Rodrigo Vivi escreveu:
> Let's stop this usage before it spreads so much.
>
> 1. This check is not part of usual searches happening when adding
> new platform.
> 2. There is already a duplication here with INTEL_INFO(dev_priv)->gen
> and INTEL_GEN(dev_priv).
>
Let's stop this usage before it spreads so much.
1. This check is not part of usual searches happening when adding
new platform.
2. There is already a duplication here with INTEL_INFO(dev_priv)->gen
and INTEL_GEN(dev_priv).
So let's please avoid yet another way.
Fixes: b22ca995ba1c ("drm/i915: