Hi,
On Mon, Jun 27, 2011 at 01:38:54PM +0530, Jaswinder Singh wrote:
> Hi Felipe,
>
> On 27 June 2011 13:30, Felipe Balbi <balbi@xxxxxx> wrote:
>
> >> >
> >> > +static struct regulator_consumer_supply sdp4430_vusb_supply =
> >> > + REGULATOR_SUPPLY("hsusb0", "ehci-omap.0");
> >
> > this should be an array.
> Ok, I can make it an array of _one_ element.
> Though I am not sure why is that a good thing, or are we to use another
> possible VUSB supply on Panda/SDP boards ? Please suggest so that
> I can add that too.
same comment I gave before to another patch:
it makes the diff a lot easier to understand should anyone modify this
later. It's also a matter of consistency.
--
balbi
Attachment:
signature.asc
Description: Digital signature
