On 17/07/20 3:33 am, Thiago Jung Bauermann wrote: > > Hari Bathini <hbath...@linux.ibm.com> writes: > >> On 16/07/20 4:22 am, Thiago Jung Bauermann wrote: >>> >>> Hari Bathini <hbath...@linux.ibm.com> writes: >>> <snip> >>>> + * each representing a memory range. >>>> + */ >>>> + ranges = (len >> 2) / (n_mem_addr_cells + n_mem_size_cells); >>>> + >>>> + for (i = 0; i < ranges; i++) { >>>> + base = of_read_number(prop, n_mem_addr_cells); >>>> + prop += n_mem_addr_cells; >>>> + end = base + of_read_number(prop, n_mem_size_cells) - 1; >> >> prop is not used after the above. >> >>> You need to `prop += n_mem_size_cells` here. >> >> But yeah, adding it would make it look complete in some sense.. > > Isn't it used in the next iteration of the loop? Memory@XXX/reg typically has only one range. I was looking at it from that perspective which is not right. Will update. Thanks Hari
- [PATCH v3 01/12] kexec_file: allow archs to handle s... Hari Bathini
- Re: [PATCH v3 01/12] kexec_file: allow archs to... Thiago Jung Bauermann
- [PATCH v3 03/12] powerpc/kexec_file: add helper func... Hari Bathini
- Re: [PATCH v3 03/12] powerpc/kexec_file: add he... Thiago Jung Bauermann
- Re: [PATCH v3 03/12] powerpc/kexec_file: ad... Hari Bathini
- Re: [PATCH v3 03/12] powerpc/kexec_file: ad... Hari Bathini
- [PATCH v3 06/12] ppc64/kexec_file: restrict memory u... Hari Bathini
- Re: [PATCH v3 06/12] ppc64/kexec_file: restrict... Thiago Jung Bauermann
- Re: [PATCH v3 06/12] ppc64/kexec_file: rest... Hari Bathini
- Re: [PATCH v3 06/12] ppc64/kexec_file: ... Thiago Jung Bauermann
- Re: [PATCH v3 06/12] ppc64/kexec_fi... Hari Bathini
- [PATCH v3 02/12] powerpc/kexec_file: mark PPC64 spec... Hari Bathini
- Re: [PATCH v3 02/12] powerpc/kexec_file: mark P... Thiago Jung Bauermann
- Re: [PATCH v3 02/12] powerpc/kexec_file: ma... Hari Bathini
- Re: [PATCH v3 02/12] powerpc/kexec_file... Thiago Jung Bauermann
- [PATCH v3 07/12] ppc64/kexec_file: add support to re... Hari Bathini
- Re: [PATCH v3 07/12] ppc64/kexec_file: add supp... Thiago Jung Bauermann
- Re: [PATCH v3 07/12] ppc64/kexec_file: add ... Hari Bathini
- Re: [PATCH v3 07/12] ppc64/kexec_file: ... Thiago Jung Bauermann
- [PATCH v3 08/12] ppc64/kexec_file: setup the stack f... Hari Bathini
- Re: [PATCH v3 08/12] ppc64/kexec_file: setup th... Thiago Jung Bauermann