Re: [PATCH v1 4/6] spi: cadence-quadspi: Add QSPI support for Intel LGM SoC

Ramuthevar, Vadivel MuruganX
2020-10-19T06:26:45Z

Hi Mark,

On 17/10/2020 12:33 am, Mark Brown wrote:
> On Fri, Oct 16, 2020 at 05:31:36PM +0800, Ramuthevar,Vadivel MuruganX wrote:
>
>> + depends on OF && (ARM || ARM64 || X86 || COMPILE_TEST)
>
>> + {
>> + .compatible = "intel,lgm-qspi",
>> + },
>
> This is an x86 SoC (or SoC series) - is it really going to use DT for
> the firmware interfaces?
Thank you for the review comments...
Intel LGM SoC does uses DT based firmware blob.
It's not specifically a problem, just
> surprising to see something other than ACPI. Or is the intention to use
> PRP0001?
Yes, You're right most of them uses ACPI based, but LGM SoC doesn't.

Regards
Vadivel
There's a new comaptible here which wasn't really the use case
> for PRP0001. Like I say not really a problem, just curious.
>