On Tue, Jul 03, 2018 at 02:59:22PM +0300, Dan Carpenter wrote:
> The gnss_allocate_device() function returns a mix of NULL and error
> pointers on error. It should only return one or the other. Since the
> callers both check for NULL, I've modified it to return NULL on error.
>
> Fixes: 10f146639fee ("gnss: add receiver type support")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Ouch, thanks for catching this!
I think you got the wrong commit for the fixes tag above, however. That
should be:
Fixes: 2b6a44035143 ("gnss: add GNSS receiver subsystem")
Not sure if you want to send a v2, or if Greg can fix that when
applying. Apart from that:
Acked-by: Johan Hovold <johan@xxxxxxxxxx>
Johan
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
