On Tue, Jun 26, 2018 at 10:20:04AM +0100, Srinivas Kandagatla wrote: > Recent discussion on https://lkml.org/lkml/2018/6/18/1282 suggested > that qdsp6 code is using less common of_platform apis which would > not give dt based module loading support. > > Suggestion from Rob was to add compatible strings to the dais nodes > This patchset adds this. Also this patchset has 3 trivial fixes which > is removing redundant owner, remove unused struct member and fix a > error check in q6routing. > > Changes since v1: > - Addressed few trivial comments from Niklas Hello Srini! I didn't see a response to my review comment in: https://marc.info/?l=devicetree&m=152998662313684 "Since adding the compatible is a prerequisite for using of_platform_populate()/of_platform_depopulate(), I think that the patches that make use of of_platform_populate()/of_platform_depopulate() should come after the patches that adds the of_table entries." If the code is bisectable with the current ordering of the patches, then feel free to ignore this review comment. Kind regards, Niklas > > Thanks, > srini > > Srinivas Kandagatla (12): > ASoC: q6adm: dt-bindings: add compatible string to routing > ASoC: q6asm: dt-bindings: add compatible string to dais > ASoC: q6afe: dt-bindings: add compatible string to dais > ASoC: qdsp6: q6adm: use of_platform_populate/depopulate() > ASoC: qdsp6: q6asm: use of_platform_populate/depopulate() > ASoC: qdsp6: q6afe: use of_platform_populate/depopulate() > ASoC: qdsp6: q6afe-dai: support dt based module loading > ASoC: qdsp6: q6asm-dai: support dt based module loading > ASoC: qdsp6: q6routing: support dt based module loading > ASoC: qcom: apq8096: remove redundant owner assignment > ASoC: qdsp6: q6routing: add proper error check > ASoC: qdsp6: q6asm: remove unused struct q6asm member > > Documentation/devicetree/bindings/sound/qcom,q6adm.txt | 6 ++++++ > Documentation/devicetree/bindings/sound/qcom,q6afe.txt | 6 ++++++ > Documentation/devicetree/bindings/sound/qcom,q6asm.txt | 6 ++++++ > sound/soc/qcom/apq8096.c | 1 - > sound/soc/qcom/qdsp6/q6adm.c | 16 ++-------------- > sound/soc/qcom/qdsp6/q6afe-dai.c | 7 +++++++ > sound/soc/qcom/qdsp6/q6afe.c | 16 ++-------------- > sound/soc/qcom/qdsp6/q6asm-dai.c | 7 +++++++ > sound/soc/qcom/qdsp6/q6asm.c | 17 ++--------------- > sound/soc/qcom/qdsp6/q6routing.c | 9 ++++++++- > 10 files changed, 46 insertions(+), 45 deletions(-) > > -- > 2.16.2 > -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
