Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754461AbaDYOQu (ORCPT ); Fri, 25 Apr 2014 10:16:50 -0400 Received: from ns.mm-sol.com ([37.157.136.199]:36961 "EHLO extserv.mm-sol.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753216AbaDYOQq (ORCPT ); Fri, 25 Apr 2014 10:16:46 -0400 Message-ID: <1398435350.4724.244.camel@iivanov-dev> Subject: Re: [RFC PATCH] mfd: pm8x41: Naive function devices registration From: "Ivan T. Ivanov" To: Rob Herring Cc: Courtney Cavin , Josh Cartwright , Samuel Ortiz , Lee Jones , "linux-kernel@vger.kernel.org" , linux-arm-msm Date: Fri, 25 Apr 2014 17:15:50 +0300 In-Reply-To: References: <1398429171-8566-1-git-send-email-iivanov@mm-sol.com> <1398432541.4724.243.camel@iivanov-dev> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.1-2ubuntu2~saucy1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2014-04-25 at 08:43 -0500, Rob Herring wrote: > On Fri, Apr 25, 2014 at 8:29 AM, Ivan T. Ivanov wrote: > > On Fri, 2014-04-25 at 08:00 -0500, Rob Herring wrote: > >> On Fri, Apr 25, 2014 at 7:32 AM, Ivan T. Ivanov wrote: > >> > From: "Ivan T. Ivanov" > >> > > >> > Currently functions that exist in both the controller at the > >> > same address offset can not be specified with the same names. > >> > > >> > Adding Unique Slave ID device address to prefix function > >> > device names fixes this. > >> > > >> > Function devices are SPMI devices, so register them on > >> > SPMI bus. > >> > > >> > Signed-off-by: Ivan T. Ivanov > >> > --- > >> > drivers/mfd/pm8x41.c | 61 ++++++++++++++++++++++++++++++++++++++++++++++++---- > >> > >> No, this should be fixed in the core, not the driver. > > > > I think that at core level they are no issues. > > By core, I mean the device naming conventions used by the DT platform > device code. There is a problem and it should be handled. > > As I mentioned in the other thread, either we should not use the > address on non-translatable addresses like this or append the parent > address. Ok, I see. Regards, Ivan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/