On 5/16/2020 11:07 PM, Ajit Pandey wrote:
On 5/14/2020 10:14 PM, Mark Brown wrote:
On Thu, May 14, 2020 at 10:08:12PM +0530, Ajit Pandey wrote:
Done the required cleanups to update lpass-cpu binding with newer yaml formats.
Please do YAML conversions as the last thing in any patch series, there is quite a big backlog of reviews of YAML conversions so they can slow down the code development. It's good to do them, just try to make sure other patches don't depend on them.
Please submit patches using subject lines reflecting the style for the subsystem, this makes it easier for people to identify relevant patches. Look at what existing commits in the area you're changing are doing and make sure your subject lines visually resemble what they're doing. There's no need to resubmit to fix this alone.
Ok.. we'll do the yaml conversion as a separate patch chain so that it won't block
merging of this feature in ASoC core.
@Mark, I was planning to post v3 patchset series for this and keep yaml change as
last patch in the series as suggested. However, we have one update in the driver to
support another compatible string. If we keep Documentation change at the last, the
driver change will have checkpatch errors. Can you please suggest if should make
first Documentation change in text file itself before driver change and finally have a
patch to convert it to yaml at the end?
Thanks,
Rohit