On Thu, 2018-10-11 at 16:11 +0200, Dirk Eibach wrote: > > Hello, > > we have a 16 bit value here, so we have to define whether bit0(containin the > information for IO0.0) is in the first or the second byte. Since the PCA9555 > does this encoding little endian, the conversion is allright. >
You used it as some number but this is really just a bunch I/O pins. I haven't seen any endian conversion in the kernel driver, have you? This is a bigger question than this driver really, so: Does IO pins have an endian? Jocke > Cheers > Dirk > Am Do., 11. Okt. 2018 um 07:42 Uhr schrieb Heiko Schocher <h...@denx.de>: > > > > Hello Joakim, > > > > Am 10.10.2018 um 19:34 schrieb Joakim Tjernlund: > > > This commit broke our pca953x usage(on ppc). > > > > > > I wonder why gpio pins here has an endian, its not a number. > > > If there must be an endian connected with this, should it not > > > be a cpu_to_be16 instead, which will retain compatibility ? > > > > Hmm.. good question, I think you are right. May dirk can do a test? > > I have no pca953x with 16bit for doing a test. > > > > bye, > > Heiko > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: h...@denx.de _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
