lkml.org 
[lkml]   [2022]   [Jul]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 7/8] PCI: qcom: Clean up IP configurations
On Mon, Jul 18, 2022 at 01:39:32PM +0300, Dmitry Baryshkov wrote:
> On 14/07/2022 10:13, Johan Hovold wrote:
> > The various IP versions have different configurations that are encoded
> > in separate sets of operation callbacks. Currently, there is no need for
> > also maintaining corresponding sets of data parameters, but it is
> > conceivable that these may again be found useful (e.g. to implement
> > minor variations of the operation callbacks).
> >
> > Rename the default configuration structures after the IP version they
> > apply to so that they can more easily be reused by different SoCs.
> >
> > Note that SoC specific configurations can be added later if need arises
> > (e.g. cfg_sc8280xp).
> >
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
>
>
> If we have nothing left in the qcom_pcie_cfg other than the .ops, what
> about dropping the qcom_pcie_cfg completely and using the qcom_pcie_ops
> as match data?

As I mention above I decided to keep the config structures as they can
be used to implement minor variations of the ops.

> This patch is nevertheless:
>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Thanks for reviewing.

Johan

\
 
 \ /
  Last update: 2022-07-18 14:02    [W:0.085 / U:3.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site