All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa@gmail.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Alan Stern <stern@rowland.harvard.edu>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	kyungmin.park@samsung.com, balbi@ti.com, jg1.han@samsung.com,
	s.nawrocki@samsung.com, kgene.kim@samsung.com,
	grant.likely@linaro.org, tony@atomide.com, arnd@arndb.de,
	swarren@nvidia.com, devicetree-discuss@lists.ozlabs.org,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-omap@vger.kernel.org,
	linux-usb@vger.kernel.org, linux-media@vger.kernel.org,
	linux-fbdev@vger.kernel.org, akpm@linux-foundation.org,
	balajitk@ti.com, george.cherian@ti.com, nsekhar@ti.com
Subject: Re: [PATCH 01/15] drivers: phy: add generic PHY framework
Date: Sun, 21 Jul 2013 12:31:05 +0200	[thread overview]
Message-ID: <3839600.WiC1OLF35o@flatron> (raw)
In-Reply-To: <20130721025910.GA23043@kroah.com>

Hi,

On Saturday 20 of July 2013 19:59:10 Greg KH wrote:
> On Sat, Jul 20, 2013 at 10:32:26PM -0400, Alan Stern wrote:
> > On Sat, 20 Jul 2013, Greg KH wrote:
> > > > >>That should be passed using platform data.
> > > > >
> > > > >Ick, don't pass strings around, pass pointers.  If you have
> > > > >platform
> > > > >data you can get to, then put the pointer there, don't use a
> > > > >"name".
> > > > 
> > > > I don't think I understood you here :-s We wont have phy pointer
> > > > when we create the device for the controller no?(it'll be done in
> > > > board file). Probably I'm missing something.
> > > 
> > > Why will you not have that pointer?  You can't rely on the "name" as
> > > the device id will not match up, so you should be able to rely on
> > > the pointer being in the structure that the board sets up, right?
> > > 
> > > Don't use names, especially as ids can, and will, change, that is
> > > going
> > > to cause big problems.  Use pointers, this is C, we are supposed to
> > > be
> > > doing that :)
> > 
> > Kishon, I think what Greg means is this:  The name you are using must
> > be stored somewhere in a data structure constructed by the board file,
> > right?  Or at least, associated with some data structure somehow.
> > Otherwise the platform code wouldn't know which PHY hardware
> > corresponded to a particular name.
> > 
> > Greg's suggestion is that you store the address of that data structure
> > in the platform data instead of storing the name string.  Have the
> > consumer pass the data structure's address when it calls phy_create,
> > instead of passing the name.  Then you don't have to worry about two
> > PHYs accidentally ending up with the same name or any other similar
> > problems.
> 
> Close, but the issue is that whatever returns from phy_create() should
> then be used, no need to call any "find" functions, as you can just use
> the pointer that phy_create() returns.  Much like all other class api
> functions in the kernel work.

I think there is a confusion here about who registers the PHYs.

All platform code does is registering a platform/i2c/whatever device, 
which causes a driver (located in drivers/phy/) to be instantiated. Such 
drivers call phy_create(), usually in their probe() callbacks, so 
platform_code has no way (and should have no way, for the sake of 
layering) to get what phy_create() returns.

IMHO we need a lookup method for PHYs, just like for clocks, regulators, 
PWMs or even i2c busses because there are complex cases when passing just 
a name using platform data will not work. I would second what Stephen said 
[1] and define a structure doing things in a DT-like way.

Example;

[platform code]

static const struct phy_lookup my_phy_lookup[] = {
	PHY_LOOKUP("s3c-hsotg.0", "otg", "samsung-usbphy.1", "phy.2"),
	/* etc... */
};

static void my_machine_init(void)
{
	/* ... */
	phy_register_lookup(my_phy_lookup, ARRAY_SIZE(my_phy_lookup));
	/* ... */
}

/* Notice nothing stuffed in platform data. */

[provider code - samsung-usbphy driver]

static int samsung_usbphy_probe(...)
{
	/* ... */
		for (i = 0; i < PHY_COUNT; ++i) {
			usbphy->phy[i].name = "phy";
			usbphy->phy[i].id = i;
			/* ... */
			ret = phy_create(&usbphy->phy);
			/* err handling */
		}
	/* ... */
}

[consumer code - s3c-hsotg driver]

static int s3c_hsotg_probe(...)
{
	/* ... */
	phy = phy_get(&pdev->dev, "otg");
	/* ... */
}

[1] http://thread.gmane.org/gmane.linux.ports.arm.kernel/252813

Best regards,
Tomasz


WARNING: multiple messages have this Message-ID (diff)
From: Tomasz Figa <tomasz.figa@gmail.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 01/15] drivers: phy: add generic PHY framework
Date: Sun, 21 Jul 2013 10:31:05 +0000	[thread overview]
Message-ID: <3839600.WiC1OLF35o@flatron> (raw)
In-Reply-To: <20130721025910.GA23043@kroah.com>

Hi,

On Saturday 20 of July 2013 19:59:10 Greg KH wrote:
> On Sat, Jul 20, 2013 at 10:32:26PM -0400, Alan Stern wrote:
> > On Sat, 20 Jul 2013, Greg KH wrote:
> > > > >>That should be passed using platform data.
> > > > >
> > > > >Ick, don't pass strings around, pass pointers.  If you have
> > > > >platform
> > > > >data you can get to, then put the pointer there, don't use a
> > > > >"name".
> > > > 
> > > > I don't think I understood you here :-s We wont have phy pointer
> > > > when we create the device for the controller no?(it'll be done in
> > > > board file). Probably I'm missing something.
> > > 
> > > Why will you not have that pointer?  You can't rely on the "name" as
> > > the device id will not match up, so you should be able to rely on
> > > the pointer being in the structure that the board sets up, right?
> > > 
> > > Don't use names, especially as ids can, and will, change, that is
> > > going
> > > to cause big problems.  Use pointers, this is C, we are supposed to
> > > be
> > > doing that :)
> > 
> > Kishon, I think what Greg means is this:  The name you are using must
> > be stored somewhere in a data structure constructed by the board file,
> > right?  Or at least, associated with some data structure somehow.
> > Otherwise the platform code wouldn't know which PHY hardware
> > corresponded to a particular name.
> > 
> > Greg's suggestion is that you store the address of that data structure
> > in the platform data instead of storing the name string.  Have the
> > consumer pass the data structure's address when it calls phy_create,
> > instead of passing the name.  Then you don't have to worry about two
> > PHYs accidentally ending up with the same name or any other similar
> > problems.
> 
> Close, but the issue is that whatever returns from phy_create() should
> then be used, no need to call any "find" functions, as you can just use
> the pointer that phy_create() returns.  Much like all other class api
> functions in the kernel work.

I think there is a confusion here about who registers the PHYs.

All platform code does is registering a platform/i2c/whatever device, 
which causes a driver (located in drivers/phy/) to be instantiated. Such 
drivers call phy_create(), usually in their probe() callbacks, so 
platform_code has no way (and should have no way, for the sake of 
layering) to get what phy_create() returns.

IMHO we need a lookup method for PHYs, just like for clocks, regulators, 
PWMs or even i2c busses because there are complex cases when passing just 
a name using platform data will not work. I would second what Stephen said 
[1] and define a structure doing things in a DT-like way.

Example;

[platform code]

static const struct phy_lookup my_phy_lookup[] = {
	PHY_LOOKUP("s3c-hsotg.0", "otg", "samsung-usbphy.1", "phy.2"),
	/* etc... */
};

static void my_machine_init(void)
{
	/* ... */
	phy_register_lookup(my_phy_lookup, ARRAY_SIZE(my_phy_lookup));
	/* ... */
}

/* Notice nothing stuffed in platform data. */

[provider code - samsung-usbphy driver]

static int samsung_usbphy_probe(...)
{
	/* ... */
		for (i = 0; i < PHY_COUNT; ++i) {
			usbphy->phy[i].name = "phy";
			usbphy->phy[i].id = i;
			/* ... */
			ret = phy_create(&usbphy->phy);
			/* err handling */
		}
	/* ... */
}

[consumer code - s3c-hsotg driver]

static int s3c_hsotg_probe(...)
{
	/* ... */
	phy = phy_get(&pdev->dev, "otg");
	/* ... */
}

[1] http://thread.gmane.org/gmane.linux.ports.arm.kernel/252813

Best regards,
Tomasz


WARNING: multiple messages have this Message-ID (diff)
From: tomasz.figa@gmail.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 01/15] drivers: phy: add generic PHY framework
Date: Sun, 21 Jul 2013 12:31:05 +0200	[thread overview]
Message-ID: <3839600.WiC1OLF35o@flatron> (raw)
In-Reply-To: <20130721025910.GA23043@kroah.com>

Hi,

On Saturday 20 of July 2013 19:59:10 Greg KH wrote:
> On Sat, Jul 20, 2013 at 10:32:26PM -0400, Alan Stern wrote:
> > On Sat, 20 Jul 2013, Greg KH wrote:
> > > > >>That should be passed using platform data.
> > > > >
> > > > >Ick, don't pass strings around, pass pointers.  If you have
> > > > >platform
> > > > >data you can get to, then put the pointer there, don't use a
> > > > >"name".
> > > > 
> > > > I don't think I understood you here :-s We wont have phy pointer
> > > > when we create the device for the controller no?(it'll be done in
> > > > board file). Probably I'm missing something.
> > > 
> > > Why will you not have that pointer?  You can't rely on the "name" as
> > > the device id will not match up, so you should be able to rely on
> > > the pointer being in the structure that the board sets up, right?
> > > 
> > > Don't use names, especially as ids can, and will, change, that is
> > > going
> > > to cause big problems.  Use pointers, this is C, we are supposed to
> > > be
> > > doing that :)
> > 
> > Kishon, I think what Greg means is this:  The name you are using must
> > be stored somewhere in a data structure constructed by the board file,
> > right?  Or at least, associated with some data structure somehow.
> > Otherwise the platform code wouldn't know which PHY hardware
> > corresponded to a particular name.
> > 
> > Greg's suggestion is that you store the address of that data structure
> > in the platform data instead of storing the name string.  Have the
> > consumer pass the data structure's address when it calls phy_create,
> > instead of passing the name.  Then you don't have to worry about two
> > PHYs accidentally ending up with the same name or any other similar
> > problems.
> 
> Close, but the issue is that whatever returns from phy_create() should
> then be used, no need to call any "find" functions, as you can just use
> the pointer that phy_create() returns.  Much like all other class api
> functions in the kernel work.

I think there is a confusion here about who registers the PHYs.

All platform code does is registering a platform/i2c/whatever device, 
which causes a driver (located in drivers/phy/) to be instantiated. Such 
drivers call phy_create(), usually in their probe() callbacks, so 
platform_code has no way (and should have no way, for the sake of 
layering) to get what phy_create() returns.

IMHO we need a lookup method for PHYs, just like for clocks, regulators, 
PWMs or even i2c busses because there are complex cases when passing just 
a name using platform data will not work. I would second what Stephen said 
[1] and define a structure doing things in a DT-like way.

Example;

[platform code]

static const struct phy_lookup my_phy_lookup[] = {
	PHY_LOOKUP("s3c-hsotg.0", "otg", "samsung-usbphy.1", "phy.2"),
	/* etc... */
};

static void my_machine_init(void)
{
	/* ... */
	phy_register_lookup(my_phy_lookup, ARRAY_SIZE(my_phy_lookup));
	/* ... */
}

/* Notice nothing stuffed in platform data. */

[provider code - samsung-usbphy driver]

static int samsung_usbphy_probe(...)
{
	/* ... */
		for (i = 0; i < PHY_COUNT; ++i) {
			usbphy->phy[i].name = "phy";
			usbphy->phy[i].id = i;
			/* ... */
			ret = phy_create(&usbphy->phy);
			/* err handling */
		}
	/* ... */
}

[consumer code - s3c-hsotg driver]

static int s3c_hsotg_probe(...)
{
	/* ... */
	phy = phy_get(&pdev->dev, "otg");
	/* ... */
}

[1] http://thread.gmane.org/gmane.linux.ports.arm.kernel/252813

Best regards,
Tomasz

  parent reply	other threads:[~2013-07-21 10:31 UTC|newest]

Thread overview: 371+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-18  6:46 [PATCH 00/15] PHY framework Kishon Vijay Abraham I
2013-07-18  6:58 ` Kishon Vijay Abraham I
2013-07-18  6:46 ` Kishon Vijay Abraham I
2013-07-18  6:46 ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 01/15] drivers: phy: add generic " Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  7:20   ` Greg KH
2013-07-18  7:20     ` Greg KH
2013-07-18  7:20     ` Greg KH
2013-07-18  8:59     ` Kishon Vijay Abraham I
2013-07-18  9:11       ` Kishon Vijay Abraham I
2013-07-18  8:59       ` Kishon Vijay Abraham I
2013-07-18  8:59       ` Kishon Vijay Abraham I
2013-07-18 15:49       ` Greg KH
2013-07-18 15:49         ` Greg KH
2013-07-18 15:49         ` Greg KH
2013-07-19  5:37         ` Kishon Vijay Abraham I
2013-07-19  5:49           ` Kishon Vijay Abraham I
2013-07-19  5:37           ` Kishon Vijay Abraham I
2013-07-19  5:37           ` Kishon Vijay Abraham I
2013-07-19  5:43           ` Greg KH
2013-07-19  5:43             ` Greg KH
2013-07-19  5:43             ` Greg KH
2013-07-19  5:55             ` Kishon Vijay Abraham I
2013-07-19  5:56               ` Kishon Vijay Abraham I
2013-07-19  5:55               ` Kishon Vijay Abraham I
2013-07-19  5:55               ` Kishon Vijay Abraham I
2013-07-19  6:29               ` Greg KH
2013-07-19  6:29                 ` Greg KH
2013-07-19  6:29                 ` Greg KH
2013-07-19  6:29                 ` Greg KH
2013-07-19  6:36                 ` Kishon Vijay Abraham I
2013-07-19  6:48                   ` Kishon Vijay Abraham I
2013-07-19  6:36                   ` Kishon Vijay Abraham I
2013-07-19  6:36                   ` Kishon Vijay Abraham I
2013-07-19 15:54                   ` Stephen Warren
2013-07-19 15:54                     ` Stephen Warren
2013-07-19 15:54                     ` Stephen Warren
2013-07-20  3:15                     ` Kishon Vijay Abraham I
2013-07-20  3:27                       ` Kishon Vijay Abraham I
2013-07-20  3:15                       ` Kishon Vijay Abraham I
2013-07-20  3:15                       ` Kishon Vijay Abraham I
2013-07-19 23:50                   ` Greg KH
2013-07-19 23:50                     ` Greg KH
2013-07-19 23:50                     ` Greg KH
2013-07-20  3:19                     ` Kishon Vijay Abraham I
2013-07-20  3:31                       ` Kishon Vijay Abraham I
2013-07-20  3:19                       ` Kishon Vijay Abraham I
2013-07-20  3:19                       ` Kishon Vijay Abraham I
2013-07-20 22:00                       ` Greg KH
2013-07-20 22:00                         ` Greg KH
2013-07-20 22:00                         ` Greg KH
2013-07-20 22:00                         ` Greg KH
2013-07-21  2:32                         ` Alan Stern
2013-07-21  2:32                           ` Alan Stern
2013-07-21  2:32                           ` Alan Stern
2013-07-21  2:32                           ` Alan Stern
2013-07-21  2:59                           ` Greg KH
2013-07-21  2:59                             ` Greg KH
2013-07-21  2:59                             ` Greg KH
2013-07-21 10:22                             ` Sascha Hauer
2013-07-21 10:22                               ` Sascha Hauer
2013-07-21 10:22                               ` Sascha Hauer
2013-07-21 15:48                               ` Greg KH
2013-07-21 15:48                                 ` Greg KH
2013-07-21 15:48                                 ` Greg KH
2013-07-21 17:14                                 ` Sylwester Nawrocki
2013-07-21 17:14                                   ` Sylwester Nawrocki
2013-07-21 17:14                                   ` Sylwester Nawrocki
2013-07-21 19:22                                   ` Alan Stern
2013-07-21 19:22                                     ` Alan Stern
2013-07-21 19:22                                     ` Alan Stern
2013-07-21 19:22                                     ` Alan Stern
2013-07-22  7:25                                     ` Kishon Vijay Abraham I
2013-07-22  7:37                                       ` Kishon Vijay Abraham I
2013-07-22  7:25                                       ` Kishon Vijay Abraham I
2013-07-22  7:25                                       ` Kishon Vijay Abraham I
2013-07-22 14:44                                       ` Alan Stern
2013-07-22 14:44                                         ` Alan Stern
2013-07-22 14:44                                         ` Alan Stern
2013-07-22 14:44                                         ` Alan Stern
2013-07-23  5:47                                         ` Kishon Vijay Abraham I
2013-07-23  5:59                                           ` Kishon Vijay Abraham I
2013-07-23  5:47                                           ` Kishon Vijay Abraham I
2013-07-23  5:47                                           ` Kishon Vijay Abraham I
2013-07-23  7:29                                         ` Tomasz Figa
2013-07-23  7:29                                           ` Tomasz Figa
2013-07-23  7:29                                           ` Tomasz Figa
2013-07-23  7:55                                           ` Tomasz Figa
2013-07-23  7:55                                             ` Tomasz Figa
2013-07-23  7:55                                             ` Tomasz Figa
2013-07-23 14:37                                             ` Alan Stern
2013-07-23 14:37                                               ` Alan Stern
2013-07-23 14:37                                               ` Alan Stern
2013-07-23 14:37                                               ` Alan Stern
2013-07-23 14:50                                               ` Tomasz Figa
2013-07-23 14:50                                                 ` Tomasz Figa
2013-07-23 14:50                                                 ` Tomasz Figa
2013-07-23 14:50                                                 ` Tomasz Figa
2013-07-23 15:18                                               ` Kishon Vijay Abraham I
2013-07-23 15:30                                                 ` Kishon Vijay Abraham I
2013-07-23 15:18                                                 ` Kishon Vijay Abraham I
2013-07-23 15:18                                                 ` Kishon Vijay Abraham I
2013-07-23 16:18                                                 ` Greg KH
2013-07-23 16:18                                                   ` Greg KH
2013-07-23 16:18                                                   ` Greg KH
2013-07-23 16:18                                                   ` Greg KH
2013-07-23 16:28                                                   ` Kishon Vijay Abraham I
2013-07-23 16:40                                                     ` Kishon Vijay Abraham I
2013-07-23 16:28                                                     ` Kishon Vijay Abraham I
2013-07-23 16:28                                                     ` Kishon Vijay Abraham I
2013-07-23 16:35                                                     ` Greg KH
2013-07-23 16:35                                                       ` Greg KH
2013-07-23 16:35                                                       ` Greg KH
2013-07-23 16:35                                                       ` Greg KH
2013-07-23 16:50                                                   ` Tomasz Figa
2013-07-23 16:50                                                     ` Tomasz Figa
2013-07-23 16:50                                                     ` Tomasz Figa
2013-07-23 16:50                                                     ` Tomasz Figa
2013-07-23 17:37                                                     ` Greg KH
2013-07-23 17:37                                                       ` Greg KH
2013-07-23 17:37                                                       ` Greg KH
2013-07-23 17:37                                                       ` Greg KH
2013-07-23 17:44                                                       ` Mark Brown
2013-07-23 17:44                                                         ` Mark Brown
2013-07-23 17:44                                                         ` Mark Brown
2013-07-23 17:44                                                         ` Mark Brown
2013-07-23 18:01                                                         ` Greg KH
2013-07-23 18:01                                                           ` Greg KH
2013-07-23 18:01                                                           ` Greg KH
2013-07-23 18:01                                                           ` Greg KH
2013-07-23 19:31                                                           ` Mark Brown
2013-07-23 19:31                                                             ` Mark Brown
2013-07-23 19:31                                                             ` Mark Brown
2013-07-23 19:31                                                             ` Mark Brown
2013-07-23 19:44                                                             ` Greg KH
2013-07-23 19:44                                                               ` Greg KH
2013-07-23 19:44                                                               ` Greg KH
2013-07-23 19:44                                                               ` Greg KH
2013-07-23 20:07                                                               ` Tomasz Figa
2013-07-23 20:07                                                                 ` Tomasz Figa
2013-07-23 20:07                                                                 ` Tomasz Figa
2013-07-23 20:07                                                                 ` Tomasz Figa
2013-07-23 20:50                                                                 ` Greg KH
2013-07-23 20:50                                                                   ` Greg KH
2013-07-23 20:50                                                                   ` Greg KH
2013-07-23 20:50                                                                   ` Greg KH
2013-07-23 21:05                                                                   ` Tomasz Figa
2013-07-23 21:05                                                                     ` Tomasz Figa
2013-07-23 21:05                                                                     ` Tomasz Figa
2013-07-23 21:05                                                                     ` Tomasz Figa
2013-07-23 21:23                                                                     ` Greg KH
2013-07-23 21:23                                                                       ` Greg KH
2013-07-23 21:23                                                                       ` Greg KH
2013-07-23 21:23                                                                       ` Greg KH
2013-07-23 23:48                                                               ` Mark Brown
2013-07-23 23:48                                                                 ` Mark Brown
2013-07-23 23:48                                                                 ` Mark Brown
2013-07-23 23:48                                                                 ` Mark Brown
2013-07-23 17:48                                                       ` Tomasz Figa
2013-07-23 17:48                                                         ` Tomasz Figa
2013-07-23 17:48                                                         ` Tomasz Figa
2013-07-23 17:48                                                         ` Tomasz Figa
2013-07-23 18:04                                                         ` Greg KH
2013-07-23 18:04                                                           ` Greg KH
2013-07-23 18:04                                                           ` Greg KH
2013-07-23 18:04                                                           ` Greg KH
2013-07-23 20:46                                                           ` Tomasz Figa
2013-07-23 20:46                                                             ` Tomasz Figa
2013-07-23 20:46                                                             ` Tomasz Figa
2013-07-23 20:46                                                             ` Tomasz Figa
2013-07-23 19:36                                                     ` Alan Stern
2013-07-23 19:36                                                       ` Alan Stern
2013-07-23 19:36                                                       ` Alan Stern
2013-07-23 19:36                                                       ` Alan Stern
2013-07-23 20:20                                                       ` Tomasz Figa
2013-07-23 20:20                                                         ` Tomasz Figa
2013-07-23 20:20                                                         ` Tomasz Figa
2013-07-23 20:20                                                         ` Tomasz Figa
2013-07-23 20:53                                                         ` Alan Stern
2013-07-23 20:53                                                           ` Alan Stern
2013-07-23 20:53                                                           ` Alan Stern
2013-07-23 20:53                                                           ` Alan Stern
2013-07-23 21:02                                                           ` Tomasz Figa
2013-07-23 21:02                                                             ` Tomasz Figa
2013-07-23 21:02                                                             ` Tomasz Figa
2013-07-23 21:02                                                             ` Tomasz Figa
2013-07-23 21:14                                                             ` Alan Stern
2013-07-23 21:14                                                               ` Alan Stern
2013-07-23 21:14                                                               ` Alan Stern
2013-07-23 21:14                                                               ` Alan Stern
2013-07-23 21:31                                                               ` Tomasz Figa
2013-07-23 21:31                                                                 ` Tomasz Figa
2013-07-23 21:31                                                                 ` Tomasz Figa
2013-07-23 21:31                                                                 ` Tomasz Figa
2013-07-24 18:32                                                                 ` Arnd Bergmann
2013-07-24 18:32                                                                   ` Arnd Bergmann
2013-07-24 18:32                                                                   ` Arnd Bergmann
2013-07-24 18:32                                                                   ` Arnd Bergmann
2013-07-25  5:11                                                                   ` Kishon Vijay Abraham I
2013-07-25  5:11                                                                     ` Kishon Vijay Abraham I
2013-07-25  7:54                                                                     ` Arnd Bergmann
2013-07-25  7:54                                                                       ` Arnd Bergmann
2013-07-25  7:54                                                                       ` Arnd Bergmann
2013-07-25  7:54                                                                       ` Arnd Bergmann
2013-07-25  9:29                                                                   ` Sylwester Nawrocki
2013-07-25  9:29                                                                     ` Sylwester Nawrocki
2013-07-25  9:29                                                                     ` Sylwester Nawrocki
2013-07-25  9:29                                                                     ` Sylwester Nawrocki
2013-07-25  9:29                                                                   ` Mark Brown
2013-07-25  9:29                                                                     ` Mark Brown
2013-07-25  9:29                                                                     ` Mark Brown
2013-07-25  9:29                                                                     ` Mark Brown
2013-07-25 10:11                                                                     ` Kishon Vijay Abraham I
2013-07-25 10:11                                                                       ` Kishon Vijay Abraham I
2013-07-25 10:16                                                                   ` Laurent Pinchart
2013-07-25 10:16                                                                     ` Laurent Pinchart
2013-07-25 10:16                                                                     ` Laurent Pinchart
2013-07-25 10:16                                                                     ` Laurent Pinchart
2013-07-25 10:26                                                                     ` Florian Fainelli
2013-07-25 10:26                                                                       ` Florian Fainelli
2013-07-25 11:00                                                                     ` Arnd Bergmann
2013-07-25 11:00                                                                       ` Arnd Bergmann
2013-07-25 11:00                                                                       ` Arnd Bergmann
2013-07-25 11:00                                                                       ` Arnd Bergmann
2013-07-25 11:10                                                                       ` Laurent Pinchart
2013-07-25 11:10                                                                         ` Laurent Pinchart
2013-07-25 11:10                                                                         ` Laurent Pinchart
2013-07-25 11:10                                                                         ` Laurent Pinchart
2013-07-25 12:09                                                                       ` Mark Brown
2013-07-25 12:09                                                                         ` Mark Brown
2013-07-25 12:09                                                                         ` Mark Brown
2013-07-25 12:09                                                                         ` Mark Brown
2013-07-23 17:34                                               ` Mark Brown
2013-07-23 17:34                                                 ` Mark Brown
2013-07-23 17:34                                                 ` Mark Brown
2013-07-23 17:34                                                 ` Mark Brown
2013-07-22 15:04                                       ` Greg KH
2013-07-22 15:04                                         ` Greg KH
2013-07-22 15:04                                         ` Greg KH
2013-07-23  5:34                                         ` Kishon Vijay Abraham I
2013-07-23  5:46                                           ` Kishon Vijay Abraham I
2013-07-23  5:34                                           ` Kishon Vijay Abraham I
2013-07-23  5:34                                           ` Kishon Vijay Abraham I
2013-07-21 10:31                             ` Tomasz Figa [this message]
2013-07-21 10:31                               ` Tomasz Figa
2013-07-21 10:31                               ` Tomasz Figa
2013-07-21 11:07                               ` Kishon Vijay Abraham I
2013-07-21 11:19                                 ` Kishon Vijay Abraham I
2013-07-21 11:07                                 ` Kishon Vijay Abraham I
2013-07-21 11:07                                 ` Kishon Vijay Abraham I
2013-07-21 11:12                                 ` Tomasz Figa
2013-07-21 11:12                                   ` Tomasz Figa
2013-07-21 11:12                                   ` Tomasz Figa
2013-07-21 15:46                                   ` Greg KH
2013-07-21 15:46                                     ` Greg KH
2013-07-21 15:46                                     ` Greg KH
2013-07-21 15:46                                     ` Greg KH
2013-07-30  7:11                                     ` Felipe Balbi
2013-07-30  7:11                                       ` Felipe Balbi
2013-07-30  7:11                                       ` Felipe Balbi
2013-07-30  7:11                                       ` Felipe Balbi
2013-07-31  5:44                                       ` Kishon Vijay Abraham I
2013-07-31  5:56                                         ` Kishon Vijay Abraham I
2013-07-31  5:44                                         ` Kishon Vijay Abraham I
2013-07-31  5:44                                         ` Kishon Vijay Abraham I
2013-07-31  6:15                                         ` Felipe Balbi
2013-07-31  6:15                                           ` Felipe Balbi
2013-07-31  6:15                                           ` Felipe Balbi
2013-07-31  6:15                                           ` Felipe Balbi
2013-08-13 10:44                                           ` Kishon Vijay Abraham I
2013-08-13 10:56                                             ` Kishon Vijay Abraham I
2013-08-13 10:44                                             ` Kishon Vijay Abraham I
2013-08-13 10:44                                             ` Kishon Vijay Abraham I
2013-08-13 11:37                                             ` Tomasz Figa
2013-08-13 11:37                                               ` Tomasz Figa
2013-08-13 11:37                                               ` Tomasz Figa
2013-08-13 12:05                                               ` Kishon Vijay Abraham I
2013-08-13 12:17                                                 ` Kishon Vijay Abraham I
2013-08-13 12:05                                                 ` Kishon Vijay Abraham I
2013-08-13 12:05                                                 ` Kishon Vijay Abraham I
2013-08-13 22:19                                                 ` Sylwester Nawrocki
2013-08-13 22:19                                                   ` Sylwester Nawrocki
2013-08-13 22:19                                                   ` Sylwester Nawrocki
2013-08-13 23:04                                                   ` Tomasz Figa
2013-08-13 23:04                                                     ` Tomasz Figa
2013-08-13 23:04                                                     ` Tomasz Figa
2013-08-14 15:05                                                     ` Kishon Vijay Abraham I
2013-08-14 15:17                                                       ` Kishon Vijay Abraham I
2013-08-14 15:05                                                       ` Kishon Vijay Abraham I
2013-08-14 15:05                                                       ` Kishon Vijay Abraham I
2013-08-19  5:28                                                       ` Kishon Vijay Abraham I
2013-08-19  5:40                                                         ` Kishon Vijay Abraham I
2013-08-19  5:28                                                         ` Kishon Vijay Abraham I
2013-08-19  5:28                                                         ` Kishon Vijay Abraham I
2013-08-20 12:26                                                         ` Felipe Balbi
2013-08-20 12:26                                                           ` Felipe Balbi
2013-08-20 12:26                                                           ` Felipe Balbi
2013-08-20 12:26                                                           ` Felipe Balbi
2013-07-18  6:46 ` [PATCH 02/15] usb: phy: omap-usb2: use the new " Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  7:21   ` Greg KH
2013-07-18  7:21     ` Greg KH
2013-07-18  7:21     ` Greg KH
2013-07-18  9:00     ` Kishon Vijay Abraham I
2013-07-18  9:12       ` Kishon Vijay Abraham I
2013-07-18  9:00       ` Kishon Vijay Abraham I
2013-07-18  9:00       ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 03/15] usb: phy: twl4030: " Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 04/15] ARM: OMAP: USB: Add phy binding information Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  7:02   ` Tony Lindgren
2013-07-18  7:02     ` Tony Lindgren
2013-07-18  7:02     ` Tony Lindgren
2013-07-18  7:02     ` Tony Lindgren
2013-07-18  6:46 ` [PATCH 05/15] ARM: dts: omap: update usb_otg_hs data Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  7:05   ` Tony Lindgren
2013-07-18  7:05     ` Tony Lindgren
2013-07-18  7:05     ` Tony Lindgren
2013-07-18  6:46 ` [PATCH 06/15] usb: musb: omap2430: use the new generic PHY framework Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 07/15] usb: phy: omap-usb2: remove *set_suspend* callback from omap-usb2 Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 08/15] usb: phy: twl4030-usb: remove *set_suspend* and *phy_init* ops Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 09/15] phy: Add driver for Exynos MIPI CSIS/DSIM DPHYs Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 10/15] video: exynos_mipi_dsim: Use the generic PHY driver Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 11/15] exynos4-is: Use the generic MIPI CSIS " Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 12/15] ARM: Samsung: Remove the MIPI PHY setup code Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 13/15] phy: Add driver for Exynos DP PHY Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 14/15] video: exynos_dp: remove non-DT support for Exynos Display Port Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46 ` [PATCH 15/15] video: exynos_dp: Use the generic PHY driver Kishon Vijay Abraham I
2013-07-18  6:58   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I
2013-07-18  6:46   ` Kishon Vijay Abraham I

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3839600.WiC1OLF35o@flatron \
    --to=tomasz.figa@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=arnd@arndb.de \
    --cc=balajitk@ti.com \
    --cc=balbi@ti.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=george.cherian@ti.com \
    --cc=grant.likely@linaro.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jg1.han@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=kishon@ti.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=nsekhar@ti.com \
    --cc=s.nawrocki@samsung.com \
    --cc=stern@rowland.harvard.edu \
    --cc=swarren@nvidia.com \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.