On 30/05/16 09:30, Neil Armstrong wrote:
On 05/27/2016 10:17 AM, Neil Armstrong wrote:
While looking for other ARMv8 based platform, I found that the RK3368 platform has the same SCPI implementation as Amlogic. They extended it with DDR, system and thermal commands. Look at : https://github.com/geekboxzone/mmallow_kernel/blob/geekbox/drivers/mailbox/scpi_cmd.h https://github.com/geekboxzone/mmallow_kernel/blob/geekbox/driver/mailbox/scpi_protocol.c So the SCPI must have a framework to allow different protocol versions, and must allow command extension. Grouping Rockchip and Amlogic should be done, thus needing a generic name like vendor_scpi or with a version. Sudeep, could you somehow find out which version of the protocol AmLogic and Rockchip based their SCPI development ?
As Caesar Wang replied, they had a previous version of SCPI and I suggested on how to extend the command set previously in private. Not sure whats the progress on that Anyways I had a look at geekbox driver and it looks mostly based on the initial driver I wrote for initial SCPI versions. I am worried that your extension might help people to develop their own protocol instead of following the existing standards. SCPI is published now, so I vendors use that rather than making up their own. Also ARM is trying to standardize something in this area like PSCI, but that may take little more time as it's under discussion with vendors. Though this initial version of SCPI is not published, I am sure it is almost same as v1.0 except that the CMD is not part of payload like v1.0. In v1.0 it's part of payload and the mailbox is used just for doorbell mechanism. IMO, we can add some compatible to indicate the pre v1.0 protocol and add the support to the existing driver itself. Let me know your thoughts. -- Regards, Sudeep
