From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Wed, 03 Apr 2013 11:12:31 +0200 Subject: [U-Boot] [PATCH] spi: mxc_spi: Fix ECSPI reset handling In-Reply-To: <1363853024-26432-1-git-send-email-dirk.behme@de.bosch.com> References: <1363853024-26432-1-git-send-email-dirk.behme@de.bosch.com> Message-ID: <515BF27F.2030404@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 21/03/2013 09:03, Dirk Behme wrote: > Reviewing the ECSPI reset handling shows two issues: > Hi Dirk, agree completely, only a very minor question.. > + > + reg_ctrl = reg_read(®s->ctrl); As you says, it makes no sense to read back the value of the register, also because reg_ctrl is overwritten some lines later ;-) 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-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================