how will you accommodate new battery types information then?

Thanks,
Rajanikanth


On 2 July 2012 15:55, Lee Jones <lee.jo...@linaro.org> wrote:
> On 02/07/12 10:49, Rajanikanth HV wrote:
>>
>> In order to accommodate allowable Batteries with in the ab8500 parameters
>> limit battery type information got bloated. (Ref: board-mop500-bm.c)
>
>
> Then maybe we should move the settings from platform data and into the
> driver itself? I'm not keen on the way it looks now.
>
> Arnd, do you have an opinion?
>
>
>> Recommended battery for snowball is: 3.7v 2600ma
>>       Ref:
>> http://www.giga-concept.fr/media/uploads/products/documents/2008/09/9100.pdf
>>
>> However, i currently i am using "SonyEricsson BST41", 3.6v. 1500mAh, 5.4w
>>
>>
>> Thanks,
>> Rajanikanth
>>
>>
>>
>> On 2 July 2012 14:30, Lee Jones <lee.jo...@linaro.org> wrote:
>>>
>>> On 29/06/12 20:44, Rajanikanth HV wrote:
>>>>
>>>>
>>>> thanks for your review comments, i will post it on public branch,
>>>> how about a review from abx500 bm data structure perspective
>>>> ref: arch/arm/mach-ux500/board-mop500-bm.c
>>>
>>>
>>>
>>> I already did:
>>>
>>>
>>>>> It looks like you've just taken values for all of the possible
>>>>> combinations.
>>>>> Is this to support different batteries, or different
>>>>> platforms/machines?
>>>
>>>
>>>
>>> The values you put into the Device Tree should be Snowball specific, but
>>> it
>>> looks like you've loaded it up with _everything_. You need to make sure
>>> that
>>> all of the information you put in makes sense when booting the Snowball
>>> board. Any values which will never be used need to be extracted.
>>>
>>>
>>> Kind regards,
>>> Lee
>>>
>>> --
>>> Lee Jones
>>> Linaro ST-Ericsson Landing Team Lead
>>> M: +44 77 88 633 515
>>> Linaro.org │ Open source software for ARM SoCs
>>> Follow Linaro: Facebook | Twitter | Blog
>>>
>>>
>
>
> --
> Lee Jones
> Linaro ST-Ericsson Landing Team Lead
> M: +44 77 88 633 515
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
>
>

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to