Re: [PATCH] custom PM support for 8250

From: Grigory Tolstolytkin (gtolstolytkin_at_dev.rtsoft.ru)
Date: 09/05/05

  • Next message: Mathieu: "Re: Brand-new notebook useless with Linux..."
    Date:	Mon, 05 Sep 2005 13:59:38 +0400
    To: Russell King <rmk+lkml@arm.linux.org.uk>
    
    

    Hi Russel,

    I tried the patch provided by Vitaly Wool. And it works correctly. And
    now I'm successful with the PM support for my own serial8250 driver. Are
    you planning to commit the Vitlaly's changes into the mainstream? I
    guess it'll be helpful for the other people too. What I want is to know
    whether this changes will be supported by the Community or not. It's
    important for the project I'm worin on, cause I'm planning to push it
    into Open Source ;)

    Thanks,
    Grigory.

    Russell King wrote:

    >On Wed, Aug 31, 2005 at 03:10:09PM +0400, Vitaly Wool wrote:
    >
    >
    >>please find the patch that allows passing the pointer to custom power
    >>management routine (via platform_device) to 8250 serial driver.
    >>Please note that the interface to the outer world (i. e. exported
    >>functions) remained the same.
    >>
    >>
    >
    >I'd rather change the structure passed via the platform device to
    >something like:
    >
    >struct platform_serial_data {
    > void (*pm)(struct uart_port *port, unsigned int state, unsigned int old);
    > int nr_ports;
    > struct plat_serial8250_port *ports;
    >};
    >
    >which also eliminates the empty plat_serial8250_port terminator from
    >all the serial8250 platform devices (which appears to have caused some
    >folk problems.)
    >
    >It does mean that a set of 8250 ports (grouped by each platform device)
    >have a common power management method - which seems a logical restriction.
    >
    >
    >

    -
    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/


  • Next message: Mathieu: "Re: Brand-new notebook useless with Linux..."

    Relevant Pages

    • Re: [PATCH] custom PM support for 8250
      ... in this radical change:) ... and work it out to the new patch. ... >have a common power management method - which seems a logical restriction. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: 2.6.0-test 2 & matroxfb or orinoco wifi card
      ... Hmmm, I found the patch, but that didn't help. ... > support and it is probably ... as my boss prefers me to be awake at work. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [PATCH][2.4 Backport] x445 usb legacy fix
      ... The solution is to disable the BIOS's USB Legacy support early in boot ... Unlike the 2.6 version of the patch, this one also includes a ported ... extern int cyclone_setup; ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] intrinsic automount and mountpoint degradation support
      ... It also adds support for such mountpoints to be ... Apart from documenting the major functions and data structures, ... documentation makes the patch review process much more fruitful. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [linux-usb-devel] 2.6: USB disk unusable level of data corruption
      ... > support for that is. ... But I see Greg's checked in a big patch ... > they fixed its jumpers. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)