From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anatolij Gustschin Date: Fri, 15 May 2009 02:44:51 +0200 Subject: [U-Boot] [PATCH] video: ct6900: Add an option to skip video initialization In-Reply-To: <200905141717.43316.sr@denx.de> References: <1242278732-23803-1-git-send-email-sr@denx.de> <200905141613.20127.sr@denx.de> <20090514150146.22A2C832E416@gemini.denx.de> <200905141717.43316.sr@denx.de> Message-ID: <4A0CBB03.9040009@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Stefan Roese wrote: > On Thursday 14 May 2009 17:01:46 Wolfgang Denk wrote: >>>> If this should be the highest level in the call chain, then any >>>> conditional calling should be done here. >>> OK, I can move the conditional calling to this level. >> I'm not sure. >> >>>> But - don't we already have >>>> a board_video_init() which could return proper status to abort >>>> initialization? >>> This is only implemented for some video drivers (SM501, SED13806, >>> MB826xx) to return the base address for the graphics chip. The PCI >>> drivers like the ct6900 don't use this interface. Here we have other >>> means to get the base address. >>> >>> I'll move the conditional calling to the highest level and resubmit. >> I'd probably rahter see that you add a board_video_init() to the >> ct6900 driver, and have this bail out. > > And what should this function return? The base address is already available in > the driver. Should we move this to all boards using this driver? I don't think > so. > >> Anatolij, what do you thik is the best solution here? > > Yes? What's your preference here? i suggest to move the board_video_skip() to the top of drv_video_init() in drivers/video/cfb_console.c. What are the arguments against doing it? Best regards, Anatolij -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de