Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934973AbbGHIJp (ORCPT ); Wed, 8 Jul 2015 04:09:45 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:60823 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934926AbbGHIJW (ORCPT ); Wed, 8 Jul 2015 04:09:22 -0400 Message-ID: <559CDA9E.7050401@ti.com> Date: Wed, 8 Jul 2015 11:09:02 +0300 From: Roger Quadros User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Marc Kleine-Budde , , CC: , , , Subject: Re: [PATCH 1/2] net: can: c_can: Fix default pinmux glitch at init References: <1436279277-3386-1-git-send-email-rogerq@ti.com> <1436279277-3386-2-git-send-email-rogerq@ti.com> <559BE34F.8080508@pengutronix.de> <559BE3BE.2020608@ti.com> <559BE433.9020308@ti.com> <559BE579.2020109@pengutronix.de> In-Reply-To: <559BE579.2020109@pengutronix.de> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1378 Lines: 43 Marc, On 07/07/15 17:43, Marc Kleine-Budde wrote: > On 07/07/2015 04:37 PM, Roger Quadros wrote: >>>>> diff --git a/drivers/net/can/c_can/c_can.c >>>>> b/drivers/net/can/c_can/c_can.c >>>>> index 041525d..66e98e7 100644 >>>>> --- a/drivers/net/can/c_can/c_can.c >>>>> +++ b/drivers/net/can/c_can/c_can.c >>>>> @@ -605,7 +605,18 @@ static int c_can_start(struct net_device *dev) >>>>> priv->can.state = CAN_STATE_ERROR_ACTIVE; >>>>> >>>>> /* activate pins */ >>>>> - pinctrl_pm_select_default_state(dev->dev.parent); >>>>> +#ifdef CONFIG_PINCTRL >>>> >>>> Please remove the ifdef, AFAICS there are static inline noop functions >>>> if CONFIG_PINCTRL switched off. >>> >>> yes, you are right. >> >> On second thoughts >> >> device->pins are not defined if CONFIG_PINCTRL is not set. >> so we can't remove the #ifdef. > > Too bad :( > > okay - should I add stable@v.k.o on Cc? I'm not sure if it would help. This patch by itself won't fix anything. It needs to go in together with the DTS change in patch 2. Maybe if Tony can Ack the second patch then both should be applicable on v4.0+ for stable. cheers, -roger -- 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/