On Thu, May 11, 2023 at 10:09:45PM +0530, Mohammad Rafi Shaik wrote: > > On 3/3/2023 3:27 AM, Mathieu Poirier wrote: > > On Thu, Feb 23, 2023 at 08:35:57PM +0530, Mohammad Rafi Shaik wrote: > > > Update section header name check and corresponding documentation. > > > Changes since v4: > > > -- Rephrase commit message. > > Asked for clarifications on V4 that were never given to me. This patchset will > > not move forward until those have been resolved. > The present Qualcomm DSP binary contains resource table name as > ".resource_table.ac_bin_process.", My questions still haven't been answered: 1. Why do we have to change the kernel because of the way a company specific tool generates an ELF? 2. Why is the "ac_bin_process" part needed at all? > so the current logic with strcmp will fail with present comparision as the > binary name is not only .resource_table but resource_table.ac_bin_process So > to overcome this issue we modified the way of checking the resource table > name to make it generic. > > strstarts(name_table + name, ".resource_table"); > > This logic will perform a string comparison with name ".resouce_table"as our > binary name is .resource_table.ac_bin_process it succeeds > > > > Changes since v3: > > > -- Rephrase commit message. > > > Changes since v2: > > > -- Update the commit message with example. > > > -- Update the documentation text appropriately. > > > Changes since v1: > > > -- Update the commit message. > > > -- Use strstarts instead of strstr. > > > -- Update documentation file. > > > > > > Srinivasa Rao Mandadapu (2): > > > remoteproc: elf_loader: Update resource table name check > > > docs: remoteproc: Update section header name requirement > > > > > > Documentation/staging/remoteproc.rst | 5 ++++- > > > drivers/remoteproc/remoteproc_elf_loader.c | 2 +- > > > 2 files changed, 5 insertions(+), 2 deletions(-) > > > > > > -- > > > 2.25.1 > > >
