On 6/30/2020 4:21 PM, Mark Brown wrote:
On Sat, Jun 27, 2020 at 10:23:22AM +0530, Sameer Pujar wrote:Following is the summary of current series. * Add YAML DT binding documentation for above mentioned modules. * Helper function for ACIF programming is exposed for Tegra210 and later. * Add ASoC driver components for each of the above modules. * Add DT entries for above components for Tegra210, Tegra186 and Tegra194. * Enable these components for Jetson Nano/TX1/TX2/Xavier * Enhance simple-card DPCM driver to suit Tegra Audio applications with few changes in core. * To begin with, enable sound card support for Tegra210 based platforms like Jetson Nano/TX1.This series is too big and covers a few different topics which makes it difficult to manage, please split it up. The most obvious thing here is that it contains both the drivers you're adding and a bunch of changes to the generic audio cards (which are a single throwaway line in your cover letter) - splitting out the generic card changes into a separate series would result in more manageable sizes.
My initial thought was, it would be better to present a entire picture where complete audio path can be tested or reviewed and hence added simple-card patches in the series. I will split this up in next revision.

