RE: [PATCH v3 2/2] RX-51: Enable isp1704 power on/off

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: kalle.jokiniemi@xxxxxxxxx [mailto:kalle.jokiniemi@xxxxxxxxx]
> Sent: Wednesday, March 30, 2011 12:12 PM
> To: keshava_mgowda@xxxxxx; linux-usb@xxxxxxxxxxxxxxx;
cbouatmailru@xxxxxxxxx
> Cc: balbi@xxxxxx; Heikki.Krogerus@xxxxxxxxx; sshtylyov@xxxxxxxxxx;
tony@xxxxxxxxxxx; khilman@xxxxxx;
> linux-omap@xxxxxxxxxxxxxxx; jhnikula@xxxxxxxxx
> Subject: RE: [PATCH v3 2/2] RX-51: Enable isp1704 power on/off
>
> Hi,
>
>  > -----Original Message-----
>  > From: ext Keshava Munegowda [mailto:keshava_mgowda@xxxxxx]
>  > Sent: 30. maaliskuuta 2011 9:39
>  > To: Jokiniemi Kalle (Nokia-MS/Tampere); linux-usb@xxxxxxxxxxxxxxx;
>  > cbouatmailru@xxxxxxxxx
>  > Cc: Felipe Balbi; Krogerus Heikki (Nokia-MS/Helsinki);
sshtylyov@xxxxxxxxxx;
>  > tony@xxxxxxxxxxx; Kevin Hilman; linux-omap@xxxxxxxxxxxxxxx;
>  > jhnikula@xxxxxxxxx
>  > Subject: RE: [PATCH v3 2/2] RX-51: Enable isp1704 power on/off
>  >
>  > > +
>  > >  static struct platform_device rx51_charger_device = {
>  > > -	.name = "isp1704_charger",
>                             ^space here
>
>
>
>  > > +	.name	= "isp1704_charger",
>                               ^tab here
>  >
>  > I don't understand what difference between above two lines?
>  > Is your mail client causing this? Or get send-mail doing this?
>
> No, changed space for tab so that all the members are indented same way.
> I had that (apparently obscure) explanation about this in the commit log
;)

Then, I am sorry; I couldn't able to notice this!



>
>  >
>  >
>  > > +	.dev	= {
>                           ^tab here also
>
>  > > +		.platform_data = &rx51_charger_data,
>  > > +	},
>  > >  };
>  > >
>  > > +static void __init rx51_charger_init(void)
>  > > +{
>  > > +	WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO,
>  > > +		GPIOF_OUT_INIT_LOW, "isp1704_reset"));
>  > > +
>  > > +	platform_device_register(&rx51_charger_device);
>  > > +}
>  > > +
>  > >  #if defined(CONFIG_KEYBOARD_GPIO) ||
>  > defined(CONFIG_KEYBOARD_GPIO_MODULE)
>  > >
>  > >  #define RX51_GPIO_CAMERA_LENS_COVER	110
>  > > @@ -981,6 +1004,6 @@ void __init rx51_peripherals_init(void)
>  > >  	if (partition)
>  > >  		omap2_hsmmc_init(mmc);
>  > >
>  > > -	platform_device_register(&rx51_charger_device);
>  > > +	rx51_charger_init();
>  > >  }
>  > >
>  > > --
>  > > 1.7.1
>  > >
>  > > --
>  > > To unsubscribe from this list: send the line "unsubscribe
linux-usb" in
>  > > the body of a message to majordomo@xxxxxxxxxxxxxxx
>  > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux