From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peng Fan <van.freenix@gmail.com>
Date: Wed, 16 Mar 2016 09:27:01 +0800
Subject: [U-Boot] [PATCH 1/2] dm: gpio: mxc: fix mxc_gpio_bank_get_value
In-Reply-To: <CAOMZO5DOMtH6OEZ9yjh27B5bPc7yCYYnr+9O1mqHJv8Tvi8ssA@mail.gmail.com>
References: <1458020937-994-1-git-send-email-van.freenix@gmail.com>
 <CAOMZO5DOMtH6OEZ9yjh27B5bPc7yCYYnr+9O1mqHJv8Tvi8ssA@mail.gmail.com>
Message-ID: <20160316012658.GA10066@linux-7smt.suse>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Fabio,

On Tue, Mar 15, 2016 at 09:53:15AM -0300, Fabio Estevam wrote:
>On Tue, Mar 15, 2016 at 2:48 AM, Peng Fan <van.freenix@gmail.com> wrote:
>> When configured a gpio to output direction, directly reading PSR register
>> can not return the output value, since we did not set SION bit for gpio
>> iomux. So, we can use data register to reflect what value is outputed.
>
>Then why not simply set the SION bit instead?

If set the SION bit, we need to change the pinmux settings in device tree,
however device tree are introduced from linux kernel. I would not like
to change the pinmux setting value.

So I choose to use data register but not PSR register.

Regards,
Peng.