On 8/30/2019 2:25 AM, Stephen Hemminger wrote:
> On Thu, 22 Aug 2019 14:05:41 +0300
> Aya Levin <a...@mellanox.com> wrote:
>
>> Add pr_out_dump_reporter prefix to the helper function's name and
>> encapsulate the print in it.
>>
>> Fixes: 2f1242efe9d0 ("devlink: Add devlink health show command")
>> Signed-off-by: Aya Levin <a...@mellanox.com>
>> Acked-by: Jiri Pirko <j...@mellanox.com>
>
>
> Looks fine, but devlink needs to be converted from doing JSON
> printing its own way and use common iproute2 libraries.
Sorry for the late response.
You are correct, it is in our plans to complete a full transition to
common iproute2 helpers in the following weeks.
I got your point, and will not submit more patches adding new print
functions using the current API and will wait for submission after the
transition to the common API.
This patch will be re-submitted after the transition.
>
