we need kernel module radio-iris-transport.ko for FM it means we should build kernel code in flame

Hi Michael, now we can build kernel code for FM module. I'm not sure if we should flast boot.img to flame since we build the code from codeaurora not from partner.

Comment on attachment 8412414 [details] build kernel module for FM Yeah, we need to create a branch for the flame specific kernel and import that kernel.

Not a regression, so this is not a QA blocker.

fyi. ( no real need info )

(In reply to viral [:viralwang] from comment #0) > we need kernel module radio-iris-transport.ko for FM > it means we should build kernel code in flame Viral, I tested successfully the FM Radio on my kernel built from T2M sources. Looks like IRIS is enabled there: > arch/arm/configs/msm8610_defconfig:CONFIG_RADIO_IRIS=y > arch/arm/configs/msm8610_defconfig:CONFIG_RADIO_IRIS_TRANSPORT=m

(In reply to Alexandre LISSY :gerard-majax from comment #5) > (In reply to viral [:viralwang] from comment #0) > > we need kernel module radio-iris-transport.ko for FM > > it means we should build kernel code in flame > > Viral, I tested successfully the FM Radio on my kernel built from T2M > sources. Looks like IRIS is enabled there: > > arch/arm/configs/msm8610_defconfig:CONFIG_RADIO_IRIS=y > > arch/arm/configs/msm8610_defconfig:CONFIG_RADIO_IRIS_TRANSPORT=m Cool, I think FM should be fine in we built the module radio-iris-transport.ko! Please feel free if your would like to take the bug :)

This is RIL related bug.

Given comment 7, back to the RIL component. If you meant that this is not a RIL bug, please file in a component that will have someone handle the build.

comment 7 was a mistake. This isn't a RIL bug.

I emailed viral and vance to confirm if this was a 2.0 blocker and he said having a built in kernel should be fine here(refer to comment #6) Removing the nomination for now, please reach out to viral if you have concerns on the approach.

out/target/product/flame/system/lib/modules/radio-iris-transport.ko This will get fixed as soon as we enable building the kernel itself, i.e., when bug 1004195 gets fixed.

this bug looks fixed now since bug 1004195 already land.

(In reply to viral [:viralwang] from comment #12) > this bug looks fixed now since bug 1004195 already land. Yes, you're right, now that we landed building the kernel properly and that we have WiFi driver building also, we are producing good system.img including the proper radio. I'm listening to the FM radio right at this moment of closing this bug :).