Hi Mauro,
On 11/11/2014 12:07 PM, Mauro Carvalho Chehab wrote:
> Em Tue, 21 Oct 2014 17:07:00 +0200
> Sebastian Reichel <s...@kernel.org> escreveu:
>
>> This switches back to the normal regulator API (but use
>> managed variant) in preparation for device tree support.
>
> This patch broke compilation. Please be sure that none of the patches in
> the series would break it, as otherwise git bisect would be broken.
Weird, as reported by Sebastian, it works for me.
However, after applying this patch I get these new warnings:
CC drivers/media/radio/si4713/radio-usb-si4713.o
drivers/media/radio/si4713/si4713.c: In function ‘si4713_probe’:
drivers/media/radio/si4713/si4713.c:1617:1: warning: label ‘free_gpio’ defined
but not used [-Wunused-label]
free_gpio:
^
drivers/media/radio/si4713/si4713.c:1451:12: warning: unused variable ‘i’
[-Wunused-variable]
int rval, i;
^
So it's probably not a good idea to merge this patch anyway until this is fixed.
Sebastian, can you fix these warnings and repost?
Thanks!
Hans
>
> Thanks,
> Mauro
>
> drivers/media/radio/si4713/si4713.c: In function 'si4713_powerup':
> drivers/media/radio/si4713/si4713.c:369:10: error: 'struct si4713_device' has
> no member named 'supplies'
>
> ^
> drivers/media/radio/si4713/si4713.c:370:35: error: 'struct si4713_device' has
> no member named 'supplies'
> if (sdev->vdd) {
> ^
> drivers/media/radio/si4713/si4713.c:370:51: error: 'struct si4713_device' has
> no member named 'supply_data'
> if (sdev->vdd) {
> ^
> drivers/media/radio/si4713/si4713.c:402:10: error: 'struct si4713_device' has
> no member named 'supplies'
> v4l2_dbg(1, debug, &sdev->sd, "Device in power up mode\n");
> ^
> drivers/media/radio/si4713/si4713.c:403:36: error: 'struct si4713_device' has
> no member named 'supplies'
> sdev->power_state = POWER_ON;
> ^
> drivers/media/radio/si4713/si4713.c:403:52: error: 'struct si4713_device' has
> no member named 'supply_data'
> sdev->power_state = POWER_ON;
> ^
> drivers/media/radio/si4713/si4713.c: In function 'si4713_powerdown':
> drivers/media/radio/si4713/si4713.c:435:11: error: 'struct si4713_device' has
> no member named 'supplies'
> int err;
> ^
> drivers/media/radio/si4713/si4713.c:436:37: error: 'struct si4713_device' has
> no member named 'supplies'
> u8 resp[SI4713_PWDN_NRESP];
> ^
> drivers/media/radio/si4713/si4713.c:437:16: error: 'struct si4713_device' has
> no member named 'supply_data'
>
> ^
> drivers/media/radio/si4713/si4713.c: In function 'si4713_probe':
> drivers/media/radio/si4713/si4713.c:1444:7: error: 'struct si4713_device' has
> no member named 'supplies'
> /* si4713_probe - probe for the device */
> ^
> drivers/media/radio/si4713/si4713.c:1447:22: error: 'struct si4713_device'
> has no member named 'supplies'
> {
> ^
> drivers/media/radio/si4713/si4713.c:1448:7: error: 'struct si4713_device' has
> no member named 'supply_data'
> struct si4713_device *sdev;
> ^
> drivers/media/radio/si4713/si4713.c:1450:46: error: 'struct si4713_device'
> has no member named 'supplies'
> struct v4l2_ctrl_handler *hdl;
> ^
> drivers/media/radio/si4713/si4713.c:1451:11: error: 'struct si4713_device'
> has no member named 'supply_data'
> int rval, i;
> ^
> drivers/media/radio/si4713/si4713.c:1583:26: error: 'struct si4713_device'
> has no member named 'supplies'
>
> ^
> drivers/media/radio/si4713/si4713.c:1583:42: error: 'struct si4713_device'
> has no member named 'supply_data'
>
> ^
> drivers/media/radio/si4713/si4713.c: In function 'si4713_remove':
> drivers/media/radio/si4713/si4713.c:1607:26: error: 'struct si4713_device'
> has no member named 'supplies'
> goto free_irq;
> ^
> drivers/media/radio/si4713/si4713.c:1607:42: error: 'struct si4713_device'
> has no member named 'supply_data'
> goto free_irq;
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
