* [U-Boot] [PATCH] video: exynos_fb: Add the missing #else clause
@ 2013-04-04 14:25 Ajay Kumar
2013-04-21 17:03 ` Ajay kumar
0 siblings, 1 reply; 4+ messages in thread
From: Ajay Kumar @ 2013-04-04 14:25 UTC (permalink / raw)
To: u-boot
This patch fixes a bug introduced while adding DT support
to Exynos FIMD driver:
commit c23f3157d69bbb6c044256870f745f195b12431e
Author: Ajay Kumar <ajaykumar.rs@samsung.com>
Date: Thu Feb 21 23:53:01 2013 +0000
video: exynos_fb: add DT support for FIMD driver
Even though this part of code was missing, things were working
fine on Exynos5250 because, samsung_get_base_fimd() used
to give the correct base address for Exynos5250 FIMD.
Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
---
drivers/video/exynos_fimd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/video/exynos_fimd.c b/drivers/video/exynos_fimd.c
index 3359949..7243ea3 100644
--- a/drivers/video/exynos_fimd.c
+++ b/drivers/video/exynos_fimd.c
@@ -280,8 +280,9 @@ void exynos_fimd_lcd_init(vidinfo_t *vid)
node, "reg");
if (fimd_ctrl == NULL)
debug("Can't get the FIMD base address\n");
-#endif
+#else
fimd_ctrl = (struct exynos_fb *)samsung_get_base_fimd();
+#endif
offset = exynos_fimd_get_base_offset();
--
1.8.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] video: exynos_fb: Add the missing #else clause
2013-04-04 14:25 [U-Boot] [PATCH] video: exynos_fb: Add the missing #else clause Ajay Kumar
@ 2013-04-21 17:03 ` Ajay kumar
2013-04-22 1:00 ` Donghwa Lee
0 siblings, 1 reply; 4+ messages in thread
From: Ajay kumar @ 2013-04-21 17:03 UTC (permalink / raw)
To: u-boot
ping.
On Thu, Apr 4, 2013 at 7:55 PM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote:
> This patch fixes a bug introduced while adding DT support
> to Exynos FIMD driver:
>
> commit c23f3157d69bbb6c044256870f745f195b12431e
> Author: Ajay Kumar <ajaykumar.rs@samsung.com>
> Date: Thu Feb 21 23:53:01 2013 +0000
>
> video: exynos_fb: add DT support for FIMD driver
>
> Even though this part of code was missing, things were working
> fine on Exynos5250 because, samsung_get_base_fimd() used
> to give the correct base address for Exynos5250 FIMD.
>
> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
> ---
> drivers/video/exynos_fimd.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/video/exynos_fimd.c b/drivers/video/exynos_fimd.c
> index 3359949..7243ea3 100644
> --- a/drivers/video/exynos_fimd.c
> +++ b/drivers/video/exynos_fimd.c
> @@ -280,8 +280,9 @@ void exynos_fimd_lcd_init(vidinfo_t *vid)
> node,
> "reg");
> if (fimd_ctrl == NULL)
> debug("Can't get the FIMD base address\n");
> -#endif
> +#else
> fimd_ctrl = (struct exynos_fb *)samsung_get_base_fimd();
> +#endif
>
> offset = exynos_fimd_get_base_offset();
>
> --
> 1.8.0
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] video: exynos_fb: Add the missing #else clause
2013-04-21 17:03 ` Ajay kumar
@ 2013-04-22 1:00 ` Donghwa Lee
2013-06-21 0:15 ` Minkyu Kang
0 siblings, 1 reply; 4+ messages in thread
From: Donghwa Lee @ 2013-04-22 1:00 UTC (permalink / raw)
To: u-boot
It looks good to me.
Acked-by: Donghwa Lee <dh09.lee@samsung.com>
Thank you,
Donghwa Lee
On Mon, Apr 22, 2013 at 02:03, Ajay kumar wrote:
> ping.
>
>
> On Thu, Apr 4, 2013 at 7:55 PM, Ajay Kumar <ajaykumar.rs@samsung.com
> <mailto:ajaykumar.rs@samsung.com>> wrote:
>
> This patch fixes a bug introduced while adding DT support
> to Exynos FIMD driver:
>
> commit c23f3157d69bbb6c044256870f745f195b12431e
> Author: Ajay Kumar <ajaykumar.rs@samsung.com
> <mailto:ajaykumar.rs@samsung.com>>
> Date: Thu Feb 21 23:53:01 2013 +0000
>
> video: exynos_fb: add DT support for FIMD driver
>
> Even though this part of code was missing, things were working
> fine on Exynos5250 because, samsung_get_base_fimd() used
> to give the correct base address for Exynos5250 FIMD.
>
> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com
> <mailto:ajaykumar.rs@samsung.com>>
> ---
> drivers/video/exynos_fimd.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/video/exynos_fimd.c b/drivers/video/exynos_fimd.c
> index 3359949..7243ea3 100644
> --- a/drivers/video/exynos_fimd.c
> +++ b/drivers/video/exynos_fimd.c
> @@ -280,8 +280,9 @@ void exynos_fimd_lcd_init(vidinfo_t *vid)
> node, "reg");
> if (fimd_ctrl == NULL)
> debug("Can't get the FIMD base address\n");
> -#endif
> +#else
> fimd_ctrl = (struct exynos_fb *)samsung_get_base_fimd();
> +#endif
>
> offset = exynos_fimd_get_base_offset();
>
> --
> 1.8.0
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de <mailto:U-Boot@lists.denx.de>
> http://lists.denx.de/mailman/listinfo/u-boot
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] video: exynos_fb: Add the missing #else clause
2013-04-22 1:00 ` Donghwa Lee
@ 2013-06-21 0:15 ` Minkyu Kang
0 siblings, 0 replies; 4+ messages in thread
From: Minkyu Kang @ 2013-06-21 0:15 UTC (permalink / raw)
To: u-boot
On 22/04/13 10:00, Donghwa Lee wrote:
> It looks good to me.
>
> Acked-by: Donghwa Lee <dh09.lee@samsung.com>
>
> Thank you,
> Donghwa Lee
>
>
> On Mon, Apr 22, 2013 at 02:03, Ajay kumar wrote:
>> ping.
>>
>>
>> On Thu, Apr 4, 2013 at 7:55 PM, Ajay Kumar <ajaykumar.rs at samsung.com <mailto:ajaykumar.rs@samsung.com>> wrote:
>>
>> This patch fixes a bug introduced while adding DT support
>> to Exynos FIMD driver:
>>
>> commit c23f3157d69bbb6c044256870f745f195b12431e
>> Author: Ajay Kumar <ajaykumar.rs at samsung.com <mailto:ajaykumar.rs@samsung.com>>
>> Date: Thu Feb 21 23:53:01 2013 +0000
>>
>> video: exynos_fb: add DT support for FIMD driver
>>
>> Even though this part of code was missing, things were working
>> fine on Exynos5250 because, samsung_get_base_fimd() used
>> to give the correct base address for Exynos5250 FIMD.
>>
>> Signed-off-by: Ajay Kumar <ajaykumar.rs at samsung.com <mailto:ajaykumar.rs@samsung.com>>
>> ---
>> drivers/video/exynos_fimd.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/video/exynos_fimd.c b/drivers/video/exynos_fimd.c
>> index 3359949..7243ea3 100644
>> --- a/drivers/video/exynos_fimd.c
>> +++ b/drivers/video/exynos_fimd.c
>> @@ -280,8 +280,9 @@ void exynos_fimd_lcd_init(vidinfo_t *vid)
>> node, "reg");
>> if (fimd_ctrl == NULL)
>> debug("Can't get the FIMD base address\n");
>> -#endif
>> +#else
>> fimd_ctrl = (struct exynos_fb *)samsung_get_base_fimd();
>> +#endif
>>
>> offset = exynos_fimd_get_base_offset();
>>
>> --
>> 1.8.0
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de <mailto:U-Boot@lists.denx.de>
>> http://lists.denx.de/mailman/listinfo/u-boot
>>
>>
>
applied to u-boot-samsung.
thanks,
Minkyu Kang.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-06-21 0:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-04 14:25 [U-Boot] [PATCH] video: exynos_fb: Add the missing #else clause Ajay Kumar
2013-04-21 17:03 ` Ajay kumar
2013-04-22 1:00 ` Donghwa Lee
2013-06-21 0:15 ` Minkyu Kang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox