From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751693AbcEKKeU (ORCPT ); Wed, 11 May 2016 06:34:20 -0400 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:50425 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751372AbcEKKeT (ORCPT ); Wed, 11 May 2016 06:34:19 -0400 Message-ID: <1462962853.2924.19.camel@pengutronix.de> Subject: Re: [RFC PATCH 13/21] reset: uniphier: add reset driver for UniPhier PH1-LD4 SoC From: Philipp Zabel To: Masahiro Yamada Cc: linux-clk@vger.kernel.org, Arnd Bergmann , linux-arm-kernel , Linux Kernel Mailing List Date: Wed, 11 May 2016 12:34:13 +0200 In-Reply-To: References: <1462873862-30940-1-git-send-email-yamada.masahiro@socionext.com> <1462873862-30940-14-git-send-email-yamada.masahiro@socionext.com> <1462883138.9155.3.camel@pengutronix.de> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.9-1+b1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2001:67c:670:100:96de:80ff:fec2:9969 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Masahiro, Am Mittwoch, den 11.05.2016, 11:52 +0900 schrieb Masahiro Yamada: > Hi Philipp, > > > 2016-05-10 21:25 GMT+09:00 Philipp Zabel : > > Hi Masahiro, > > > > Am Dienstag, den 10.05.2016, 18:50 +0900 schrieb Masahiro Yamada: > >> This series is just for review. > >> Please do not apply this patch. > >> > >> Signed-off-by: Masahiro Yamada > > > > No need for all these tiny drivers. If you probe from DT as Lee > > requests, you can let of_device_id.data point to a compatible specific > > uniphier_reset_data. > > > > I will need to fill the table a bit more to complete this work, > but I do not think this reset driver will become as big as the clk driver. > (I guess it will be less than 1000 lines.) > > If you prefer a single file for the whole of this SoC family, > I can squash all the files into one. > (I can split it if I need to do so in the future.) I'm fine with split files if you think the complete lists are too long, but in that case export the uniphier_reset_data I don't care about the lists being in a single file so much as I'd prefer to avoid the duplicated boilerplate of one module_platform_driver per SoC family member. > > >