Skip to contentSkip to conversation

Un-used DSS regulator data

omap-for-linus Santosh Shilimkar 2 participants 3 messages Feb 28, 2011

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

Tomi/Sumit,

Today while building Tony's 'omap-for-linus' branch I came across below build warning.
warning: 'sdp3430_vdda_dac_supply' defined but not used

It seems the DSS regulator data is not being used. Are you missing any patch which will use it eventually?

If not I propose to remove that data to kill the warning. Below is the patch for the same. Also attached the same

Regards,
Santosh

Quoted 1 line

From 037f1122d0bfe77883e6cf937009fd955be253a0 Mon Sep 17 00:00:00 2001

From: Santosh Shilimkar <[email protected]>
Date: Mon, 28 Feb 2011 13:54:19 +0530
Subject: [PATCH] omap: 3430sdp-board: Fix warning:
'sdp3430_vdda_dac_supply' defined but not used

The DSS regulator data not seems to be used. So remove the un-used sdp3430_vdda_dac_supply to avoid build warning

Cc Senthilvadivu Guruswamy <[email protected]>
Cc Sumit Semwal <[email protected]>
Cc Tomi Valkeinen <[email protected]>
---
 arch/arm/mach-omap2/board-3430sdp.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)
(quoted diff) +0 -0

b/arch/arm/mach-omap2/board-3430sdp.c
index 8950ecc..19a36a3 100644

arch/arm/mach-omap2/board-3430sdp.c +0 -3
@@ -307,9 +307,6 @@ static struct omap_dss_board_info sdp3430_dss_data = {
 	.default_device	= &sdp3430_lcd_device,
 };
 
-static struct regulator_consumer_supply sdp3430_vdda_dac_supply =
-	REGULATOR_SUPPLY("vdda_dac", "omapdss");
-
 static struct omap_board_config_kernel sdp3430_config[] __initdata = {
 };
 
Signature · 1 line
1.6.0.4

From 037f1122d0bfe77883e6cf937009fd955be253a0 Mon Sep 17 00:00:00 2001
From: Santosh Shilimkar <[email protected]>
Date: Mon, 28 Feb 2011 13:54:19 +0530
Subject: [PATCH] omap: 3430sdp-board: Fix warning: 'sdp3430_vdda_dac_supply' defined but not used

The DSS regulator data not seems to be used. So remove the un-used sdp3430_vdda_dac_supply to avoid build warning

Cc Senthilvadivu Guruswamy <[email protected]>
Cc Sumit Semwal <[email protected]>
Cc Tomi Valkeinen <[email protected]>
---
 arch/arm/mach-omap2/board-3430sdp.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)
arch/arm/mach-omap2/board-3430sdp.c +0 -3
index 8950ecc..19a36a3 100644
@@ -307,9 +307,6 @@ static struct omap_dss_board_info sdp3430_dss_data = {
 	.default_device	= &sdp3430_lcd_device,
 };
 
-static struct regulator_consumer_supply sdp3430_vdda_dac_supply =
-	REGULATOR_SUPPLY("vdda_dac", "omapdss");
-
 static struct omap_board_config_kernel sdp3430_config[] __initdata = {
 };
 
Signature · 1 line
1.6.0.4
#1 lore raw

Hi,

Quoted 8 lines · Santosh

Tomi/Sumit,

Today while building Tony's 'omap-for-linus' branch I came across below build warning.
warning: 'sdp3430_vdda_dac_supply' defined but not used

It seems the DSS regulator data is not being used. Are you missing any patch which will use it eventually?

It was an accidental left over from conflict resolving. I have a patch in DSS tree which removes the regulator supply.

Tomi

#2 lore raw
Quoted 17 lines

-----Original Message-----
From: Tomi Valkeinen [mailto:[email protected]]
Sent: Monday, February 28, 2011 2:30 PM
To: Shilimkar, Santosh
Cc: Semwal, Sumit; Guruswamy, Senthilvadivu; linux- [email protected]; [email protected]
Subject: Re: [omap-for-linus] Un-used DSS regulator data

Hi,

Quoted 6 lines · Santosh

Tomi/Sumit,

Today while building Tony's 'omap-for-linus' branch I came across below build warning.
warning: 'sdp3430_vdda_dac_supply' defined but not used

Quoted 6 lines
Quoted 2 lines

It seems the DSS regulator data is not being used. Are you missing any patch which will use it eventually?

It was an accidental left over from conflict resolving. I have a patch in DSS tree which removes the regulator supply.

Ok.