RE: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-28 Thread Seungwon Jeon
Hi Thomas, Thank you for your effort. Some reviews seems like to be omitted. Please check more. On Sunday, August 26, 2012 Thomas Abraham thomas.abra...@linaro.org wrote: Samsung Exynos SoC's extend the dw-mshc controller for additional clock and bus control. Add support for these extensions

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-28 Thread Thomas Abraham
Hi Seungwon, On 28 August 2012 12:36, Seungwon Jeon tgih@samsung.com wrote: Hi Thomas, Thank you for your effort. Some reviews seems like to be omitted. Please check more. On Sunday, August 26, 2012 Thomas Abraham thomas.abra...@linaro.org wrote: Samsung Exynos SoC's extend the dw-mshc

RE: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-28 Thread Seungwon Jeon
On Tuesday, August 28, 2012 Thomas Abraham thomas.abra...@linaro.org wrote: Hi Seungwon, On 28 August 2012 12:36, Seungwon Jeon tgih@samsung.com wrote: Hi Thomas, Thank you for your effort. Some reviews seems like to be omitted. Please check more. On Sunday, August 26, 2012

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-28 Thread Thomas Abraham
On 28 August 2012 16:13, Seungwon Jeon tgih@samsung.com wrote: On Tuesday, August 28, 2012 Thomas Abraham thomas.abra...@linaro.org wrote: Hi Seungwon, On 28 August 2012 12:36, Seungwon Jeon tgih@samsung.com wrote: [...] + /* + * Exynos4412 and Exynos5250 extends the use

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Jaehoon Chung
Hi Thomas, + gpio = of_get_named_gpio(slot_np, wp-gpios, 0); + if (gpio_is_valid(gpio)) { + if (devm_gpio_request(host-dev, gpio, dw-mci-wp)) + dev_info(host-dev, gpio [%d] request failed\n, + gpio); + }

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Thomas Abraham
On 27 August 2012 14:28, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, + gpio = of_get_named_gpio(slot_np, wp-gpios, 0); + if (gpio_is_valid(gpio)) { + if (devm_gpio_request(host-dev, gpio, dw-mci-wp)) + dev_info(host-dev, gpio [%d] request

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Jaehoon Chung
Hi Thomas, On 08/28/2012 01:48 PM, Thomas Abraham wrote: On 27 August 2012 14:28, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, + gpio = of_get_named_gpio(slot_np, wp-gpios, 0); + if (gpio_is_valid(gpio)) { + if (devm_gpio_request(host-dev, gpio, dw-mci-wp)) +

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Thomas Abraham
On 28 August 2012 10:25, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, On 08/28/2012 01:48 PM, Thomas Abraham wrote: On 27 August 2012 14:28, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, + gpio = of_get_named_gpio(slot_np, wp-gpios, 0); + if (gpio_is_valid(gpio))

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Thomas Abraham
On 28 August 2012 10:40, Thomas Abraham thomas.abra...@linaro.org wrote: On 28 August 2012 10:25, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, On 08/28/2012 01:48 PM, Thomas Abraham wrote: On 27 August 2012 14:28, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, + gpio

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Jaehoon Chung
On 08/28/2012 02:10 PM, Thomas Abraham wrote: On 28 August 2012 10:40, Thomas Abraham thomas.abra...@linaro.org wrote: On 28 August 2012 10:25, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, On 08/28/2012 01:48 PM, Thomas Abraham wrote: On 27 August 2012 14:28, Jaehoon Chung

Re: [PATCH v4 9/9] mmc: dw_mmc: add support for exynos specific implementation of dw-mshc

2012-08-27 Thread Thomas Abraham
On 28 August 2012 10:46, Jaehoon Chung jh80.ch...@samsung.com wrote: On 08/28/2012 02:10 PM, Thomas Abraham wrote: On 28 August 2012 10:40, Thomas Abraham thomas.abra...@linaro.org wrote: On 28 August 2012 10:25, Jaehoon Chung jh80.ch...@samsung.com wrote: Hi Thomas, On 08/28/2012 01:48 PM,