On 2023/5/26 23:33, Mark Brown wrote: > On Fri, May 26, 2023 at 02:25:27PM +0800, William Qiu wrote: > >> then: >> properties: >> + clocks: >> + maxItems: 3 >> + >> + clock-names: >> + items: >> + - const: qspi-ref >> + - const: qspi-ahb >> + - const: qspi-apb >> + > > Are these really the names that the clocks have in the IP? It seems > weird that they'd include the IP name in there and not just be ref, ahb > and apb. Hi Mark, These three clocks are the internal clocks in the IP. The AHB clock is the main system clock used to transfer data over the AHB bus between an external master and the QSPI controller. The APB clock is used to access the register map of the QSPI controller, perform controller and device configuration.service interrupts and control certain run time modes. The reference clock is used to serialize the data and drive the external SPI interface. I'm going to change the names of these three clocks to hclk, pclk, and ref_clk, as defined in the data book. What do you think? Thanks for taking time to review this patch series and give useful suggestions. Best regards, William
