[PATCH v3 1/3] dt-bindings: spi: apple,spi: Add binding for Apple SPI controllers
Nick Chan
towinchenmi at gmail.com
Sat Nov 2 06:15:05 PDT 2024
On 2/11/2024 16:39, Mark Kettenis wrote:
>> Date: Sat, 2 Nov 2024 10:36:56 +0800
>> Content-Language: en-MW
>>
>> On 2/11/2024 03:26, Janne Grunau via B4 Relay wrote:
>>
>> [...]
>>> +properties:
>>> + compatible:
>>> + items:
>>> + - enum:
>>> + - apple,t8103-spi
>>> + - apple,t8112-spi
>>> + - apple,t6000-spi
>>> + - const: apple,spi
>> Apple A7-A11 SoCs seems to use a Samsung SPI block, so apple,spi is too
>> generic. Fallback to something like apple,t8103-spi instead.
>
> Well, if that is a Samsung SPI block it probably should have a
> "generic" compatible that starts with "samsung,". The M1/M2
> controllers have a different SPI block (presumably) designed by Apple
> themselves. So I think it is (still) appropriate that that one is
> "apple,spi".
I just looked into the SPI on A7-A11 SoC in more detail instead of just
going off the ADT compatible. It seems a very big chunk of the registers
offsets and bits seems to be the same as the ones in M1. So, feel free to
ignore my comment above.
Acked-by: Nick Chan <towinchenmi at gmail.com>
>
> Also, (upstream) U-Boot already uses the "apple,spi" compatible. So
> changing it for purity sake just causes pain.
Well, if upstream U-Boot is using it, then I agree that "apple,spi"
should continue to be used.
>
Nick Chan
More information about the linux-arm-kernel
mailing list