[PATCH v3 02/18] remoteproc: qcom: Move minidump specific data to qcom_minidump.h

Krzysztof Kozlowski krzysztof.kozlowski at linaro.org
Thu May 4 04:38:32 PDT 2023


On 03/05/2023 19:02, Mukesh Ojha wrote:
> Move minidump specific data types and macros to a separate internal
> header(qcom_minidump.h) so that it can be shared among different
> Qualcomm drivers.

No, this is not internal header. You moved it to global header.

There is no reason driver internals should be exposed to other unrelated
subsystems.

> 
> There is no change in functional behavior after this.

It is. You made all these internal symbols available to others.

> 

This comes without justification why other drivers needs to access
private and internal data. It does not look correct design. NAK.

Best regards,
Krzysztof




More information about the linux-arm-kernel mailing list