From: "Michael ." <email@example.com> To: Bjorn Helgaas <firstname.lastname@example.org> Cc: Dominik Brodowski <email@example.com>, firstname.lastname@example.org, email@example.com, Trevor Jacobs <firstname.lastname@example.org>, Kris Cleveland <email@example.com>, Jeff <firstname.lastname@example.org>, Morgan Klym <email@example.com>, Ulf Hansson <firstname.lastname@example.org>, Philip Langdale <email@example.com>, Pierre Ossman <firstname.lastname@example.org>, Maxim Levitsky <email@example.com>, firstname.lastname@example.org Subject: Re: PCI device function not being enumerated [Was: PCMCIA not working on Panasonic Toughbook CF-29] Date: Sun, 23 Feb 2020 05:14:13 +1100 Message-ID: <CAFjuqNgOO89nVhju1R3m0q_P+y97vr+xo6--0yy34P63LFBYemail@example.com> (raw) In-Reply-To: <20200222165617.GA207731@google.com> Hi Bjorn, yes this is still unfixed. I'm sorry that I haven't been able to pursue this but the weather in Australia has been horrendous since October last year. Your proposals sound good but are way beyond my knowledge and skill level to implement. I, and my friends, are happy to help in any way we can. Cheers. Michael. P.S. I apologise for the double reply On 23/02/2020, Bjorn Helgaas <firstname.lastname@example.org> wrote: > On Tue, Oct 29, 2019 at 12:02:50PM -0500, Bjorn Helgaas wrote: >> [+cc Ulf, Philip, Pierre, Maxim, linux-mmc; see  for beginning of >> thread,  for problem report and the patch Michael tested] >> >> On Tue, Oct 29, 2019 at 07:58:27PM +1100, Michael . wrote: >> > Bjorn and Dominik. >> > I am happy to let you know the patch did the trick, it compiled well >> > on 5.4-rc4 and my friends in the CC list have tested the modified >> > kernel and confirmed that both slots are now working as they should. >> > As a group of dedicated Toughbook users and Linux users please accept >> > our thanks your efforts and assistance is greatly appreciated. >> > >> > Now that we know this patch works what kernel do you think it will be >> > released in? Will it make 5.4 or will it be put into 5.5 development >> > for further testing? >> >> That patch was not intended to be a fix; it was just to test my guess >> that the quirk might be related. >> >> Removing the quirk solved the problem *you're* seeing, but the quirk >> was added in the first place to solve some other problem, and if we >> simply remove the quirk, we may reintroduce the original problem. >> >> So we have to look at the history and figure out some way to solve >> both problems. I cc'd some people who might have insight. Here are >> some commits that look relevant: >> >> 5ae70296c85f ("mmc: Disabler for Ricoh MMC controller") >> 03cd8f7ebe0c ("ricoh_mmc: port from driver to pci quirk") >> >> >>  >> https://lore.kernel.org/r/CAFjuqNi+knSb9WVQOahCVFyxsiqoGgwoM7Z1aqDBebNzp_-jYw@mail.gmail.com/ >>  https://lore.kernel.org/r/20191021160952.GA229204@google.com/ > > I guess this problem is still unfixed? I hate the fact that we broke > something that used to work. > > Maybe we need some sort of DMI check in ricoh_mmc_fixup_rl5c476() so > we skip it for Toughbooks? Or maybe we limit the quirk to the > machines where it was originally needed? > > Bjorn >
next prev parent reply index Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <CAFjuqNh1=B7Ft6v7nzo3BW70EbAvK=Eko_4yqrJ4Z4N3w_Y+Xw@mail.gmail.com> [not found] ` <CAFjuqNjLJw8J0nU2oo8rDfDUBavHLC7D0=AAwM62tp6=kHHk-A@mail.gmail.com> [not found] ` <20191015064801.GA104469@owl.dominikbrodowski.net> [not found] ` <CAFjuqNgxAuf+JTkWqhimDspzPd0+s5yGJro=Zi164uoxu4CmOA@mail.gmail.com> [not found] ` <CANfzparZ17SMzE1qzzF=Rixu=aYpf1RiKqR4KXXS0S+u7Q3TwQ@mail.gmail.com> 2019-10-20 9:08 ` Dominik Brodowski 2019-10-21 16:09 ` Bjorn Helgaas 2019-10-21 18:17 ` Michael . 2019-10-21 18:47 ` Dominik Brodowski 2019-10-21 18:59 ` Michael . [not found] ` <CAFjuqNg8_G-B5Owz1NBxa-nw620hXwcn9WkE4sfETVR81MWatA@mail.gmail.com> [not found] ` <CAFjuqNiToDC9BSJVdahdkPz8Ejb-rM3VRkyoBNUTJr8goemail@example.com> [not found] ` <20191025075536.GA14531@owl.dominikbrodowski.net> 2019-10-25 18:34 ` Michael . 2019-10-29 8:58 ` Michael . 2019-10-29 17:02 ` Bjorn Helgaas 2020-02-22 16:56 ` Bjorn Helgaas 2020-02-22 18:14 ` Michael . [this message] 2020-02-25 15:03 ` Ulf Hansson 2020-02-25 19:15 ` Bjorn Helgaas 2020-02-25 23:46 ` bluerocksaddles 2020-02-26 1:13 ` Arvind Sankar 2020-02-26 1:50 ` Michael . 2020-02-26 3:12 ` Trevor Jacobs 2020-02-26 4:51 ` Arvind Sankar 2020-02-26 5:20 ` Philip Langdale 2020-02-26 6:10 ` Michael . 2020-02-26 19:46 ` bluerocksaddles 2020-07-28 1:50 ` Michael . 2020-08-02 5:58 ` Michael .
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAFjuqNgOO89nVhju1R3m0q_P+y97vr+xo6--0yy34P63LFBYfirstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Linux-PCI Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-pci/0 linux-pci/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-pci linux-pci/ https://lore.kernel.org/linux-pci \ email@example.com public-inbox-index linux-pci Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.linux-pci AGPL code for this site: git clone https://public-inbox.org/public-inbox.git