From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v2] pinctrl: msm8998: Squash TSIF pins together Date: Mon, 22 Jul 2019 17:53:00 -0600 Message-ID: <20190722235300.GA24879@bogus> References: <503b2ae8-ead6-70cd-7b21-ce5f5166a23a@free.fr> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <503b2ae8-ead6-70cd-7b21-ce5f5166a23a@free.fr> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Marc Gonzalez Cc: DT , Jeffrey Hugo , MSM , Linus Walleij , Jonathan =?iso-8859-1?Q?Neusch=E4fer?= , Bjorn Andersson , gpio , Linux ARM List-Id: devicetree@vger.kernel.org On Thu, Jul 04, 2019 at 10:57:06AM +0200, Marc Gonzalez wrote: > TSIF is the Transport Stream Interface. > First, rename tsif1 to tsif0, and tsif2 to tsif1. > Then squash all 5 tsif0 pins into a single function. > Same for tsif1. Doesn't this break backwards compatibility? If so, you should say so and say why that's okay for this platform. In any case, whether it's a problem or not is up to the platform maintainer(s). > > Signed-off-by: Marc Gonzalez > --- > Changes from v1: > - Reword commit message for clarity (hopefully) > - Drop unrelated change in qcom,msm8998-pinctrl.txt > - CC DT > --- > .../bindings/pinctrl/qcom,msm8998-pinctrl.txt | 5 +- Acked-by: Rob Herring > drivers/pinctrl/qcom/pinctrl-msm8998.c | 76 +++++-------------- > 2 files changed, 20 insertions(+), 61 deletions(-)