From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Wed, 13 Jun 2012 13:18:24 +0200 Subject: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg In-Reply-To: References: <4FD75739.90007@gmail.com> <4FD75A9E.60405@de.bosch.com> Message-ID: <201206131318.24535.marex@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 Dear Liu Hui-R64343, > >-----Original Message----- > >From: Dirk Behme [mailto:dirk.behme at de.bosch.com] > >Sent: Tuesday, June 12, 2012 11:05 PM > >To: Fabio Estevam; Liu Hui-R64343 > >Cc: Vikram Narayanan; u-boot at lists.denx.de > >Subject: Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in > >imximage.cfg > > > >On 12.06.2012 16:50, Vikram Narayanan wrote: > >> If in case this is valid according to the latest datasheet, ignore this > >> patch. > >> > >> -- > >> According to REV C manual, the register IOMUXC_IOMUXC_GPR4 has bits 4 > >> and 5 read-only and the value is always set as zero. > >> So write '0' to these bits instead of writing '1'. > > > >Jason, Fabio: What do you think? You should be the datasheet 'masters' ;) > > Yes, according to the RM, > 5 > Reserved > This read-only field is reserved and always has the value 0. > 4 > Reserved > This read-only field is reserved and always has the value 0. > > So, write 1 should have no effect. I really dislike how "write 1 should have no effect" sounds. Can you please check with HW people? [..] Best regards, Marek Vasut