From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sylwester Nawrocki Subject: Re: [PATCH v7 0/9] Generic PHY Framework Date: Mon, 24 Jun 2013 21:27:20 +0200 Message-ID: <51C89D98.1070806@samsung.com> References: <1371113039-5784-1-git-send-email-kishon@ti.com> <51BEE203.6090103@samsung.com> <20130618094942.GY5461@arwen.pp.htv.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <20130618094942.GY5461@arwen.pp.htv.fi> Sender: linux-kernel-owner@vger.kernel.org To: balbi@ti.com Cc: Kishon Vijay Abraham I , grant.likely@linaro.org, tony@atomide.com, arnd@arndb.de, swarren@nvidia.com, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-usb@vger.kernel.org, gregkh@linuxfoundation.org, akpm@linux-foundation.org, rob.herring@calxeda.com, rob@landley.net, b-cousson@ti.com, linux@arm.linux.org.uk, benoit.cousson@linaro.org, mchehab@redhat.com, cesarb@cesarb.net, davem@davemloft.net, rnayak@ti.com, shawn.guo@linaro.org, santosh.shilimkar@ti.com, devicetree-discuss@lists.ozlabs.org, linux-doc@vger.kernel.org, nsekhar@ti.com List-Id: linux-omap@vger.kernel.org Hi, On 06/18/2013 11:49 AM, Felipe Balbi wrote: > On Mon, Jun 17, 2013 at 12:16:35PM +0200, Sylwester Nawrocki wrote: >> I have already used this API for our MIPI CSI-2/DSIM DPHYs driver, >> the RFC patch series can be found at [1]. >> >> Thanks, >> Sylwester >> >> [1] http://www.spinics.net/lists/arm-kernel/msg251666.html > > one comment to that series: > > let's make the phy names standardized, how about phy-exynos-video-mipi.c > or phy-mipi-csi-dsim.c ? Yes, I have been thinking about that, wasn't sure exactly what pattern to chose. I would make it phy-exynos-mipi-video.c, phy-exynos-dsim-csis.c feels a bit odd. phy-mipi-csis-dsim.c might be to generic as MIPI CSIS stands for MIPI CSI Slave and MIPI DSIM - MIPI DSI Master. And there might be (actually are) IP blocks in an SoC that use the other MIPI Aliance standards. For the HDMI PHY I guess just phy-exynos-hdmi.c could be used. Thanks, Sylwester From mboxrd@z Thu Jan 1 00:00:00 1970 From: s.nawrocki@samsung.com (Sylwester Nawrocki) Date: Mon, 24 Jun 2013 21:27:20 +0200 Subject: [PATCH v7 0/9] Generic PHY Framework In-Reply-To: <20130618094942.GY5461@arwen.pp.htv.fi> References: <1371113039-5784-1-git-send-email-kishon@ti.com> <51BEE203.6090103@samsung.com> <20130618094942.GY5461@arwen.pp.htv.fi> Message-ID: <51C89D98.1070806@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On 06/18/2013 11:49 AM, Felipe Balbi wrote: > On Mon, Jun 17, 2013 at 12:16:35PM +0200, Sylwester Nawrocki wrote: >> I have already used this API for our MIPI CSI-2/DSIM DPHYs driver, >> the RFC patch series can be found at [1]. >> >> Thanks, >> Sylwester >> >> [1] http://www.spinics.net/lists/arm-kernel/msg251666.html > > one comment to that series: > > let's make the phy names standardized, how about phy-exynos-video-mipi.c > or phy-mipi-csi-dsim.c ? Yes, I have been thinking about that, wasn't sure exactly what pattern to chose. I would make it phy-exynos-mipi-video.c, phy-exynos-dsim-csis.c feels a bit odd. phy-mipi-csis-dsim.c might be to generic as MIPI CSIS stands for MIPI CSI Slave and MIPI DSIM - MIPI DSI Master. And there might be (actually are) IP blocks in an SoC that use the other MIPI Aliance standards. For the HDMI PHY I guess just phy-exynos-hdmi.c could be used. Thanks, Sylwester