[PATCH 3/6] mv643xx.c: Add basic device tree support.
Andrew Lunn
andrew at lunn.ch
Tue Jul 31 04:27:42 EDT 2012
On Tue, Jul 31, 2012 at 09:19:52AM +0100, Ian Molton wrote:
> On 31/07/12 08:14, Arnaud Patard (Rtp) wrote:
> > Ian Molton <ian.molton at codethink.co.uk> writes:
> > Hi,
> >
> > [...]
> >
> >> @@ -2873,7 +2913,31 @@ static int mv643xx_eth_probe(struct platform_device *pdev)
> >> struct resource *res;
> >> int err;
> >>
> >> - pd = pdev->dev.platform_data;
> >> + if (pdev->dev.of_node) {
> >> + struct device_node *np = NULL;
> >> +
> >> + /* when all users of this driver use FDT, we can remove this */
> >> + pd = kzalloc(sizeof(*pd), GFP_ATOMIC);
> >> + if (!pd) {
> >> + dev_dbg(&pdev->dev, "Could not allocate platform data\n");
> >> + return -ENOMEM;
> >> + }
> >> +
> >> + of_property_read_u32(pdev->dev.of_node,
> >> + "port_number", &pd->port_number);
> >> + of_property_read_u32(pdev->dev.of_node,
> >> + "phy_addr", &pd->phy_addr);
> > I guess we need something for tx_csum_limit in the device tree too. It's
> > important for kirkwood and dove.
>
> It was my intention to keep the patch as non-invasive as possible - none
> of the platforms using D-T at present require this.
>
> Furthermore, git grep shows nothing in mach-kirkwood or mach-dove using
> at all...
>
> Are you sure this is required? It can always be added if anything
> actually uses it :)
Hi Ian
Try using jumbo packets on kirkwood.
http://comments.gmane.org/gmane.linux.ports.arm.kernel/178785
http://www.spinics.net/lists/arm-kernel/msg186152.html
Andrew
More information about the linux-arm-kernel
mailing list