From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Ardelean, Alexandru" Subject: Re: [PATCH v4 0/9] staging: iio: ad7780: move out of staging Date: Fri, 1 Mar 2019 08:35:17 +0000 Message-ID: <72a54cd5f58aeb9507b95b7e33ca3d9a38c853e9.camel@analog.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Content-ID: <5442297840C98448AB8000052EEEA3FE@analog.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: "lars@metafoo.de" , "robh+dt@kernel.org" , "Popa, Stefan Serban" , "knaack.h@gmx.de" , "jic23@kernel.org" , "Hennerich, Michael" , "renatogeh@gmail.com" , "mark.rutland@arm.com" , "giuliano.belinassi@usp.br" , "pmeerw@pmeerw.net" , "gregkh@linuxfoundation.org" Cc: "linux-iio@vger.kernel.org" , "devel@driverdev.osuosl.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "kernel-usp@googlegroups.com" List-Id: devicetree@vger.kernel.org On Thu, 2019-02-28 at 11:23 -0300, Renato Lui Geh wrote: > The patch-series is a bit big. I guess that the intent is to move this out-of-staging, but various patches are holding this in it's place. For patch series above a certain size, you could get many re-spins [V2,3,4... so on]. You could send some of the changes as individual patches, or group them in series of 1,2 or 3 patches. That way, you "parallelize" patch sending, and when you get reviews on each patch, you can re-spin them individually. You'll find over time that certain patches get accepted on V1, others on V2 and some on V7 [ hopefully, there isn't any frustration at that point ]. Well, this is a technique I use to distribute some of my upstream-patch- work, so that I can switch easier between internal-work & upstreaming-work. Coming back to this patch-series. My general input, is that the patches are fine over-all; some are just cosmetics/noise/a-different-way-of-doing-things-for-this-driver, and those usually can be left to preference [of the maintainer usually]. I do suggest to not hurry when re-spinning patches, and not change too much the number of patches in a new series. That can complicate things sometimes. But, if doing small patch-series or individual patches, you won't have this problem too much. Thanks Alex > > This series of patches contains the following: > - Adds user input for the 'gain' and 'filter' GPIO pins for the ad778x > family chips; > - Filter reading for the ad778x; > - Sets pattern macro values and mask for PATTERN status bits; > - Adds ID values for the ad7170, ad7171, ad7780 and ad7781 for ID > status bits checking; > - Moves regulator initialization to after GPIO init to maintain > consistency between probe and remove; > - Copyright edits, adding SPDX identifier and new copyright holder; > - Moves the ad7780 driver out of staging to the mainline; > - Adds device tree binding for the ad7780 driver. > > Renato Lui Geh (9): > staging: iio: ad7780: add gain & filter gpio support > staging: iio: ad7780: add filter reading to ad778x > staging: iio: ad7780: set pattern values and masks directly > staging:iio:ad7780: add chip ID values and mask > staging: iio: ad7780: move regulator to after GPIO init > staging: iio: ad7780: add SPDX identifier > staging: iio: ad7780: add new copyright holder > staging: iio: ad7780: moving ad7780 out of staging > staging: iio: ad7780: add device tree binding > > Changelog: > *v3 > - SPDX and regulator init as patches > - Renamed filter to odr and ad778x_filter to ad778x_odr_avail > - Removed unnecessary regulator disabling > - Removed unnecessary AD_SD_CHANNEL macro > - Changed unsigned int to unsigned long long to avoid overflow > *v4 > - Split gain & filter patch into two, with the new commit adding only > filter reading > - Changed pattern values to direct values, and added pattern mask > - Added ID values and mask > - Added new copyright holder > - Added device tree binding to the ad7780 driver > > .../bindings/iio/adc/adi,ad7780.txt | 48 +++ > drivers/iio/adc/Kconfig | 12 + > drivers/iio/adc/Makefile | 1 + > drivers/iio/adc/ad7780.c | 365 ++++++++++++++++++ > drivers/staging/iio/adc/Kconfig | 13 - > drivers/staging/iio/adc/Makefile | 1 - > drivers/staging/iio/adc/ad7780.c | 277 ------------- > 7 files changed, 426 insertions(+), 291 deletions(-) > create mode 100644 > Documentation/devicetree/bindings/iio/adc/adi,ad7780.txt > create mode 100644 drivers/iio/adc/ad7780.c > delete mode 100644 drivers/staging/iio/adc/ad7780.c > > -- > 2.21.0 >