From: Ulf Hansson <ulf.hansson@linaro.org>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: "Mark Rutland" <mark.rutland@arm.com>,
"Kefeng Wang" <wangkefeng.wang@huawei.com>,
"Petr Mladek" <pmladek@suse.com>,
"Tony Lindgren" <tony@atomide.com>,
"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
"Bjorn Helgaas" <bhelgaas@google.com>,
netdev <netdev@vger.kernel.org>,
"Discussions about the Letux Kernel"
<letux-kernel@openphoenux.org>,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
"Russell King" <linux@armlinux.org.uk>,
"Mike Rapoport" <rppt@linux.ibm.com>,
"Yangtao Li" <tiny.windzz@gmail.com>,
"Kalle Valo" <kvalo@codeaurora.org>,
DTML <devicetree@vger.kernel.org>,
"Sakari Ailus" <sakari.ailus@linux.intel.com>,
"Alexios Zavras" <alexios.zavras@intel.com>,
"Rob Herring" <robh+dt@kernel.org>,
"John Stultz" <john.stultz@linaro.org>,
"David Sterba" <dsterba@suse.com>,
"Thomas Gleixner" <tglx@linutronix.de>,
linux-omap <linux-omap@vger.kernel.org>,
"Allison Randal" <allison@lohutok.net>,
"Linux ARM" <linux-arm-kernel@lists.infradead.org>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
linux-wireless <linux-wireless@vger.kernel.org>,
"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
"Benoît Cousson" <bcousson@baylibre.com>,
kernel@pyra-handheld.com
Subject: Re: [PATCH v3 00/12] OpenPandora: make wl1251 connected to mmc3 sdio port of OpenPandora work again
Date: Fri, 15 Nov 2019 10:29:46 +0100 [thread overview]
Message-ID: <CAPDyKFr=Uk1i0c=3WvuOYCQ__Skpr-9mjVM2Yqst-hd8zY6OeQ@mail.gmail.com> (raw)
In-Reply-To: <5F5A5FC0-8F91-4D5B-9EF6-AF36FE38B588@goldelico.com>
On Thu, 14 Nov 2019 at 16:16, H. Nikolaus Schaller <hns@goldelico.com> wrote:
>
> Hi Ulf,
>
> > Am 14.11.2019 um 15:18 schrieb Ulf Hansson <ulf.hansson@linaro.org>:
> >
> > On Thu, 7 Nov 2019 at 11:31, H. Nikolaus Schaller <hns@goldelico.com> wrote:
> >>
> >>
> >> * add a revisit note for special wl1251 handling code because it should
> >> be solved more generic in mmc core - suggested by Ulf Hansson <ulf.hansson@linaro.org>
> >> * remove init_card callback from platform_data/hsmmc-omap.h - suggested by Ulf Hansson <ulf.hansson@linaro.org>
> >> * remove obstructive always-on for vwlan regulator - suggested by Ulf Hansson <ulf.hansson@linaro.org>
> >> * rename DT node - suggested by Rob Herring <robh@kernel.org>
> >> * fix ARM: dts: subject prefix - suggested by Tony Lindgren <tony@atomide.com>
> >> * also remove omap2_hsmmc_info and obc-y line in Makefile - suggested by Tony Lindgren <tony@atomide.com>
> >
> > No further comments from my side. Let's just agree on how to deal with
> > the ti,power-gpio, then I can apply this.
>
> I'd say it can be a separate patch since it does not fix the Pandora
> issues, but is a new and independent optimization.
>
> And in case someone complains and uses it for some out-of tree purpose
> it can be discussed or even be reverted easier if it is a separate patch.
>
> I can do it in the next days.
Okay, that sounds reasonable.
In the meantime, I have queued up the series on my next branch (for v5.5).
I fixed up a couple of complaints from checkpatch, and also added
stable tags for the first two patches in the series, as that what
missing.
Kind regards
Uffe
>
> > Thanks a lot for fixing all this mess!
>
> I hope the users also appreciate our work.
>
> Best regards,
> Nikolaus
>
> >
> > Kind regards
> > Uffe
> >
> >>
> >> PATCH V2 2019-10-19 20:41:47:
> >> * added acked-by for wl1251 patches - Kalle Valo <kvalo@codeaurora.org>
> >> * really removed old pdata-quirks code (not through #if 0)
> >> * splited out a partial revert of
> >> efdfeb079cc3b ("regulator: fixed: Convert to use GPIO descriptor only")
> >> because that was introduced after v4.19 and stops the removal of
> >> the pdata-quirks patch from cleanly applying to v4.9, v4.14, v4.19
> >> - reported by Sasha Levin <sashal@kernel.org>
> >> * added a new patch to remove old omap hsmmc since pdata quirks
> >> were last user - suggested by Tony Lindgren <tony@atomide.com>
> >>
> >> PATCH V1 2019-10-18 22:25:39:
> >> Here we have a set of scattered patches to make the OpenPandora WiFi work again.
> >>
> >> v4.7 did break the pdata-quirks which made the mmc3 interface
> >> fail completely, because some code now assumes device tree
> >> based instantiation.
> >>
> >> Fixes: 81eef6ca9201 ("mmc: omap_hsmmc: Use dma_request_chan() for requesting DMA channel")
> >>
> >> v4.11 did break the sdio qirks for wl1251 which made the driver no longer
> >> load, although the device was found as an sdio client.
> >>
> >> Fixes: 884f38607897 ("mmc: core: move some sdio IDs out of quirks file")
> >>
> >> To solve these issues:
> >> * we convert mmc3 and wl1251 initialization from pdata-quirks
> >> to device tree
> >> * we make the wl1251 driver read properties from device tree
> >> * we fix the mmc core vendor ids and quirks
> >> * we fix the wl1251 (and wl1271) driver to use only vendor ids
> >> from header file instead of (potentially conflicting) local
> >> definitions
> >>
> >>
> >> H. Nikolaus Schaller (12):
> >> Documentation: dt: wireless: update wl1251 for sdio
> >> net: wireless: ti: wl1251 add device tree support
> >> ARM: dts: pandora-common: define wl1251 as child node of mmc3
> >> mmc: host: omap_hsmmc: add code for special init of wl1251 to get rid
> >> of pandora_wl1251_init_card
> >> omap: pdata-quirks: revert pandora specific gpiod additions
> >> omap: pdata-quirks: remove openpandora quirks for mmc3 and wl1251
> >> omap: remove omap2_hsmmc_info in old hsmmc.[ch] and update Makefile
> >> mmc: host: omap-hsmmc: remove init_card pdata callback from pdata
> >> mmc: sdio: fix wl1251 vendor id
> >> mmc: core: fix wl1251 sdio quirks
> >> net: wireless: ti: wl1251 use new SDIO_VENDOR_ID_TI_WL1251 definition
> >> net: wireless: ti: remove local VENDOR_ID and DEVICE_ID definitions
> >>
> >> .../bindings/net/wireless/ti,wl1251.txt | 26 +++
> >> arch/arm/boot/dts/omap3-pandora-common.dtsi | 36 +++-
> >> arch/arm/mach-omap2/Makefile | 3 -
> >> arch/arm/mach-omap2/common.h | 1 -
> >> arch/arm/mach-omap2/hsmmc.c | 171 ------------------
> >> arch/arm/mach-omap2/hsmmc.h | 32 ----
> >> arch/arm/mach-omap2/pdata-quirks.c | 105 -----------
> >> drivers/mmc/core/quirks.h | 7 +
> >> drivers/mmc/host/omap_hsmmc.c | 30 ++-
> >> drivers/net/wireless/ti/wl1251/sdio.c | 23 ++-
> >> drivers/net/wireless/ti/wlcore/sdio.c | 8 -
> >> include/linux/mmc/sdio_ids.h | 2 +
> >> include/linux/platform_data/hsmmc-omap.h | 3 -
> >> 13 files changed, 111 insertions(+), 336 deletions(-)
> >> delete mode 100644 arch/arm/mach-omap2/hsmmc.c
> >> delete mode 100644 arch/arm/mach-omap2/hsmmc.h
> >>
> >> --
> >> 2.23.0
> >>
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
prev parent reply other threads:[~2019-11-15 9:30 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-07 10:30 [PATCH v3 00/12] OpenPandora: make wl1251 connected to mmc3 sdio port of OpenPandora work again H. Nikolaus Schaller
2019-11-07 10:30 ` [PATCH v3 01/12] Documentation: dt: wireless: update wl1251 for sdio H. Nikolaus Schaller
2019-11-13 13:33 ` Rob Herring
2019-11-14 14:11 ` Ulf Hansson
2019-11-07 10:30 ` [PATCH v3 02/12] net: wireless: ti: wl1251 add device tree support H. Nikolaus Schaller
2019-11-14 14:12 ` Ulf Hansson
2019-11-07 10:30 ` [PATCH v3 03/12] ARM: dts: pandora-common: define wl1251 as child node of mmc3 H. Nikolaus Schaller
2019-11-08 17:10 ` Tony Lindgren
2019-11-07 10:30 ` [PATCH v3 04/12] mmc: host: omap_hsmmc: add code for special init of wl1251 to get rid of pandora_wl1251_init_card H. Nikolaus Schaller
2019-11-07 10:30 ` [PATCH v3 05/12] omap: pdata-quirks: revert pandora specific gpiod additions H. Nikolaus Schaller
2019-11-08 17:10 ` Tony Lindgren
2019-11-07 10:30 ` [PATCH v3 06/12] omap: pdata-quirks: remove openpandora quirks for mmc3 and wl1251 H. Nikolaus Schaller
2019-11-08 17:11 ` Tony Lindgren
2019-11-07 10:30 ` [PATCH v3 07/12] omap: remove omap2_hsmmc_info in old hsmmc.[ch] and update Makefile H. Nikolaus Schaller
2019-11-08 17:11 ` Tony Lindgren
2019-11-07 10:30 ` [PATCH v3 08/12] mmc: host: omap-hsmmc: remove init_card pdata callback from pdata H. Nikolaus Schaller
2019-11-07 10:30 ` [PATCH v3 09/12] mmc: sdio: fix wl1251 vendor id H. Nikolaus Schaller
2019-11-07 10:30 ` [PATCH v3 10/12] mmc: core: fix wl1251 sdio quirks H. Nikolaus Schaller
2019-11-07 10:30 ` [PATCH v3 11/12] net: wireless: ti: wl1251 use new SDIO_VENDOR_ID_TI_WL1251 definition H. Nikolaus Schaller
2019-11-07 10:30 ` [PATCH v3 12/12] net: wireless: ti: remove local VENDOR_ID and DEVICE_ID definitions H. Nikolaus Schaller
2019-11-14 14:18 ` [PATCH v3 00/12] OpenPandora: make wl1251 connected to mmc3 sdio port of OpenPandora work again Ulf Hansson
2019-11-14 15:15 ` H. Nikolaus Schaller
2019-11-15 9:29 ` Ulf Hansson [this message]
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='CAPDyKFr=Uk1i0c=3WvuOYCQ__Skpr-9mjVM2Yqst-hd8zY6OeQ@mail.gmail.com' \
--to=ulf.hansson@linaro.org \
--cc=alexios.zavras@intel.com \
--cc=allison@lohutok.net \
--cc=bcousson@baylibre.com \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=dsterba@suse.com \
--cc=gregkh@linuxfoundation.org \
--cc=hns@goldelico.com \
--cc=john.stultz@linaro.org \
--cc=kernel@pyra-handheld.com \
--cc=kvalo@codeaurora.org \
--cc=letux-kernel@openphoenux.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=netdev@vger.kernel.org \
--cc=pmladek@suse.com \
--cc=rafael.j.wysocki@intel.com \
--cc=robh+dt@kernel.org \
--cc=rppt@linux.ibm.com \
--cc=sakari.ailus@linux.intel.com \
--cc=tglx@linutronix.de \
--cc=tiny.windzz@gmail.com \
--cc=tony@atomide.com \
--cc=wangkefeng.wang@huawei.com \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).