On 12 December 2011 06:43, Evgeny Voevodin <e.voevo...@samsung.com> wrote: > --- /dev/null > +++ b/hw/exynos4210_sdhc.c > @@ -0,0 +1,1666 @@ > +/* > + * Samsung exynos4210 SD/MMC host controller > + * (SD host controller specification ver. 2.0 compliant)
Is there anything in this implementation which is exynos specific, or is it purely an implementation of the standard? We should separate out (and name appropriately) the code which is implementing the SD host standard specification, and if necessary add a thin wrapper for any exynos-specific bits. -- PMM