From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753263AbdLERHp (ORCPT ); Tue, 5 Dec 2017 12:07:45 -0500 Received: from mail.kernel.org ([198.145.29.99]:46286 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753079AbdLERHm (ORCPT ); Tue, 5 Dec 2017 12:07:42 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6C535218B3 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=atull@kernel.org X-Google-Smtp-Source: AGs4zMbgv1FXosEKSOy1xsa7mYbm3U/sEWiYScTdZ1U5jGI5c1FhbXet23Ba1yPzeomwwAmP7hHrs0aLFKlFuorbDYw= MIME-Version: 1.0 In-Reply-To: References: <20171204191357.3211-1-atull@kernel.org> From: Alan Tull Date: Tue, 5 Dec 2017 11:07:00 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 0/2] of: dynamic: restrict overlay by targets To: Frank Rowand Cc: Moritz Fischer , Rob Herring , Pantelis Antoniou , "devicetree@vger.kernel.org" , linux-kernel , linux-fpga@vger.kernel.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 On Mon, Dec 4, 2017 at 7:14 PM, Frank Rowand wrote: > Hi Alan, > > In the RFC thread "of: Add whitelist", I did not understand the use case and > asked you some questions (30 Nov 2017 07:46:36 -0500), that you seem to have > overlooked (or my mail server failed to deliver your answer to me). Can you > please answer that question so I can better understand this patch set is > needed for. Hi Frank, Sorry I missed those, I've replied to the original questions now. Alan > > Thanks, > > Frank > > > On 12/04/17 14:13, Alan Tull wrote: >> Restrict which nodes are valid targets for a DT overlay. >> >> Add a flag bit to struct device_node allowing nodes to be marked as >> valid target for overlays. >> >> A driver that is always intended to handle DT overlays can >> enable overlays by calling a function for its DT node. >> >> For individual nodes that need to be opened up for a specific use, >> adding the property "overlay-allowed" enables overlays targeting >> that node. I'll need to document the DT property, not sure where >> specifically. New file bindings/overlay.txt? >> >> This patchset differs from the RFC: >> * Added a flag bit and got rid of the whitelist >> * Renamed the functions that enable a node >> * Added a DT property >> >> Alan Tull (2): >> of: overlay: add flag enabling overlays and enable fpga-region >> overlays >> of: dynamic: add overlay-allowed DT property >> >> drivers/fpga/of-fpga-region.c | 4 ++++ >> drivers/of/base.c | 4 ++-- >> drivers/of/dynamic.c | 3 +++ >> drivers/of/fdt.c | 3 +++ >> drivers/of/of_private.h | 2 ++ >> drivers/of/overlay.c | 26 ++++++++++++++++++++++++++ >> include/linux/of.h | 19 +++++++++++++++++++ >> 7 files changed, 59 insertions(+), 2 deletions(-) >> >