On Thu, Jul 6, 2017 at 5:58 AM, Colin King <colin.king@xxxxxxxxxxxxx> wrote: > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > The arrays pctl0_data and pctl1_data do not need to be in global scope, > so them both static. > > Cleans up sparse warnings: > symbol 'pctl0_data' was not declared. Should it be static? > symbol 'pctl1_data' was not declared. Should it be static? > > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Applied. thanks! Alex > --- > drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c b/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c > index 9804318f3488..4c079207d699 100644 > --- a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c > +++ b/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c > @@ -249,7 +249,7 @@ struct pctl_data { > uint32_t data; > }; > > -const struct pctl_data pctl0_data[] = { > +static const struct pctl_data pctl0_data[] = { > {0x0, 0x7a640}, > {0x9, 0x2a64a}, > {0xd, 0x2a680}, > @@ -274,7 +274,7 @@ const struct pctl_data pctl0_data[] = { > #define PCTL0_STCTRL_REG_SAVE_RANGE0_BASE 0xa640 > #define PCTL0_STCTRL_REG_SAVE_RANGE0_LIMIT 0xa833 > > -const struct pctl_data pctl1_data[] = { > +static const struct pctl_data pctl1_data[] = { > {0x0, 0x39a000}, > {0x3b, 0x44a040}, > {0x81, 0x2a08d}, > -- > 2.11.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html