Re: [PATCH v9 01/13] Documentation: Add SLIMbus summary

2017-12-11 Thread Jonathan Corbet
On Thu, 7 Dec 2017 23:22:51 + Srinivas Kandagatla wrote: > > It seems you have kerneldoc comments for your data structures and at least > > some of your exported symbols. If you really want to document this stuff > > well, I'd suggest finishing out those

Re: [alsa-devel] [PATCH v9 01/13] Documentation: Add SLIMbus summary

2017-12-08 Thread Srinivas Kandagatla
On 08/12/17 08:44, Vinod Koul wrote: Do we really need a separate subdirectory for a single file? May be not, TBH, I did take some inspiration from soundwire patches. FWIW, SoundWire patches have more Documentation. We have 4 files atm, though they are not part of current series, so a

Re: [alsa-devel] [PATCH v9 01/13] Documentation: Add SLIMbus summary

2017-12-08 Thread Vinod Koul
On Thu, Dec 07, 2017 at 11:22:51PM +, Srinivas Kandagatla wrote: > Thankyou for taking time to review the patch, > > On 07/12/17 17:32, Jonathan Corbet wrote: > >On Thu, 7 Dec 2017 10:27:08 + > >srinivas.kandaga...@linaro.org wrote: > > > >A couple of overall comments... > > > >>

Re: [PATCH v9 01/13] Documentation: Add SLIMbus summary

2017-12-07 Thread Srinivas Kandagatla
Thankyou for taking time to review the patch, On 07/12/17 17:32, Jonathan Corbet wrote: On Thu, 7 Dec 2017 10:27:08 + srinivas.kandaga...@linaro.org wrote: A couple of overall comments... Documentation/driver-api/index.rst | 1 + Documentation/driver-api/slimbus/index.rst

Re: [PATCH v9 01/13] Documentation: Add SLIMbus summary

2017-12-07 Thread Jonathan Corbet
On Thu, 7 Dec 2017 10:27:08 + srinivas.kandaga...@linaro.org wrote: A couple of overall comments... > Documentation/driver-api/index.rst | 1 + > Documentation/driver-api/slimbus/index.rst | 15 > Documentation/driver-api/slimbus/summary.rst | 106 >

[PATCH v9 01/13] Documentation: Add SLIMbus summary

2017-12-07 Thread srinivas . kandagatla
From: Sagar Dharia SLIMbus (Serial Low Power Interchip Media Bus) is a specification developed by MIPI (Mobile Industry Processor Interface) alliance. SLIMbus is a 2-wire implementation, which is used to communicate with peripheral components like audio-codec. The