2008-08-02 23:04:18

by Alexander Beregalov

[permalink] [raw]
Subject: [PATCH] drivers/video/console/promcon.c: fix build error

From: Alexander Beregalov <[email protected]>

drivers/video/console/promcon.c: fix build error

drivers/video/console/promcon.c:158: error: implicit declaration of
function 'con_protect_unimap'

Introduced by commit 4744b43431e8613f920c5cba88346756f53c5165

Signed-off-by: Alexander Beregalov <[email protected]>
Cc: David Woodhouse <[email protected]>
Cc: Tim Bird <[email protected]>
---

include/linux/vt_kern.h | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/linux/vt_kern.h b/include/linux/vt_kern.h
index 8c8119f..1c78d56 100644
--- a/include/linux/vt_kern.h
+++ b/include/linux/vt_kern.h
@@ -86,6 +86,7 @@ int con_copy_unimap(struct vc_data *dst_vc, struct vc_data *src_vc);
#define con_copy_unimap(d, s) (0)
#define con_get_unimap(vc, ct, uct, list) (-EINVAL)
#define con_free_unimap(vc) do { ; } while (0)
+#define con_protect_unimap(vc, rdonly) do { ; } while (0)

#define vc_translate(vc, c) (c)
#endif


2008-08-03 08:57:17

by David Woodhouse

[permalink] [raw]
Subject: Re: [PATCH] drivers/video/console/promcon.c: fix build error

On Sun, 2008-08-03 at 03:03 +0400, Alexander Beregalov wrote:
> From: Alexander Beregalov <[email protected]>
>
> drivers/video/console/promcon.c: fix build error
>
> drivers/video/console/promcon.c:158: error: implicit declaration of
> function 'con_protect_unimap'
>
> Introduced by commit 4744b43431e8613f920c5cba88346756f53c5165

Commit a29ccf6f823a84d89e1c7aaaf221cf7282022024, surely?

And in fact it looks like we could remove those parts of promcon.c
completely if !CONFIG_CONSOLE_TRANSLATIONS. But your patch is certainly
correct, and I've applied it to my 'random-2.6.git' tree which I've
already asked Linus to pull.

--
David Woodhouse Open Source Technology Centre
[email protected] Intel Corporation

2008-08-03 09:21:19

by Alexander Beregalov

[permalink] [raw]
Subject: Re: [PATCH] drivers/video/console/promcon.c: fix build error

2008/8/3 David Woodhouse <[email protected]>:
> On Sun, 2008-08-03 at 03:03 +0400, Alexander Beregalov wrote:
>> From: Alexander Beregalov <[email protected]>
>>
>> drivers/video/console/promcon.c: fix build error
>>
>> drivers/video/console/promcon.c:158: error: implicit declaration of
>> function 'con_protect_unimap'
>>
>> Introduced by commit 4744b43431e8613f920c5cba88346756f53c5165
>
> Commit a29ccf6f823a84d89e1c7aaaf221cf7282022024, surely?
Yes, my mistake.
>
> And in fact it looks like we could remove those parts of promcon.c
> completely if !CONFIG_CONSOLE_TRANSLATIONS. But your patch is certainly
> correct, and I've applied it to my 'random-2.6.git' tree which I've
> already asked Linus to pull.
Thanks.