lkml.org 
[lkml]   [2013]   [Dec]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 02/10] net: stmmac: Honor DT parameter to force DMA store and forward mode
    Resending reply due to incorrect MIME type.

    On Sat, Dec 7, 2013 at 5:26 AM, David Miller <davem@davemloft.net> wrote:
    >
    > From: Chen-Yu Tsai <wens@csie.org>
    > Date: Sat, 7 Dec 2013 01:29:35 +0800
    >
    > > @@ -47,6 +47,7 @@ static int stmmac_probe_config_dt(struct platform_device *pdev,
    > > plat->bus_id = 0;
    > >
    > > of_property_read_u32(np, "snps,phy-addr", &plat->phy_addr);
    > > + plat->force_sf_dma_mode = of_property_read_bool(np, "snps,force_sf_dma_mode");
    >
    > Will this do the right thing for when the property is not present?
    > Right now the force_sf_dma_mode value is always false.

    of_property_read_bool will return false when the property is not present.

    > In fact won't it override the explicit settings done elsewhere in the
    > driver?

    Point taken. The current implementation will override settings passed from
    platform data. ORing the two would be better.


    Thanks


    \
     
     \ /
      Last update: 2013-12-07 02:41    [W:4.039 / U:0.552 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site