From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Sun, 14 Nov 2010 18:08:58 +0100 Subject: [U-Boot] [PATCH] FSL_ESDHC: Make the base address configurable at runtime In-Reply-To: <201011141801.48221.marek.vasut@gmail.com> References: <1289750462-6119-1-git-send-email-marek.vasut@gmail.com> <201011141735.30504.marek.vasut@gmail.com> <20101114165909.09C65134F54@gemini.denx.de> <201011141801.48221.marek.vasut@gmail.com> Message-ID: <4CE017AA.808@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 11/14/2010 06:01 PM, Marek Vasut wrote: > Hi Wolfgang, > > it means Stefano explained to me he already tried pushing in such a patch and > the patch was rejected. Therefore, I assume you should ignore this patch. To complete the explanation: there is already a way to make address configurable and we do not need to change fsl_esdhc_mmc_init. The fsl_esdhc_initialize(bd_t *bis, struct fsl_esdhc_cfg *cfg) was added to the driver when I pushed the MX51 stuff, allowing us to set up the base address, Not only, the struct fsl_esdhc_cfg could be extended in the future, if we will need it (but not at the moment, as I know). This way is already used by the mx51evk (both SDHC controller are supported) and by the vision2 (only one SDHC controller is supported). Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================