You forgot to say:

   Pointy Hat to:  ian

I'm not quite sure how I fumbled the return-type change when I applied
the patches from our local repo; thanks to Christoph for spotting this.

-- Ian

On Sun, 2013-04-07 at 13:03 +0000, Hans Petter Selasky wrote:
> Author: hselasky
> Date: Sun Apr  7 13:03:57 2013
> New Revision: 249232
> URL: http://svnweb.freebsd.org/changeset/base/249232
> 
> Log:
>   Fix regression issue after r248910.
>   
>   PR:         arm/177685
>   Submitted by:       Christoph Mallon <christoph.mal...@gmx.de>
> 
> Modified:
>   head/sys/arm/at91/at91_pio.c
>   head/sys/arm/at91/at91_piovar.h
>   head/sys/dev/usb/controller/at91dci_atmelarm.c
> 
> Modified: head/sys/arm/at91/at91_pio.c
> ==============================================================================
> --- head/sys/arm/at91/at91_pio.c      Sun Apr  7 11:05:38 2013        
> (r249231)
> +++ head/sys/arm/at91/at91_pio.c      Sun Apr  7 13:03:57 2013        
> (r249232)
> @@ -554,12 +554,12 @@ at91_pio_gpio_clear(uint32_t pio, uint32
>       PIO[PIO_CODR / 4] = data_mask;
>  }
>  
> -uint8_t
> +uint32_t
>  at91_pio_gpio_get(uint32_t pio, uint32_t data_mask)
>  {
>       uint32_t *PIO = (uint32_t *)(AT91_BASE + pio);
>  
> -     return ((PIO[PIO_PDSR / 4] & data_mask));
> +     return (PIO[PIO_PDSR / 4] & data_mask);
>  }
>  
>  void
> 
> Modified: head/sys/arm/at91/at91_piovar.h
> ==============================================================================
> --- head/sys/arm/at91/at91_piovar.h   Sun Apr  7 11:05:38 2013        
> (r249231)
> +++ head/sys/arm/at91/at91_piovar.h   Sun Apr  7 13:03:57 2013        
> (r249232)
> @@ -39,7 +39,7 @@ void at91_pio_gpio_output(uint32_t pio, 
>  void at91_pio_gpio_high_z(uint32_t pio, uint32_t high_z_mask, int enable);
>  void at91_pio_gpio_set(uint32_t pio, uint32_t data_mask);
>  void at91_pio_gpio_clear(uint32_t pio, uint32_t data_mask);
> -uint8_t at91_pio_gpio_get(uint32_t pio, uint32_t data_mask);
> +uint32_t at91_pio_gpio_get(uint32_t pio, uint32_t data_mask);
>  void at91_pio_gpio_set_deglitch(uint32_t pio, uint32_t data_mask,
>      int use_deglitch);
>  void at91_pio_gpio_set_interrupt(uint32_t pio, uint32_t data_mask,
> 
> Modified: head/sys/dev/usb/controller/at91dci_atmelarm.c
> ==============================================================================
> --- head/sys/dev/usb/controller/at91dci_atmelarm.c    Sun Apr  7 11:05:38 
> 2013        (r249231)
> +++ head/sys/dev/usb/controller/at91dci_atmelarm.c    Sun Apr  7 13:03:57 
> 2013        (r249232)
> @@ -91,7 +91,7 @@ at91_vbus_poll(struct at91_udp_softc *sc
>  {
>       uint8_t vbus_val;
>  
> -     vbus_val = at91_pio_gpio_get(VBUS_BASE, VBUS_MASK);
> +     vbus_val = at91_pio_gpio_get(VBUS_BASE, VBUS_MASK) != 0;
>       at91dci_vbus_interrupt(&sc->sc_dci, vbus_val);
>  
>       callout_reset(&sc->sc_vbus, hz, (void *)&at91_vbus_poll, sc);


_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to