Re: [PATCH] driver/hwtracing: use ERR_CAST instead of ERR_PTR

From: Mathieu Poirier
Date: Wed Aug 01 2018 - 13:29:24 EST


On Tue, 31 Jul 2018 at 09:40, zhong jiang <zhongjiang@xxxxxxxxxx> wrote:
>
> Use ERR_CAT inlined function to replace the ERR_PTR(PTR_ERR). It
> make the code more concise.
>
> Signed-off-by: zhong jiang <zhongjiang@xxxxxxxxxx>
> ---
> drivers/hwtracing/coresight/coresight-tmc-etr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hwtracing/coresight/coresight-tmc-etr.c b/drivers/hwtracing/coresight/coresight-tmc-etr.c
> index 2eda5de..1196364 100644
> --- a/drivers/hwtracing/coresight/coresight-tmc-etr.c
> +++ b/drivers/hwtracing/coresight/coresight-tmc-etr.c
> @@ -536,7 +536,7 @@ static void tmc_etr_sg_table_populate(struct etr_sg_table *etr_table)
> sg_table = tmc_alloc_sg_table(dev, node, nr_tpages, nr_dpages, pages);
> if (IS_ERR(sg_table)) {
> kfree(etr_table);
> - return ERR_PTR(PTR_ERR(sg_table));
> + return ERR_CAST(sg_table);

Applied - thanks.
Mathieu

> }
>
> etr_table->sg_table = sg_table;
> --
> 1.7.12.4
>