Re: [PATCH] drm/i915: Remove unecessary PORT3 definition.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



patch merged to dinq with the fixed spelling.
Thanks for the review

On Tue, Jun 6, 2017 at 11:14 AM, Manasi Navare
<manasi.d.navare@xxxxxxxxx> wrote:
> Just a very tiny nitpick about correction in the
> spelling of "unnecessary" in the title.
>
> Other than that
>
> Reviewed-by: Manasi Navare <manasi.d.navare@xxxxxxxxx>
>
> On Tue, Jun 06, 2017 at 09:06:36AM -0700, Rodrigo Vivi wrote:
>> Let's be picky and just use PICK directly.
>> So we can extend this later without creating
>> a new PORT_X por every new number of ports we
>> have to handle.
>>
>> Cc: Manasi Navare <manasi.d.navare@xxxxxxxxx>
>> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
>> ---
>>  drivers/gpu/drm/i915/i915_reg.h | 6 ++----
>>  1 file changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
>> index 1329420..c3bed89 100644
>> --- a/drivers/gpu/drm/i915/i915_reg.h
>> +++ b/drivers/gpu/drm/i915/i915_reg.h
>> @@ -58,10 +58,8 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg)
>>  #define _MMIO_TRANS(tran, a, b) _MMIO(_TRANS(tran, a, b))
>>  #define _PORT(port, a, b) ((a) + (port)*((b)-(a)))
>>  #define _MMIO_PORT(port, a, b) _MMIO(_PORT(port, a, b))
>> -#define _PIPE3(pipe, ...) _PICK(pipe, __VA_ARGS__)
>> -#define _MMIO_PIPE3(pipe, a, b, c) _MMIO(_PIPE3(pipe, a, b, c))
>> -#define _PORT3(port, ...) _PICK(port, __VA_ARGS__)
>> -#define _MMIO_PORT3(pipe, a, b, c) _MMIO(_PORT3(pipe, a, b, c))
>> +#define _MMIO_PIPE3(pipe, a, b, c) _MMIO(_PICK(pipe, a, b, c))
>> +#define _MMIO_PORT3(pipe, a, b, c) _MMIO(_PICK(pipe, a, b, c))
>>  #define _PHY3(phy, ...) _PICK(phy, __VA_ARGS__)
>>  #define _MMIO_PHY3(phy, a, b, c) _MMIO(_PHY3(phy, a, b, c))
>>
>> --
>> 1.9.1
>>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux