From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7B238C0651F for ; Thu, 4 Jul 2019 14:00:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5DB022085A for ; Thu, 4 Jul 2019 14:00:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727371AbfGDOAi (ORCPT ); Thu, 4 Jul 2019 10:00:38 -0400 Received: from host-88-217-225-28.customer.m-online.net ([88.217.225.28]:37011 "EHLO mail.dev.tdt.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727199AbfGDOAh (ORCPT ); Thu, 4 Jul 2019 10:00:37 -0400 Received: from mail.dev.tdt.de (localhost [IPv6:::1]) by mail.dev.tdt.de (Postfix) with ESMTP id 6F50421580; Thu, 4 Jul 2019 14:00:35 +0000 (UTC) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Thu, 04 Jul 2019 16:00:35 +0200 From: Florian Eckert To: Andy Shevchenko Cc: Eckert.Florian@googlemail.com, "Enrico Weigelt, metux IT consult" , Darren Hart , Andy Shevchenko , Platform Driver , Linux Kernel Mailing List Subject: Re: [PATCH 0/3] Update pcengines-apuv2 platform device In-Reply-To: References: <20190704090205.19400-1-fe@dev.tdt.de> Message-ID: X-Sender: fe@dev.tdt.de User-Agent: Roundcube Webmail/1.1.5 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Andy, thanks for feedback >> This patchset adds the following changes to this pcengines-apuv2 >> platform device. >> > > Before doing anything to this driver, what is the plan for previously > upstreamed: > > drivers/leds/leds-apu.c I think we can remove the related APU2/APU3 code stuff from this driver. The recently added pcengines-apuv2 driver does *not* support the APU1. So I think we need the related APU1 stuff if we still want to support this board. > arch/x86/platform/geode/alix.c I think this is not related because this is a different platform driver. Maybe we should move them to drivers/platform/x86? Regards Florian