lkml.org 
[lkml]   [2021]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 5/5] arm64: dts: qcom: msm8916: Enable modem and WiFi
    On Fri 19 Mar 10:19 CDT 2021, Stephan Gerhold wrote:

    > Hi Bjorn,
    >

    Hi Stephan,

    I thought I wrote you a reply, but I don't see it on lore or in my
    mailbox...

    > On Thu, Mar 11, 2021 at 04:33:18PM -0800, Bjorn Andersson wrote:
    > > Enable the modem and WiFi subsystems and specify msm8916 specific
    > > firmware path for these and the WCNSS control service.
    > >
    > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
    >
    > The changes itself look good to me, but the commit message is really
    > misleading. It does not mention anywhere that the change actually just
    > enables "modem" on apq8016-sbc instead of "msm8916". :)
    >
    > Also, WCNSS was actually enabled before already (with the default
    > firmware path). In my opinion, it would be clearer to change the
    > firmware-name for it in an extra patch.
    >

    You're right, I will rewrite the commit message.

    > > ---
    > > arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi | 12 ++++++++++++
    > > arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 +-
    > > 2 files changed, 13 insertions(+), 1 deletion(-)
    > >
    > > diff --git a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
    > > index 6aef0c2e4f0a..448e3561ef63 100644
    > > --- a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
    > > +++ b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
    > > @@ -305,6 +305,12 @@ &mdss {
    > > status = "okay";
    > > };
    > >
    > > +&mpss {
    > > + status = "okay";
    > > +
    > > + firmware-name = "qcom/msm8916/mba.mbn", "qcom/msm8916/modem.mbn";
    > > +};
    > > +
    > > &pm8916_resin {
    > > status = "okay";
    > > linux,code = <KEY_VOLUMEDOWN>;
    > > @@ -312,6 +318,8 @@ &pm8916_resin {
    > >
    > > &pronto {
    > > status = "okay";
    > > +
    > > + firmware-name = "qcom/msm8916/wcnss.mbn";
    > > };
    > >
    >
    > How do I get a .mbn from the wcnss.{mdt,.b??} files provided in the
    > DB410c firmware package? I guess I should just run them through
    > https://github.com/andersson/pil-squasher?
    >

    Yes, .mdt + .bNN files can be put back together as a .mbn by running
    them through the pil-squasher.

    It's worth mentioning that the suffix doesn't matter for the loader, so
    symlinking the .mdt to .mbn (in either way) will keep the kernel
    happy - if that simplifies transitions in either way.

    > Also, is the single file format (mbn) preferred now? Not sure if there
    > is any significant difference except having less files laying around.
    >

    We've had numerous cases where people has managed to mix the parts from
    different versions of the firmware, sometimes with the nice result of
    the loader complaining that some particular .bNN file is missing
    other times just with the signature check (mysteriously) failing.

    Lumping the parts into a single .mbn resolves all these issues and
    hence I prefer this. It is however not a requirement.

    Regards,
    Bjorn

    \
     
     \ /
      Last update: 2021-05-31 19:33    [W:2.142 / U:0.792 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site