Re: [PATCH] interconnect: qcom: osm-l3: drop unuserd header inclusion

[Bjorn Andersson <andersson@kernel.org>]

On Tue, Jan 03, 2023 at 05:11:59AM +0200, Dmitry Baryshkov wrote:
> The commit 4529992c9474 ("interconnect: qcom: osm-l3: Use
> platform-independent node ids") made osm-l3 driver use
> platform-independent IDs, removing the need to include platform headers.
> 
> Fixes: 4529992c9474 ("interconnect: qcom: osm-l3: Use platform-independent node ids")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  drivers/interconnect/qcom/osm-l3.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/drivers/interconnect/qcom/osm-l3.c b/drivers/interconnect/qcom/osm-l3.c
> index 5fa171087425..0c907765783e 100644
> --- a/drivers/interconnect/qcom/osm-l3.c
> +++ b/drivers/interconnect/qcom/osm-l3.c
> @@ -14,13 +14,6 @@
>  
>  #include <dt-bindings/interconnect/qcom,osm-l3.h>
>  
> -#include "sc7180.h"
> -#include "sc7280.h"
> -#include "sc8180x.h"
> -#include "sdm845.h"
> -#include "sm8150.h"
> -#include "sm8250.h"
> -

Thanks for cleaning this up Dmitry. The <platform>_MASTER_OSM_L3_APPS
and <platform>_SLAVE_OSM_L3 in these header files could also be removed,
if you would like.

Either way, you have my:

Reviewed-by: Bjorn Andersson <andersson@kernel.org>

Regards,
Bjorn

>  #define LUT_MAX_ENTRIES			40U
>  #define LUT_SRC				GENMASK(31, 30)
>  #define LUT_L_VAL			GENMASK(7, 0)
> -- 
> 2.39.0
>