From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Subject: Re: [PATCH v4 2/2] misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP init driver Date: Mon, 18 Dec 2017 14:13:26 +0100 Message-ID: References: <29198c0a-783e-8aa0-00e4-44b1fa1acef7@infradead.org> <1513322656-4571-1-git-send-email-dshah@xilinx.com> <1513322656-4571-3-git-send-email-dshah@xilinx.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnd Bergmann , Dhaval Shah Cc: gregkh , Rob Herring , Mark Rutland , Randy Dunlap , DTML , Linux Kernel Mailing List , Michal Simek , Hyun Kwon , Dhaval Shah List-Id: devicetree@vger.kernel.org On 15.12.2017 14:26, Arnd Bergmann wrote: > In Fri, Dec 15, 2017 at 8:24 AM, Dhaval Shah wrote: >> Xilinx ZYNQMP logicoreIP Init driver is based on the new >> LogiCoreIP design created. This driver provides the processing system >> and programmable logic isolation. Set the frequency based on the clock >> information get from the logicoreIP register set. >> >> It is put in drivers/misc as there is no subsystem for this logicoreIP. >> >> Signed-off-by: Dhaval Shah > > After giving this some more thought, I'd suggest you move the driver to > drivers/soc/xilinx or drivers/soc/zynq instead of drivers/misc/, and have > it merged by Michal Simek as a driver patch that will go through arm-soc. I have not a problem of creating drivers/soc/xilinx/ location for this driver. It is not zynq (arm32) but zynqmp(arm64) device where this driver can be used. As far as I understand it is memory mapped soft IP which could be also accessed by soft core CPU. It means drivers/soc/xilinx could be shared by all xilinx platforms anyway. We have been discussing that openrisc cases and for sure if someone wants to enable this driver there using misc location would be one option but I also think that using drivers/soc/xilinx location is not a bad option because it is very unlikely that anybody tries it. Arnd: misc or drivers/soc/xilinx? Thanks, Michal -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html