From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754269AbeBUNYO (ORCPT ); Wed, 21 Feb 2018 08:24:14 -0500 Received: from mail.kernel.org ([198.145.29.99]:56320 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753180AbeBUNYM (ORCPT ); Wed, 21 Feb 2018 08:24:12 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B9591217A0 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=robh@kernel.org X-Google-Smtp-Source: AH8x225oDH7an3a+FMuEDI39cELszblfpHdBYMFvM+hM49ILnAft8S044xdvJ+6/VbIp5taDGq/iWDWgG6FnoSdO18k= MIME-Version: 1.0 In-Reply-To: References: <1518701959-22899-1-git-send-email-tali.perry1@gmail.com> <1518701959-22899-2-git-send-email-tali.perry1@gmail.com> <20180219144905.bfrycjnas7dcuo34@rob-hp-laptop> From: Rob Herring Date: Wed, 21 Feb 2018 07:23:50 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 1/1] clk: npcm750: update text with fixed clocks To: Brendan Higgins Cc: Tali Perry , Mark Rutland , Russell King , Avi Fishman , Tomer Maimon , Rick Altherr , devicetree , Linux Kernel Mailing List , Linux ARM , OpenBMC Maillist , Nancy Yuen , Patrick Venture Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 20, 2018 at 10:47 PM, Brendan Higgins wrote: > On Mon, Feb 19, 2018 at 6:49 AM, Rob Herring wrote: >> On Thu, Feb 15, 2018 at 02:38:12PM -0800, Brendan Higgins wrote: >>> On Thu, Feb 15, 2018 at 5:39 AM, Tali Perry wrote: >>> > >>> > Signed-off-by: Tali Perry >>> > >>> >>> >>> I think this should probably be rolled into [PATCH v2 1/1] npcm750: add fixed >>> clocks (moved from drivers/clk/clk-npcm7xx.c): >>> https://www.spinics.net/lists/arm-kernel/msg634678.html >> >> No, binding docs, dts files and driver code should all be separate >> patches. > > My mistake. This patch has a dt-bindings include file; should the include file > go in here, with the dtsi changes, or in its own separate patch? It defines the binding, so with the binding documentation. Rob