Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Sachin Kamat
Hi Jaehoon, On 13 March 2013 09:50, Jaehoon Chung wrote: > Hi All, > > this patch has already got acked-by of Seungwon and me. It is needed for the second patch of this series "[PATCH 2/2] mmc: dw_mmc: exynos: Remove unnecessary use of of_match_ptr()" https://patchwork.kernel.org/patch/2156621/

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Jaehoon Chung
Hi All, this patch has already got acked-by of Seungwon and me. Best Regards, Jaehoon Chung On 03/13/2013 12:04 AM, Sachin Kamat wrote: > On 12 March 2013 20:30, Chris Ball wrote: >> Hi, >> >> On Tue, Mar 12 2013, Sachin Kamat wrote: >>> There is also another patch in this series [1] that needs

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Sachin Kamat
On 12 March 2013 20:30, Chris Ball wrote: > Hi, > > On Tue, Mar 12 2013, Sachin Kamat wrote: >> There is also another patch in this series [1] that needs your attention. >> [1] https://patchwork.kernel.org/patch/2156621/ > > I'd rather have one of the dw_mmc maintainers handle ACKing patches > bef

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Chris Ball
Hi, On Tue, Mar 12 2013, Sachin Kamat wrote: > There is also another patch in this series [1] that needs your attention. > [1] https://patchwork.kernel.org/patch/2156621/ I'd rather have one of the dw_mmc maintainers handle ACKing patches before I apply them. If they haven't reviewed it by the t

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Sachin Kamat
>> -int dw_mci_exynos_probe(struct platform_device *pdev) >> +static int dw_mci_exynos_probe(struct platform_device *pdev) >> { >> const struct dw_mci_drv_data *drv_data; >> const struct of_device_id *match; > > Thanks, pushed to mmc-next for 3.10. Thanks Chris. There is also another

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Chris Ball
Hi, On Mon, Feb 18 2013, Sachin Kamat wrote: > Silences the following sparse warning: > drivers/mmc/host/dw_mmc-exynos.c:218:5: warning: > symbol 'dw_mci_exynos_probe' was not declared. Should it be static? > > Signed-off-by: Sachin Kamat > --- > drivers/mmc/host/dw_mmc-exynos.c |2 +- > 1 f

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-12 Thread Sachin Kamat
On 4 March 2013 10:06, Sachin Kamat wrote: > Hi Chris, > > On 18 February 2013 16:21, Seungwon Jeon wrote: >> Hi Sachin, >> >> You're right. >> 'dw_mci_exynos_probe' is only used here. >> >> Acked-by: Seungwon Jeon >> >> On Monday, February 18, 2013, Jaehoon Chung wrote: >>> Acked-by: Jaehoon Ch

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-03-03 Thread Sachin Kamat
Hi Chris, On 18 February 2013 16:21, Seungwon Jeon wrote: > Hi Sachin, > > You're right. > 'dw_mci_exynos_probe' is only used here. > > Acked-by: Seungwon Jeon > > On Monday, February 18, 2013, Jaehoon Chung wrote: >> Acked-by: Jaehoon Chung >> >> On 02/18/2013 05:53 PM, Sachin Kamat wrote: >>

RE: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-02-18 Thread Seungwon Jeon
Hi Sachin, You're right. 'dw_mci_exynos_probe' is only used here. Acked-by: Seungwon Jeon On Monday, February 18, 2013, Jaehoon Chung wrote: > Acked-by: Jaehoon Chung > > On 02/18/2013 05:53 PM, Sachin Kamat wrote: > > Silences the following sparse warning: > > drivers/mmc/host/dw_mmc-exynos.

Re: [PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-02-18 Thread Jaehoon Chung
Acked-by: Jaehoon Chung On 02/18/2013 05:53 PM, Sachin Kamat wrote: > Silences the following sparse warning: > drivers/mmc/host/dw_mmc-exynos.c:218:5: warning: > symbol 'dw_mci_exynos_probe' was not declared. Should it be static? > > Signed-off-by: Sachin Kamat > --- > drivers/mmc/host/dw_mmc-

[PATCH 1/2] mmc: dw_mmc: Make dw_mci_exynos_probe static

2013-02-18 Thread Sachin Kamat
Silences the following sparse warning: drivers/mmc/host/dw_mmc-exynos.c:218:5: warning: symbol 'dw_mci_exynos_probe' was not declared. Should it be static? Signed-off-by: Sachin Kamat --- drivers/mmc/host/dw_mmc-exynos.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/d