Re: [Spice-devel] [spice v3] utils: Remove the LL suffix from NSEC_PER_SEC

2019-06-15 Thread Frediano Ziglio
> > This constant fits in a 32 bit signed integer so it does not need the > suffix. However some of the derived constants don't so use an uint64_t > cast to avoid the long vs long long confusion (such as in print > statements). > Also some of the expressions these constants are used in may overflo

[Spice-devel] [PATCH 06/59] drm/prime: Actually remove DRIVER_PRIME everywhere

2019-06-15 Thread Daniel Vetter
Split out to make the functional changes stick out more. v2: amdgpu gained DRIVER_SYNCOBJ_TIMELINE. v3: amdgpu lost DRIVER_SYNCOBJ_TIMELINE. Signed-off-by: Daniel Vetter Cc: amd-...@lists.freedesktop.org Cc: etna...@lists.freedesktop.org Cc: freedr...@lists.freedesktop.org Cc: intel-...@lists.f

Re: [Spice-devel] [PATCH 06/59] drm/prime: Actually remove DRIVER_PRIME everywhere

2019-06-15 Thread Sam Ravnborg
Hi Daniel. Minor nitpick.. > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 65d599065709..4fd09a9ad67a 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -3193,7 +3193,7 @@ static struct drm_driver driver = { >

Re: [Spice-devel] [PATCH x11spice v2] Changes to build on the 32 bit raspberry pi.

2019-06-15 Thread Frediano Ziglio
> > Signed-off-by: Jeremy White > --- > v2 Use (u) (u) thing instead of (u) ((u) thing) >Simplify some casting where possible > --- > src/scan.c| 6 +++--- > src/session.c | 8 > src/spice.c | 18 +++--- > 3 files changed, 18 insertions(+), 14 deletions(-) > >