public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Arnaud Patard <arnaud.patard@rtp-net.org>
Cc: linux-omap-open-source@linux.omap.com
Subject: Re: [PATCH] add ssi/Kconfig to arch/arm/Kconfig
Date: Mon, 5 Mar 2007 02:01:38 -0800	[thread overview]
Message-ID: <20070305100136.GC17801@atomide.com> (raw)
In-Reply-To: <85r6s4damj.fsf@orfeo.duckcorp.org>

* Arnaud Patard <arnaud.patard@rtp-net.org> [070304 12:25]:
> Dirk Behme <dirk.behme@googlemail.com> writes:
> 
> > Arnaud Patard (Rtp) wrote:
> >> If one wants to use ssi/Kconfig on arm, it should be added to
> >> arch/arm/Kconfig.
> >
> > Yes, I found this as well by getting
> >
> > sound/arm/Kconfig:55:warning: 'select' used by config symbol
> > 'SND_OMAP_TSC2101' refer to undefined symbol 'OMAP_TSC2101'
> > sound/arm/Kconfig:56:warning: 'select' used by config symbol
> > 'SND_OMAP_TSC2101' refer to undefined symbol 'OMAP_UWIRE'
> >
> > Kconfig warnings.
> 
> same here :)
> 
> >
> > While touching this, what do you think about doing some more
> > clean up:
> >
> > - omap_uwire.c is now in drivers/spi [], remove (old?) one from
> > drivers/ssi
> 
> I had the same patch but you send it before me :)
> 
> >
> > - Replace OMAP_UWIRE in Kconfig by SPI_OMAP_UWIRE pointing
> > to drivers/spi
> >
> > ?
> >
> 
> Even if I'm not a omap dev, I'll be happy to see the ssi stuff go to
> /dev/null. Unfortunately, grep is telling us that there are still users
> of the old omap_uwire. Moreover, it seems that it's possible to build a
> kernel without the OMAP_UWIRE and with the drivers using it. The drivers
> using the old ssi stuff should either depends on it or select it...
> (see drivers/video/omap/lcd_{p,h}2.c).
> Until this is solved, we won't be able to remove it :(

I agree, let's get rid of the old drivers/ssi, anything using it should
be updated to use drivers/spi.

Regards,

Tony

  reply	other threads:[~2007-03-05 10:01 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-04 12:30 [PATCH] add ssi/Kconfig to arch/arm/Kconfig Arnaud Patard
2007-03-04 17:11 ` Dirk Behme
2007-03-04 20:20   ` Arnaud Patard
2007-03-05 10:01     ` Tony Lindgren [this message]
2007-03-05 19:56       ` Dirk Behme
2007-03-05 20:32         ` Brian Swetland
2007-03-08 10:36         ` Imre Deak

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=20070305100136.GC17801@atomide.com \
    --to=tony@atomide.com \
    --cc=arnaud.patard@rtp-net.org \
    --cc=linux-omap-open-source@linux.omap.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