diff mbox series

[v3,02/10] mfd: cs5535-mfd: Remove mfd_cell->id hack

Message ID 20191024163832.31326-3-lee.jones@linaro.org
State Superseded
Headers show
Series Simplify MFD Core | expand

Commit Message

Lee Jones Oct. 24, 2019, 4:38 p.m. UTC
The current implementation abuses the platform 'id' mfd_cell member
to index into the correct resources entry.  Seeing as enough resource
slots are already available, let's just loop through all available
bars and allocate them to their appropriate slot, even if they happen
to be zero.

Signed-off-by: Lee Jones <lee.jones@linaro.org>

---
 drivers/mfd/cs5535-mfd.c | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

-- 
2.17.1

Comments

Daniel Thompson Oct. 25, 2019, 8:36 a.m. UTC | #1
On Thu, Oct 24, 2019 at 05:38:24PM +0100, Lee Jones wrote:
> The current implementation abuses the platform 'id' mfd_cell member

> to index into the correct resources entry.  Seeing as enough resource

> slots are already available, let's just loop through all available

> bars and allocate them to their appropriate slot, even if they happen

> to be zero.

> 

> Signed-off-by: Lee Jones <lee.jones@linaro.org>


Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>


> ---

>  drivers/mfd/cs5535-mfd.c | 14 ++------------

>  1 file changed, 2 insertions(+), 12 deletions(-)

> 

> diff --git a/drivers/mfd/cs5535-mfd.c b/drivers/mfd/cs5535-mfd.c

> index cda7f5b942e7..b35f1efa01f6 100644

> --- a/drivers/mfd/cs5535-mfd.c

> +++ b/drivers/mfd/cs5535-mfd.c

> @@ -63,25 +63,21 @@ static struct resource cs5535_mfd_resources[NR_BARS];

>  

>  static struct mfd_cell cs5535_mfd_cells[] = {

>  	{

> -		.id = SMB_BAR,

>  		.name = "cs5535-smb",

>  		.num_resources = 1,

>  		.resources = &cs5535_mfd_resources[SMB_BAR],

>  	},

>  	{

> -		.id = GPIO_BAR,

>  		.name = "cs5535-gpio",

>  		.num_resources = 1,

>  		.resources = &cs5535_mfd_resources[GPIO_BAR],

>  	},

>  	{

> -		.id = MFGPT_BAR,

>  		.name = "cs5535-mfgpt",

>  		.num_resources = 1,

>  		.resources = &cs5535_mfd_resources[MFGPT_BAR],

>  	},

>  	{

> -		.id = PMS_BAR,

>  		.name = "cs5535-pms",

>  		.num_resources = 1,

>  		.resources = &cs5535_mfd_resources[PMS_BAR],

> @@ -90,7 +86,6 @@ static struct mfd_cell cs5535_mfd_cells[] = {

>  		.disable = cs5535_mfd_res_disable,

>  	},

>  	{

> -		.id = ACPI_BAR,

>  		.name = "cs5535-acpi",

>  		.num_resources = 1,

>  		.resources = &cs5535_mfd_resources[ACPI_BAR],

> @@ -108,23 +103,18 @@ static const char *olpc_acpi_clones[] = {

>  static int cs5535_mfd_probe(struct pci_dev *pdev,

>  		const struct pci_device_id *id)

>  {

> -	int err, i;

> +	int err, bar;

>  

>  	err = pci_enable_device(pdev);

>  	if (err)

>  		return err;

>  

> -	/* fill in IO range for each cell; subdrivers handle the region */

> -	for (i = 0; i < ARRAY_SIZE(cs5535_mfd_cells); i++) {

> -		int bar = cs5535_mfd_cells[i].id;

> +	for (bar = 0; bar < NR_BARS; bar++) {

>  		struct resource *r = &cs5535_mfd_resources[bar];

>  

>  		r->flags = IORESOURCE_IO;

>  		r->start = pci_resource_start(pdev, bar);

>  		r->end = pci_resource_end(pdev, bar);

> -

> -		/* id is used for temporarily storing BAR; unset it now */

> -		cs5535_mfd_cells[i].id = 0;

>  	}

>  

>  	err = mfd_add_devices(&pdev->dev, PLATFORM_DEVID_NONE, cs5535_mfd_cells,

> -- 

> 2.17.1

>
diff mbox series

Patch

diff --git a/drivers/mfd/cs5535-mfd.c b/drivers/mfd/cs5535-mfd.c
index cda7f5b942e7..b35f1efa01f6 100644
--- a/drivers/mfd/cs5535-mfd.c
+++ b/drivers/mfd/cs5535-mfd.c
@@ -63,25 +63,21 @@  static struct resource cs5535_mfd_resources[NR_BARS];
 
 static struct mfd_cell cs5535_mfd_cells[] = {
 	{
-		.id = SMB_BAR,
 		.name = "cs5535-smb",
 		.num_resources = 1,
 		.resources = &cs5535_mfd_resources[SMB_BAR],
 	},
 	{
-		.id = GPIO_BAR,
 		.name = "cs5535-gpio",
 		.num_resources = 1,
 		.resources = &cs5535_mfd_resources[GPIO_BAR],
 	},
 	{
-		.id = MFGPT_BAR,
 		.name = "cs5535-mfgpt",
 		.num_resources = 1,
 		.resources = &cs5535_mfd_resources[MFGPT_BAR],
 	},
 	{
-		.id = PMS_BAR,
 		.name = "cs5535-pms",
 		.num_resources = 1,
 		.resources = &cs5535_mfd_resources[PMS_BAR],
@@ -90,7 +86,6 @@  static struct mfd_cell cs5535_mfd_cells[] = {
 		.disable = cs5535_mfd_res_disable,
 	},
 	{
-		.id = ACPI_BAR,
 		.name = "cs5535-acpi",
 		.num_resources = 1,
 		.resources = &cs5535_mfd_resources[ACPI_BAR],
@@ -108,23 +103,18 @@  static const char *olpc_acpi_clones[] = {
 static int cs5535_mfd_probe(struct pci_dev *pdev,
 		const struct pci_device_id *id)
 {
-	int err, i;
+	int err, bar;
 
 	err = pci_enable_device(pdev);
 	if (err)
 		return err;
 
-	/* fill in IO range for each cell; subdrivers handle the region */
-	for (i = 0; i < ARRAY_SIZE(cs5535_mfd_cells); i++) {
-		int bar = cs5535_mfd_cells[i].id;
+	for (bar = 0; bar < NR_BARS; bar++) {
 		struct resource *r = &cs5535_mfd_resources[bar];
 
 		r->flags = IORESOURCE_IO;
 		r->start = pci_resource_start(pdev, bar);
 		r->end = pci_resource_end(pdev, bar);
-
-		/* id is used for temporarily storing BAR; unset it now */
-		cs5535_mfd_cells[i].id = 0;
 	}
 
 	err = mfd_add_devices(&pdev->dev, PLATFORM_DEVID_NONE, cs5535_mfd_cells,