On 14.08.2023 18:28, Bryan O'Donoghue wrote:
> Pass the bandwidth table as a platform parameter not if/else derived
> pointer to the static table.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
> ---
> drivers/media/platform/qcom/camss/camss.c | 29 +++++++----------------
> drivers/media/platform/qcom/camss/camss.h | 3 ++-
> 2 files changed, 11 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/media/platform/qcom/camss/camss.c b/drivers/media/platform/qcom/camss/camss.c
> index 68eb45b2c0aaa..1a195eb4298a5 100644
> --- a/drivers/media/platform/qcom/camss/camss.c
> +++ b/drivers/media/platform/qcom/camss/camss.c
> @@ -1484,21 +1484,15 @@ static int camss_configure_pd(struct camss *camss)
> static int camss_icc_get(struct camss *camss)
> {
> const struct resources_icc *icc_res;
> - int nbr_icc_paths = 0;
> int i;
>
> - if (camss->version == CAMSS_8250) {
> - icc_res = &icc_res_sm8250[0];
> - nbr_icc_paths = ICC_SM8250_COUNT;
> - }
> + icc_res = camss->res->icc_res;
Would initializing at declaration time fit in 100 chars?
lgtm otherwise
Konrad
