Re: [flashrom] [PATCH] Don't compile internal code if not selected

[Idwer Vollering]

2009/12/24 Carl-Daniel Hailfinger
> internal.c was always compiled in because it hosted the function
> internal_delay(). Move that function to udelay.c and compile internal.c
> only if really needed.
> physmap.c is only needed if the programmer is internal or a PCI card.
> Make its compilation conditional.
>
> Signed-off-by: Carl-Daniel Hailfinger
> Index: flashrom-programmer_internal_refactor_delay/flash.h
>
> --- flashrom-programmer_internal_refactor_delay/flash.h (Revision 821)
> +++ flashrom-programmer_internal_refactor_delay/flash.h (Arbeitskopie)
> @@ -277,6 +277,7 @@
>  /* udelay.c */
>  void myusec_delay(int usecs);
>  void myusec_calibrate_delay(void);
> +void internal_delay(int usecs);
>
> #if NEED_PCI == 1
>  /* pcidev.c */
> @@ -365,7 +366,6 @@
>  uint8_t mmio_readb(void *addr);
>  uint16_t mmio_readw(void *addr);
>  uint32_t mmio_readl(void *addr);
> -void internal_delay(int usecs);
>  int noop_shutdown(void);
>  void *fallback_map(const char *descr, unsigned long phys_addr, size_t
> len);
>  void fallback_unmap(void *virt_addr, size_t len);
> Index: flashrom-programmer_internal_refactor_delay/Makefile
> ===================================================================
> --- flashrom-programmer_internal_refactor_delay/Makefile        (Revision
> 821)
> +++ flashrom-programmer_internal_refactor_delay/Makefile
>  (Arbeitskopie)
> @@ -47,7 +47,7 @@
>        sst28sf040.o am29f040b.o mx29f002.o m29f400bt.o pm29f002.o
> w39v040c.o \
>        w49f002u.o 82802ab.o pm49fl00x.o sst49lf040.o en29f002a.o m29f002.o
> \
>        sst49lfxxxc.o sst_fwhub.o flashchips.o layout.o spi.o \
>   flashrom.o print.o physmap.o internal.o udelay.o
>   flashrom.o print.o udelay.o
>
> all: pciutils features dep $(PROGRAM)
>
> @@ -168,7 +168,7 @@
>
> ifeq ($(NEED_PCI), yes)
>  FEATURE_CFLAGS += -D'NEED_PCI=1'
> -OBJS += pcidev.o
> +OBJS += pcidev.o physmap.o internal.o
>  endif
>
> ifeq ($(CONFIG_PRINT_WIKI), yes)
> Index: flashrom-programmer_internal_refactor_delay/internal.c
> ===================================================================
> --- flashrom-programmer_internal_refactor_delay/internal.c      (Revision
> 821)
> +++ flashrom-programmer_internal_refactor_delay/internal.c
>  (Arbeitskopie)
> @@ -248,18 +248,6 @@
>        return *(volatile uint32_t *) addr;
>  }
>
> -void internal_delay(int usecs)
> -{
>   /* If the delay is >1 s, use usleep because timing does not need to
>    * be so precise.
>    */
>   if (usecs > 1000000) {
>           usleep(usecs);
>   } else {
>           myusec_delay(usecs);
>   }
>
> -}
>
> /* No-op shutdown() for programmers which don't need special handling */
>  int noop_shutdown(void)
>  {
> Index: flashrom-programmer_internal_refactor_delay/udelay.c
> ===================================================================
> --- flashrom-programmer_internal_refactor_delay/udelay.c        (Revision
> 821)
> +++ flashrom-programmer_internal_refactor_delay/udelay.c
>  (Arbeitskopie)
> @@ -63,3 +63,16 @@
>                     (unsigned long)micro, timeusec);
>        printf("OK.\n");
>  }
>
> +void internal_delay(int usecs)
> +{
>   /* If the delay is >1 s, use usleep because timing does not need to
>    * be so precise.
>    */
>   if (usecs > 1000000) {
>           usleep(usecs);
>   } else {
>           myusec_delay(usecs);
>   }
>
> +}
>
> flashrom mailing list
> [e-mail]
> http://www.flashrom.org/mailman/listinfo/flashrom

Acked-by: Idwer Vollering <[e-mail] >