From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joel Stanley Date: Wed, 3 Oct 2018 09:49:41 +0200 Subject: [PATCH i2c-next v4 2/3] i2c: aspeed: Add 'timeout' DT property reading code In-Reply-To: <20181001202748.8030-3-jae.hyun.yoo@linux.intel.com> References: <20181001202748.8030-1-jae.hyun.yoo@linux.intel.com> <20181001202748.8030-3-jae.hyun.yoo@linux.intel.com> Message-ID: List-Id: To: linux-aspeed@lists.ozlabs.org MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hello Jae, On Mon, 1 Oct 2018 at 22:28, Jae Hyun Yoo wrote: > > This commit adds reading code of the 'timeout' DT property to set > bus timeout value in adapter configuration. This value still > can be configured through an I2C_TIMEOUT ioctl on cdev too. > > Signed-off-by: Jae Hyun Yoo > --- > drivers/i2c/busses/i2c-aspeed.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/i2c/busses/i2c-aspeed.c b/drivers/i2c/busses/i2c-aspeed.c > index 8dc9161ced38..6d31f54a6653 100644 > --- a/drivers/i2c/busses/i2c-aspeed.c > +++ b/drivers/i2c/busses/i2c-aspeed.c > @@ -885,6 +885,7 @@ static int aspeed_i2c_probe_bus(struct platform_device *pdev) > struct clk *parent_clk; > struct resource *res; > int irq, ret; > + u32 timeout_ms; > > bus = devm_kzalloc(&pdev->dev, sizeof(*bus), GFP_KERNEL); > if (!bus) > @@ -918,6 +919,11 @@ static int aspeed_i2c_probe_bus(struct platform_device *pdev) > bus->bus_frequency = 100000; > } > > + ret = of_property_read_u32(pdev->dev.of_node, "timeout", > + &timeout_ms); > + if (ret) > + timeout_ms = 0; /* then adap.timeout will be set by i2c-core */ Is it possible to implement the parsing of this property in the i2c core instead?