On Tue, Jul 23, 2019 at 04:03:02PM +0100, Mark Brown wrote: > On Tue, Jul 23, 2019 at 04:01:25PM +0100, Charles Keepax wrote: > > > Ah.. yes I follow. Yes this is intentional, all the DT fields in > > question are optional so the driver should proceed if even if they > > are corrupt or missing. > > If they're present but unparsable you should probably say something > about that really. That's clearly different to optional. The helper logs an error message itself since essentially the only difference in the messages is the name of the property, there is just no need to process the return value since we would take the same code path regardless of failure or success. Thanks, Charles _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel