lkml.org 
[lkml]   [2012]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    SubjectRe: [ 35/83] ARM: imx: ehci: fix host power mask bit
    Date
    This patch is definitely needed -- it is obviously setting the wrong bit --  
    but I am still doing testing to see what other complications may arise from
    actually setting the RIGHT bit (i.e. I've seen some odd results so far and
    suspect some other code may have been changed historically to account for
    the wrong behaviour this bit was causing).

    We're currently on vacation for Thanksgiving over here in the USA and I
    won't be back in the office til next week.

    If you would, hold off on adding the patch to the stable kernel until I have
    a bit more time to check out a couple of things.

    Thanks,
    Michael Burkey

    ----- Original Message -----
    From: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
    To: <linux-kernel@vger.kernel.org>; <stable@vger.kernel.org>
    Cc: "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>;
    <alan@lxorguk.ukuu.org.uk>; "Michael Burkey" <mdburkey@gmail.com>;
    "Christoph Fritz" <chf.fritz@googlemail.com>; "Eric Bénard"
    <eric@eukrea.com>; "Sascha Hauer" <s.hauer@pengutronix.de>
    Sent: Wednesday, November 21, 2012 7:41 PM
    Subject: [ 35/83] ARM: imx: ehci: fix host power mask bit


    > 3.6-stable review patch. If anyone has any objections, please let me
    > know.
    >
    > ------------------
    >
    > From: Christoph Fritz <chf.fritz@googlemail.com>
    >
    > commit 3d5e2abe6e265acc5e1fda810301243e9bac92b2 upstream.
    >
    > This patch sets HPM (Host power mask bit) to bit 16 according to i.MX
    > Reference Manual. Falsely it was set to bit 8, but this controls pull-up
    > Impedance.
    >
    > Reported-by: Michael Burkey <mdburkey@gmail.com>
    > Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com>
    > Acked-by: Eric Bénard <eric@eukrea.com>
    > Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
    > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    >
    > ---
    > arch/arm/mach-imx/ehci-imx25.c | 2 +-
    > arch/arm/mach-imx/ehci-imx35.c | 2 +-
    > 2 files changed, 2 insertions(+), 2 deletions(-)
    >
    > --- a/arch/arm/mach-imx/ehci-imx25.c
    > +++ b/arch/arm/mach-imx/ehci-imx25.c
    > @@ -30,7 +30,7 @@
    > #define MX25_H1_SIC_SHIFT 21
    > #define MX25_H1_SIC_MASK (0x3 << MX25_H1_SIC_SHIFT)
    > #define MX25_H1_PP_BIT (1 << 18)
    > -#define MX25_H1_PM_BIT (1 << 8)
    > +#define MX25_H1_PM_BIT (1 << 16)
    > #define MX25_H1_IPPUE_UP_BIT (1 << 7)
    > #define MX25_H1_IPPUE_DOWN_BIT (1 << 6)
    > #define MX25_H1_TLL_BIT (1 << 5)
    > --- a/arch/arm/mach-imx/ehci-imx35.c
    > +++ b/arch/arm/mach-imx/ehci-imx35.c
    > @@ -30,7 +30,7 @@
    > #define MX35_H1_SIC_SHIFT 21
    > #define MX35_H1_SIC_MASK (0x3 << MX35_H1_SIC_SHIFT)
    > #define MX35_H1_PP_BIT (1 << 18)
    > -#define MX35_H1_PM_BIT (1 << 8)
    > +#define MX35_H1_PM_BIT (1 << 16)
    > #define MX35_H1_IPPUE_UP_BIT (1 << 7)
    > #define MX35_H1_IPPUE_DOWN_BIT (1 << 6)
    > #define MX35_H1_TLL_BIT (1 << 5)
    >
    >

    --
    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: 2012-11-25 00:21    [W:2.106 / U:0.088 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site