On Wed, 2018-04-18 at 15:35 +0530, Amit Pundir wrote:

> +                     if (phy->next_read_size >
> FDP_NCI_I2C_MAX_PAYLOAD) {

> +                             dev_dbg(&client->dev, "%s: corrupted
> packet\n",
> +                                     __func__);

If Android people would follow the kernel in reasonable time they may
have noticed Dynamic Debug functionality and how it works.

In this case the __func__ is superfluous.

> +                             phy->next_read_size = 5;
> +                             goto flush;
> +                     }
>               } else {
>                       phy->next_read_size =
> FDP_NCI_I2C_MIN_PAYLOAD;
>  

-- 
Andy Shevchenko <andriy.shevche...@linux.intel.com>
Intel Finland Oy

Reply via email to