On Mon, Oct 14, 2019 at 12:34 PM Srinivas Kandagatla
Thanks Rob for taking time to review,
On 14/10/2019 18:12, Rob Herring wrote:
On Fri, Oct 11, 2019 at 04:44:22PM +0100, Srinivas Kandagatla wrote:Sure! I can do that in next version!
This patch adds bindings for Qualcomm soundwire controller.
Qualcomm SoundWire Master controller is present in most Qualcomm SoCs
either integrated as part of WCD audio codecs via slimbus or
as part of SOC I/O.
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
.../bindings/soundwire/qcom,sdw.txt | 167 ++++++++++++++++++
1 file changed, 167 insertions(+)
create mode 100644 Documentation/devicetree/bindings/soundwire/qcom,sdw.txt
Next time, do a DT schema.
I meant the next binding you write, not v4. However, ...
Thanks, Looks like I missed that here.+= SoundWire devices
+Each subnode of the bus represents SoundWire device attached to it.
+The properties of these nodes are defined by the individual bindings.
Is there some sort of addressing that needs to be defined?
it should be something like this,
#address-cells = <2>;
#size-cells = <0>;
Will add the in next version.
You need a common soundwire binding for this. You also need to define
the format of 'reg' and unit addresses. And it needs to be a schema.
So perhaps this binding too should be.