Skip to content

Commit 3a50f41

Browse files
Pekkarialexdeucher
Pekkari
authored and
alexdeucher
committed
drm/radeon: replace 1-element arrays with flexible-array members
Reported by coccinelle, the following patch will move the following 1 element arrays to flexible arrays. drivers/gpu/drm/radeon/atombios.h:5523:32-48: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:5545:32-48: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:5461:34-44: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4447:30-40: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4236:30-41: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7095:28-45: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:3896:27-37: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:5443:16-25: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:5454:34-43: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4603:21-32: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4628:32-46: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:6285:29-39: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4296:30-36: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4756:28-36: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:4064:22-35: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7327:9-24: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7332:32-53: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7362:26-41: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7369:29-44: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7349:24-32: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) drivers/gpu/drm/radeon/atombios.h:7355:27-35: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays) Signed-off-by: José Pekkarinen <jose.pekkarinen@foxhound.fi> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent 49afe91 commit 3a50f41

1 file changed

Lines changed: 21 additions & 21 deletions

File tree

‎drivers/gpu/drm/radeon/atombios.h‎

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3893,7 +3893,7 @@ typedef struct _ATOM_GPIO_PIN_ASSIGNMENT
38933893
typedef struct _ATOM_GPIO_PIN_LUT
38943894
{
38953895
ATOM_COMMON_TABLE_HEADER sHeader;
3896-
ATOM_GPIO_PIN_ASSIGNMENT asGPIO_Pin[1];
3896+
ATOM_GPIO_PIN_ASSIGNMENT asGPIO_Pin[];
38973897
}ATOM_GPIO_PIN_LUT;
38983898

38993899
/****************************************************************************/
@@ -4061,7 +4061,7 @@ typedef struct _ATOM_SRC_DST_TABLE_FOR_ONE_OBJECT //usSrcDstTableOffset
40614061
UCHAR ucNumberOfSrc;
40624062
USHORT usSrcObjectID[1];
40634063
UCHAR ucNumberOfDst;
4064-
USHORT usDstObjectID[1];
4064+
USHORT usDstObjectID[];
40654065
}ATOM_SRC_DST_TABLE_FOR_ONE_OBJECT;
40664066

40674067

@@ -4233,7 +4233,7 @@ typedef struct _ATOM_CONNECTOR_DEVICE_TAG_RECORD
42334233
ATOM_COMMON_RECORD_HEADER sheader;
42344234
UCHAR ucNumberOfDevice;
42354235
UCHAR ucReserved;
4236-
ATOM_CONNECTOR_DEVICE_TAG asDeviceTag[1]; //This Id is same as "ATOM_DEVICE_XXX_SUPPORT", 1 is only for allocation
4236+
ATOM_CONNECTOR_DEVICE_TAG asDeviceTag[]; //This Id is same as "ATOM_DEVICE_XXX_SUPPORT", 1 is only for allocation
42374237
}ATOM_CONNECTOR_DEVICE_TAG_RECORD;
42384238

42394239

@@ -4293,7 +4293,7 @@ typedef struct _ATOM_OBJECT_GPIO_CNTL_RECORD
42934293
ATOM_COMMON_RECORD_HEADER sheader;
42944294
UCHAR ucFlags; // Future expnadibility
42954295
UCHAR ucNumberOfPins; // Number of GPIO pins used to control the object
4296-
ATOM_GPIO_PIN_CONTROL_PAIR asGpio[1]; // the real gpio pin pair determined by number of pins ucNumberOfPins
4296+
ATOM_GPIO_PIN_CONTROL_PAIR asGpio[]; // the real gpio pin pair determined by number of pins ucNumberOfPins
42974297
}ATOM_OBJECT_GPIO_CNTL_RECORD;
42984298

42994299
//Definitions for GPIO pin state
@@ -4444,7 +4444,7 @@ typedef struct _ATOM_BRACKET_LAYOUT_RECORD
44444444
UCHAR ucWidth;
44454445
UCHAR ucConnNum;
44464446
UCHAR ucReserved;
4447-
ATOM_CONNECTOR_LAYOUT_INFO asConnInfo[1];
4447+
ATOM_CONNECTOR_LAYOUT_INFO asConnInfo[];
44484448
}ATOM_BRACKET_LAYOUT_RECORD;
44494449

44504450
/****************************************************************************/
@@ -4600,7 +4600,7 @@ typedef struct _ATOM_I2C_VOLTAGE_OBJECT_V3
46004600
UCHAR ucVoltageControlAddress;
46014601
UCHAR ucVoltageControlOffset;
46024602
ULONG ulReserved;
4603-
VOLTAGE_LUT_ENTRY asVolI2cLut[1]; // end with 0xff
4603+
VOLTAGE_LUT_ENTRY asVolI2cLut[]; // end with 0xff
46044604
}ATOM_I2C_VOLTAGE_OBJECT_V3;
46054605

46064606
// ATOM_I2C_VOLTAGE_OBJECT_V3.ucVoltageControlFlag
@@ -4625,7 +4625,7 @@ typedef struct _ATOM_LEAKAGE_VOLTAGE_OBJECT_V3
46254625
UCHAR ucLeakageEntryNum; // indicate the entry number of LeakageId/Voltage Lut table
46264626
UCHAR ucReserved[2];
46274627
ULONG ulMaxVoltageLevel;
4628-
LEAKAGE_VOLTAGE_LUT_ENTRY_V2 asLeakageIdLut[1];
4628+
LEAKAGE_VOLTAGE_LUT_ENTRY_V2 asLeakageIdLut[];
46294629
}ATOM_LEAKAGE_VOLTAGE_OBJECT_V3;
46304630

46314631

@@ -4753,7 +4753,7 @@ typedef struct _ATOM_POWER_SOURCE_INFO
47534753
{
47544754
ATOM_COMMON_TABLE_HEADER asHeader;
47554755
UCHAR asPwrbehave[16];
4756-
ATOM_POWER_SOURCE_OBJECT asPwrObj[1];
4756+
ATOM_POWER_SOURCE_OBJECT asPwrObj[];
47574757
}ATOM_POWER_SOURCE_INFO;
47584758

47594759

@@ -5440,7 +5440,7 @@ typedef struct _ATOM_FUSION_SYSTEM_INFO_V2
54405440
typedef struct _ATOM_I2C_DATA_RECORD
54415441
{
54425442
UCHAR ucNunberOfBytes; //Indicates how many bytes SW needs to write to the external ASIC for one block, besides to "Start" and "Stop"
5443-
UCHAR ucI2CData[1]; //I2C data in bytes, should be less than 16 bytes usually
5443+
UCHAR ucI2CData[]; //I2C data in bytes, should be less than 16 bytes usually
54445444
}ATOM_I2C_DATA_RECORD;
54455445

54465446

@@ -5451,14 +5451,14 @@ typedef struct _ATOM_I2C_DEVICE_SETUP_INFO
54515451
UCHAR ucSSChipID; //SS chip being used
54525452
UCHAR ucSSChipSlaveAddr; //Slave Address to set up this SS chip
54535453
UCHAR ucNumOfI2CDataRecords; //number of data block
5454-
ATOM_I2C_DATA_RECORD asI2CData[1];
5454+
ATOM_I2C_DATA_RECORD asI2CData[];
54555455
}ATOM_I2C_DEVICE_SETUP_INFO;
54565456

54575457
//==========================================================================================
54585458
typedef struct _ATOM_ASIC_MVDD_INFO
54595459
{
54605460
ATOM_COMMON_TABLE_HEADER sHeader;
5461-
ATOM_I2C_DEVICE_SETUP_INFO asI2CSetup[1];
5461+
ATOM_I2C_DEVICE_SETUP_INFO asI2CSetup[];
54625462
}ATOM_ASIC_MVDD_INFO;
54635463

54645464
//==========================================================================================
@@ -5520,7 +5520,7 @@ typedef struct _ATOM_ASIC_INTERNAL_SS_INFO
55205520
typedef struct _ATOM_ASIC_INTERNAL_SS_INFO_V2
55215521
{
55225522
ATOM_COMMON_TABLE_HEADER sHeader;
5523-
ATOM_ASIC_SS_ASSIGNMENT_V2 asSpreadSpectrum[1]; //this is point only.
5523+
ATOM_ASIC_SS_ASSIGNMENT_V2 asSpreadSpectrum[]; //this is point only.
55245524
}ATOM_ASIC_INTERNAL_SS_INFO_V2;
55255525

55265526
typedef struct _ATOM_ASIC_SS_ASSIGNMENT_V3
@@ -5542,7 +5542,7 @@ typedef struct _ATOM_ASIC_SS_ASSIGNMENT_V3
55425542
typedef struct _ATOM_ASIC_INTERNAL_SS_INFO_V3
55435543
{
55445544
ATOM_COMMON_TABLE_HEADER sHeader;
5545-
ATOM_ASIC_SS_ASSIGNMENT_V3 asSpreadSpectrum[1]; //this is pointer only.
5545+
ATOM_ASIC_SS_ASSIGNMENT_V3 asSpreadSpectrum[]; //this is pointer only.
55465546
}ATOM_ASIC_INTERNAL_SS_INFO_V3;
55475547

55485548

@@ -6282,7 +6282,7 @@ typedef union _ATOM_MEMORY_SETTING_ID_CONFIG_ACCESS
62826282

62836283
typedef struct _ATOM_MEMORY_SETTING_DATA_BLOCK{
62846284
ATOM_MEMORY_SETTING_ID_CONFIG_ACCESS ulMemoryID;
6285-
ULONG aulMemData[1];
6285+
ULONG aulMemData[];
62866286
}ATOM_MEMORY_SETTING_DATA_BLOCK;
62876287

62886288

@@ -7092,7 +7092,7 @@ typedef struct _ATOM_DISP_OUT_INFO_V3
70927092
UCHAR ucCoreRefClkSource; // value of CORE_REF_CLK_SOURCE
70937093
UCHAR ucDispCaps;
70947094
UCHAR ucReserved[2];
7095-
ASIC_TRANSMITTER_INFO_V2 asTransmitterInfo[1]; // for alligment only
7095+
ASIC_TRANSMITTER_INFO_V2 asTransmitterInfo[]; // for alligment only
70967096
}ATOM_DISP_OUT_INFO_V3;
70977097

70987098
//ucDispCaps
@@ -7324,12 +7324,12 @@ typedef struct _CLOCK_CONDITION_SETTING_ENTRY{
73247324
USHORT usMaxClockFreq;
73257325
UCHAR ucEncodeMode;
73267326
UCHAR ucPhySel;
7327-
ULONG ulAnalogSetting[1];
7327+
ULONG ulAnalogSetting[];
73287328
}CLOCK_CONDITION_SETTING_ENTRY;
73297329

73307330
typedef struct _CLOCK_CONDITION_SETTING_INFO{
73317331
USHORT usEntrySize;
7332-
CLOCK_CONDITION_SETTING_ENTRY asClkCondSettingEntry[1];
7332+
CLOCK_CONDITION_SETTING_ENTRY asClkCondSettingEntry[];
73337333
}CLOCK_CONDITION_SETTING_INFO;
73347334

73357335
typedef struct _PHY_CONDITION_REG_VAL{
@@ -7346,27 +7346,27 @@ typedef struct _PHY_CONDITION_REG_VAL_V2{
73467346
typedef struct _PHY_CONDITION_REG_INFO{
73477347
USHORT usRegIndex;
73487348
USHORT usSize;
7349-
PHY_CONDITION_REG_VAL asRegVal[1];
7349+
PHY_CONDITION_REG_VAL asRegVal[];
73507350
}PHY_CONDITION_REG_INFO;
73517351

73527352
typedef struct _PHY_CONDITION_REG_INFO_V2{
73537353
USHORT usRegIndex;
73547354
USHORT usSize;
7355-
PHY_CONDITION_REG_VAL_V2 asRegVal[1];
7355+
PHY_CONDITION_REG_VAL_V2 asRegVal[];
73567356
}PHY_CONDITION_REG_INFO_V2;
73577357

73587358
typedef struct _PHY_ANALOG_SETTING_INFO{
73597359
UCHAR ucEncodeMode;
73607360
UCHAR ucPhySel;
73617361
USHORT usSize;
7362-
PHY_CONDITION_REG_INFO asAnalogSetting[1];
7362+
PHY_CONDITION_REG_INFO asAnalogSetting[];
73637363
}PHY_ANALOG_SETTING_INFO;
73647364

73657365
typedef struct _PHY_ANALOG_SETTING_INFO_V2{
73667366
UCHAR ucEncodeMode;
73677367
UCHAR ucPhySel;
73687368
USHORT usSize;
7369-
PHY_CONDITION_REG_INFO_V2 asAnalogSetting[1];
7369+
PHY_CONDITION_REG_INFO_V2 asAnalogSetting[];
73707370
}PHY_ANALOG_SETTING_INFO_V2;
73717371

73727372
typedef struct _GFX_HAVESTING_PARAMETERS {

0 commit comments

Comments
 (0)