All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Lankhorst, Maarten" <maarten.lankhorst@intel.com>
To: "intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>,
	"Kumar, Mahesh1" <mahesh1.kumar@intel.com>
Cc: "Zanoni, Paulo R" <paulo.r.zanoni@intel.com>
Subject: Re: [PATCH 4/8] drm/i915: Addition wrapper for fixed16.16 operation
Date: Wed, 21 Jun 2017 09:40:50 +0000	[thread overview]
Message-ID: <1498038049.17446.0.camel@intel.com> (raw)
In-Reply-To: <20170621061416.7263-5-mahesh1.kumar@intel.com>

Mahesh Kumar schreef op wo 21-06-2017 om 11:44 [+0530]:
> This patch introduce addition wrapper for fixed point 16.16
> operations.
> Which will be used by later patches to avoid direct member variables
> access of fixed_16_16_t structure.
> 
> add_fixed16 : takes 2 fixed_16_16_t variable & returns fixed_16_16_t
> add_fixed16_u32 : takes fixed_16_16_t & u32 variable & returns
> fixed_16_16_t

Much easier to review. :)

Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> for
first 4 patches.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2017-06-21  9:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-21  6:14 [PATCH 0/8] Fixed16.16 wrapper cleanup & wm optimization Mahesh Kumar
2017-06-21  6:14 ` [PATCH 1/8] drm/i915: take-out common clamping code of fixed16 wrappers Mahesh Kumar
2017-06-21  6:14 ` [PATCH 2/8] drm/i915: Always perform internal fixed16 division in 64 bits Mahesh Kumar
2017-06-21  6:14 ` [PATCH 3/8] drm/i915: cleanup fixed-point wrappers naming Mahesh Kumar
2017-06-21  6:14 ` [PATCH 4/8] drm/i915: Addition wrapper for fixed16.16 operation Mahesh Kumar
2017-06-21  9:40   ` Lankhorst, Maarten [this message]
2017-06-21 14:32     ` Mahesh Kumar
2017-06-21  6:14 ` [PATCH 5/8] drm/i915/skl+: WM calculation don't require height Mahesh Kumar
2017-06-21  6:14 ` [PATCH 6/8] drm/i915/skl+: unify cpp value in WM calculation Mahesh Kumar
2017-06-21  6:14 ` [PATCH 7/8] drm/i915/skl+: Optimize " Mahesh Kumar
2017-06-21  6:14 ` [PATCH 8/8] drm/i915/gen10: Calculate and enable transition WM Mahesh Kumar
2017-06-21  6:29 ` ✓ Fi.CI.BAT: success for Fixed16.16 wrapper cleanup & wm optimization (rev3) Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1498038049.17446.0.camel@intel.com \
    --to=maarten.lankhorst@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=mahesh1.kumar@intel.com \
    --cc=paulo.r.zanoni@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.