On Mon, Aug 1, 2016 at 4:30 PM, Jean-Francois Moine <moinejf@xxxxxxx> wrote: > On Mon, 1 Aug 2016 02:43:06 +0100 > André Przywara <andre.przywara@xxxxxxx> wrote: > >> As this became quite a long read, here a TL;DR: >> - We consider using an SCPI based clock system for the A64, alongside >> allwinner,simple-gates and fixed clocks. We try to avoid any Allwinner >> specific clocks (apart from the simple-gates). >> - ARM Trusted Firmware provides the SCPI implementation - for now, later >> we may move this into a possible arisc firmware. >> - We upstream some basic DT first, possibly omitting any controversial >> clock parts at all. >> >> Let me know what you think! > > Hi André, > > This looks interesting. > As I understand, the clock enable/rate setting functions would be in > the arisc. The arisc firmware would be loaded only once in the Soc and > would contain the code for handling this specific SoC. > From my calculations, this would save about 1Mb of clock descriptions > in the kernel for a universal Allwinner kernel. > > But I don't see why you are keeping the simple-gates. The bus gate may > be ungated/gated when the clock is enabled/disabled, and that's what > Allwinner's software does. For peripherals that have a separate mod clock, having them separate is a good thing. One example might be the audio codecs. You could ungate the bus gate to access its registers to program it, but only enable the mod clock when you actually play something. Regards ChenYu -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html