On 3/11/20 7:35 AM, Ang, Chee Hong wrote: [...] >>>> Hmm, so you're just using misc_ops to still issue generic writes. >>>> From the discussion with Marek in the last version, I would have >>>> thought you wanted to create a higher level API instead of still tunnelling >> reads and writes? >> >> Any response to this? > Sorry, I missed this one > Actually I have created higher level API in ATF but I switch back to generic > writes > because the higher level API in ATF doesn't apply to Gen5/A10 platforms.
ATF doesn't apply to Gen5/A10 either though ? > Here is what I will do in my revision in system manager driver: > 1) drop misc_read/misc_write and use misc_ioctl instead in system manager > 2) misc_ioctl() will support configuring EMAC/SDMMC > 3) For SoC64 running at EL2 (non-secure), misc_iotctl() will invoke the ATF's > 'high level' API > 4) For Gen/A10 and SoC64 running at EL3 (secure), the driver just configure > the EMAC/SDMMC registers in misc_iotcl() > Is this better ? Can't you configure everything in secure-mode and just not configure anything anymore in non-secure mode ? -- Best regards, Marek Vasut
