On Tuesday, June 25, 2019 7:15:28 AM CEST luanshi wrote: > Function __ghes_check_estatus() is always called after > __ghes_peek_estatus(), but it is already called in __ghes_peek_estatus(). > So we should remove some needless __ghes_check_estatus() calls. > > Signed-off-by: Liguang Zhang <zhangliguang@xxxxxxxxxxxxxxxxx> > Reviewed-by: James Morse <james.morse@xxxxxxx> > --- > drivers/acpi/apei/ghes.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c > index 993940d..a66e00f 100644 > --- a/drivers/acpi/apei/ghes.c > +++ b/drivers/acpi/apei/ghes.c > @@ -345,7 +345,7 @@ static int __ghes_peek_estatus(struct ghes *ghes, > return -ENOENT; > } > > - return __ghes_check_estatus(ghes, estatus); > + return 0; > } > > static int __ghes_read_estatus(struct acpi_hest_generic_status *estatus, > Applied, thanks!
