On Mon, 2015-05-04 at 03:17 +0100, Bryan O'Donoghue wrote:
> --- a/arch/x86/platform/intel-quark/Makefile
> +++ b/arch/x86/platform/intel-quark/Makefile
> obj-$(CONFIG_INTEL_IMR) += imr.o
(Your change to drivers/platform/x86/Kconfig now makes it possible that
imr.o will be part of a module. More on that below.)
> +obj-$(CONFIG_INTEL_ESRAM) += esram.o
INTEL_ESRAM is a bool Kconfig symbol, so esram.o will never be part of a
module, right?
> --- /dev/null
> +++ b/arch/x86/platform/intel-quark/esram.c
> +MODULE_DEVICE_TABLE(x86cpu, esram_ids);
> +module_init(esram_init);
> +module_exit(esram_exit);
> +
> +MODULE_AUTHOR("Bryan O'Donoghue <pure.logic@xxxxxxxxxxxxxxxxx>");
> +MODULE_DESCRIPTION("Intel Embedded SRAM overlay driver");
> +MODULE_LICENSE("Dual BSD/GPL");
> +
(Trailing empty line.)
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> config INTEL_IMR
> - bool "Intel Isolated Memory Region support"
> + tristate "Intel Isolated Memory Region support"
It's not obvious, at least to me, why this symbol is changed to
tristate. Does that follow from the other changes?
> default n
> depends on X86_INTEL_QUARK && IOSF_MBI
> ---help---
> +config INTEL_ESRAM
> + bool "Intel Embedded SRAM (eSRAM) support"
If I'm reading the corresponding Makefile change correctly, esram.o will
never be part of a module. But esram.c is added with some module
specific boilerplate. (Note, again, that I'm not sure whether
KBUILD_MODNAME is module specific.)
Was your intention perhaps to make this a tristate symbol?
> + default n
> + depends on X86_INTEL_QUARK && IOSF_MBI
> + select GENERIC_ALLOCATOR
> + ---help---
> + This options provides an API to allocate memory from Embedded SRAM
> + (eSRAM) present on Quark X1000 SoC processors.
> + eSRAM is a 512 KiB block of low-latency SRAM organized as
> + 128 * 4 KiB pages or as one 512 KiB chunk of memory. This driver
> + enables eSRAM in per-page overlay mode and provides a gen_pool
> + allocator which allows allocation of memory from the eSRAM pool.
> +
> + If you are running on a Galileo/Quark say Y here.
Thanks,
Paul Bolle
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
