Re: [PATCH v2 07/12] OMAP: Serial: Allow UART parameters to be configured from board file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 4, 2011 at 3:55 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> * Govindraj <govindraj.ti@xxxxxxxxx> [110504 03:03]:
>> On Wed, May 4, 2011 at 3:25 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
>> > * Govindraj.R <govindraj.raja@xxxxxx> [110429 05:39]:
>> >> @@ -250,7 +251,7 @@ static void __init omap_2430sdp_init(void)
>> >>       omap2430_i2c_init();
>> >>
>> >>       platform_add_devices(sdp2430_devices, ARRAY_SIZE(sdp2430_devices));
>> >> -     omap_serial_init();
>> >> +     omap_serial_init(NULL);
>> >>       omap2_hsmmc_init(mmc);
>> >>       omap2_usbfs_init(&sdp2430_usb_config);
>> > ...
>> >
>> >>  static inline void board_serial_init(void)
>> >>  {
>> >> -     omap_serial_init();
>> >> +     omap_serial_init(NULL);
>> >>  }
>> >
>> > This change seems like "crazy churn" and probably not needed
>> > if it's always null. Boards using platform_data can use
>> > omap_serial_init_port instead, right?
>>
>> We can intiliaze only one uart port with omap_serial_init_port.
>>
>> omap_serial_init initializes all uart port.
>>
>> During any client device integration using dma mode
>> (for example Bluetooth) we can pass the dma config
>> params based on the board & client device.
>
> But you're changing all omap_serial_init() calls to
> omap_serial_init(NULL) so there are no users for that.
>
> It should be a separate patch if it's needed.
>

Agree.

Will add seperate func. in serial.c file for board that need to pass these
params and retain the old func. omap_serial_init() syntax to avoid all the
additions in board-*.c files.

--
Thanks,
Govindraj.R

> Tony
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux