Messages in this thread | | | Date | Mon, 13 Jun 2011 20:32:15 +0900 | Subject | Re: [PATCH] unicore32: fix build error for find bitops | From | Akinobu Mita <> |
| |
2011/6/13 Guan Xuetao <gxt@mprc.pku.edu.cn>: > On Mon, 2011-06-13 at 18:33 +0900, Akinobu Mita wrote: >> 2011/6/13 Guan Xuetao <gxt@mprc.pku.edu.cn>: >> > Thanks, please review following patch: >> >> Looks good to me. > Could I add your acked-by information on this patch?
Of course, yes.
>> > diff --git a/arch/unicore32/kernel/ksyms.c >> > b/arch/unicore32/kernel/ksyms.c >> > index a897080..d98bd81 100644 >> > --- a/arch/unicore32/kernel/ksyms.c >> > +++ b/arch/unicore32/kernel/ksyms.c >> > @@ -24,8 +24,8 @@ >> > >> > #include "ksyms.h" >> > >> > -EXPORT_SYMBOL(__uc32_find_next_zero_bit); >> > -EXPORT_SYMBOL(__uc32_find_next_bit); >> > +EXPORT_SYMBOL(find_next_zero_bit); >> > +EXPORT_SYMBOL(find_next_bit); >> >> I'm wondering why there are no EXPORT_SYMBOL for find_first_bit >> and find_first_zero_bit. > IMO, only functions that are needed by modules should be exported. > When building modules under unicore32 & pkunity-soc, we only need > find_next_bit and find_next_zero_bit exported at present, and if new > functions are required, I am going to add them in the same time. > In fact, I think, the 'arch-generic' ksyms.c is a better idea.
OK, I see. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |