From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sun, 7 Oct 2012 20:09:37 +0200 Subject: [U-Boot] [PATCH 02/10] split mpc8xx hooks from cmd_ide.c In-Reply-To: <1349625374-12391-3-git-send-email-morpheus.ibis@gmail.com> References: <1349625374-12391-1-git-send-email-morpheus.ibis@gmail.com> <1349625374-12391-3-git-send-email-morpheus.ibis@gmail.com> Message-ID: <201210072009.37936.marex@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 Dear Pavel Herrmann, > move most of mpc8xx hooks from cmd_ide.c into ide_preinit() and newly > created ide_init_postreset() (invoked after calling ide_reset after > ide_preinit), some cleanup to make checkpatch happy, enable IDE init hooks > in configs of affected boards. > Confusingly, these hooks are used by more than just mpc8xx-based boards, > and therefore are placed in arch/ppc/lib/ > > note: checkpatch still emits warnings about using volatile > > Signed-off-by: Pavel Herrmann > --- [...] > --- a/include/ide.h > +++ b/include/ide.h > @@ -54,6 +54,14 @@ void ide_init(void); > ulong ide_read(int device, lbaint_t blknr, ulong blkcnt, void *buffer); > ulong ide_write(int device, lbaint_t blknr, ulong blkcnt, const void > *buffer); > > +#ifdef CONFIG_IDE_PREINIT Is there some documentation for these new config variables? > +int ide_preinit(void); > +#endif > + > +#ifdef CONFIG_IDE_INIT_POSTRESET > +int ide_init_postreset(void); > +#endif > + > #if defined(CONFIG_OF_IDE_FIXUP) > int ide_device_present(int dev); > #endif