On Thu, 2020-04-16 at 19:53 +0200, David Hildenbrand wrote:
> > >
> > Hm, I'm happy to make the changes, but EINVAL to me suggests there is a
> > problem in the way this was called by the user. And in this case there
> > really might not be much the user can change in case fo buggy firmware.
>
> Yeah, but introducing new return codes callers might not expected might
> create IMHO other issues.
>
> > Same thing with the WARN - make the potential firmware bug much more
> > obvious and visible.
> >
>
> Yeah, but I doubt this is really necessary. No strong feelings.
>
Forgot to
Acked-by: David Hildenbrand <david(a)redhat.com>
Thanks for the review David. I'll change the return code, and keep the
WARN, and send a new version.