[PATCH] Mediatek staging: Use individual config flags in Makefile
George Hilliard
thirtythreeforty at gmail.com
Thu Jan 10 07:58:03 UTC 2019
On Thu, Jan 10, 2019 at 12:39 AM Greg KH <greg at kroah.com> wrote:
>
> On Thu, Jan 10, 2019 at 12:26:28AM -0700, thirtythreeforty at gmail.com wrote:
> > From: George Hilliard <thirtythreeforty at gmail.com>
> >
> > These drivers are useful on other MT76xx SoCs, which have compatible
> > peripherals. The drivers are selectable in Kconfig, but they were
> > quietly excluded from the build because the SOC_MT7621 chip was not
> > selected. So, make the main staging Makefile use the same flags as
> > everything else for these drivers.
> >
> > I have personally confirmed that the mt7621-spi driver works on the
> > MT7688, which was what prompted this change.
> >
> > Cc: devel at driverdev.osuosl.org
> > Cc: Matthias Brugger <matthias.bgg at gmail.com>
> > Cc: John Crispin <blogic at openwrt.org>
> > Signed-off-by: George Hilliard <thirtythreeforty at gmail.com>
> > ---
> > drivers/staging/Makefile | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/staging/Makefile b/drivers/staging/Makefile
> > index 5868631e8f1b..50c980c16c0d 100644
> > --- a/drivers/staging/Makefile
> > +++ b/drivers/staging/Makefile
> > @@ -42,10 +42,10 @@ obj-$(CONFIG_BCM2835_VCHIQ) += vc04_services/
> > obj-$(CONFIG_DRM_VBOXVIDEO) += vboxvideo/
> > obj-$(CONFIG_PI433) += pi433/
> > obj-$(CONFIG_SOC_MT7621) += mt7621-pci/
> > -obj-$(CONFIG_SOC_MT7621) += mt7621-pinctrl/
> > -obj-$(CONFIG_SOC_MT7621) += mt7621-spi/
> > +obj-$(CONFIG_PINCTRL_RT2880) += mt7621-pinctrl/
> > +obj-$(CONFIG_SPI_MT7621) += mt7621-spi/
> > obj-$(CONFIG_SOC_MT7621) += mt7621-dma/
>
> What about the DMA driver?
It has two separate drivers in it, actually. Briefly:
config DMA_RALINK
tristate "RALINK DMA support"
depends on RALINK && !SOC_RT288X
config MTK_HSDMA
tristate "MTK HSDMA support"
depends on RALINK && SOC_MT7621
I think they should be broken out into two separate directories to
make them amenable to this change. Should I do this?
>
> > -obj-$(CONFIG_SOC_MT7621) += mt7621-mmc/
> > +obj-$(CONFIG_MTK_MMC) += mt7621-mmc/
> > obj-$(CONFIG_SOC_MT7621) += mt7621-eth/
> > obj-$(CONFIG_SOC_MT7621) += mt7621-dts/
>
> And these drivers? Why not change all of these at once?
mt7621-dts ignored because it truly does depend on the MT7621 (only
thing in there is the device tree for the GnuBee).
mt7621-eth ignored because I forgot it. I'll add to the next revision.
>
> thanks,
>
> greg k-h
More information about the devel
mailing list