On Mon 06 Apr 23:45 PDT 2020, Akash Asthana wrote:
> Hi Bjorn,
>
> On 4/1/2020 5:02 AM, Bjorn Andersson wrote:
> > On Tue 31 Mar 04:09 PDT 2020, Akash Asthana wrote:
> >
> > > Add necessary macros and structure variables to support ICC BW
> > > voting from individual SE drivers.
> > >
> > > Signed-off-by: Akash Asthana <akashast@xxxxxxxxxxxxxx>
> > > ---
> > > Changes in V2:
> > > - As per Bjorn's comment dropped enums for ICC paths, given the three
> > > paths individual members
> > >
> > > Changes in V3:
> > > - Add geni_icc_get, geni_icc_vote_on and geni_icc_vote_off as helper API.
> > > - Add geni_icc_path structure in common header
> > >
> > > drivers/soc/qcom/qcom-geni-se.c | 98 +++++++++++++++++++++++++++++++++++++++++
> > > include/linux/qcom-geni-se.h | 36 +++++++++++++++
> > > 2 files changed, 134 insertions(+)
> > >
> > > diff --git a/drivers/soc/qcom/qcom-geni-se.c b/drivers/soc/qcom/qcom-geni-se.c
> > > index 7d622ea..9344c14 100644
> > > --- a/drivers/soc/qcom/qcom-geni-se.c
> > > +++ b/drivers/soc/qcom/qcom-geni-se.c
> > > @@ -720,6 +720,104 @@ void geni_se_rx_dma_unprep(struct geni_se *se, dma_addr_t iova, size_t len)
> > > }
> > > EXPORT_SYMBOL(geni_se_rx_dma_unprep);
> > > +int geni_icc_get(struct geni_se *se, const char *icc_core, const char *icc_cpu,
> > > + const char *icc_ddr)
> > > +{
> > > + if (icc_core) {
> > Afaict it's only this that might be passed as NULL, so please drop these
> > conditionals (keep the last one).
> IIUC you're suggesting to drop if (icc_core/cpu) but keep if (icc_ddr) ?
Correct
Thanks,
Bjorn
