Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932314AbbBZXdE (ORCPT ); Thu, 26 Feb 2015 18:33:04 -0500 Received: from mail-gw3-out.broadcom.com ([216.31.210.64]:21764 "EHLO mail-gw3-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754035AbbBZXdC (ORCPT ); Thu, 26 Feb 2015 18:33:02 -0500 X-IronPort-AV: E=Sophos;i="5.09,655,1418112000"; d="scan'208";a="57991889" Message-ID: <54EFAD2A.9060501@broadcom.com> Date: Thu, 26 Feb 2015 15:32:58 -0800 From: Ray Jui User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Wolfram Sang , CC: , , , Benjamin Herrenschmidt , Ludovic Desroches , Yingjoe Chen , Eddie Huang , Scott Branden , , Subject: Re: [RFC V2 12/12] i2c: bcm-iproc: make use of the new infrastructure for quirks References: <1424880126-15047-1-git-send-email-wsa@the-dreams.de> <1424880126-15047-13-git-send-email-wsa@the-dreams.de> In-Reply-To: <1424880126-15047-13-git-send-email-wsa@the-dreams.de> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2599 Lines: 72 On 2/25/2015 8:02 AM, Wolfram Sang wrote: > From: Wolfram Sang > > Signed-off-by: Wolfram Sang > --- > drivers/i2c/busses/i2c-bcm-iproc.c | 15 +++++++-------- > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-bcm-iproc.c b/drivers/i2c/busses/i2c-bcm-iproc.c > index d3c89157b33774..f9f2c2082151e2 100644 > --- a/drivers/i2c/busses/i2c-bcm-iproc.c > +++ b/drivers/i2c/busses/i2c-bcm-iproc.c > @@ -160,14 +160,6 @@ static int bcm_iproc_i2c_xfer_single_msg(struct bcm_iproc_i2c_dev *iproc_i2c, > u32 val; > unsigned long time_left = msecs_to_jiffies(I2C_TIMEOUT_MESC); > > - /* need to reserve one byte in the FIFO for the slave address */ > - if (msg->len > M_TX_RX_FIFO_SIZE - 1) { > - dev_err(iproc_i2c->device, > - "only support data length up to %u bytes\n", > - M_TX_RX_FIFO_SIZE - 1); > - return -EOPNOTSUPP; > - } > - > /* check if bus is busy */ > if (!!(readl(iproc_i2c->base + M_CMD_OFFSET) & > BIT(M_CMD_START_BUSY_SHIFT))) { > @@ -287,6 +279,12 @@ static const struct i2c_algorithm bcm_iproc_algo = { > .functionality = bcm_iproc_i2c_functionality, > }; > > +static struct i2c_adapter_quirks bcm_iproc_i2c_quirks = { > + /* need to reserve one byte in the FIFO for the slave address */ > + .max_read_len = M_TX_RX_FIFO_SIZE - 1, > + .max_write_len = M_TX_RX_FIFO_SIZE - 1, > +}; > + > static int bcm_iproc_i2c_cfg_speed(struct bcm_iproc_i2c_dev *iproc_i2c) > { > unsigned int bus_speed; > @@ -413,6 +411,7 @@ static int bcm_iproc_i2c_probe(struct platform_device *pdev) > i2c_set_adapdata(adap, iproc_i2c); > strlcpy(adap->name, "Broadcom iProc I2C adapter", sizeof(adap->name)); > adap->algo = &bcm_iproc_algo; > + adap->quirks = &bcm_iproc_i2c_quirks; > adap->dev.parent = &pdev->dev; > adap->dev.of_node = pdev->dev.of_node; > > Change on the iproc i2c driver looks good to me. Sanity tested the change from Wolfram's i2c/quirks branch on Cygnus 958300K combo board. Sanity tested with an attempt to transfer large amount of I2C data to ensure the transfer is denied by the i2c-core: / # cat /dev/i2c-0 [ 657.310261] i2c i2c-0: quirk: msg too long (addr 0x0000, size 4096, read) Reviewed-by: Ray Jui Tested-by: Ray Jui Thanks, Ray -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/