linux-spi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Flávio Alberto Lopes Soares" <flavioalsoares-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: Analog Digital Converter - ADS1256 extreme newbie questions
Date: Sun, 12 Sep 2010 11:39:29 -0300	[thread overview]
Message-ID: <AANLkTikXq6Wm0kA_cHiAyZc+5kOJQBvP_2gOSbnihv5i@mail.gmail.com> (raw)
In-Reply-To: <20100910171858.GA11284-MrY2KI0G/OVr83L8+7iqerDks+cytr/Z@public.gmane.org>

Hi Grant,
thank you for your help.

I misunderstood the "struct spi_device *spi" that functions in protocol
drivers receive as parameters, for example in the probe function, I thought
that spi_device struct represent the spi master controller but as I saw
in Documentation/DocBook/device-drivers/ch09.html it represents itself and
not the SPI controller.

But, comparing drivers/staging/iio/accel/kxsd9.c
and drivers/misc/eeprom/at25.c why kxsd9 driver doesn't uses the
platform_data struct ? There's some functional difference in these 2 drivers
in the way that they access the SPI subsystem ?
I'm thinking of use the kxsd9.c as a "master" to write my driver because the
apparent simplicity way of access the SPI subsystem, I like it.

Thanks for all help
Flavio Alberto Lopes Soares

2010/9/10 Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>

> On Fri, Sep 10, 2010 at 10:11:35AM -0300, Flávio Alberto Lopes Soares
> wrote:
> > Hello all,
> > I'm working 9 years as a Linux C/C++ programmer but I spent 99,9% of this
> > time working in user space programming, now I need to write a driver to
> use
> > the ADS1256 Analog-Digital converter (
> > http://focus.ti.com/docs/prod/folders/print/ads1256.html ) in an ARM
> board
> > using S3C2440 processor, my lecture start point
> > was Documentation/spi/spi-sumary from 2.6.32.2 kernel tree version and
> > drivers/misc/eeprom/at25.c code (this is an eeprom driver but I want look
> > the SPI general view), reading the text and the code I found some
> conceptual
> > doubts, my initial idea was create a character device driver and attach
> the
> > device read operation with a ADS1256 start lecture and pass the ADC
> channel
> > data to this device, this concept is correct ? There's some standard
> > interface to do this if my idea is wrong ?
> >
> > Looking in at25.c in at25_probe(struct spi_device *spi) function this
> > "struct spi_device *spi" parameter represents the S3C2440 SPI controller
> and
> > came from SPI core subsystem ? If yes how the SPI controller knows that
> this
> > is an eeprom device in the snippet bellow ?
> > ...
> > const struct spi_eeprom *chip;
> > ...
> > /* Chip description */
> > chip = spi->dev.platform_data;
> >
> > who provides this platform_data structure and who knows it must be a
> struct
> > spi_eeprom object ?
>
> Hi Flávio.
>
> The platform_data structure is provided by the code that registers the
> spi_device; usually this is some form of machine specific platform
> code in an arch/*/ subdirectory.  The driver knows that platform_data
> points to the drivers structure because the code that registered the
> device must be responsible to provide the right kind of platform_data
> pointer for the driver that will be bound to it.
>
> For an example, look at arch/arm/mach-omap2/board-3430sdp.c which has
> an spi_board_info structure which passes both the requested driver
> (.modalias = "ads7846") and the data structure required by the ads7846
> driver (.platform_data = "&tsc2046_config").
>
> The matching driver is drivers/input/touchscreen/ads7846.c
>
> You'll notice that board-3430sdp.c and ads7846.c both use the
> ads7846_platform_data structure.
>
> g.
>
------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing
http://p.sf.net/sfu/novell-sfdev2dev

  parent reply	other threads:[~2010-09-12 14:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-10 13:11 Analog Digital Converter - ADS1256 extreme newbie questions Flávio Alberto Lopes Soares
     [not found] ` <AANLkTi=iQVwpd-PymaWSKapfntmf4aRrt51Jutsz=D_g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-09-10 17:18   ` Grant Likely
     [not found]     ` <20100910171858.GA11284-MrY2KI0G/OVr83L8+7iqerDks+cytr/Z@public.gmane.org>
2010-09-12 14:39       ` Flávio Alberto Lopes Soares [this message]
     [not found]         ` <AANLkTikXq6Wm0kA_cHiAyZc+5kOJQBvP_2gOSbnihv5i-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-09-12 20:30           ` Grant Likely
  -- strict thread matches above, loose matches on Subject: below --
2010-09-10  5:13 Flávio Alberto Lopes Soares

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=AANLkTikXq6Wm0kA_cHiAyZc+5kOJQBvP_2gOSbnihv5i@mail.gmail.com \
    --to=flavioalsoares-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.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
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).