newbie: phy platform interrupt config

2 messages, 2 authors, 2012-02-09 · open the first message on its own page

newbie: phy platform interrupt config

From: Tim Sander <hidden>
Date: 2012-01-24 14:43:18

Hi

I would like to set the interrupt for the phy on the i.mx35 pcm43 in the file 
arch/arm/mach-imx/mach-pcm043.c. Unfortunatly i didn't find a way to specify 
it. So i want to set the field (struct phy_device*)->irq.

What i managed so far is to request the gpio for the interrupt:
#define IMX_PHY_IRQ IMX_GPIO_NR(2,7)
gpio_request(IMX_PHY_IRQ, "FEC PHY irq");
gpio_direction_input(IMX_PHY_IRQ);

Now i guess the result of "gpio_to_irq(IMX_PHY_IRQ)" should go into some kind 
of argument to imx35_add_fec(pdata) which is a macro 
imx_add_fec(&imx35_fec_data, pdata).
pdata is of type (?) struct fec_platform_data which contains phy_interface_t 
and unsigned char array. So i don't see a way to set the phy interrupt there.

I tried search for a driver which set the phy irq but i didn't find any and 
setting the phy in the phy_fixup function is also not working since the 
interrupt seems not get registered when opening the device.

So how do i set the phy interrupt line in the platform file?

Best regards
Tim

PS:
The cc list is from scripts/get_maintainer.pl -f drivers/net/phy/ which are 
hopefully the right maintainers plus Andy Fleming which commited the 
documentation for libphy.

Re: newbie: phy platform interrupt config

From: Andy Fleming <hidden>
Date: 2012-02-09 06:29:23

On Tue, Jan 24, 2012 at 8:37 AM, Tim Sander [off-list ref] wrote:
Hi

I would like to set the interrupt for the phy on the i.mx35 pcm43 in the file
arch/arm/mach-imx/mach-pcm043.c. Unfortunatly i didn't find a way to specify
it. So i want to set the field (struct phy_device*)->irq.

What i managed so far is to request the gpio for the interrupt:
#define IMX_PHY_IRQ IMX_GPIO_NR(2,7)
gpio_request(IMX_PHY_IRQ, "FEC PHY irq");
gpio_direction_input(IMX_PHY_IRQ);

Now i guess the result of "gpio_to_irq(IMX_PHY_IRQ)" should go into some kind
of argument to imx35_add_fec(pdata) which is a macro
imx_add_fec(&imx35_fec_data, pdata).
pdata is of type (?) struct fec_platform_data which contains phy_interface_t
and unsigned char array. So i don't see a way to set the phy interrupt there.

I tried search for a driver which set the phy irq but i didn't find any and
setting the phy in the phy_fixup function is also not working since the
interrupt seems not get registered when opening the device.

So how do i set the phy interrupt line in the platform file?
Look in drivers/of/of_mdio.c.

It's the job of the mdio driver to set the IRQs for the device. I
don't honestly know why I designed it that way. I suspect it's because
the MDIO controller's driver is the entity most likely to communicate
directly with the platform code.

So wherever you set up your mdio bus, set the IRQs for all the PHYs on
the bus there.

Andy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help