From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sylwester Nawrocki Subject: Re: [PATCH 6/6] mmc: sdhci-s3c: Add device tree support Date: Thu, 05 Jan 2012 17:22:56 +0100 Message-ID: <4F05CE60.5090402@samsung.com> References: <1320266163-15292-1-git-send-email-thomas.abraham@linaro.org> <1320266163-15292-7-git-send-email-thomas.abraham@linaro.org> <20111107211726.GC28491@ponder.secretlab.ca> <4F047235.2090503@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7BIT Return-path: In-reply-to: Sender: linux-samsung-soc-owner@vger.kernel.org To: Thomas Abraham Cc: kgene.kim@samsung.com, Grant Likely , linux-mmc@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, cjb@laptop.org, rob.herring@calxeda.com, linux-samsung-soc@vger.kernel.org, ben-linux@fluff.org List-Id: linux-mmc@vger.kernel.org Hi Thomas, On 01/05/2012 04:45 PM, Thomas Abraham wrote: >>>> On Thu, Nov 03, 2011 at 02:06:03AM +0530, Thomas Abraham wrote: >>>>> Add device tree based discovery support for Samsung's sdhci controller >>>>> >>>>> Cc: Ben Dooks >>>>> Signed-off-by: Thomas Abraham >>>>> --- >>>>> +Example: >>>>> + sdhci@12530000 { >>>>> + compatible = "samsung,exynos4210-sdhci"; >>>>> + reg = <0x12530000 0x100>; >>>>> + interrupts = <139>; >>>>> + samsung,sdhci-bus-width = <4>; >>>>> + linux,mmc_cap_4_bit_data; >>>> >>>> Following on from my reply on patch 5, this is an example of exactly >>>> what I'm talking about. This node both sets bus-width to '4', and >>>> sets the 4_bit_data flag. Don't you think that the driver would be >>>> smart enough to set the 4_bit_data flag when the bus width was set to >>>> 4? >>> >>> Yes, that is true. I will modify the driver based on your comments and >>> resubmit this patch. >> >> Are we going to have those patches in 3.3-rc1 or only in 3.4 ? > > Sorry for the delaying in completing this patchset. I will redo this > patchset and submit it soon. But 3.3-rc1 looks unlikely. Ok, thanks. It's fine, there is no rush. I was just curious because we have reused this sdhci driver update series and it seemed essential for initial DT support on Exynos4 machines. Just FYI I have prepared initial DT support patches for the S5P MIPI-CSI2 (s5p-mipi-csis) receiver driver. But the whole camera subsystem needs more work, i.e. resource dependencies (clock) between all involved devices (like image sensors and video capture platform devices) need to be resolved and method of registering sub-devices in the V4L2 core will likely need to be changed. -- Thanks, Sylwester From mboxrd@z Thu Jan 1 00:00:00 1970 From: s.nawrocki@samsung.com (Sylwester Nawrocki) Date: Thu, 05 Jan 2012 17:22:56 +0100 Subject: [PATCH 6/6] mmc: sdhci-s3c: Add device tree support In-Reply-To: References: <1320266163-15292-1-git-send-email-thomas.abraham@linaro.org> <1320266163-15292-7-git-send-email-thomas.abraham@linaro.org> <20111107211726.GC28491@ponder.secretlab.ca> <4F047235.2090503@samsung.com> Message-ID: <4F05CE60.5090402@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Thomas, On 01/05/2012 04:45 PM, Thomas Abraham wrote: >>>> On Thu, Nov 03, 2011 at 02:06:03AM +0530, Thomas Abraham wrote: >>>>> Add device tree based discovery support for Samsung's sdhci controller >>>>> >>>>> Cc: Ben Dooks >>>>> Signed-off-by: Thomas Abraham >>>>> --- >>>>> +Example: >>>>> + sdhci at 12530000 { >>>>> + compatible = "samsung,exynos4210-sdhci"; >>>>> + reg = <0x12530000 0x100>; >>>>> + interrupts = <139>; >>>>> + samsung,sdhci-bus-width = <4>; >>>>> + linux,mmc_cap_4_bit_data; >>>> >>>> Following on from my reply on patch 5, this is an example of exactly >>>> what I'm talking about. This node both sets bus-width to '4', and >>>> sets the 4_bit_data flag. Don't you think that the driver would be >>>> smart enough to set the 4_bit_data flag when the bus width was set to >>>> 4? >>> >>> Yes, that is true. I will modify the driver based on your comments and >>> resubmit this patch. >> >> Are we going to have those patches in 3.3-rc1 or only in 3.4 ? > > Sorry for the delaying in completing this patchset. I will redo this > patchset and submit it soon. But 3.3-rc1 looks unlikely. Ok, thanks. It's fine, there is no rush. I was just curious because we have reused this sdhci driver update series and it seemed essential for initial DT support on Exynos4 machines. Just FYI I have prepared initial DT support patches for the S5P MIPI-CSI2 (s5p-mipi-csis) receiver driver. But the whole camera subsystem needs more work, i.e. resource dependencies (clock) between all involved devices (like image sensors and video capture platform devices) need to be resolved and method of registering sub-devices in the V4L2 core will likely need to be changed. -- Thanks, Sylwester