Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-15 Thread Stefano Babic
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 o

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Vikram Narayanan
Hello Marek, On 6/13/2012 10:45 PM, Marek Vasut wrote: Dear Vikram Narayanan, Hello Marek, On 6/13/2012 5:57 PM, Marek Vasut wrote: Dear Vikram Narayanan, If in case this is valid according to the latest datasheet, ignore this patch. -- According to REV C manual, the register IOMUXC_IOMUX

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Marek Vasut
Dear Vikram Narayanan, > Hello Marek, > > On 6/13/2012 5:57 PM, Marek Vasut wrote: > > Dear Vikram Narayanan, > > > >> 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

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Vikram Narayanan
Hello Marek, On 6/13/2012 5:57 PM, Marek Vasut wrote: Dear Vikram Narayanan, 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

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Marek Vasut
Dear Liu Hui-R64343, > Hi, Marek, > > >-Original Message- > >From: Marek Vasut [mailto:ma...@denx.de] > >Sent: Wednesday, June 13, 2012 8:25 PM > >To: u-boot@lists.denx.de > >Cc: Liu Hui-R64343; Dirk Behme > >Subject: Re: [U-Boot] [PATC

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Liu Hui-R64343
Hi, Marek, >-Original Message- >From: Marek Vasut [mailto:ma...@denx.de] >Sent: Wednesday, June 13, 2012 8:25 PM >To: u-boot@lists.denx.de >Cc: Liu Hui-R64343; Dirk Behme >Subject: Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in >imximage.cfg &

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Marek Vasut
Dear Vikram Narayanan, > 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'. I'm

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Marek Vasut
Dear Liu Hui-R64343, [...] > >> >> 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 datashee

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Liu Hui-R64343
Hi, Marek, >-Original Message- >From: Marek Vasut [mailto:ma...@denx.de] >Sent: Wednesday, June 13, 2012 7:18 PM >To: u-boot@lists.denx.de >Cc: Liu Hui-R64343; Dirk Behme; Fabio Estevam >Subject: Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in >imximag

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Fabio Estevam
On Wed, Jun 13, 2012 at 8:18 AM, Marek Vasut wrote: > > I really dislike how "write 1 should have no effect" sounds. Can you please > check with HW people? I agree. Jason, please check this with the design team. Vikram's patch look correct. Regards, Fabio Estevam

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-13 Thread Marek Vasut
Dear Liu Hui-R64343, > >-Original Message- > >From: Dirk Behme [mailto:dirk.be...@de.bosch.com] > >Sent: Tuesday, June 12, 2012 11:05 PM > >To: Fabio Estevam; Liu Hui-R64343 > >Cc: Vikram Narayanan; u-boot@lists.denx.de > >Subject: Re: [U-Boot] [PATCH]

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-12 Thread Liu Hui-R64343
>-Original Message- >From: Dirk Behme [mailto:dirk.be...@de.bosch.com] >Sent: Tuesday, June 12, 2012 11:05 PM >To: Fabio Estevam; Liu Hui-R64343 >Cc: Vikram Narayanan; u-boot@lists.denx.de >Subject: Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in >imximage

Re: [U-Boot] [PATCH] mx6: Avoid writing to read-only bits in imximage.cfg

2012-06-12 Thread Dirk Behme
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 writi