On Wed, 1 Nov 2023 at 01:53, Timur Tabi <timur@xxxxxxxxxx> wrote: > > On Tue, Oct 31, 2023 at 12:20 AM Dave Airlie <airlied@xxxxxxxxx> wrote: > > +#define NV_GSP_REG_NUM_ENTRIES 2 > > + > > +static const struct nv_gsp_registry_entries r535_registry_entries[NV_GSP_REG_NUM_ENTRIES] = { > > + { "RMSecBusResetEnable", 1 }, > > + { "RMForcePcieConfigSave", 1 }, > > +}; > > How about : > > static const struct nv_gsp_registry_entries r535_registry_entries[] = { > { "RMSecBusResetEnable", 1 }, > { "RMForcePcieConfigSave", 1 }, > }; > > #define NV_GSP_REG_NUM_ENTRIES ARRAY_SIZE(r535_registry_entries) Good plan. I'll change that now. Dave.