Hi,

 > -----Original Message-----
 > From: ext Keshava Munegowda [mailto:keshava_mgo...@ti.com]
 > Sent: 30. maaliskuuta 2011 9:39
 > To: Jokiniemi Kalle (Nokia-MS/Tampere); linux-...@vger.kernel.org;
 > cbouatmai...@gmail.com
 > Cc: Felipe Balbi; Krogerus Heikki (Nokia-MS/Helsinki); sshtyl...@mvista.com;
 > t...@atomide.com; Kevin Hilman; linux-omap@vger.kernel.org;
 > jhnik...@gmail.com
 > 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 ;)

 > 
 > 
 > > +  .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 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-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to