Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758936AbdCVHcj (ORCPT ); Wed, 22 Mar 2017 03:32:39 -0400 Received: from conssluserg-02.nifty.com ([210.131.2.81]:55277 "EHLO conssluserg-02.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758514AbdCVHb4 (ORCPT ); Wed, 22 Mar 2017 03:31:56 -0400 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-02.nifty.com v2M7VaWm007102 X-Nifty-SrcIP: [209.85.161.169] MIME-Version: 1.0 In-Reply-To: <1490106791-3553-1-git-send-email-piotrs@cadence.com> References: <1490106736-2242-1-git-send-email-piotrs@cadence.com> <1490106791-3553-1-git-send-email-piotrs@cadence.com> From: Masahiro Yamada Date: Wed, 22 Mar 2017 16:31:35 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [v5 3/4] mmc: sdhci-cadence: Update PHY delay configuration To: Piotr Sroka Cc: linux-mmc , Adrian Hunter , Ulf Hansson , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1101 Lines: 34 2017-03-21 23:33 GMT+09:00 Piotr Sroka : > DTS properties are used instead of fixed data > because PHY settings can be different for different chips/boards. > > Signed-off-by: Piotr Sroka > --- > Changes for v2: > - dts part was removed from this patch > - most delays were moved from dts file > to data associated with an SoC specific compatible > - remove unrelated changes > --- > Changes for v3: > - move all delays back to dts because they are also boards dependent > - prefix all of the Cadence-specific properties with cdns prefix > - put checking delay properties inside the for loop > instead of using a lot of single if expressions > --- > Changes for v4: > - remove unecessary declaration of sdhci_cdns_match > - format fix (blank line removed) > --- > Changes for v5: > - use driver version from next branch, with applied enhanced strobe feature support. > - change name of property to be consistent with timing modes > available in Linux > --- Reviewed-by: Masahiro Yamada -- Best Regards Masahiro Yamada