| Message ID | 20230103031159.1060075-1-dmitry.baryshkov@linaro.org |
|---|---|
| State | Accepted |
| Commit | f730038fe6a6de170268fd779b2c029aa70a928b |
| Headers | show |
| Series | interconnect: qcom: osm-l3: drop unuserd header inclusion | expand |
On 03/01/2023 05:33, Bjorn Andersson wrote: > 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. Nice suggestion, thank you. I've send a separate patch. > > 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 >>
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" - #define LUT_MAX_ENTRIES 40U #define LUT_SRC GENMASK(31, 30) #define LUT_L_VAL GENMASK(7, 0)
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(-)