linux-kernel - Re: IDE DVD and CD drive seem to not work with Linux desktop 3.3.0-030300-generic

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAMfrEMbuGY5Gkts_YO1weeij6s-7i87Dx2Oy8KUcKR9zS0cLdA@mail.gmail.com>
Date:	Fri, 23 Mar 2012 17:33:14 -0500
From:	Chris Holland <bandidoirlandes@...il.com>
To:	Matthew Garrett <mjg59@...f.ucam.org>
Cc:	Robert Hancock <hancockrwd@...il.com>,
	linux-kernel@...r.kernel.org, jbarnes@...tuousgeek.org
Subject: Re: IDE DVD and CD drive seem to not work with Linux desktop 3.3.0-030300-generic

On Fri, Mar 23, 2012 at 3:12 PM, Matthew Garrett <mjg59@...f.ucam.org> wrote:
> On Fri, Mar 23, 2012 at 03:10:14PM -0500, Chris Holland wrote:
>
>> I think I had the commit 4949be16822e92a18ea0cc1616319926628092ee
>> removed when I applied the patch.
>>
>> I did a
>> git reset --hard v3.3
>> applied the patch and am rebuilding.
>
> Thanks - let me know if it helps!
>
> --
> Matthew Garrett | mjg59@...f.ucam.org

That did not work for me after I build I still dont get the cd and dvd in dmesg

And cat /var/log/dmesg | egrep '(CD|DVD)'
[    0.000000] Linux version 3.3.0+ (root@...ktop) (gcc version 4.6.1
(Ubuntu/Linaro 4.6.1-9ubuntu3) ) #10 SMP Fri Mar 23 16:09:17 CDT 2012
[    0.000000]   CD000-EFFFF uncachable

What I did

git reset --hard v3.3
git apply patch1
make oldconfig
fakeroot make deb-pkg
install reboot

git diff
diff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c
index 24f049e..04eb680 100644
--- a/drivers/pci/pcie/aspm.c
+++ b/drivers/pci/pcie/aspm.c
@@ -185,6 +185,7 @@ static void
pcie_aspm_configure_common_clock(struct pcie_link_state *link)
        int ppos, cpos, same_clock = 1;
        u16 reg16, parent_reg, child_reg[8];
        unsigned long start_jiffies;
+       bool retrain = false;
        struct pci_dev *child, *parent = link->pdev;
        struct pci_bus *linkbus = parent->subordinate;
        /*
@@ -215,7 +216,10 @@ static void
pcie_aspm_configure_common_clock(struct pcie_link_state *link)
                        reg16 |= PCI_EXP_LNKCTL_CCC;
                else
                        reg16 &= ~PCI_EXP_LNKCTL_CCC;
-               pci_write_config_word(child, cpos + PCI_EXP_LNKCTL, reg16);
+               if (reg16 != child_reg[PCI_FUNC(child->devfn)]) {
+                       retrain = true;
+                       pci_write_config_word(child, cpos +
PCI_EXP_LNKCTL, reg16);
+               }
        }

        /* Configure upstream component */
@@ -225,9 +229,15 @@ static void
pcie_aspm_configure_common_clock(struct pcie_link_state *link)
                reg16 |= PCI_EXP_LNKCTL_CCC;
        else
                reg16 &= ~PCI_EXP_LNKCTL_CCC;
-       pci_write_config_word(parent, ppos + PCI_EXP_LNKCTL, reg16);
+       if (reg16 != parent_reg) {
+               retrain = true;
+               pci_write_config_word(parent, ppos + PCI_EXP_LNKCTL, reg16);
+       }

        /* Retrain link */
+       if (!retrain)
+               return;
+
        reg16 |= PCI_EXP_LNKCTL_RL;
        pci_write_config_word(parent, ppos + PCI_EXP_LNKCTL, reg16);

@@ -436,13 +446,16 @@ static void pcie_aspm_cap_init(struct
pcie_link_state *link, int blacklist)

 static void pcie_config_aspm_dev(struct pci_dev *pdev, u32 val)
 {
-       u16 reg16;
+       u16 reg16, orig_reg16;
        int pos = pci_pcie_cap(pdev);

        pci_read_config_word(pdev, pos + PCI_EXP_LNKCTL, &reg16);
+       orig_reg16 = reg16;
        reg16 &= ~0x3;
        reg16 |= val;
-       pci_write_config_word(pdev, pos + PCI_EXP_LNKCTL, reg16);
+
+       if (reg16 != orig_reg16)
+               pci_write_config_word(pdev, pos + PCI_EXP_LNKCTL, reg16);
 }

 static void pcie_config_aspm_link(struct pcie_link_state *link, u32 state)


git status
# On branch master
# Your branch is behind 'origin/master' by 3856 commits, and can be
fast-forwarded.
#
# Changes not staged for commit:
#   (use "git add <file>..." to update what will be committed)
#   (use "git checkout -- <file>..." to discard changes in working directory)
#
#	modified:   drivers/pci/pcie/aspm.c
#
# Untracked files:
#   (use "git add <file>..." to include in what will be committed)
#
#	patch1
no changes added to commit (use "git add" and/or "git commit -a")
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/