On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> Make the type checking macro name consistent with the TYPE_*
> constant.
> 
> Signed-off-by: Eduardo Habkost <[email protected]>
> ---
> Cc: Mark Cave-Ayland <[email protected]>
> Cc: Laurent Vivier <[email protected]>
> Cc: "Hervé Poussineau" <[email protected]>
> Cc: Aleksandar Rikalo <[email protected]>
> Cc: Aleksandar Markovic <[email protected]>
> Cc: Aurelien Jarno <[email protected]>
> Cc: Jiaxun Yang <[email protected]>
> Cc: Paolo Bonzini <[email protected]>
> Cc: Fam Zheng <[email protected]>
> Cc: Artyom Tarasenko <[email protected]>
> Cc: [email protected]
> ---
>  include/hw/scsi/esp.h | 2 +-
>  hw/dma/sparc32_dma.c  | 2 +-
>  hw/m68k/q800.c        | 2 +-
>  hw/mips/jazz.c        | 2 +-
>  hw/scsi/esp.c         | 6 +++---
>  hw/sparc/sun4m.c      | 2 +-
>  6 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/include/hw/scsi/esp.h b/include/hw/scsi/esp.h
> index f281625dd5..20800dbf5b 100644
> --- a/include/hw/scsi/esp.h
> +++ b/include/hw/scsi/esp.h
> @@ -67,7 +67,7 @@ struct ESPState {
>  
>  #define TYPE_ESP "esp"
>  typedef struct SysBusESPState SysBusESPState;
> -DECLARE_INSTANCE_CHECKER(SysBusESPState, ESP_STATE,
> +DECLARE_INSTANCE_CHECKER(SysBusESPState, ESP,
>                           TYPE_ESP)

"ESPDMA" would less likely clash with eventual 'ESP' definition.

My 2 cents.


Reply via email to