On 14/01/2022 06:41, Alim Akhtar wrote: > Hi Krzysztof, > >> -----Original Message----- >> From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@xxxxxxxxxxxxx] >> Sent: Thursday, January 13, 2022 6:02 PM >> To: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>; linux-arm- >> kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx >> Cc: soc@xxxxxxxxxx; linux-clk@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; >> olof@xxxxxxxxx; linus.walleij@xxxxxxxxxx; catalin.marinas@xxxxxxx; >> robh+dt@xxxxxxxxxx; s.nawrocki@xxxxxxxxxxx; linux-samsung- >> soc@xxxxxxxxxxxxxxx; pankaj.dubey@xxxxxxxxxxx >> Subject: Re: [PATCH 00/23] Add support for Tesla Full Self-Driving (FSD) SoC >> >> On 13/01/2022 13:11, Alim Akhtar wrote: >>> This patch set adds basic support for the Tesla Full Self-Driving >>> (FSD) SoC. This SoC contains three clusters of four Cortex-A72 CPUs, >>> as well as several IPs. >>> >>> Patches 1 to 8 provide support for the clock controller (which is >>> designed similarly to Exynos SoCs). >>> >>> The remaining changes provide pinmux support, initial device tree >>> support, and SPI, ADC, and MCT IP functionality. >> >> Does FSD have some version number? The FDS, especially in compatibles, >> looks quite generic, so what will happen if a newer SoC comes later? You >> would have: >> - tesla,fsd-pinctrl >> - tesla,fsd-xxxx-pinctrl (where xxxx could be some new version) >> >> This will be extra confusing, because fsd-pinctrl looks like the generic one, >> while it is specific... >> > AFAIK, there is no version for FSD SoC (like we see on Exynos or any other SoC) > In case something comes in future, may be just adopt as Olof suggested in the other thread like fsd2 etc.. >> Best regards, >> Krzysztof The naming is still confusing. The SoC is FSD, compatible is "fsd" but entire sub-architecture is also FSD called. Therefore it looks like creating entire sub-architecture for only one SoC, which actually in multiple pieces is or looks like Samsung Exynos (designed by Samsung, using several blocks from Exynos SoC). Best regards, Krzysztof
