> -----Original Message----- > From: Stephen Boyd <swboyd@xxxxxxxxxxxx> > Sent: Friday, October 29, 2021 3:32 AM > To: agross@xxxxxxxxxx; bjorn.andersson@xxxxxxxxxx; > mathieu.poirier@xxxxxxxxxx; ohad@xxxxxxxxxx; p.zabel@xxxxxxxxxxxxxx; > pillair@xxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx > Cc: linux-arm-msm@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx; sibis@xxxxxxxxxxxxxx; mpubbise@xxxxxxxxxxxxxx; > kuabhs@xxxxxxxxxxxx > Subject: RE: [PATCH v6 0/3] Add support for sc7280 WPSS PIL loading > > Quoting pillair@xxxxxxxxxxxxxx (2021-10-28 06:08:39) > > > > > > > -----Original Message----- > > > From: Stephen Boyd <swboyd@xxxxxxxxxxxx> > > > Sent: Friday, October 8, 2021 12:05 AM > > > To: Rakesh Pillai <pillair@xxxxxxxxxxxxxx>; agross@xxxxxxxxxx; > > > bjorn.andersson@xxxxxxxxxx; mathieu.poirier@xxxxxxxxxx; > > > ohad@xxxxxxxxxx; p.zabel@xxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx > > > Cc: linux-arm-msm@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > > > linux- kernel@xxxxxxxxxxxxxxx; sibis@xxxxxxxxxxxxxx; > > > mpubbise@xxxxxxxxxxxxxx; kuabhs@xxxxxxxxxxxx > > > Subject: Re: [PATCH v6 0/3] Add support for sc7280 WPSS PIL loading > > > > > > Quoting Rakesh Pillai (2021-10-03 23:48:50) > > > > Add support for PIL loading of WPSS co-processor for SC7280 SOCs. > > > > > > > > Changes from v4/v5: > > > > - Add yaml conversion for adsp/cdsp dt-bindings > > > > - Change clock names in wpss dt-bindings > > > > - Correct mistake in signed-off enail ID > > > > > > Can you keep a running tally here of the full progression of the series? > > > That helps to look back and make sure we don't make a comment that > > > has already been made before. > > > > > > One more request. Can you add support for 'firmware-name' like there > > > is in Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt so > > > that we can install firmware into some namespaced/versioned place > > > instead of having to put wpss files into /lib/firmware? > > > > Hi Stephen, > > I have posted v7 with the support for firmware-name to be provided in the > DT entry. > > Thanks. I didn't see it in my inbox. No Cc for me? > > > > > > It would also be nice to load a > > > single firmware file instead of having to split the file into many pieces. > > > > This would require lot of changes and lot of code duplication from request > firmware. > > Also the base ath11k firmware files have been posted as split files. > > > > Other firmwares have done it so it seems technically possible. So nothing is > preventing it? Yes it should be possible. I can probably add that support and post it as a different patch series, so as to not club it with the current patch series. Does that sound okay ? Thanks, Rakesh Pillai.
