Re: [PATCH] cfi_flash: Fix devicetree address determination

2020-09-24 Thread Simon Glass
Hi André, On Wed, 23 Sep 2020 at 09:39, André Przywara wrote: > > On 23/09/2020 06:26, Stefan Roese wrote: > > Hi Simon, > > > > On 22.09.20 15:51, Simon Glass wrote: > >> Hi Stefan, > >> > >> On Mon, 21 Sep 2020 at 07:28, Stefan Roese wrote: > >>> > >>> Hi Andre, > >>> > >>> (added Simon) > >>>

Re: [PATCH] cfi_flash: Fix devicetree address determination

2020-09-23 Thread André Przywara
On 23/09/2020 06:26, Stefan Roese wrote: > Hi Simon, > > On 22.09.20 15:51, Simon Glass wrote: >> Hi Stefan, >> >> On Mon, 21 Sep 2020 at 07:28, Stefan Roese wrote: >>> >>> Hi Andre, >>> >>> (added Simon) >>> >>> On 18.09.20 19:45, Andre Przywara wrote: The cfi-flash driver uses an open-code

Re: [PATCH] cfi_flash: Fix devicetree address determination

2020-09-22 Thread Stefan Roese
Hi Simon, On 22.09.20 15:51, Simon Glass wrote: Hi Stefan, On Mon, 21 Sep 2020 at 07:28, Stefan Roese wrote: Hi Andre, (added Simon) On 18.09.20 19:45, Andre Przywara wrote: The cfi-flash driver uses an open-coded version of the generic algorithm to decode and translate multiple frames of

Re: [PATCH] cfi_flash: Fix devicetree address determination

2020-09-22 Thread Simon Glass
Hi Stefan, On Mon, 21 Sep 2020 at 07:28, Stefan Roese wrote: > > Hi Andre, > > (added Simon) > > On 18.09.20 19:45, Andre Przywara wrote: > > The cfi-flash driver uses an open-coded version of the generic > > algorithm to decode and translate multiple frames of a "reg" property. > > > > This star

Re: [PATCH] cfi_flash: Fix devicetree address determination

2020-09-21 Thread Stefan Roese
Hi Andre, (added Simon) On 18.09.20 19:45, Andre Przywara wrote: The cfi-flash driver uses an open-coded version of the generic algorithm to decode and translate multiple frames of a "reg" property. This starts off the wrong foot by using the address-cells and size-cells properties of *this* v

[PATCH] cfi_flash: Fix devicetree address determination

2020-09-18 Thread Andre Przywara
The cfi-flash driver uses an open-coded version of the generic algorithm to decode and translate multiple frames of a "reg" property. This starts off the wrong foot by using the address-cells and size-cells properties of *this* very node, and not of the parent. This somewhat happened to work back