lkml.org 
[lkml]   [2007]   [Jun]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] PHY fixed driver: rework release path and updatephy_id notation
    Date
    From
    On Sat, 2007-06-09 at 20:21 +0400, Vitaly Bordug wrote:
    > device_bind_driver() error code returning has been fixed.
    > release() function has been written, so that to free resources
    > in correct way; the release path is now clean.
    >
    > Before the rework, it used to cause
    > Device 'fixed@100:1' does not have a release() function, it is broken
    > and must be fixed.
    > BUG: at drivers/base/core.c:104 device_release()
    >
    > Call Trace:
    > [<ffffffff802ec380>] kobject_cleanup+0x53/0x7e
    > [<ffffffff802ec3ab>] kobject_release+0x0/0x9
    > [<ffffffff802ecf3f>] kref_put+0x74/0x81
    > [<ffffffff8035493b>] fixed_mdio_register_device+0x230/0x265
    > [<ffffffff80564d31>] fixed_init+0x1f/0x35
    > [<ffffffff802071a4>] init+0x147/0x2fb
    > [<ffffffff80223b6e>] schedule_tail+0x36/0x92
    > [<ffffffff8020a678>] child_rip+0xa/0x12
    > [<ffffffff80311714>] acpi_ds_init_one_object+0x0/0x83
    > [<ffffffff8020705d>] init+0x0/0x2fb
    > [<ffffffff8020a66e>] child_rip+0x0/0x12
    >
    >
    > Also changed the notation of the fixed phy definition on
    > mdio bus to the form of <speed>+<duplex> to make it able to be used by
    > gianfar and ucc_geth that define phy_id strictly as "%d:%d"
    >
    > Signed-off-by: Vitaly Bordug <vitb@kernel.crashing.org>

    Hi Vitaly

    I am about to test this but I am having trouble figuring out how to
    make ucc_geth find this PHY. Need some hints/template what to write
    in the dts or board init code.

    Also, I need several fixed PHYs, maybe you should export
    fixed_mdio_register_device() ?

    Jocke


    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-06-13 18:25    [W:0.023 / U:29.736 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site