All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cyril Chemparathy <cyril-l0cyMroinI0@public.gmane.org>
To: "Nori, Sekhar" <nsekhar-l0cyMroinI0@public.gmane.org>
Cc: "davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org"
	<davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org>,
	"dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org"
	<dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>,
	"sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org"
	<sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
	"linus.ml.walleij-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
	<linus.ml.walleij-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org"
	<broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>,
	"grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org"
	<grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>,
	"rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org"
	<rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org>,
	"alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org"
	<alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org>,
	"spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org"
	<spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>,
	"akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org"
	<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	"lrg-kDsPt+C1G03kYMGBc/C6ZA@public.gmane.org"
	<lrg-kDsPt+C1G03kYMGBc/C6ZA@public.gmane.org>
Subject: Re: [PATCH v7 11/12] davinci: add tnetv107x evm backlight device
Date: Thu, 09 Dec 2010 09:58:41 -0500	[thread overview]
Message-ID: <4D00EEA1.8070201@ti.com> (raw)
In-Reply-To: <B85A65D85D7EB246BE421B3FB0FBB5930247CD84F4-/tLxBxkBPtCIQmiDNMet8wC/G2K4zDHf@public.gmane.org>

On 12/09/2010 06:00 AM, Nori, Sekhar wrote:
> On Thu, Dec 09, 2010 at 14:25:49, Nori, Sekhar wrote:
> 
>> This call should simply return if machine is not tnetv107x EVM.
>>
>> I didn't follow the entire series but wondering why
>> platform device registration should be a late init call.
>> Typically the driver probe can be made a late init call
>> in case of init sequence dependencies.
> 
> I meant driver init, not probe.

This was done to handle driver dependencies, and is meant to be
temporary (until a real driver dependency framework comes in).

Please see [1] for a bit more of background on changing drivers to use
module_init().

Regards
- Cyril.

[1]
http://linux.davincidsp.com/pipermail/davinci-linux-open-source/2010-November/020987.html

WARNING: multiple messages have this Message-ID (diff)
From: cyril@ti.com (Cyril Chemparathy)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 11/12] davinci: add tnetv107x evm backlight device
Date: Thu, 09 Dec 2010 09:58:41 -0500	[thread overview]
Message-ID: <4D00EEA1.8070201@ti.com> (raw)
In-Reply-To: <B85A65D85D7EB246BE421B3FB0FBB5930247CD84F4@dbde02.ent.ti.com>

On 12/09/2010 06:00 AM, Nori, Sekhar wrote:
> On Thu, Dec 09, 2010 at 14:25:49, Nori, Sekhar wrote:
> 
>> This call should simply return if machine is not tnetv107x EVM.
>>
>> I didn't follow the entire series but wondering why
>> platform device registration should be a late init call.
>> Typically the driver probe can be made a late init call
>> in case of init sequence dependencies.
> 
> I meant driver init, not probe.

This was done to handle driver dependencies, and is meant to be
temporary (until a real driver dependency framework comes in).

Please see [1] for a bit more of background on changing drivers to use
module_init().

Regards
- Cyril.

[1]
http://linux.davincidsp.com/pipermail/davinci-linux-open-source/2010-November/020987.html

  parent reply	other threads:[~2010-12-09 14:58 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-07 14:51 [PATCH v7 00/12] tnetv107x ssp drivers Cyril Chemparathy
2010-12-07 14:51 ` Cyril Chemparathy
     [not found] ` <1291733522-3626-1-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-12-07 14:51   ` [PATCH v7 01/12] misc: add driver for sequencer serial port Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
     [not found]     ` <1291733522-3626-2-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-12-11  0:13       ` Kevin Hilman
2010-12-11  0:13         ` Kevin Hilman
2010-12-07 14:51   ` [PATCH v7 02/12] davinci: add tnetv107x ssp platform device Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
2010-12-07 14:51   ` [PATCH v7 03/12] davinci: add ssp config for tnetv107x evm board Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
2010-12-07 14:51   ` [PATCH v7 04/12] spi: add ti-ssp spi master driver Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
     [not found]     ` <1291733522-3626-5-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-12-31  6:26       ` Grant Likely
2010-12-31  6:26         ` Grant Likely
2010-12-07 14:51   ` [PATCH v7 05/12] davinci: add spi devices on tnetv107x evm Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
2010-12-07 14:51   ` [PATCH v7 06/12] regulator: add driver for tps6524x regulator Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
     [not found]     ` <1291733522-3626-7-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-12-07 15:22       ` Mark Brown
2010-12-07 15:22         ` Mark Brown
     [not found]         ` <20101207152222.GD9689-HF5t3jzXg/6ND3a5+9QAFujbO/Zr0HzV@public.gmane.org>
2010-12-07 16:46           ` Cyril Chemparathy
2010-12-07 16:46             ` Cyril Chemparathy
2010-12-07 14:51   ` [PATCH v7 07/12] davinci: add tnetv107x evm regulators Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
2010-12-07 14:51   ` [PATCH v7 08/12] gpio: add ti-ssp gpio driver Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
     [not found]     ` <1291733522-3626-9-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-12-31  6:16       ` Grant Likely
2010-12-31  6:16         ` Grant Likely
2010-12-07 14:51   ` [PATCH v7 09/12] davinci: add tnetv107x evm ti-ssp gpio device Cyril Chemparathy
2010-12-07 14:51     ` Cyril Chemparathy
2010-12-07 14:52   ` [PATCH v7 10/12] backlight: add support for tps6116x controller Cyril Chemparathy
2010-12-07 14:52     ` Cyril Chemparathy
2010-12-07 14:52   ` [PATCH v7 11/12] davinci: add tnetv107x evm backlight device Cyril Chemparathy
2010-12-07 14:52     ` Cyril Chemparathy
     [not found]     ` <1291733522-3626-12-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-12-09  8:55       ` Nori, Sekhar
2010-12-09  8:55     ` Nori, Sekhar
2010-12-09 11:00       ` Nori, Sekhar
     [not found]         ` <B85A65D85D7EB246BE421B3FB0FBB5930247CD84F4-/tLxBxkBPtCIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2010-12-09 14:58           ` Cyril Chemparathy [this message]
2010-12-09 14:58             ` Cyril Chemparathy
     [not found]             ` <4D00EEA1.8070201-l0cyMroinI0@public.gmane.org>
2010-12-10 16:01               ` Nori, Sekhar
2010-12-10 16:01                 ` Nori, Sekhar
     [not found]                 ` <B85A65D85D7EB246BE421B3FB0FBB5930247CD8CE8-/tLxBxkBPtCIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2010-12-10 16:56                   ` Kevin Hilman
2010-12-10 16:56                     ` Kevin Hilman
     [not found]       ` <B85A65D85D7EB246BE421B3FB0FBB5930247CD83E1-/tLxBxkBPtCIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2010-12-09 11:00         ` Nori, Sekhar
2010-12-07 14:52   ` [PATCH v7 12/12] davinci: add tnetv107x evm i2c eeprom device Cyril Chemparathy
2010-12-07 14:52     ` Cyril Chemparathy
2010-12-11  0:19 ` [PATCH v7 00/12] tnetv107x ssp drivers Kevin Hilman
2010-12-11  0:19   ` Kevin Hilman
     [not found]   ` <87r5dp6tzv.fsf-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>
2010-12-11 12:42     ` Mark Brown
2010-12-11 12:42       ` Mark Brown
     [not found]       ` <20101211124242.GE9380-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2010-12-30 21:05         ` Grant Likely
2010-12-30 21:05           ` Grant Likely

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D00EEA1.8070201@ti.com \
    --to=cyril-l0cymroini0@public.gmane.org \
    --cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
    --cc=alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org \
    --cc=broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org \
    --cc=davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org \
    --cc=dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org \
    --cc=linus.ml.walleij-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=lrg-kDsPt+C1G03kYMGBc/C6ZA@public.gmane.org \
    --cc=nsekhar-l0cyMroinI0@public.gmane.org \
    --cc=rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org \
    --cc=sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.