From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6B66CC433F5 for ; Mon, 6 Dec 2021 08:29:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239377AbhLFIdL (ORCPT ); Mon, 6 Dec 2021 03:33:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46238 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239361AbhLFIdH (ORCPT ); Mon, 6 Dec 2021 03:33:07 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3E43BC061359; Mon, 6 Dec 2021 00:29:38 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 0992EB80EFA; Mon, 6 Dec 2021 08:29:37 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 52477C341C4; Mon, 6 Dec 2021 08:29:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1638779375; bh=Qpk06R2m9n8EzDlZnPqkeXf0kQJLMZ7QcJIUjhYiQe0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=kFa9V1dRNeWM8rerUSgJUZ425RnogekENZT8h2AP8/6kPIR+ohiGTkddnoPlooCMq Xg6pGLFkTh5CUmqGSZsidkn0n8CmFsB+aZ56tbUfo7ESYZTrsdDwYV4Qk0vAOvFX3m A1OVjokBlGxmybM39bXpEdd+V8PmuEsIoRa2rBHg= Date: Mon, 6 Dec 2021 09:29:33 +0100 From: Greg KH To: Tony Huang =?utf-8?B?6buD5oe35Y6a?= Cc: Tony Huang , "derek.kiernan@xilinx.com" , "dragan.cvetic@xilinx.com" , "arnd@arndb.de" , "linux-kernel@vger.kernel.org" , "robh+dt@kernel.org" , "devicetree@vger.kernel.org" , Wells Lu =?utf-8?B?5ZGC6Iqz6aiw?= Subject: Re: [PATCH v2 2/2] misc: Add iop driver for Sunplus SP7021 Message-ID: References: <9bb79f74ff1b08a5f9a1f6707b3b41484506468a.1638499659.git.tonyhuang.sunplus@gmail.com> <5ab2710b37784d51a3f4f90f625ae907@sphcmbx02.sunplus.com.tw> <2e7d5df0cbe74f2697709ed3e8c94b0b@sphcmbx02.sunplus.com.tw> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <2e7d5df0cbe74f2697709ed3e8c94b0b@sphcmbx02.sunplus.com.tw> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 06, 2021 at 08:22:01AM +0000, Tony Huang 黃懷厚 wrote: > Dear Greg KH: > > > -----Original Message----- > > From: Greg KH > > Sent: Monday, December 6, 2021 4:07 PM > > To: Tony Huang 黃懷厚 > > Cc: Tony Huang ; derek.kiernan@xilinx.com; > > dragan.cvetic@xilinx.com; arnd@arndb.de; linux-kernel@vger.kernel.org; > > robh+dt@kernel.org; devicetree@vger.kernel.org; Wells Lu 呂芳騰 > > > > Subject: Re: [PATCH v2 2/2] misc: Add iop driver for Sunplus SP7021 > > > > On Mon, Dec 06, 2021 at 06:48:46AM +0000, Tony Huang 黃懷厚 wrote: > > > Dear Greg KH, Arnd: > > > > > > > -----Original Message----- > > > > From: Greg KH > > > > Sent: Friday, December 3, 2021 6:39 PM > > > > To: Tony Huang > > > > Cc: derek.kiernan@xilinx.com; dragan.cvetic@xilinx.com; > > > > arnd@arndb.de; linux-kernel@vger.kernel.org; robh+dt@kernel.org; > > > > devicetree@vger.kernel.org; Wells Lu 呂芳騰 ; > > > > Tony Huang 黃懷厚 > > > > > > > > Subject: Re: [PATCH v2 2/2] misc: Add iop driver for Sunplus SP7021 > > > > > > > > On Fri, Dec 03, 2021 at 11:48:45AM +0800, Tony Huang wrote: > > > > > IOP (IO Processor) embedded inside SP7021 which is used as > > > > > Processor for I/O control, RTC wake-up and cooperation with CPU & > > > > > PMC in power management purpose. > > > > > The IOP core is DQ8051, so also named IOP8051, it supports > > > > > dedicated JTAG debug pins which share with SP7021. > > > > > In standby mode operation, the power spec reach 400uA. > > > > > > > > > > Signed-off-by: Tony Huang > > > > > --- > > > > > Changes in v2: > > > > > - Addressed comments from Arnd Bergmann. > > > > > - Addressed comments from Greg KH. > > > > > - Addressed comments from kernel test robot. > > > > > > > > > > MAINTAINERS | 1 + > > > > > drivers/misc/Kconfig | 1 + > > > > > drivers/misc/Makefile | 1 + > > > > > drivers/misc/iop/Kconfig | 13 ++ > > > > > drivers/misc/iop/Makefile | 6 + > > > > > > > > Why do you need a subdirectory for a single .c file? > > > > > > > > > > 1. Currently my bin file is placed in the root file system. I need to > > > wait for the kernel booting to succeed before loading the bin code. > > > > What "bin file"? > > > > IOP MODULE EXECUTES 8051 CODE > Source code should reserve SDRAM memory area for IOP module code. 8051 bin file normal code and standby code can be placed in this area. The location area can be select by user. > Normal code: Monitor CPU commands. > Standby code: For RTC wake up, cooperation with CPU&PMC in power management When the system enters standby mode, 8051 bin file should be moved to I_Cache. > I_Cache has 16K only. Standby code cannot exceed 16K. > When the IOP module is mounted, CPU load 8051 codes (normal.bin) into memory. > Iop_base_addr_l and iop_base_addr_h specify address. > During system boot up, when the IOP is mounted, it will load 8051 normal code and start execute 8051 code. So this is firmware, just put it in the normal firmware location. thanks, greg k-h