From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932799AbcHKJYu (ORCPT ); Thu, 11 Aug 2016 05:24:50 -0400 Received: from mailapp01.imgtec.com ([195.59.15.196]:18804 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752307AbcHKJYs (ORCPT ); Thu, 11 Aug 2016 05:24:48 -0400 Date: Thu, 11 Aug 2016 10:24:35 +0100 From: Eric Engestrom To: Gustavo Padovan CC: Greg Kroah-Hartman , , Daniel Stone , Daniel Vetter , Riley Andrews , , , Arve =?utf-8?B?SGrDuG5uZXbDpWc=?= , , Gustavo Padovan , John Harrison Subject: Re: [PATCH v2 0/6] de-stage SW_SYNC validation frawework Message-ID: <20160811092435.GN10429@imgtec.com> References: <1470691462-19140-1-git-send-email-gustavo@padovan.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <1470691462-19140-1-git-send-email-gustavo@padovan.org> User-Agent: Mutt/1.6.2 (2016-07-01) X-Originating-IP: [10.60.4.28] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 08, 2016 at 06:24:16PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan > > Hi Greg, > > This is the last step in the Sync Framwork de-stage task. It de-stage > the SW_SYNC validation framework and the sync_debug info debugfs file. > > The first 2 patches are clean up and improvements and the rest is preparation > to de-stage and then finally the actual de-stage. > > v2: > - add documentation about the SW_SYNC ioctl API (comments from Pavel Machek) > - remove for now patch to add sync_pt name to debugfs > > Please review, > > Gustavo > > --- > Gustavo Padovan (6): > staging/android: remove doc from sw_sync > staging/android: do not let userspace trigger WARN_ON > staging/android: move trace/sync.h to sync_trace.h > staging/android: prepare sw_sync files for de-staging > staging/android: add Doc for SW_SYNC ioctl interface > dma-buf/sw_sync: de-stage SW_SYNC > > drivers/dma-buf/Kconfig | 13 ++ > drivers/dma-buf/Makefile | 1 + > drivers/dma-buf/sw_sync.c | 349 +++++++++++++++++++++++++++++++++++ > drivers/dma-buf/sync_debug.c | 230 +++++++++++++++++++++++ > drivers/dma-buf/sync_debug.h | 69 +++++++ > drivers/dma-buf/sync_trace.h | 32 ++++ > drivers/staging/android/Kconfig | 13 -- > drivers/staging/android/Makefile | 1 - > drivers/staging/android/sw_sync.c | 344 ---------------------------------- > drivers/staging/android/sync_debug.c | 230 ----------------------- > drivers/staging/android/sync_debug.h | 84 --------- > drivers/staging/android/trace/sync.h | 32 ---- > 12 files changed, 694 insertions(+), 704 deletions(-) > create mode 100644 drivers/dma-buf/sw_sync.c > create mode 100644 drivers/dma-buf/sync_debug.c > create mode 100644 drivers/dma-buf/sync_debug.h > create mode 100644 drivers/dma-buf/sync_trace.h > delete mode 100644 drivers/staging/android/sw_sync.c > delete mode 100644 drivers/staging/android/sync_debug.c > delete mode 100644 drivers/staging/android/sync_debug.h > delete mode 100644 drivers/staging/android/trace/sync.h When you send your next revision, could you use `git format-patch -M`? A good 95% of the lines in these patches aren't actually modified, just moved around, which makes it much harder to spot the actual changes :) Cheers, Eric > > -- > 2.5.5