From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753795AbdHUM6l (ORCPT ); Mon, 21 Aug 2017 08:58:41 -0400 Received: from mail-it0-f54.google.com ([209.85.214.54]:37980 "EHLO mail-it0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753491AbdHUM6j (ORCPT ); Mon, 21 Aug 2017 08:58:39 -0400 MIME-Version: 1.0 X-Originating-IP: [149.6.131.75] In-Reply-To: References: <20170615185418.13980-1-javier@dowhile0.org> <20170615185418.13980-12-javier@dowhile0.org> From: Javier Martinez Canillas Date: Mon, 21 Aug 2017 14:58:37 +0200 Message-ID: Subject: Re: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM To: Michal Simek Cc: Will Deacon , Mark Rutland , Catalin Marinas , Linux Kernel , Wolfram Sang , Rob Herring , =?UTF-8?Q?S=C3=B6ren_Brinkmann?= , "devicetree@vger.kernel.org" , Rob Herring , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Michal, On Mon, Aug 21, 2017 at 2:04 PM, Michal Simek wrote: > On 21.8.2017 13:27, Javier Martinez Canillas wrote: >> [snip] >> >>>> >>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts >>>> index ef1b9e573af0..53907b7a7fd9 100644 >>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts >>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts >>>> @@ -55,7 +55,7 @@ >>>> status = "okay"; >>>> clock-frequency = <400000>; >>>> eeprom@54 { >>>> - compatible = "at,24c64"; >>>> + compatible = "atmel,24c64"; >>>> reg = <0x54>; >>>> }; >>>> }; >>>> @@ -64,7 +64,7 @@ >>>> status = "okay"; >>>> clock-frequency = <400000>; >>>> eeprom@55 { >>>> - compatible = "at,24c64"; >>>> + compatible = "atmel,24c64"; >>>> reg = <0x55>; >>>> }; >>>> }; >>>> >>> >>> Acked-by: Michal Simek >>> >> >> Any reasons why this patch was never picked? It was acked two months ago. > > Normally when you send series then it should be applied as the whole > series not just one piece of that. > Right, I mentioned in the cover letter that the patches could be applied independently. Sorry for the confusion. > Anyway I see that changes are applied and this one is missing that's why > I will add to zynqmp/dt branch and going to send pull request today. > Great, thanks a lot for your help. > Thanks, > Michal Best regards, Javier From mboxrd@z Thu Jan 1 00:00:00 1970 From: javier@dowhile0.org (Javier Martinez Canillas) Date: Mon, 21 Aug 2017 14:58:37 +0200 Subject: [RESEND PATCH v5 11/16] arm64: zynqmp: Add generic compatible string for I2C EEPROM In-Reply-To: References: <20170615185418.13980-1-javier@dowhile0.org> <20170615185418.13980-12-javier@dowhile0.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello Michal, On Mon, Aug 21, 2017 at 2:04 PM, Michal Simek wrote: > On 21.8.2017 13:27, Javier Martinez Canillas wrote: >> [snip] >> >>>> >>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts >>>> index ef1b9e573af0..53907b7a7fd9 100644 >>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts >>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-ep108.dts >>>> @@ -55,7 +55,7 @@ >>>> status = "okay"; >>>> clock-frequency = <400000>; >>>> eeprom at 54 { >>>> - compatible = "at,24c64"; >>>> + compatible = "atmel,24c64"; >>>> reg = <0x54>; >>>> }; >>>> }; >>>> @@ -64,7 +64,7 @@ >>>> status = "okay"; >>>> clock-frequency = <400000>; >>>> eeprom at 55 { >>>> - compatible = "at,24c64"; >>>> + compatible = "atmel,24c64"; >>>> reg = <0x55>; >>>> }; >>>> }; >>>> >>> >>> Acked-by: Michal Simek >>> >> >> Any reasons why this patch was never picked? It was acked two months ago. > > Normally when you send series then it should be applied as the whole > series not just one piece of that. > Right, I mentioned in the cover letter that the patches could be applied independently. Sorry for the confusion. > Anyway I see that changes are applied and this one is missing that's why > I will add to zynqmp/dt branch and going to send pull request today. > Great, thanks a lot for your help. > Thanks, > Michal Best regards, Javier