Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755187AbaFQEHr (ORCPT ); Tue, 17 Jun 2014 00:07:47 -0400 Received: from mail-ve0-f171.google.com ([209.85.128.171]:61395 "EHLO mail-ve0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751040AbaFQEHp (ORCPT ); Tue, 17 Jun 2014 00:07:45 -0400 MIME-Version: 1.0 In-Reply-To: References: <1402464739-19044-1-git-send-email-tushar.b@samsung.com> <1402464739-19044-4-git-send-email-tushar.b@samsung.com> <539ED2C0.8030405@gmail.com> Date: Mon, 16 Jun 2014 21:07:44 -0700 Message-ID: Subject: Re: [PATCH 3/3] ARM: dts: Enable audio support for Peach-pi board From: Doug Anderson To: Tushar Behera Cc: "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , linux-samsung-soc , "linux-arm-kernel@lists.infradead.org" , Mike Turquette , Tomasz Figa , Russell King , Kumar Gala , Ian Campbell , Mark Rutland , Pawel Moll , Rob Herring , Kukjin Kim , Kevin Hilman , Mark Brown Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Tushar, On Mon, Jun 16, 2014 at 8:36 PM, Tushar Behera wrote: > On Mon, Jun 16, 2014 at 10:19 PM, Doug Anderson wrote: >> Tushar, >> >> On Mon, Jun 16, 2014 at 4:19 AM, Tushar Behera wrote: >>> On 06/13/2014 10:33 PM, Doug Anderson wrote: >>>> Tushar, >>>> >>>> On Tue, Jun 10, 2014 at 10:32 PM, Tushar Behera wrote: >>>>> Peach-pi board has MAX98090 audio codec connected on HSI2C-7 bus. >>>> >>>> If you want to be a stickler about it, peach-pi actually has a >>>> max98091. That requires code changes to the i2c driver, though. >>>> ...and unfortunately listing two compatible strings for i2c devices is >>>> broken. :( >>>> >>> Hi Doug, >>> >>> You are right. I checked the boot logs, the detected codec type is >>> MAX98091. Since both these CODECs are supported through a single driver >>> and the detection of chip is done during runtime, I would suggest we go >>> ahead with "max98090" compatible string. I will update the commit >>> message accordingly. >>> >>> Does that sound okay to you? >> >> As per my understanding you shouldn't do this. You should have two patches: >> >> 1. Add "max98091". You could simply post Wonjoon's patch from >> >> >> 2. Change the device tree to refer to "max98091" >> >> The argument that the "current kernel driver has a single driver" is >> an argument that you're not supposed to make for device tree. The >> same device tree is supposed to work for U-Boot, BSD, or any other >> platform. On those platforms it might not be a shared driver. >> > > My argument is that the device type is getting detected during > runtime, hence there is no need to differentiate between these two. > > But if you prefer that way, I will repost. Yes please. True that it is possible to detect 98090 vs. 98091. ...but it's also possible to detect exynos5250 vs. exynos5420 vs. exynos5800. ...yet they have different compatible strings. -- 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/