Re: [PATCH v3 4/4] spi: qcom-geni: Enable SPI on SA8255p Qualcomm platforms

[Praveen Talari]

Hi Konrad

On 17-06-2026 16:35, Konrad Dybcio wrote:
> On 6/4/26 8:50 AM, Praveen Talari wrote:
> > The Qualcomm automotive SA8255p SoC relies on firmware to configure
> > platform resources, including clocks, interconnects and TLMM.
> > The driver requests resources operations over SCMI using power
> > and performance protocols.
> >
> > The SCMI power protocol enables or disables resources like clocks,
> > interconnect paths, and TLMM (GPIOs) using runtime PM framework APIs,
> > such as resume/suspend, to control power states(on/off).
> >
> > The SCMI performance protocol manages SPI frequency, with each
> > frequency rate represented by a performance level. The driver uses
> > geni_se_set_perf_opp() API to request the desired frequency rate.
> >
> > As part of geni_se_set_perf_opp(), the OPP for the requested frequency
> > is obtained using dev_pm_opp_find_freq_floor() and the performance
> > level is set using dev_pm_opp_set_opp().
> >
> > Signed-off-by: Praveen Talari <praveen.talari@xxxxxxxxxxxxxxxx>
> > ---
> [...]
>
> > -	ret = geni_se_resources_init(&mas->se);
> > +	mas->dev_data = device_get_match_data(&pdev->dev);
> This really needs to be null-checked

Sure, will add in next patch set.


Thanks,

Praveen talari

> Perhaps also the functions within that are called unconditionally
>
> Konrad