From: Henk Stegeman <henk.stegeman@gmail.com>
To: Juergen Beisert <jbe@pengutronix.de>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: Davicom DM9000A on MPC5200B (powerpc) works using a dirty offsetting and byte trick
Date: Mon, 9 Mar 2009 10:32:19 +0100 [thread overview]
Message-ID: <ae4f76fd0903090232n21152b02of5ebcac86c60e53a@mail.gmail.com> (raw)
In-Reply-To: <200903071109.49726.jbe@pengutronix.de>
Juergen,
I don't understand how this would work,
Now I do one byte-swap, which works.
-I byteswap in software, for 16-bit cycles by byte swapping and for 8
bit cycles by adding an offset of 1.
(The byte swapping on the chipselect is off)
Your advice includes two byteswaps, one by re-routing the data bus and
one by enabling the byte swap on the chip-select.
Or does one of them not really swap bytes?
Henk
On Sat, Mar 7, 2009 at 11:09 AM, Juergen Beisert <jbe@pengutronix.de> wrote=
:
> Henk,
>
> On Freitag, 20. Februar 2009, Henk Stegeman wrote:
>> - Any suggestions to what could be wrong here? Or does the MPC5200 in
>> this case only byte swap u16 reads, but a u8 read is unchanged?
>
> You should not follow the Freescale bus signal names when you connect you=
r
> external little endian device. Otherwise the offsets are always wrong.
>
> Do it in this way instead:
>
> =A0 =A0MPC =A0 =A0 =A0 =A0 =A0LE Device
> =A0D[0..7] =A0 <-> =A0D[24..31]
> =A0D[8..15] =A0<-> =A0D[16..23]
> =A0D[16..23] <-> =A0D[8..15]
> =A0D[24..31] <-> =A0D[0..7]
>
> If you connect your device in such a way, just enable CS's byte swap feat=
ure
> depending on your bus size and you are done (no additional software
> manipulation required). Now you can write bytes, words or longs and you w=
ill
> always write the correct data into the corresponding device register.
>
> Hope it helps
> Juergen
>
> --
> Pengutronix e.K. =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =
=A0| Juergen Beisert =A0 =A0 =A0 =A0 =A0 =A0 |
> Linux Solutions for Science and Industry =A0 =A0 =A0| Phone: +49-8766-939=
228 =A0 =A0 |
> Vertretung Sued/Muenchen, Germany =A0 =A0 =A0 =A0 =A0 =A0 | Fax: =A0 +49-=
5121-206917-5555 |
> Amtsgericht Hildesheim, HRA 2686 =A0 =A0 =A0 =A0 =A0 =A0 =A0| http://www.=
pengutronix.de/ =A0|
>
next prev parent reply other threads:[~2009-03-09 9:32 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-20 9:51 Davicom DM9000A on MPC5200B (powerpc) works using a dirty offsetting and byte trick Henk Stegeman
2009-03-06 15:02 ` Grant Likely
2009-03-07 10:09 ` Juergen Beisert
2009-03-09 9:32 ` Henk Stegeman [this message]
2009-03-09 11:09 ` Juergen Beisert
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=ae4f76fd0903090232n21152b02of5ebcac86c60e53a@mail.gmail.com \
--to=henk.stegeman@gmail.com \
--cc=jbe@pengutronix.de \
--cc=linuxppc-dev@ozlabs.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).