Re: [alsa-devel] [PATCH] ASoC: nau8822: new codec driver
OK, I will resend a new patch.
However, from my last patch file, I can see DT file where is located at "Documentation/devicetree/bindings/sound/nau8822.txt".
And don't you see any DT file?
On 9/26/2018 1:52 AM, Mark Brown wrote:
On Tue, Sep 25, 2018 at 09:56:13AM -0700, Mark Brown wrote:
On Tue, Sep 25, 2018 at 10:50:19AM +0800, AC30 CTLin0 wrote:
May I learn the status of patch review due to our customer ask us about upstream status?
I don't know, I don't seem to have a copy of this patch in my review queue. Please resend:
Found a copy in the list archives, but still no idea where it went in my review queue. My best guess is that it got eaten by spam filtering somewhere but I'm not 100% sure. Sorry about that. :( The driver looks pretty clean, a few small points:
- The Mute Switch controls should just be called Switch so that applications know how to handle them.
- PGA Boost +20dB should just be "PGA Boost Volume" with the +20dB specified by having it be a SOC_DOUBLE_R_TLV().
- The switch statements in hw_params() should have default: cases that return errors so that if an incorrect parameter makes it through we detect it.
- There's a DT compatible string but no DT binding file is supplied, should just be a very simple one as there's no parameters.
These are all small, hopefully should be quick to update.
=========================================================================================== The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.
On Thu, Sep 27, 2018 at 06:44:45PM +0800, AC30 CTLin0 wrote:
OK, I will resend a new patch.
However, from my last patch file, I can see DT file where is located at "Documentation/devicetree/bindings/sound/nau8822.txt".
And don't you see any DT file?
Ah, that's weird - I'd not expect the binding to have been applied for a new driver without the code. In that case everything is fine and no need to send them again, perhaps just add a note saying that the binding is already there in case I forget.
participants (2)
-
AC30 CTLin0
-
Mark Brown