Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> writes:
>> include/linux/scatterlist.h | 5 ++
>> lib/scatterlist.c | 189 ++++++++++++++++++++++++++++++++++++++++++++
>> 2 files changed, 194 insertions(+)
>
> It's quite a bit of code for a fairly specialised thing. How ugly
> would it be to put this in a new .c file and have subsystems select it
> in Kconfig?
I have no idea about the "ugliness", but why not ...
If nobody objects, and in order to submit a proper patch, there are decisions to
make :
- what will be the scope of this new .c file ?
- only sg_plit() ?
- all sg specialized functions, ie. sg_lib.c ?
- will include/linux/scatterlist.h have an "ifdefed" portion for what X.c
offers ?
- what naming for X.c and the config entry ?
What about adding this to lib/Makefile, and one ifdef to scatterlist.h ? :
obj-$(CONFIG_SG_LIB) += sg_lib.o
Cheers.
--
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
